Closed Bug 570783 Opened 14 years ago Closed 14 years ago

progress listeners' OnRefreshAttempted being called twice instead of once for progress listeners and once for tab progress listeners

Categories

(SeaMonkey :: Tabbed Browser, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: morac, Assigned: misak.bugzilla)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

http://mxr.mozilla.org/comm-central/source/suite/browser/tabbrowser.xml#613

OnRefreshAttempted is currently calling the progress listener's onRefreshAttempted callback function twice instead of one call to the progress listeners and another call to the tab progress listeners.

Line 631 needs to be changed from:

this.mTabBrowser.mProgressListeners.forEach(

to:

this.mTabBrowser.mTabsProgressListeners.forEach(
Blocks: SMtabAPI
Attached patch patchSplinter Review
One liner patch.
Assignee: nobody → misak.bugzilla
Status: NEW → ASSIGNED
Attachment #452246 - Flags: superreview?(neil)
Attachment #452246 - Flags: review?(neil)
Attachment #452246 - Flags: superreview?(neil)
Attachment #452246 - Flags: superreview+
Attachment #452246 - Flags: review?(neil)
Attachment #452246 - Flags: review+
Pushed: http://hg.mozilla.org/comm-central/rev/c20397a441e2
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: