Changeset 141608 in webkit


Ignore:
Timestamp:
Feb 1, 2013 10:55:09 AM (11 years ago)
Author:
commit-queue@webkit.org
Message:

Source/WebKit/chromium: [Chromium] Ignore whitespace in spellcheck
https://bugs.webkit.org/show_bug.cgi?id=108510

Patch by Rouslan Solomakhin <rouslan@chromium.org> on 2013-02-01
Reviewed by Ryosuke Niwa.

  • src/ContextMenuClientImpl.cpp:

(WebKit::selectMisspellingAsync): Ignore whitespace when selecting the misspelling.

LayoutTests: [Chromium] Expect spellcheck to ignore whitespace
https://bugs.webkit.org/show_bug.cgi?id=108510

Patch by Rouslan Solomakhin <rouslan@chromium.org> on 2013-02-01
Reviewed by Ryosuke Niwa.

  • platform/chromium/TestExpectations: Expect spellcheck to ignore whitespace.
Location:
trunk
Files:
4 edited

Legend:

Unmodified
Added
Removed
  • trunk/LayoutTests/ChangeLog

    r141607 r141608  
     12013-02-01  Rouslan Solomakhin  <rouslan@chromium.org>
     2
     3        [Chromium] Expect spellcheck to ignore whitespace
     4        https://bugs.webkit.org/show_bug.cgi?id=108510
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        * platform/chromium/TestExpectations: Expect spellcheck to ignore whitespace.
     9
    1102013-02-01  Pavel Feldman  <pfeldman@chromium.org>
    211
  • trunk/LayoutTests/platform/chromium/TestExpectations

    r141578 r141608  
    43394339
    43404340# Spellchecker behavior tests.
    4341 webkit.org/b/108370 editing/spelling/spelling-double-clicked-word.html [ Failure ]
    43424341webkit.org/b/108370 editing/spelling/spelling-double-clicked-word-with-underscores.html [ Failure ]
    43434342webkit.org/b/108370 editing/spelling/spelling-with-punctuation-selection.html [ Failure ]
    43444343webkit.org/b/108370 editing/spelling/spelling-with-underscore-selection.html [ Failure ]
    4345 webkit.org/b/108370 editing/spelling/spelling-with-whitespace-selection.html [ Failure ]
    43464344
    43474345webkit.org/b/108424 media/video-error-does-not-exist.html [ Failure Crash ]
  • trunk/Source/WebKit/chromium/ChangeLog

    r141595 r141608  
     12013-02-01  Rouslan Solomakhin  <rouslan@chromium.org>
     2
     3        [Chromium] Ignore whitespace in spellcheck
     4        https://bugs.webkit.org/show_bug.cgi?id=108510
     5
     6        Reviewed by Ryosuke Niwa.
     7
     8        * src/ContextMenuClientImpl.cpp:
     9        (WebKit::selectMisspellingAsync): Ignore whitespace when selecting the misspelling.
     10
    1112013-02-01  Sheriff Bot  <webkit.review.bot@gmail.com>
    212
  • trunk/Source/WebKit/chromium/src/ContextMenuClientImpl.cpp

    r141519 r141608  
    167167    }
    168168
    169     if (!WebCore::areRangesEqual(markerRange.get(), selectionRange.get()))
     169    if (markerRange->text().stripWhiteSpace() != selectionRange->text().stripWhiteSpace())
    170170        return String();
    171171
Note: See TracChangeset for help on using the changeset viewer.