Summary: | Settings::editingBehaviorType() incorrectly returns mac for the gtk build when running tests | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Benjamin (Ben) Kalman <kalman> | ||||
Component: | Tools / Tests | Assignee: | Antonio Gomes <tonikitoo> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | abarth, eric, mrobinson, rniwa, tonikitoo, webkit.review.bot | ||||
Priority: | P2 | ||||||
Version: | 528+ (Nightly build) | ||||||
Hardware: | PC | ||||||
OS: | Linux | ||||||
Bug Depends on: | |||||||
Bug Blocks: | 49873, 51261 | ||||||
Attachments: |
|
Description
Benjamin (Ben) Kalman
2010-12-15 21:04:45 PST
Some helpful links: see http://trac.webkit.org/browser/trunk/LayoutTests/platform/gtk/editing/selection/extend-selection-home-end-expected.txt (gtk results) and http://trac.webkit.org/browser/trunk/LayoutTests/platform/mac/editing/selection/extend-selection-home-end-expected.txt (mac results) are the same when they shouldn't be. Rather, should be like http://trac.webkit.org/browser/trunk/LayoutTests/editing/selection/extend-selection-home-end-expected.txt I will look at this today. Created attachment 76841 [details] patch v1 (committed r74269, r=mrobinson) 1) Make GTK+ reset its editing behavior to UNIX (default) instead of Mac in DumpRenderTree::resetDefaultsToConsistentValues 2) Removes gtk and win expectation files for editing/selection/extend-selection-home-end.html since they are equal to the cross platform result. Comment on attachment 76841 [details] patch v1 (committed r74269, r=mrobinson) LGTM. Thanks for fixing this bug :) Comment on attachment 76841 [details] patch v1 (committed r74269, r=mrobinson) Clearing flags on attachment: 76841 Committed r74269: <http://trac.webkit.org/changeset/74269> http://trac.webkit.org/changeset/74269 might have broken GTK Linux 64-bit Debug (In reply to comment #6) > http://trac.webkit.org/changeset/74269 might have broken GTK Linux 64-bit Debug Fixed in http://trac.webkit.org/changeset/74273 with the test rebasilined, and filed bug 51261 for the proper fix. |