Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(20)

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

Created:
11 years, 11 months ago by Finnur
Modified:
9 years, 6 months 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
11 years, 11 months ago (2009-01-28 00:55:09 UTC) #1
jcampan
11 years, 11 months ago (2009-01-28 01:01:39 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698