Make apz_test_utils.js work with GeckoView

RESOLVED FIXED in Firefox 63

Status

()

defect
P3
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: snorp, Assigned: snorp)

Tracking

(Blocks 1 bug)

unspecified
mozilla63
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 attachment)

Right now it uses Services.wm.getMostRecentWindow('navigator:browser') in a couple places, but GeckoView uses navigator:geckoview.

Comment 2

Last year
mozreview-review
Comment on attachment 8984606 [details]
Bug 1467918 - Fix APZ utils to work with GeckoView

https://reviewboard.mozilla.org/r/250484/#review256764

Seems a bit hacky, but since it's just test code, it's fine.

::: commit-message-028db:1
(Diff revision 1)
> +Bug 1467918 - Fix APZ utils to work with GeckoView r=botond

"APZ utils" -> "APZ test utils"
Attachment #8984606 - Flags: review?(botond) → review+
Is this good to land?
Assignee: nobody → snorp
Flags: needinfo?(snorp)
Priority: -- → P3
Whiteboard: [gfx-noted]
It is, modulo the commit message. I was on PTO last week but will try to land this today.
Flags: needinfo?(snorp)

Comment 5

Last year
Pushed by jwillcox@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/b7230ab18e62
Fix APZ test utils to work with GeckoView r=botond

Comment 6

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/b7230ab18e62
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.