Some independent cleanups for bug 1423013

RESOLVED FIXED in Firefox 66

Status

()

enhancement
P3
normal
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: hiro, Assigned: hiro)

Tracking

Trunk
mozilla66
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox66 fixed)

Details

Attachments

(8 attachments)

Assignee

Description

5 months ago
No description provided.
Assignee

Comment 4

5 months ago
It's been unused since bug 1337325.

Depends on D15338
Assignee

Comment 5

5 months ago
There is no reason to use nsAtoms*, it's error prone, i.e. the function
can take arbitrary nsAtoms*.

Depends on D15339
Attachment #9033266 - Attachment description: Bug 1516368 - Make ChangeOrigin::Other more descriptive. r=botond → Bug 1516368 - Unite ChangeOrigin::eRestore and eOther as ChangeOrigin::eMainThread. r=botond

Comment 9

5 months ago
Pushed by hikezoe@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/3790e0649bd7
Rename aViewportSize to aViewportOrContentSize. r=botond
https://hg.mozilla.org/integration/autoland/rev/40b774aa7492
Use mOuter->PresShell() directly. r=botond
https://hg.mozilla.org/integration/autoland/rev/463079c73a31
Align boolean member variables in ScrollFrameHelper. r=botond
https://hg.mozilla.org/integration/autoland/rev/bb9b55fa1161
Drop nsIDOMWindowUtils.isResolutionSet. r=botond
https://hg.mozilla.org/integration/autoland/rev/f11eea40912a
Change SetResolutionAndScaleTo to take an enum instead of an nsAtoms*. r=botond
https://hg.mozilla.org/integration/autoland/rev/a17c856e81b5
Unite ChangeOrigin::eRestore and eOther as ChangeOrigin::eMainThread. r=botond
https://hg.mozilla.org/integration/autoland/rev/ff2870cad3e6
Align boolean members in nsIPresShell. r=botond
https://hg.mozilla.org/integration/autoland/rev/592d38d2892f
De-virtualize nsIPresShell::IsResolutionUpdated and SetResolutionUpdated. r=botond
You need to log in before you can comment on or make changes to this bug.