Bug 94730 - Layout Test fast/repaint/japanese-rl-selection-repaint-in-regions.html is failing after r126304
Summary: Layout Test fast/repaint/japanese-rl-selection-repaint-in-regions.html is fai...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Tools / Tests (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac All
: P2 Normal
Assignee: Andrei Bucur
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-08-22 11:10 PDT by Kenneth Russell
Modified: 2012-08-28 23:29 PDT (History)
10 users (show)

See Also:


Attachments
Patch (28.27 KB, patch)
2012-08-28 10:12 PDT, Andrei Bucur
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from gce-cq-03 (566.98 KB, application/zip)
2012-08-28 11:25 PDT, WebKit Review Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Kenneth Russell 2012-08-22 11:10:37 PDT
The following layout test is failing on Chromium Mac:

fast/repaint/japanese-rl-selection-repaint-in-regions.html

It looks like it started failing with different image results after http://trac.webkit.org/changeset/126304/ . See:

http://test-results.appspot.com/dashboards/flakiness_dashboard.html#showExpectations=true&tests=fast%2Frepaint%2Fjapanese-rl-selection-repaint-in-regions.html

and look specifically at the Webkit Mac10.6, Webkit Mac10.7 and Webkit Mac10.7 (dbg) image results. (The other platforms were expected to have differences.)

From reading the description of the above changeset it doesn't sound like it was supposed to affect rendering at all, only correctness, so I think this change in behavior warrants investigation.

Not seeing failures on the Apple bots on build.webkit.org, interestingly.

I'm going to mark this as an expected failure in LayoutTests/platform/chromium/TestExpectations for the moment.
Comment 1 Kenneth Russell 2012-08-22 11:14:12 PDT
Committed r126322: <http://trac.webkit.org/changeset/126322>
Comment 2 Andrei Bucur 2012-08-28 10:12:33 PDT
Created attachment 161005 [details]
Patch
Comment 3 Dave Hyatt 2012-08-28 10:25:41 PDT
Comment on attachment 161005 [details]
Patch

r=me
Comment 4 WebKit Review Bot 2012-08-28 11:25:42 PDT
Comment on attachment 161005 [details]
Patch

Rejecting attachment 161005 [details] from commit-queue.

New failing tests:
http/tests/cache/post-with-cached-subresources.php
Full output: http://queues.webkit.org/results/13654451
Comment 5 WebKit Review Bot 2012-08-28 11:25:46 PDT
Created attachment 161020 [details]
Archive of layout-test-results from gce-cq-03

The attached test failures were seen while running run-webkit-tests on the commit-queue.
Bot: gce-cq-03  Port: <class 'webkitpy.common.config.ports.ChromiumXVFBPort'>  Platform: Linux-2.6.39-gcg-201203291735-x86_64-with-Ubuntu-10.04-lucid
Comment 6 WebKit Review Bot 2012-08-28 19:27:35 PDT
Comment on attachment 161005 [details]
Patch

Rejecting attachment 161005 [details] from review queue.

hyatt@apple.com does not have reviewer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have reviewer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have reviewer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your reviewer rights.
Comment 7 WebKit Review Bot 2012-08-28 19:28:12 PDT
Comment on attachment 161005 [details]
Patch

Rejecting attachment 161005 [details] from commit-queue.

abarth@webkit.org does not have committer permissions according to http://trac.webkit.org/browser/trunk/Tools/Scripts/webkitpy/common/config/committers.py.

- If you do not have committer rights please read http://webkit.org/coding/contributing.html for instructions on how to use bugzilla flags.

- If you have committer rights please correct the error in Tools/Scripts/webkitpy/common/config/committers.py by adding yourself to the file (no review needed).  The commit-queue restarts itself every 2 hours.  After restart the commit-queue will correctly respect your committer rights.
Comment 8 Adam Barth 2012-08-28 19:30:43 PDT
Sorry for the noise on this bug.  I'm not sure what's wrong with the bots.  :(
Comment 9 Adam Barth 2012-08-28 23:15:17 PDT
Comment on attachment 161005 [details]
Patch

Re-applying Hyatt's r+
Comment 10 WebKit Review Bot 2012-08-28 23:29:12 PDT
Comment on attachment 161005 [details]
Patch

Clearing flags on attachment: 161005

Committed r126961: <http://trac.webkit.org/changeset/126961>
Comment 11 WebKit Review Bot 2012-08-28 23:29:17 PDT
All reviewed patches have been landed.  Closing bug.