Bug 135361

Summary: REGRESSION(r164133): Selection disappears after scrolling on nytimes.com
Product: WebKit Reporter: zalan <zalan>
Component: Layout and RenderingAssignee: zalan <zalan>
Status: RESOLVED FIXED    
Severity: Normal CC: commit-queue, rniwa
Priority: P2 Keywords: InRadar, Regression
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

Description zalan 2014-07-28 15:43:01 PDT
1. Load nytimes.com
2. Select all content
3. start scrolling up and down (it may require a few iterations but I was able to repro it almost always)

Expected result: content stays selected.
Actual result: after scrolling the content gets deselected.



<rdar://problem/17824793>
Comment 1 zalan 2014-07-28 15:54:36 PDT
Created attachment 235626 [details]
Patch
Comment 2 Ryosuke Niwa 2014-07-28 16:03:54 PDT
Comment on attachment 235626 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=235626&action=review

> LayoutTests/fast/dynamic/selection-gets-cleared-when-part-of-it-gets-removed-expected.html:3
> +  <title>This tests that selection stays on when part of the selection gets removed.</title>

Nit: wrong indentations!
Ditto elsewhere in the file.

> LayoutTests/fast/dynamic/selection-gets-cleared-when-part-of-it-gets-removed.html:3
> +  <title>This tests that selection stays on when part of the selection gets removed.</title>

Nit: wrong indentations!
Ditto elsewhere in the file.

> LayoutTests/fast/dynamic/selection-gets-cleared-when-part-of-it-gets-removed.html:15
> +    

Nit: whitespaces.

> LayoutTests/fast/dynamic/selection-gets-cleared-when-part-of-it-gets-removed.html:21
> +    

Nit: whitespaces.
Comment 3 zalan 2014-07-28 16:29:48 PDT
Created attachment 235638 [details]
Patch
Comment 4 WebKit Commit Bot 2014-07-28 18:46:14 PDT
Comment on attachment 235638 [details]
Patch

Clearing flags on attachment: 235638

Committed r171718: <http://trac.webkit.org/changeset/171718>
Comment 5 WebKit Commit Bot 2014-07-28 18:46:17 PDT
All reviewed patches have been landed.  Closing bug.