Closed Bug 606691 Opened 14 years ago Closed 14 years ago

combined location bar buttons should not appear in popups without toolbar items

Categories

(Firefox :: General, defect)

x86
Windows 7
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: alice0775, Assigned: fryn)

References

Details

Attachments

(1 file)

Build Identifier: http://hg.mozilla.org/mozilla-central/rev/092596c1faef Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b8pre) Gecko/20101022 Firefox/4.0b8pre ID:20101023101209 STR: Load attachment 464766 [details], open the popup, Middle-click the reload button. A non-visible tab generated. The following error shows in error console. Error: newBrowser is undefined Source file: chrome://browser/content/tabbrowser.xml Line: 754 var updatePageReport = false; if (!oldBrowser || (oldBrowser.pageReport && !newBrowser.pageReport) || >> (!oldBrowser.pageReport && newBrowser.pageReport)) updatePageReport = true; Error: content is null Source file: chrome://browser/content/browser.js Line: 5313 function pageShowEventHandlers(event) { // Filter out events that are not about the document load we are interested in >> if (event.originalTarget == content.document) { charsetLoadListener(event); XULBrowserWindow.asyncUpdateUI(); } }
The reload button shouldn't be visible here.
Blocks: 544816
Assignee: nobody → fryn
Status: NEW → ASSIGNED
this removes the integrated buttons from those types of popups, as per dao's comment.
Attachment #486963 - Flags: review?(dao)
Attachment #486963 - Flags: review?(dao) → review+
Summary: Middle-click the reload button shouldn't add tabs to popups → combined location bar buttons should not appear in popups without toolbar items
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: