Summary: | REGRESSION (r180907): inspector/timeline tests frequently time out or fail | ||
---|---|---|---|
Product: | WebKit | Reporter: | Brent Fulgham <bfulgham> |
Component: | Web Inspector | Assignee: | Matt Baker <mattbaker> |
Status: | NEW --- | ||
Severity: | Normal | CC: | ap, bfulgham, ggaren, graouts, inspector-bugzilla-changes, jonowells, mark.lam, webkit-bug-importer |
Priority: | P2 | Keywords: | InRadar |
Version: | 528+ (Nightly build) | ||
Hardware: | All | ||
OS: | All |
Description
Brent Fulgham
2015-03-02 20:10:16 PST
This looks like a regression from <http://trac.webkit.org/r180907>: https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard.html#showAllRuns=true&tests=inspector%2Ftimeline%2Fexception-in-injected-script-while-recording.html Should we roll out? (In reply to comment #2) > This looks like a regression from <http://trac.webkit.org/r180907>: > > https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard. > html#showAllRuns=true&tests=inspector%2Ftimeline%2Fexception-in-injected- > script-while-recording.html > > Should we roll out? I think so. if we leave it skipped I think it may just join the dozen other inspector-related bugs and timeouts that have been left unaddressed. (In reply to comment #3) > (In reply to comment #2) > > This looks like a regression from <http://trac.webkit.org/r180907>: > > > > https://webkit-test-results.appspot.com/dashboards/flakiness_dashboard. > > html#showAllRuns=true&tests=inspector%2Ftimeline%2Fexception-in-injected- > > script-while-recording.html > > > > Should we roll out? > > I think so. if we leave it skipped I think it may just join the dozen other > inspector-related bugs and timeouts that have been left unaddressed. I disagree. I added a test inspector/timeline/exception-in-injected-script-while-recording.html which is based on inspector/timeline/debugger-paused-while-recording.html. Note that the original test was previously excluded, and I fixed a bug for it and unskipped the test (see http://trac.webkit.org/changeset/180901). I was warned that the test was seen to be flaky before, but I since I fixed one issue for it, and I wasn't able to reproduce the flakiness locally, I unskipped the test. The proper remedy is to restore the skipping inspector/timeline again. I will do this while I take a second look at why the tests can flake now that I have examples of their flakiness to study. I've restored the skipping of the inspector/timeline directory in r180922: <http://trac.webkit.org/r180922>. From the bots, it looks like the flaky failures are a lot more reproducible if we run with webkit2 instead of webkit1. I was able to reproduce this locally. These tests now ASSERT under InspectorTimelineAgent::willComposite. Likely due to that not handling nesting well. |