Closed Bug 458963 Opened 16 years ago Closed 16 years ago

persistTabAttribute isn't a no-op when the attribute has already been registered

Categories

(Firefox :: Session Restore, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3.1b2

People

(Reporter: zeniko, Assigned: zeniko)

Details

Attachments

(1 file)

      No description provided.
Attached patch fixSplinter Review
This two-line patch makes sure that the list of tracked attributes doesn't grow unnecessarily when extensions repeatedly call persistTabAttribute from an overlay (instead of from a component) or when two extensions want the same attribute to be tracked.
Assignee: nobody → zeniko
Attachment #342144 - Flags: review?(dietrich)
Attachment #342144 - Flags: review?(dietrich) → review+
Keywords: checkin-needed
Pushed to mozilla-central: <http://hg.mozilla.org/mozilla-central/rev/1b7319402823>
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 3.1b2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: