1459774 - Make cpstartup Talos test load the target over http:// rather than chrome://

Status: RESOLVED FIXED

(Testing :: Talos, enhancement)

Description

[Tim Guan-tin Chien [:timdream] (please needinfo)]

See

https://bugzilla.mozilla.org/show_bug.cgi?id=1458375#c12

I am pretty sure this is unrelated to the empty scrollbar binding, but I will try to make sure of that before check-in this.

Comment 1

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Attachment: Bug 1459774 - Make cpstartup Talos test load the target over http:// rather than chrome://.

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

Comment 2

[Tim Guan-tin Chien [:timdream] (please needinfo)]

Comment on attachment 8973881 [details]
Bug 1459774 - Make cpstartup Talos test load the target over http:// rather than chrome://.

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

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Comment on attachment 8973881 [details]
Bug 1459774 - Make cpstartup Talos test load the target over http:// rather than chrome://.

https://reviewboard.mozilla.org/r/242248/#review248098

doing a drive by review, thanks for submitting this!

Comment 4

[Tim Guan-tin Chien [:timdream] (please needinfo)]

This change by itself does not change cpstartup numbers.

https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=2e7bac585cfa&newProject=try&newRevision=020474f30922e22cbc87f72bf33b5d6003963eb6&framework=1&filter=cpstartup

Applying the empty binding removal still slow things down, though.

https://treeherder.mozilla.org/perf.html#/compare?originalProject=try&originalRevision=020474f30922&newProject=try&newRevision=dbc4d9adee78&framework=1&filter=cpstartup

I guess we can land this first regardless.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/346cf7a79bf8
Make cpstartup Talos test load the target over http:// rather than chrome://. r=jmaher

Comment 6

[Arthur Iakab [arthur_iakab]]

https://hg.mozilla.org/mozilla-central/rev/346cf7a79bf8