Multiple tabs in Pop-up windows (created through Panorama)

VERIFIED FIXED in Firefox 10

Status

Firefox Graveyard
Panorama
VERIFIED FIXED
6 years ago
a year ago

People

(Reporter: AndreiD[QA], Assigned: raymondlee)

Tracking

({regression})

Trunk
Firefox 10
regression

Details

(Whiteboard: [fixed-in-fx-team])

Attachments

(1 attachment, 1 obsolete attachment)

(Reporter)

Description

6 years ago
Build ID: 
Win: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:9.0a1) Gecko/20110920 Firefox/9.0a1
Linux: Mozilla/5.0 (X11; Linux x86_64; rv:9.0a1) Gecko/20110920 Firefox/9.0a1
Mac: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:9.0a1) Gecko/20110921 Firefox/9.0a1

STR:
1. Launch Firefox on a clean profile and go to http://www.popuptest.com/popuptest1.html - if the case, set permissions to allow pop-ups.
2. Resize one of the open pop-up windows, making it bigger
3. Using the ctrl+shift+e/ cmd + shift +e enter panorama. 
4. While in panorama create another empty group and click over it to create a tab

Actual result:
4. In step 4, a new tab is created; in the newly created tab, some browser functions are invalid (create a new tab, type a weblink, et cetera)

Expected result:
4. In step 4, a new group AND/OR tab should not be created since the window is not a normal browser one (pop-up)


*Notes:
This bug is reproducible on Linux and Mac as well, so setting OS to all.
I tested this on the Firefox channels on Windows and the bug is reproducible also on:
Beta: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:7.0) Gecko/20100101 Firefox/7.0
Aurora: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:8.0a2) Gecko/20110918 Firefox/8.0a2
You can find a screen cast video of the issue at the provided link:
https://skydrive.live.com/?cid=bc75b2c6fced798f&id=BC75B2C6FCED798F%21344#cid=BC75B2C6FCED798F&id=BC75B2C6FCED798F%21345
(Reporter)

Comment 1

6 years ago
Pushlog: http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=422bbd8245a7&tochange=1a89509e25e4

Last good nightly: 2011-03-31
First bad nightly: 2011-04-01
Component: General → Panorama
Keywords: regression
QA Contact: general → panorama
(Assignee)

Updated

6 years ago
Assignee: nobody → raymond
Status: NEW → ASSIGNED
(Assignee)

Comment 2

6 years ago
In popup windows, many things are disabled and hidden e.g. sidebar, menu bar, etc.  IMO, it makes more sense to not allowing user to enter Panorama since there would only be one tab in a popup window.

Thoughts?
Status: ASSIGNED → NEW
(In reply to Raymond Lee [:raymondlee] from comment #2)
> In popup windows, many things are disabled and hidden e.g. sidebar, menu
> bar, etc.  IMO, it makes more sense to not allowing user to enter Panorama
> since there would only be one tab in a popup window.

Totally agreed, we shouldn't allow Panorama to be loaded in a popup window.
(Assignee)

Comment 4

6 years ago
Created attachment 564134 [details] [diff] [review]
v1

Trivial patch
Attachment #564134 - Flags: review?(ttaubert)
Comment on attachment 564134 [details] [diff] [review]
v1

Review of attachment 564134 [details] [diff] [review]:
-----------------------------------------------------------------

Great, thanks!

::: browser/base/content/browser-tabview.js
@@ +174,5 @@
>    // If the frame already exists, calls the callback immediately. 
>    _initFrame: function TabView__initFrame(callback) {
>      let hasCallback = typeof callback == "function";
>  
> +    // prevent frame to be initialized for popup window

Nit: "for popup windows".
Attachment #564134 - Flags: review?(ttaubert) → review+
(Assignee)

Comment 6

6 years ago
Created attachment 564598 [details] [diff] [review]
Patch for checkin

(In reply to Tim Taubert [:ttaubert] from comment #5)
> ::: browser/base/content/browser-tabview.js
> @@ +174,5 @@
> >    // If the frame already exists, calls the callback immediately. 
> >    _initFrame: function TabView__initFrame(callback) {
> >      let hasCallback = typeof callback == "function";
> >  
> > +    // prevent frame to be initialized for popup window
> 
> Nit: "for popup windows".

Fixed

Pushed to try and waiting for the results
https://tbpl.mozilla.org/?tree=Try&rev=142447f72b41
Attachment #564134 - Attachment is obsolete: true
(Assignee)

Comment 7

6 years ago
Passed Try!
Status: NEW → ASSIGNED
Keywords: checkin-needed
https://hg.mozilla.org/integration/fx-team/rev/aa487dd537e3
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
https://hg.mozilla.org/mozilla-central/rev/aa487dd537e3
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 10
(Reporter)

Comment 10

6 years ago
The issue is not reproducible anymore on the latest Nightly, due to the fix, on:
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:10.0a1) Gecko/20111009 Firefox/10.0a1 (Win)
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.6; rv:10.0a1) Gecko/20111009 Firefox/10.0a1 (Mac)
Mozilla/5.0 (X11; Linux i686; rv:10.0a1) Gecko/20111009 Firefox/10.0a1 (Linux)

A user cannot enter Panorama using the shortcut keys (ctrl+shift+e -win and linux) and cmd+shift+e on Mac in a pop-up windows (following the steps in the Description)
Status: RESOLVED → VERIFIED
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.