Port |Bug 675037: fullzoom callback needs to handle null currentURI|

RESOLVED FIXED in seamonkey2.17

Status

RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: InvisibleSmiley, Assigned: InvisibleSmiley)

Tracking

Trunk
seamonkey2.17

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
Created attachment 687845 [details] [diff] [review]
patch [Checkin: Comment 3]

As mentioned on bug 817326, we should port bug 675037.

Note that the old code executed also if aBrowser was null. I think that's stupid (self._applyPrefToSetting() tries to get a browser by itself, but if it fails, it fails) so I replaced the check by one that does nothing if aBrowser is null.
Attachment #687845 - Flags: review?(philip.chee)

Comment 1

6 years ago
Comment on attachment 687845 [details] [diff] [review]
patch [Checkin: Comment 3]

> Note that the old code executed also if aBrowser was null. I think that's
> stupid (self._applyPrefToSetting() tries to get a browser by itself, but if it
> fails, it fails) so I replaced the check by one that does nothing if aBrowser
> is null.
So this gets rid of the error but I disagree with your changing the logic. I think we need to check with Neil about this.
Attachment #687845 - Flags: review?(philip.chee)
Attachment #687845 - Flags: review?(neil)
Attachment #687845 - Flags: review+

Comment 2

6 years ago
Comment on attachment 687845 [details] [diff] [review]
patch [Checkin: Comment 3]

Not sure why we were null-checking aBrowser there, it's never null. (Although Firefox's aBrowser can be null, it always uses a local browser that's not.)

r=me if you remove the aBrowser && check completely.
Attachment #687845 - Flags: review?(neil) → review+
(Assignee)

Comment 3

6 years ago
Comment on attachment 687845 [details] [diff] [review]
patch [Checkin: Comment 3]

http://hg.mozilla.org/comm-central/rev/1c212e41559a
with comment 2 addressed
Attachment #687845 - Attachment description: patch → patch [Checkin: Comment 3]
(Assignee)

Updated

6 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.17
You need to log in before you can comment on or make changes to this bug.