Closed
Bug 1337325
Opened 7 years ago
Closed 7 years ago
[geckoview] Remove moar JavaPanZoomController (JPZ) leftovers from mobile/android
Categories
(GeckoView :: General, defect)
Tracking
(firefox54 fixed)
RESOLVED
FIXED
mozilla54
Tracking | Status | |
---|---|---|
firefox54 | --- | fixed |
People
(Reporter: JanH, Assigned: JanH)
References
Details
Attachments
(1 file)
+++ This bug was initially created as a clone of Bug #1312743 +++ (In reply to Kartikaya Gupta (email:kats@mozilla.com) from comment #0) > +++ This bug was initially created as a clone of Bug #1311030 +++ > > It's like this never ends! Indeed, I think there's some more code that can be removed, namely everything related to Tab.prototype.saveSessionZoom() and restoredSessionZoom(), which nobody seems to call anymore. This also means we can remove _getGeckoZoom(), _restoreZoom and _updateZoomFromHistoryEvent(). While this means that the history listener will become temporarily unused, I *don't* want to remove it yet, since I think it might be useful for fixing at least the session store part of bug 1337264.
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8836473 [details] Bug 1337325 - Remove unused zoom and scroll code from browser.js https://reviewboard.mozilla.org/r/111896/#review113160 Yay code deletion!
Attachment #8836473 -
Flags: review?(bugmail) → review+
Assignee | ||
Comment 3•7 years ago
|
||
Since this is touching the tab object history listener as well...
Depends on: 1337940
Pushed by mozilla@buttercookie.de: https://hg.mozilla.org/integration/autoland/rev/28c5827d707c Remove unused zoom and scroll code from browser.js r=kats
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/28c5827d707c
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox54:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 54
Comment 6•7 years ago
|
||
backed this out for tc-M(31) perma failure, like https://treeherder.mozilla.org /logviewer.html#?job_id=79217617&repo=autoland&lineNumber=1937 https://hg.mozilla.org/mozilla-central/rev/c53368e7d1a3 https://hg.mozilla.org/mozilla-central/rev/9a9db410f207 https://treeherder.mozilla.org/#/jobs?repo=autoland&revision=&fromchange=a437b95a0de44c025192f5d2a98c13bc0feb6a90&filter-searchStr=Android%204.3%20API15%2B%20debug%20Mochitests%20executed%20by%20TaskCluster%20test-android-4.3-arm7-api-15%2Fdebug-mochitest-31%20tc-M(31)&tochange=69dc3f79527d7cef2d3036756ca43bbf1a2f31ff Sorry we are not sure which patch causes this failure, so I backed your patches out. Please help to check it, thanks.
Status: RESOLVED → REOPENED
Flags: needinfo?(jh+bugzilla)
Resolution: FIXED → ---
Backout by ihsiao@mozilla.com: https://hg.mozilla.org/mozilla-central/rev/1ee728de804e Backed out changeset 28c5827d707c for tc-M(31) perma failure
Assignee | ||
Comment 8•7 years ago
|
||
Given that the test "failure" is the test timing out, I don't think this one is to blame, although since the history change listener in browser.js is also touched by bug 1337940 and this is nothing urgent, it can stay backed out until bug 1337940 sticks.
Flags: needinfo?(jh+bugzilla)
Pushed by mozilla@buttercookie.de: https://hg.mozilla.org/integration/autoland/rev/3f7919d07156 Remove unused zoom and scroll code from browser.js r=kats
Comment 10•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3f7919d07156
Status: REOPENED → RESOLVED
Closed: 7 years ago → 7 years ago
Resolution: --- → FIXED
Updated•5 years ago
|
Product: Firefox for Android → GeckoView
Updated•5 years ago
|
Target Milestone: Firefox 54 → mozilla54
You need to log in
before you can comment on or make changes to this bug.
Description
•