1260359 - [e10s] Detached devtools window title is not updated when navigating

Status: VERIFIED FIXED

(DevTools :: General, defect, P2)

Description

[Julian Descottes [:jdescottes]]

STRs: 
- open https://developer.mozilla.org/en-US/
- open devtools as a detached window
- (here devtools window title should be "Inspector - Mozilla Developer Network")
- navigate to https://bugzilla.mozilla.org/

Expected:
- detached devtools window title should be "Inspector - Bugzilla Main Page"

Actual:
- detached devtools window title is still "Inspector - Mozilla Developer Network"

Comment 1

[Julian Descottes [:jdescottes]]

Actually related to e10s. 

The name getter in target.js is accessing this._tab.linkedBrowser.contentDocument which may not work when e10s is enabled.

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: MozReview Request: Bug 1260359 - fix devtools detached toolbox title update in e10s;r=jryans

The devtools host window updates its title when its target navigates.
This was done using a contentDocument CPOW, and thus failing on e10s.

The url and title are now updated in TabTarget.form upon tab navigation.
Updated existing test to cover the case of pages with a title.

Review commit: https://reviewboard.mozilla.org/r/42989/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/42989/

Comment 3

[Julian Descottes [:jdescottes]]

Try : https://treeherder.mozilla.org/#/jobs?repo=try&revision=f44380684cc5

Comment 4

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8735859 [details]
MozReview Request: Bug 1260359 - fix devtools detached toolbox title update in e10s;r=jryans

https://reviewboard.mozilla.org/r/42989/#review39633

Looks good, thanks for working on it!

::: devtools/client/framework/target.js:476
(Diff revision 1)
> +      this._form.url = aPacket.url;
> +      this._form.title = aPacket.title;

It's a little strange to mutate the form here, since previously I thought of the form as data we received as a complete protocol packet...  but in the end this is probably okay.  We can always change if it becomes confusing.

Comment 5

[Julian Descottes [:jdescottes]]

Comment on attachment 8735859 [details]
MozReview Request: Bug 1260359 - fix devtools detached toolbox title update in e10s;r=jryans

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/42989/diff/1-2/

Comment 6

[Julian Descottes [:jdescottes]]

Comment on attachment 8735859 [details]
MozReview Request: Bug 1260359 - fix devtools detached toolbox title update in e10s;r=jryans

Thanks for the review! I had another look at the fix based on your comment. I think we should avoid updating form here, as you said. 

It seems weird for me to have outdated information in this object, but it is definitely not the role of target.js to update this object.

Instead here I read the url & title from the form when receiving the payload, save them as properties on TabTarget directly, and update them when navigating.

Let me know if you prefer to go with this one.

Try: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e6b152fb6072

Comment 7

[J. Ryan Stinnett [:jryans] (Use needinfo, replies may be slow)]

Comment on attachment 8735859 [details]
MozReview Request: Bug 1260359 - fix devtools detached toolbox title update in e10s;r=jryans

https://reviewboard.mozilla.org/r/42989/#review39837

Cool, let's go with this one then.  Thanks!

Comment 8

[Pulsebot]

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

Comment 9

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/34fabc54b7a4

Comment 10

[arni2033]

Affected:   Win7_64, Nightly 48, 32bit, ID 20160320030409
Fixed on:   Win7_64, Nightly 48, 32bit, ID 20160401030216