Bug 55726

Summary:

Enable fullscreen layout tests for Chromium

Product:

WebKit

Reporter:

David Dorwin <ddorwin>

Component:

WebCore Misc.

Assignee:

Nobody <webkit-unassigned>

Status:

RESOLVED FIXED

Severity:

Normal

CC:

commit-queue, ddorwin, levin, scherkus

Priority:

Version:

528+ (Nightly build)

Hardware:

All

OS:

All

Bug Depends on:

44797, 55727

Bug Blocks:

Attachments:

Description	Flags
Enable fullscreen in Chromium's DumpRenderTree.	none
Remove test_expectations entry for fullscreen tests.	none
Fixed typo.	none
Merged with head	none
Merged with head	none

David Dorwin

Reported 2011-03-03 16:47:59 PST

The fullscreen layout tests have been failing in Chromium since they were landed. The fullscreen API is enabled for Chromium WebKit (bug 44797) and is about to be compiled but disabled by default for Chromium. This bug replaces bug 44797 for tracking the test_expectations.txt override for the fullscreen tests. Once the fullscreen API is enabled by default in Chromium, we can remove the entry(ies) from test_expectations.txt.

Attachments
Enable fullscreen in Chromium's DumpRenderTree. (1.36 KB, patch) 2011-03-29 17:22 PDT, David Dorwin	no flags	Details Formatted Diff Diff
Remove test_expectations entry for fullscreen tests. (2.68 KB, patch) 2011-03-30 14:25 PDT, David Dorwin	no flags	Details Formatted Diff Diff
Fixed typo. (2.67 KB, patch) 2011-04-04 14:29 PDT, David Dorwin	no flags	Details Formatted Diff Diff
Merged with head (2.70 KB, patch) 2011-04-04 17:03 PDT, David Dorwin	no flags	Details Formatted Diff Diff
Merged with head (2.73 KB, patch) 2011-04-06 13:45 PDT, David Dorwin	no flags	Details Formatted Diff Diff
Show Obsolete (4) View All Add attachment proposed patch, testcase, etc.

David Dorwin

Comment 1 2011-03-29 17:22:09 PDT

Created attachment 87439 [details] Enable fullscreen in Chromium's DumpRenderTree. Enable fullscreen in DumpRenderTree by default so we can enable the the fullscreen tests before the feature is enabled by default in Chromium. This is similar to what was done for the mac platform.

David Dorwin

Comment 2 2011-03-30 14:25:38 PDT

Created attachment 87614 [details] Remove test_expectations entry for fullscreen tests. There is a Chromium entry overriding this, and we should be able to remove it once this changeset as well as bug 57400 and 57483 are committed.

David Levin

Comment 3 2011-04-04 12:01:45 PDT

Comment on attachment 87614 [details] Remove test_expectations entry for fullscreen tests. View in context: https://bugs.webkit.org/attachment.cgi?id=87614&action=review > Tools/DumpRenderTree/chromium/WebPreferences.cpp:169 > + // Enable fullscreen so the fullscreen LaoutTests can run. typo: LaoutTests

David Levin

Comment 4 2011-04-04 13:00:04 PDT

Note that even though you are not a committer you can take advantage of this r+. Just fix the issue and change the ChangeLog to say "Review by David Levin." (instead of the OOPS thing). Then upload a new patch and don't mark it r?. You may want to mark it as cq? Just any committer to make it as cq+ (and you have a few around you) and it will go through.

David Dorwin

Comment 5 2011-04-04 14:29:29 PDT

Created attachment 88130 [details] Fixed typo.

David Dorwin

Comment 6 2011-04-04 17:03:19 PDT

Created attachment 88165 [details] Merged with head

David Dorwin

Comment 7 2011-04-06 13:45:11 PDT

Created attachment 88503 [details] Merged with head

WebKit Commit Bot

Comment 8 2011-04-06 15:37:27 PDT

Comment on attachment 88503 [details] Merged with head Clearing flags on attachment: 88503 Committed r83112: <http://trac.webkit.org/changeset/83112>

WebKit Commit Bot

Comment 9 2011-04-06 15:37:35 PDT

All reviewed patches have been landed. Closing bug.

Note You need to log in before you can comment on or make changes to this bug.