Closed Bug 570783 Opened 10 years ago Closed 10 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

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: 10 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.