Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(2)

Issue 19408: Layout test for fast/forms/focus2.html (Closed)

Created:
10 years, 6 months ago by Finnur
Modified:
8 years, 1 month ago
Reviewers:
jcampan
CC:
chromium-reviews_googlegroups.com
Visibility:
Public.

Description

In WebKit r39791 Glenn Wilson changed radio buttons so that they can be checked even if they don't have a name. Before, they were matching IE6 and early versions of IE7 but now match current IE7, IE8 and FF. Therefore, our expected results must reflect that the user can now focus the radio button.. Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=8807

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+7 lines, -2 lines) Patch
M webkit/data/layout_tests/platform/chromium-win/LayoutTests/fast/forms/focus2-expected.txt View 3 chunks +6 lines, -1 line 0 comments Download
M webkit/tools/layout_tests/test_lists/tests_fixable.txt View 2 chunks +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Finnur
Only one layout test fix in this changelist... /me sad
10 years, 6 months ago (2009-01-28 00:55:09 UTC) #1
jcampan
10 years, 6 months ago (2009-01-28 01:01:39 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698