webengine/osswebengine/WebKit/s60/webview/WebTextFormatMask.cpp
changeset 42 d39add9822e2
parent 38 6297cdf66332
child 65 5bfc169077b2
child 92 e1bea15f9a39
--- a/webengine/osswebengine/WebKit/s60/webview/WebTextFormatMask.cpp	Mon Jan 18 21:20:18 2010 +0200
+++ b/webengine/osswebengine/WebKit/s60/webview/WebTextFormatMask.cpp	Tue Feb 02 00:56:45 2010 +0200
@@ -192,9 +192,15 @@
         m_currentMask = m_currentMask->nextMask();
     }
 
-    // did we use up all the masks?
-    if(m_currentMask && m_currentMask->multitude() != kInfinite)
-        return false;
+       // this check doesn't seem to be proper as the check is done for 
+       // the partial text.Because the checkText() is called for every character input by user,
+       // there are remaining masks after complete text length has been checked, 
+       // that is valid case and it should not return false. 
+       // If text length is bigger than mask length then that case is handled within for loop 
+       //before this condition check. So it is redundant in current implementation
+       // did we use up all the masks?
+       /* if(m_currentMask && m_currentMask->multitude() != kInfinite)
+        return false;*/
 
     return (eb.m_start == -1);
 }