1648270 - Get rid of content parent Opener field

Status: RESOLVED FIXED

(Core :: DOM: Content Processes, task)

Description

[Kris Maglione [:kmag]]

It's unused, and adds a fair amount of complexity to already complex code

Comment 1

[Kris Maglione [:kmag]]

Attachment: Bug 1648270: Get rid of unused ContentParent 'opener' field. r=nika

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/3468660179b1
Get rid of unused ContentParent 'opener' field. r=nika

Comment 3

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a777bb34bb93
Backed out changeset 3468660179b1 for bustages at ContentParent.h. CLOSED TREE

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/840d3dc313f1
Get rid of unused ContentParent 'opener' field. r=nika

Comment 5

[Atila Butkovits]

Backed out for causing build bustages on ProtocolFuzzer.cpp.

Backout link: https://hg.mozilla.org/integration/autoland/rev/5796bdae0df7d6111f9fc7b99550a4d0bd4b6a86

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=840d3dc313f1d9fda0cf396e3c7e27634ead28a4&selectedTaskRun=ZOERKT9cSlahF6Dxq_A3hg.0&searchStr=Linux%2Cx64%2Cdebug%2Cbuild-linux64-fuzzing%2Fdebug%2C%28Bf%29

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307590822&repo=autoland&lineNumber=46780

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/bce7350f629b
Get rid of unused ContentParent 'opener' field. r=nika

Comment 7

[Kris Maglione [:kmag]]

Attachment: Bug 1648270: Follow-up: Fix botched rebase. r=bustage

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/13ea2ac60dc9
Follow-up: Fix botched rebase.

Comment 9

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/bce7350f629b
https://hg.mozilla.org/mozilla-central/rev/13ea2ac60dc9

Comment 10

[Alexandru Ionescu (needinfo me) [:alexandrui]]

(In reply to Atila Butkovits from comment #5)

Backed out for causing build bustages on ProtocolFuzzer.cpp.

Backout link: https://hg.mozilla.org/integration/autoland/rev/5796bdae0df7d6111f9fc7b99550a4d0bd4b6a86

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=840d3dc313f1d9fda0cf396e3c7e27634ead28a4&selectedTaskRun=ZOERKT9cSlahF6Dxq_A3hg.0&searchStr=Linux%2Cx64%2Cdebug%2Cbuild-linux64-fuzzing%2Fdebug%2C%28Bf%29

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307590822&repo=autoland&lineNumber=46780

== Change summary for alert #26342 (as of Fri, 26 Jun 2020 08:39:28 GMT) ==

Improvements:

5% build times osx-shippable opt nightly taskcluster-c5d.4xlarge 2,856.44 -> 2,715.68

For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=26342