Reviewed by John.
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jul 2004 23:15:02 +0000 (23:15 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 28 Jul 2004 23:15:02 +0000 (23:15 +0000)
- avoid triggering an assertion when using dead keys (like option-e)

        * WebView.subproj/WebHTMLView.m:
        (-[WebHTMLView insertText:]): Don't pass zero-length strings to
WebCore. Dead-key input seems to insert empty strings as a side
effect.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@7150 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKit/ChangeLog
WebKit/WebView.subproj/WebHTMLView.m

index 34cfb40..c46279d 100644 (file)
@@ -1,3 +1,14 @@
+2004-07-28  Maciej Stachowiak  <mjs@apple.com>
+
+        Reviewed by John.
+
+       - avoid triggering an assertion when using dead keys (like option-e)
+
+        * WebView.subproj/WebHTMLView.m:
+        (-[WebHTMLView insertText:]): Don't pass zero-length strings to
+       WebCore. Dead-key input seems to insert empty strings as a side
+       effect.
+
 2004-07-28  Trey Matteson  <trey@apple.com>
 
        Spellchecking, Part I.  Basic spellcheck is working.  Spelling panel is hooked up.
index d93ff7a..e8c0035 100644 (file)
@@ -3125,6 +3125,12 @@ static WebHTMLView *lastHitView = nil;
 - (void)insertText:(NSString *)text
 {
     WebBridge *bridge = [self _bridge];
+
+    // avoid entering empty strings because it confuses WebCore
+    if ([text length] == 0) {
+       return;
+    }
+
     if ([bridge isSelectionEditable]) {
         WebView *webView = [self _webView];
         if ([[webView _editingDelegateForwarder] webView:webView shouldInsertText:text replacingDOMRange:[bridge selectedDOMRange] givenAction:WebViewInsertActionTyped]) {