Closed
Bug 691320
Opened 13 years ago
Closed 13 years ago
typo in GroupItems.newTab from panorama
Categories
(Firefox Graveyard :: Panorama, defect)
Firefox Graveyard
Panorama
Tracking
(Not tracked)
RESOLVED
FIXED
Firefox 10
People
(Reporter: tabmix.onemen, Assigned: raymondlee)
References
Details
(Whiteboard: [fixed-in-fx-team])
Attachments
(1 file, 1 obsolete file)
1.51 KB,
patch
|
Details | Diff | Splinter Review |
in the function GroupItems.newTab there is a typo: let targetGroupItem; // find first visible non-app tab in the tabbar. gBrowser.visibleTabs.some(function(tab) { if (!tab.pinned && tab != tabItem.tab) { if (tab._tabViewTabItem) { - if (!tab._tabViewTabItem.parent && !tab._tabViewTabItem.parent.hidden) { + if (tab._tabViewTabItem.parent && !tab._tabViewTabItem.parent.hidden) { // the first visible tab belongs to a group, add the new tabItem to // that group targetGroupItem = tab._tabViewTabItem.parent; } } return true; } return false; });
Assignee | ||
Comment 1•13 years ago
|
||
Hi reporter, which version did you find the typo? Looking at the mozilla central, everything looks right? http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabview/groupitems.js#2387
Assignee | ||
Comment 2•13 years ago
|
||
(In reply to Raymond Lee [:raymondlee] from comment #1) > Hi reporter, which version did you find the typo? > > Looking at the mozilla central, everything looks right? > http://mxr.mozilla.org/mozilla-central/source/browser/base/content/tabview/ > groupitems.js#2387 I see what you mean now. the if condition should use tab._tabViewTabItem.parent.
Assignee | ||
Comment 3•13 years ago
|
||
Comment 4•13 years ago
|
||
Comment on attachment 564447 [details] [diff] [review] v1 Review of attachment 564447 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! Can you please push it to try to make sure we don't rely on that wrong condition?
Attachment #564447 -
Flags: review?(ttaubert) → review+
Assignee | ||
Comment 5•13 years ago
|
||
Pushed to try and waiting for the results https://tbpl.mozilla.org/?tree=Try&rev=5663f9b06052
Assignee | ||
Updated•13 years ago
|
Keywords: checkin-needed
Comment 7•13 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/bce52912af92
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 10
Version: unspecified → Trunk
Comment 8•13 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/bce52912af92
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Product: Firefox → Firefox Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•