diff -r 6297cdf66332 -r d39add9822e2 webengine/osswebengine/WebKit/s60/webview/WebTextFormatMask.cpp --- 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); }