 Chromium Code Reviews
 Chromium Code Reviews Issue 
            6542053:
    Merge 78168 - 2011-02-09  Ryosuke Niwa  <rniwa@webkit.org>...  (Closed)
    
  
    Issue 
            6542053:
    Merge 78168 - 2011-02-09  Ryosuke Niwa  <rniwa@webkit.org>...  (Closed) 
  | DescriptionMerge 78168 - 2011-02-09  Ryosuke Niwa  <rniwa@webkit.org>
        Reviewed by Darin Adler.
        REGRESSION(r76107): Crash in VisibleSelection::toNormalizedRange
        https://bugs.webkit.org/show_bug.cgi?id=54053
        The bug was caused by RenderBlock::positionForPoint's improperly returning a shadow node inside
        RenderFileUploadControl for hit testing and VisibleSelection::toNormalizedRange's always assuming
        the position variable "s" is not null.
        Fixed the bug by always returning a null position from RenderFileUploadControl::positionForPoint,
        and also exiting early when either "s" or "e" is null in VisibleSelection::toNormalizedRange.
        Test: fast/forms/file-input-hit-test.html
        * editing/VisibleSelection.cpp:
        (WebCore::VisibleSelection::toNormalizedRange):
        * rendering/RenderFileUploadControl.cpp:
        (WebCore::RenderFileUploadControl::positionForPoint):
        * rendering/RenderFileUploadControl.h:
2011-02-09  Ryosuke Niwa  <rniwa@webkit.org>
        Reviewed by Darin Adler.
        REGRESSION(r76107): Crash in VisibleSelection::toNormalizedRange
        https://bugs.webkit.org/show_bug.cgi?id=54053
        Added a test to ensure WebKit does not crash when a user clicks on a region immediately after a file
        upload control.
        * fast/forms/file-input-hit-test-expected.txt: Added.
        * fast/forms/file-input-hit-test.html: Added.
TBR=rniwa@webkit.org
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=79264
   Patch Set 1 #
 Messages
    Total messages: 1 (0 generated)
     | |||||||||||||||||||||||||||||||||||||||||||||||||||||||