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.
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.
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.
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
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.
Created attachment 88130 [details] Fixed typo.
Created attachment 88165 [details] Merged with head
Created attachment 88503 [details] Merged with head
Comment on attachment 88503 [details] Merged with head Clearing flags on attachment: 88503 Committed r83112: <http://trac.webkit.org/changeset/83112>
All reviewed patches have been landed. Closing bug.