Closed Bug 859962 Opened 11 years ago Closed 11 years ago

BrowserApp assumes LayerView.getPanZoomController never returns null

Categories

(Firefox for Android Graveyard :: Toolbar, defect)

All
Android
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 23

People

(Reporter: kats, Assigned: kats)

References

Details

Attachments

(1 file)

I think there might be cases where getPanZoomController returns null. I'm running into some of them with my work on the APZC unification when I switch to my new implementation.
Attachment #735338 - Flags: review?(chrislord.net)
Comment on attachment 735338 [details] [diff] [review]
Guard against null

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

LGTM.
Attachment #735338 - Flags: review?(chrislord.net) → review+
https://hg.mozilla.org/mozilla-central/rev/beaf93963e65
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: