Bug 42547 - WebKitTestRunner needs layoutTestController.setDashboardCompatibilityMode
Summary: WebKitTestRunner needs layoutTestController.setDashboardCompatibilityMode
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit2 (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Jonathan Bedard
Keywords: InRadar
Depends on:
Reported: 2010-07-18 21:48 PDT by Maciej Stachowiak
Modified: 2016-09-02 15:54 PDT (History)
2 users (show)

See Also:

Patch (14.15 KB, patch)
2016-08-08 12:38 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Archive of layout-test-results from ews125 for ios-simulator-wk2 (700.99 KB, application/zip)
2016-08-08 13:34 PDT, Build Bot
no flags Details
Patch (14.17 KB, patch)
2016-08-08 14:12 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (14.04 KB, patch)
2016-08-22 10:09 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (14.30 KB, patch)
2016-08-29 10:54 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff
Patch (17.18 KB, patch)
2016-08-29 11:51 PDT, Jonathan Bedard
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Maciej Stachowiak 2010-07-18 21:48:10 PDT
WebKitTestRunner needs layoutTestController.setDashboardCompatibilityMode
Comment 1 Maciej Stachowiak 2010-07-18 21:52:31 PDT
Comment 2 Jonathan Bedard 2016-08-08 12:38:21 PDT
Created attachment 285578 [details]
Comment 3 Build Bot 2016-08-08 13:34:23 PDT
Comment on attachment 285578 [details]

Attachment 285578 [details] did not pass ios-sim-ews (ios-simulator-wk2):
Output: http://webkit-queues.webkit.org/results/1835932

New failing tests:
Comment 4 Build Bot 2016-08-08 13:34:26 PDT
Created attachment 285585 [details]
Archive of layout-test-results from ews125 for ios-simulator-wk2

The attached test failures were seen while running run-webkit-tests on the ios-sim-ews.
Bot: ews125  Port: ios-simulator-wk2  Platform: Mac OS X 10.11.5
Comment 5 Jonathan Bedard 2016-08-08 14:12:29 PDT
Created attachment 285589 [details]
Comment 6 Darin Adler 2016-08-20 19:50:22 PDT
Comment on attachment 285589 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=285589&action=review

> Source/WebKit2/WebProcess/InjectedBundle/InjectedBundle.cpp:323
> +    const HashSet<Page*>& pages = PageGroup::pageGroup(pageGroup->identifier())->pages();
> +    for (HashSet<Page*>::iterator iter = pages.begin(); iter != pages.end(); ++iter)
> +        (*iter)->settings().setUsesDashboardBackwardCompatibilityMode(enabled);

Here’s how we should write this in new code (other code in tis file is old):

    for (auto& page : PageGroup::pageGroup(pageGroup->identifier())->pages())

We don’t use abbreviations in WebKit code, so code should be using "iter" (even though there are many other functions in this file doing it). And with a C++11 for loop we don’t need to bother with a local variable.

> Tools/WebKitTestRunner/InjectedBundle/TestRunner.cpp:420
> +    fputs("Dashboard unsupported\n", stderr);

This isn’t a good idea. The test runner should not be writing things out directly to stderr.
Comment 7 Jonathan Bedard 2016-08-22 10:09:40 PDT
Created attachment 286602 [details]
Comment 8 WebKit Commit Bot 2016-08-27 18:56:07 PDT
Comment on attachment 286602 [details]

Rejecting attachment 286602 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 286602, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
   1faa5bf..d95384d  master     -> origin/master
Partial-rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc ...
Currently at 205093 = 1faa5bf9266d53f8f9c1608d5eade57c354a00ad
r205094 = d95384d3d87658680c88eef17a66218ee4bf1d38
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/1956872
Comment 9 Jonathan Bedard 2016-08-29 10:54:49 PDT
Created attachment 287281 [details]
Comment 10 Darin Adler 2016-08-29 11:04:52 PDT
Comment on attachment 287281 [details]

View in context: https://bugs.webkit.org/attachment.cgi?id=287281&action=review

> LayoutTests/platform/ios-simulator-wk2/TestExpectations:92
> +# iOS is missing some dashboard features

Dashboard is a Mac-specific feature. We don't need the dashboard features on any other platform. This test should be treated as a Mac-only test and no comment should imply a future plan to add this to any other platform (including iOS).
Comment 11 Jonathan Bedard 2016-08-29 11:51:40 PDT
Created attachment 287298 [details]
Comment 12 WebKit Commit Bot 2016-09-02 15:54:08 PDT
Comment on attachment 287298 [details]

Clearing flags on attachment: 287298

Committed r205379: <http://trac.webkit.org/changeset/205379>
Comment 13 WebKit Commit Bot 2016-09-02 15:54:12 PDT
All reviewed patches have been landed.  Closing bug.