Attachment 139606 Details for Bug 85271 – Quick fix v1

[patch] Quick fix v1

issue86271-patch0.txt (text/plain), 1.54 KB, created by Hironori Bono on 2012-05-01 03:32:58 PDT

(hide)

Description:

Filename:

MIME Type:

Creator: Hironori Bono

Created: 2012-05-01 03:32:58 PDT

Size: 1.54 KB

patch

obsolete

>Index: Source/WebKit/chromium/ChangeLog
>===================================================================
>--- Source/WebKit/chromium/ChangeLog	(revision 115716)
>+++ Source/WebKit/chromium/ChangeLog	(working copy)
>@@ -1,3 +1,16 @@
>+2012-05-01  Hironori Bono  <hbono@chromium.org>
>+
>+        [Chromium] Regression r113405 ContextMenuImpl does not set a misspelled word
>+        https://bugs.webkit.org/show_bug.cgi?id=85271
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        This change calls selectMisspelledWord() to get a misspelled word so Chrome can
>+        set the misspelledWord variable without selecting text.
>+
>+        * src/ContextMenuClientImpl.cpp:
>+        (WebKit::ContextMenuClientImpl::getCustomMenuFromDefaultItems):
>+
> 2012-04-30  Kent Tamura  <tkent@chromium.org>
> 
>         [Chromium/Windows] Add LocalizedDateWin
>Index: Source/WebKit/chromium/src/ContextMenuClientImpl.cpp
>===================================================================
>--- Source/WebKit/chromium/src/ContextMenuClientImpl.cpp	(revision 115701)
>+++ Source/WebKit/chromium/src/ContextMenuClientImpl.cpp	(working copy)
>@@ -290,7 +290,7 @@ PlatformMenuDescription ContextMenuClien
>                     }
>                 }
>                 data.dictionarySuggestions = suggestions;
>-                data.misspelledWord = range->text();
>+                data.misspelledWord = selectMisspelledWord(defaultMenu, selectedFrame);
>             }
>         } else if (m_webView->focusedWebCoreFrame()->editor()->isContinuousSpellCheckingEnabled()) {
>             data.isSpellCheckingEnabled = true;

Actions: View | Formatted Diff | Diff

Attachments on bug 85271: 139606