Update broken call to getContentWindow() from larch merge

RESOLVED FIXED in Firefox 44

Status

Firefox OS
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: timdream, Assigned: timdream)

Tracking

unspecified
FxOS-S10 (30Oct)
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(firefox44 fixed)

Details

Attachments

(1 attachment)

https://hg.mozilla.org/mozilla-central/annotate/f029ccdee154bdc2b49a1e0fcd5b0fa0397aa97b/b2g/chrome/content/shell.js#l108 

bug 1160923 removed getContentWindow() but bug 1204965 merged some code containing that. It should be updated to either

shell.contentBrowser.contentWindow

or 

SystemAppProxy._frame.contentWindow

Apparently the branch does not contain any test to realize this feature is broken? I can help with a drive-by patch tomorrow or someone can beat me to it.

PS I really don't like these untested global functions being checked into shell.js. Not sure who can make the call here and if the said person really cares.
Comment hidden (obsolete)
Oops, accidentally submitted the form, was going to say:

Thanks for filing, sorry about merging this in a broken state - it should definitely have tests. I can't get to this immediately but may be able to take it down the road if you don't get to it.
Created attachment 8677848 [details] [diff] [review]
bug1217357.patch
Flags: needinfo?(kevingrandon)
Attachment #8677848 - Flags: review?(kevingrandon)
Comment on attachment 8677848 [details] [diff] [review]
bug1217357.patch

Review of attachment 8677848 [details] [diff] [review]:
-----------------------------------------------------------------

Not too familiar with this stuff, but this particular getter seems to be all over shell.js so it makes sense to me. Thank you very much for fixing it.
Attachment #8677848 - Flags: review?(kevingrandon) → review+
Keywords: checkin-needed

Comment 6

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3b5812d1023c
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-firefox44: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → FxOS-S10 (30Oct)
You need to log in before you can comment on or make changes to this bug.