Closed
Bug 583000
Opened 14 years ago
Closed 14 years ago
Chrome messed up in popup windows when closeWindowWithLastTab = false
Categories
(Firefox :: Tabbed Browser, defect)
Firefox
Tabbed Browser
Tracking
()
RESOLVED
FIXED
Firefox 4.0b5
People
(Reporter: c.ascheberg, Assigned: dao)
References
()
Details
(Keywords: regression)
Attachments
(3 files)
120 bytes,
text/html
|
Details | |
1.28 KB,
patch
|
Gavin
:
review+
Gavin
:
approval2.0+
|
Details | Diff | Splinter Review |
54.74 KB,
image/jpeg
|
Details |
User-Agent: Mozilla/5.0 (Windows; Windows NT 6.1; rv:2.0b3pre) Gecko/20100729 Minefield/4.0b3pre
Build Identifier: Mozilla/5.0 (Windows; Windows NT 6.1; rv:2.0b3pre) Gecko/20100729 Minefield/4.0b3pre
When opening a popup, the URL is not displayed in the urlbar
Reproducible: Always
Steps to Reproduce:
1. on einslive.de: click link "webradio" on top of page
2. switch to opened popup
Actual Results:
urlbar is empty
Expected Results:
urlbar should shown current URL
Reporter | ||
Comment 1•14 years ago
|
||
Reporter | ||
Updated•14 years ago
|
Version: unspecified → Trunk
Reporter | ||
Updated•14 years ago
|
Keywords: regression
Reporter | ||
Comment 2•14 years ago
|
||
this only seems happens when browser.tabs.closeWindowWithLastTab is set to false
Reporter | ||
Updated•14 years ago
|
blocking2.0: --- → ?
Comment 3•14 years ago
|
||
I confirm the bug, and it's surprising to me. Doesn't block because it only appears under non-standard configuration, but I would like to see it fixed regardless!
Assignee | ||
Comment 4•14 years ago
|
||
I'm getting these errors:
Warning: reference to undefined property this.tabContainer
Source File: chrome://browser/content/tabbrowser.xml
Line: 89
Error: this.browsers[i] is undefined
Source File: chrome://browser/content/tabbrowser.xml
Line: 253
Error: uncaught exception: [Exception... "Component returned failure code: 0x80070057 (NS_ERROR_ILLEGAL_VALUE) [nsIObserverService.addObserver]" nsresult: "0x80070057 (NS_ERROR_ILLEGAL_VALUE)" location: "JS frame :: chrome://browser/content/browser.js :: prepareForStartup :: line 5283" data: no]
Assignee | ||
Updated•14 years ago
|
Component: Location Bar → Tabbed Browser
OS: Windows 7 → All
QA Contact: location.bar → tabbed.browser
Hardware: x86 → All
Comment 5•14 years ago
|
||
look like that the problem is in gBrowser.tabs
changing tabs from field to property fix the porplem
- <field name="tabs" readonly="true">
- this.tabContainer.childNodes;
- </field>
+ <property name="tabs"
+ onget="return this.tabContainer.childNodes;"
+ readonly="true"/>
Assignee | ||
Comment 6•14 years ago
|
||
(In reply to comment #5)
> look like that the problem is in gBrowser.tabs
> changing tabs from field to property fix the porplem
>
> - <field name="tabs" readonly="true">
> - this.tabContainer.childNodes;
> - </field>
> + <property name="tabs"
> + onget="return this.tabContainer.childNodes;"
> + readonly="true"/>
This is more expensive, though. I don't really see what's wrong with the <field>. I also don't understand how the closeWindowWithLastTab pref affects this.
Comment 7•14 years ago
|
||
set browser.tabs.closeWindowWithLastTab=false and browser.tabs.closeButtons=1
tabbrowser-tabs constructor call adjustTabstrip
adjustTabstrip call _numPinnedTabs
then in _numPinnedTabs this.tabs is undefined
Assignee | ||
Comment 8•14 years ago
|
||
(In reply to comment #7)
> tabbrowser-tabs constructor call adjustTabstrip
> adjustTabstrip call _numPinnedTabs
This explains why closeWindowWithLastTab makes a difference. Thanks!
This leaves the question of why tabbrowser.tabContainer.childNodes fails when called from the tabContainer constructor.
Comment 9•14 years ago
|
||
not tabbrowser.tabContainer.childNodes fails tabbrowser.tabs fails.
tabContainer constructor calls tabbrowser.tabs that calls tabbrowser.tabContainer.childNodes.
this loop of tabContainer > tabbrowser > tabbrowser.tabContainer from tabContainer constructor don't look so good to me
the simple fux will be to move the call to adjustTabstrip from tabContainer constructor to tabbrowser constructor
Assignee | ||
Comment 10•14 years ago
|
||
(In reply to comment #9)
> not tabbrowser.tabContainer.childNodes fails tabbrowser.tabs fails.
That's essentially the same.
> tabContainer constructor calls tabbrowser.tabs that calls
> tabbrowser.tabContainer.childNodes.
>
> this loop of tabContainer > tabbrowser > tabbrowser.tabContainer from
> tabContainer constructor don't look so good to me
It's circular, sure, but this doesn't mean it should fail...
Comment 11•14 years ago
|
||
i guess that it fail in this circumstance since tabbrowser is not ready when tabContainer try to call it.
Assignee | ||
Comment 12•14 years ago
|
||
I imagine tabbrowser should automagically become ready when it's accessed, but maybe XBL bindings can't be constructed concurrently or something.
Assignee | ||
Comment 13•14 years ago
|
||
This is the right thing to do anyway. No point in trying to measure the tab width if there is only one tab.
Updated•14 years ago
|
Attachment #469035 -
Flags: review?(gavin.sharp)
Attachment #469035 -
Flags: review+
Attachment #469035 -
Flags: approval2.0+
Assignee | ||
Comment 14•14 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 4.0b5
Comment 16•14 years ago
|
||
http://www.pcurtis.com/popup.htm
on javascript popup window,
no Firefox button/tabbar.
title(bar) displayed.
intended ?
Assignee | ||
Comment 17•14 years ago
|
||
(In reply to comment #16)
> Created attachment 469405 [details]
> screenshot
>
> http://www.pcurtis.com/popup.htm
>
> on javascript popup window,
> no Firefox button/tabbar.
> title(bar) displayed.
>
> intended ?
Yes.
Summary: URL not displayed in popup window when closeWindowWithLastTab = false → Chrome messed up in popup windows when closeWindowWithLastTab = false
Comment 18•14 years ago
|
||
(In reply to comment #17)
OK, thanks.
Comment 20•14 years ago
|
||
Thanks for fixing this!
Mozilla/5.0 (Windows NT 6.1; rv:2.0b5pre) Gecko/20100826 Minefield/4.0b5pre Firefox/4.0b4pre ID:20100826040655
Updated•14 years ago
|
Whiteboard: [switch-to-tab]
Comment 22•14 years ago
|
||
Dennis, please don't remove my whiteboard tag. I'm using [switch-to-tab] to track all bugs which affect switch-to-tab.
Whiteboard: [switch-to-tab]
Comment 23•14 years ago
|
||
I removed it because it doesn't have anything to do with switch to tab. And since you cannot edit popup's switch to tab doesn't apply to them.
Assignee | ||
Updated•14 years ago
|
Whiteboard: [switch-to-tab]
Comment 24•14 years ago
|
||
(In reply to comment #23)
> I removed it because it doesn't have anything to do with switch to tab. And
> since you cannot edit popup's switch to tab doesn't apply to them.
This bug has an affect on switch-to-tab as per the duped bug in comment 21.
Whiteboard: [switch-to-tab]
Assignee | ||
Comment 25•14 years ago
|
||
It affected countless random things, as it broke the popup's whole chrome... Please don't add them all to the whiteboard.
Whiteboard: [switch-to-tab]
Comment 26•14 years ago
|
||
(In reply to comment #25)
> It affected countless random things, as it broke the popup's whole chrome...
> Please don't add them all to the whiteboard.
I don't track all areas...I track switch-to-tab. Having the whiteboard tag will help me track this. Please don't remove the tag again. It has no affect on your work with this bug.
Whiteboard: [switch-to-tab]
Assignee | ||
Comment 27•14 years ago
|
||
Please track bug 586054. Mark it fixed by this one or something.
This is no switch-to-tab bug. It's also mine. Please stop messing with it.
Whiteboard: [switch-to-tab]
Comment 28•14 years ago
|
||
(In reply to comment #27)
> Please track bug 586054. Mark it fixed by this one or something.
>
> This is no switch-to-tab bug. It's also mine. Please stop messing with it.
Please explain to me the harm having my whiteboard tag in this bug causes?
I am tracking bug 586054. However, there is no dependency tree so this bug will get lost without me using the whiteboard tag. It's really a necessity for me.
Comment 29•14 years ago
|
||
Can you guys sort this out outside of bugzilla - ideally with voices (if not faces) involved? This is a bugzilla slapfight right now, and you are all, each one of you, above it.
Assignee | ||
Comment 30•14 years ago
|
||
It's resolved. Bug 586054, blocked and fixed by this one, is the switch-to-tab bug to track. This avoids the confusion of tagging a non-switch-to-tab bug with [switch-to-tab].
People build and read bug lists in various ways and for various reasons. Stuff you put in the whiteboard can mislead them, since it isn't private. In this case it would have lead me to believe that I worked on a switch-to-tab bug when I didn't. If you don't expect others to understand and use your tags, you can avoid this by obscuring them.
You need to log in
before you can comment on or make changes to this bug.
Description
•