Bug 147503 - Web Inspector: Unskip / Unflake inspector tests after r187627
Summary: Web Inspector: Unskip / Unflake inspector tests after r187627
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Web Inspector (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Joseph Pecoraro
URL:
Keywords: DoNotImportToRadar
Depends on:
Blocks:
 
Reported: 2015-07-31 13:56 PDT by Joseph Pecoraro
Modified: 2015-07-31 18:23 PDT (History)
9 users (show)

See Also:


Attachments
[PATCH] Proposed Fix (17.84 KB, patch)
2015-07-31 14:07 PDT, Joseph Pecoraro
burg: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Pecoraro 2015-07-31 13:56:19 PDT
* SUMMARY
Unskip / Unflake inspector tests after r187627. This makes tests far more reliable in WebKit2. Test should no longer be Slow or Timeout unless they are incorrectly written or regress. So lets update all the TestExpectations that were skipping tests.
Comment 1 Joseph Pecoraro 2015-07-31 14:07:56 PDT
Created attachment 257957 [details]
[PATCH] Proposed Fix
Comment 2 Joseph Pecoraro 2015-07-31 14:09:20 PDT
This gets us running up to 54 of 64 tests. Each of the skipped tests has up to date bugs filed about them:

  * inspector/timeline/*exception* ASSERTs
  * inspector/debugger/*promise*uncaught* failing
  * inspector/replay failing
Comment 3 Brian Burg 2015-07-31 14:32:20 PDT
Comment on attachment 257957 [details]
[PATCH] Proposed Fix

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

r=me

Thanks for getting to the bottom of this. When you land this, please be around to watch the bots and update expectations if some lingering issues persist.

> Source/WebInspectorUI/ChangeLog:15
> +        resends the output to the new page for test output.

Nice.

> Source/WebInspectorUI/ChangeLog:21
> +        Clearing messages wasn't really serving a purpose.

Agree, it was kind of sloppy.
Comment 4 Joseph Pecoraro 2015-07-31 16:58:22 PDT
<http://trac.webkit.org/changeset/187692>

Watching the bots. I expect Windows might have some issues, in which case I'll skip all inspector tests on Windows, as I think they are all skipped on Windows in some capacity.
Comment 5 Joseph Pecoraro 2015-07-31 18:23:43 PDT
Marked 1 test as Flakey for all, then skipped tests on Windows and GTK with:
http://trac.webkit.org/changeset/187699

EFL is in a tricky spot. Most pass, some appear flakey (duplicate output), and few timeout. I'll watch it a bit.