Bug 50823 – Flaky Test: inspector/timeline-event-dispatch.html

• Status: RESOLVED
• Resolution: FIXED
• Component: Tools / Tests

[WebKit Commit Bot]

This is an automatically generated bug from the commit-queue.
inspector/timeline-event-dispatch.html has been flaky on the commit-queue.

inspector/timeline-event-dispatch.html was authored by [e-mail] and [e-mail].
http://trac.webkit.org/browser/trunk/inspector/timeline-event-dispatch.html

The commit-queue just saw inspector/timeline-event-dispatch.html flake while processing attachment 76198 [details] on bug 46895.

The bots will update this with information from each new failure.

If you would like to track this test fix with another bug, please close this bug as a duplicate.

[Tony Chang]

If this is on chromium, this is probably bug 50722

*** This bug has been marked as a duplicate of bug 50722 ***

[Eric Seidel (no email)]

No, this is on Snow Leopard.  The commit-queue is always Snow Leopard.

[Tony Chang]

Ok, then not a dupe, although we see this test timeout a lot on chromium debug builds.

[WebKit Commit Bot]

The commit-queue just saw inspector/timeline-event-dispatch.html flake while processing attachment 77158 [details] on bug 51429.
Bot: cr-jail-3  Port: Mac  Platform: Mac OS X 10.6.4

[Ilya Tikhonovsky]

Created attachment 77627 [details]
[patch] initial version

[Ilya Tikhonovsky]

It was a problem with Chromium DRT. DevToolsAgent object instance was
attached not only to the inspected page but to the DevTools window too.
As result all the inspector files were interpreted as a content of inspected page
and were transfered to the DevTools scripts panel for debugging etc.

[Pavel Feldman]

Comment on attachment 77627 [details]
[patch] initial version

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

> Tools/DumpRenderTree/chromium/TestShell.cpp:568
> +    WebView* view = WebView::create(host, attachDevTools ? m_drtDevToolsAgent.get() : 0);

It is true that WebView for devtools should have 0 as WebDevToolsAgentClient. However, even after your fix, same m_drtDevToolsAgent is being reused for different WebViews (and hence encapsulated WebDevToolsAgent instances). This is not right since it results in same host id being used by different WebViews. Are we using this method to create exactly two WebViews - one for layout test and one for DevTools? If so, your fix is Ok. In either case, I'd rather have the views created explicitly and see no generic createNewWindow call that allows reusing the devtools agent client.

[Eric Seidel (no email)]

This report was filed by snow leopard commit bots.  It's fine to do your fix too, but your fix only affects Chromium (which is unrelated to this report).

[Ilya Tikhonovsky]

(In reply to comment #8)
> This report was filed by snow leopard commit bots.  It's fine to do your fix too, but your fix only affects Chromium (which is unrelated to this report).

It is not clear for me why this bug got this comment.
----------
The commit-queue just saw inspector/timeline-event-dispatch.html flake while processing attachment 77158 [details] on bug 51429.
Bot: cr-jail-3  Port: Mac  Platform: Mac OS X 10.6.4
----------

As I understand this is a message from chromium bot on mac.
But chromium inspector's test enabled only for Windows.

Could you please throw an amount of light on this?

[Eric Seidel (no email)]

(In reply to comment #9)
> It is not clear for me why this bug got this comment.
> ----------
> The commit-queue just saw inspector/timeline-event-dispatch.html flake while processing attachment 77158 [details] [details] on bug 51429.
> Bot: cr-jail-3  Port: Mac  Platform: Mac OS X 10.6.4
> ----------
> 
> As I understand this is a message from chromium bot on mac.
> But chromium inspector's test enabled only for Windows.

Sorry for any confusion.  What that's saying is:

"cr-jail-3" is the bot name.  We run some of the commit-bots in the Chromium jail (a special section of a google datacenter.

The port is "Mac" which is the Apple Mac port.  I could make it say that more explicitly.  Which is different from Chromium Mac.

The Platform it was built on is 10.6.4, Snow Leopard.

[Ilya Tikhonovsky]

Comment on attachment 77627 [details]
[patch] initial version

wrong bug