[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    branches/KDE/4.1/kdelibs/khtml/html
From:       Germain Garand <germain () ebooksfrance ! org>
Date:       2008-09-15 4:26:56
Message-ID: 1221452816.145277.19169.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 861086 by ggarand:

automatically merged revision 861053:
one more follow-up to r859311:

now save/restore code need to differentiate null/empty too.

 M  +5 -5      html_formimpl.cpp  


--- branches/KDE/4.1/kdelibs/khtml/html/html_formimpl.cpp #861085:861086
@@ -1375,7 +1375,7 @@
             document()->view()->addFormCompletionItem(name().string(), \
value().string());  /* nobreak */
     default:
-        return value().string() + (m_unsubmittedFormChange ? 'M' : '.');
+        return value().string() + (m_unsubmittedFormChange ? 'M' : '.') + \
(value().isNull() ? 'N' : '.');  }
 }
 
@@ -1387,15 +1387,15 @@
         setChecked((state == QLatin1String("on")));
         break;
     case FILE:
-        m_value = DOMString(state.left(state.length()-1));
+        m_value = DOMString(state.left(state.length()-2));
         setChanged();
         break;
     case HIDDEN:
         // Don't mess with those...
         break;
     default:
-        setValue(DOMString(state.left(state.length()-1)));
-        m_unsubmittedFormChange = state.endsWith('M');
+        setValue(state.endsWith('N') ? DOMString() : \
DOMString(state.left(state.length()-2))); +        m_unsubmittedFormChange = \
(state.right(1) == "M");  break;
     }
 }
@@ -1789,7 +1789,7 @@
 {
     if (m_type == FILE) return;
 
-    m_value = (val.isNull() ? DOMString("") : val);
+    m_value = val;
     // ### set attribute for other types, too. no need for m_value
     // ### in those cases.
     if (m_type == RADIO || m_type == CHECKBOX)


[prev in list] [next in list] [prev in thread] [next in thread]