636149 - Home Page -> Use Current Pages; Does not respect Tab groups

Status: VERIFIED FIXED

(Firefox Graveyard :: Panorama, defect, P3)

Description

[bugzilla]

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b12pre) Gecko/20110222 Firefox/4.0b12pre
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b12pre) Gecko/20110222 Firefox/4.0b12pre

Only the current pages in the current tab group should be set as home, instead every page from all groups are, and they are all brought into the active group when clicking on the home button.

Perhaps there should be a url scheme to allow tab groups to be bookmarks or set as home pages etc.

Reproducible: Always

Steps to Reproduce:
1. separate some pages into groups
2. options > home page > use current pages
3. all pages are added

Comment 1

[juan becerra [:juanb]]

I can see the problem as described here. If the user's intention is to use the current pages as viewed in the current state in panorama, then this doesn't work as expected, because all the tabs come up in the same group. Nomination for clarification.

Comment 2

[Kevin Hanes]

Technically we're doing exactly what the button says even though some of your pages are not in your current view. I agree that we could do better here in conjunction with Panorama.

Unless it's blocking, I think this is a post-Fx4 issue.

Comment 3

[Mike Beltzner [:beltzner, not reading bugmail]]

Yeah, not a blocker.

Comment 4

[Derek Williams]

I have an issue that may be related.  When I choose Use Current Pages, Firefox adds a spam ISP site and a blank Tab, neither of which is a current page when I choose Use Current Pages.

Below is what was saved:
http://mail.derekwilliams.net/zimbra/#7|http://community.vsl.co.at/forums/|http://www.facebook.com/|http://commonsenseatheism.com/?p=15167|http://by148w.bay148.mail.live.com/default.aspx?wa=wsignin1.0|https://mail.google.com/mail/?hl=en&shva=1#inbox|http://extranet.inpho.com.au/livr/dw/dw_status.php?action=line_status|http://timeanddate.com/worldclock/|http://translate.google.com/#auto|en||http://dictionary.reference.com/

I am attaching an image of what opens.  I contacted their Abuse team, to no avail.

Comment 5

[Derek Williams]

Attachment: screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected

screenshot showing unsolicited spam tab from spam ISP when Use Current Pages selected

Comment 6

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch v1

Comment 7

[:Gavin Sharp [email: [e-mail]]]

Comment on attachment 529276 [details] [diff] [review]
patch v1

While you're at it, you could update the XXX comment to be a "FIXME: Bug 244192".

You could also replace all of this code with:

var tabs = win.gBrowser.visibleTabs;
function getTabURI(t) t.linkedBrowser.currentURI.spec;
var newVal = tabs.map(getTabURI).join("|");

Comment 8

[Tim Taubert [:ttaubert] (inactive)]

Attachment: patch for checkin

(In reply to comment #7)
> While you're at it, you could update the XXX comment to be a "FIXME: Bug
> 244192".

Done.

> You could also replace all of this code with:
> 
> var tabs = win.gBrowser.visibleTabs;
> function getTabURI(t) t.linkedBrowser.currentURI.spec;
> var newVal = tabs.map(getTabURI).join("|");

That's much better :) Done.

Comment 9

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/projects/cedar/rev/ec5747dbbfed

Comment 10

[Boris Zbarsky [:bzbarsky]]

http://hg.mozilla.org/mozilla-central/rev/ec5747dbbfed

Comment 11

[George Carstoiu]

 Mozilla/5.0 (X11; Linux i686; rv:6.0a1) Gecko/20110519 Firefox/6.0a1

Verified issue using steps from Comment 0 on Ubuntu 10.10, WinXP, Mac OS X 10.6, Win 7. Bug is no longer present.