1633488 - Simplify the zoom setup even more.

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task, P3)

Comment 1

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1633488 - Stop listening for FullZoomChange in MobileViewportManager. r=kats,botond

When the full zoom changes, it affects the layout viewport, and thus
RefreshViewportSize(false) gets called via PresShell::ResizeReflow, so
it's not needed.

This is in preparation to firing FullZoomChange only in the browser
element.

Comment 2

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1633488 - Fire zoom change events directly in the parent process instead of roundabouting through the child process. r=nika

This ensures that chrome code can always react to the state the browsing
context the parent process observes properly. It's also simpler.

We fix one test that was relying on the event being fired at the
browser. For the same reason as the previous patch in this bug, we can
use resize events instead.

Depends on D72711

Comment 3

[Emilio Cobos Álvarez (:emilio)]

Attachment: Bug 1633488 - Remove other unused bits of ZoomChild.jsm. r=nika

These are unused after the move to BrowsingContext. Will try to figure
out what to do about the enlarge events and such.

Depends on D72712

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/897f08396a2f
Stop listening for FullZoomChange in MobileViewportManager. r=botond

Comment 5

[Oana Pop-Rus]

https://hg.mozilla.org/mozilla-central/rev/897f08396a2f

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c999ecad3be8
Fire zoom change events directly in the parent process instead of roundabouting through the child process. r=nika

Comment 7

[Dorel Luca [:dluca]]

Backed out 4 changesets (bug 1633938, bug 1578008, bug 1633488) for Devtool failures in browser/base/content/test/static/browser_parsable_css.js. CLOSED TREE

Log:
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=300120222&repo=autoland&lineNumber=1908

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&group_state=expanded&revision=e690c4cf884848b968c214c38f86727ff30bc03d

Backout:
https://hg.mozilla.org/integration/autoland/rev/d3a80376591e14e1880195b31f1b2baec87f6faa

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e837430046c8
Fire zoom change events directly in the parent process instead of roundabouting through the child process. r=nika

Comment 9

[Andreea Pavel [:apavel]]

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