1112644 - make DevTools timeline tests pass with e10s

Status: RESOLVED FIXED

(DevTools :: Performance Tools (Profiler/Timeline), defect)

Description

[Tom Tromey :tromey]

The timeline tests in docshell/test/browser don't work with e10s yet.

Comment 1

[Tom Tromey :tromey]

Attachment: make timeline tests work on e10s

This changes the timeline marker tests to work with e10s.

I also took the opportunity to share code which was formerly copied
into the various test files.  I put the browser process code into
head.js and shared content code into a new "frame-head.js".

Comment 2

[Tom Tromey :tromey]

https://tbpl.mozilla.org/?tree=Try&rev=930446c6a0e9

Comment 4

[Tom Tromey :tromey]

Comment on attachment 8537922 [details] [diff] [review]
make timeline tests work on e10s

The basic idea of this patch is to split the tests into a small
bit on the browser side, that loads the real tests into the content
process.  "ok", "is", and "info" are defined on the content side
to just forward messages to the browser process.

Comment 5

[Paul Rouget [:paul]]

Is there any value in running these tests in a remote tab?

What about running them in a chrome mochitest instead? Them we would not have to deal with E10S.

Comment 6

[Tom Tromey :tromey]

(In reply to Paul Rouget [:paul] (slow to respond. Ping me on IRC) from comment #5)
> Is there any value in running these tests in a remote tab?
> 
> What about running them in a chrome mochitest instead? Them we would not
> have to deal with E10S.

I didn't realize this could work for these tests.
I'll investigate.

Comment 7

[Paul Rouget [:paul]]

Comment on attachment 8537922 [details] [diff] [review]
make timeline tests work on e10s

I don't want to block this. And I believe using a chrome mochitest might not work well as we want to use an actual tab.

Comment 8

[Tom Tromey :tromey]

Thanks, Paul.

Comment 9

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/integration/fx-team/rev/d51c8141591c

Comment 10

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d51c8141591c

Comment 11

[David Lawrence [:dkl]]

Sorry for the spam. Moving bugs to Firefox :: Developer Tools: Performance Tools (Profiler/Timeline).

dkl