Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

pinned tabs either missing or duplicated after Brave relaunch #10179

Closed
diracdeltas opened this issue Jul 27, 2017 · 28 comments
Closed

pinned tabs either missing or duplicated after Brave relaunch #10179

diracdeltas opened this issue Jul 27, 2017 · 28 comments
Labels
bug feature/pinned-tabs fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P2 Crashes. Loss of data. Severe memory leak. wontfix

Comments

@diracdeltas
Copy link
Member

diracdeltas commented Jul 27, 2017

for the last several releases, my pinned tabs have either disappeared or been duplicated after updating and relaunching. seems similar to #8543 except the tabs are gone/duplicated, not just out of order.

for instance, on updating to 0.18, the pinned tabs disappeared.

FWIW my pinned tabs are usually my work gmail + personal gmail + gcal.

OS X

STR:

  1. in your regular brave profile, pin 2 gmail tabs (logged into different accounts, such as yan@brave and yan@gmail) and 1 google calendar tab
  2. copy session-store-1 from brave to brave-development
  3. npm start
  4. the 2 gmail tabs are missing
  5. now close the browser and restart
  6. now the google calendar tab is duplicated
@NejcZdovc
Copy link
Contributor

Had a similar situation today. Brave crashed and when I relaunched it again pinned tabs were gone

@diracdeltas diracdeltas changed the title pinned tabs either missing or duplicated after Brave update pinned tabs either missing or duplicated after Brave relaunch Jul 27, 2017
@diracdeltas
Copy link
Member Author

i copied session-store-1 from 0.18.x (after adding back some pinned tabs) to brave-development, and the pinned tabs were gone when i started brave in dev mode

@darkdh
Copy link
Member

darkdh commented Jul 27, 2017

Are we seeing the same thing for pinned tab missing?
#10122

@diracdeltas
Copy link
Member Author

diracdeltas commented Jul 27, 2017

can confirm i also see the error in #10122 when tabs are missing

@diracdeltas
Copy link
Member Author

diracdeltas commented Jul 27, 2017

i just relaunched the dev profile where pinned tabs were missing and now the pinned tabs are triplicated (2 tabs -> 6 tabs). no console errors.

@srirambv
Copy link
Collaborator

@diracdeltas does quitting the browser and restart restores the original amount of pinned tabs? #10053 could be related to this as well

@diracdeltas
Copy link
Member Author

@srirambv quitting and restarting seems to always duplicate the pinned tabs in 0.18.14.

@melvinstubbe
Copy link

I had a similar issue with my two pinned tabs disappearing after an update. Tried to pin them again but they were gone after a browser restart. The pinned tabs re-appeared after a cleanup of the browser history, but now the second of the two pinned tabs (twitter in my case) is pinned twice. Unpinning one of those results in having one regular twitter tab open (as expected), but the other pinned twitter tab disappearing. After a browser restart both pinned twitter tabs are back. Cleaning the history again does not change this.

@petemill
Copy link
Member

This sounds different to #10510 or #9635 as those bugs were not reporting duplicating tabs, but only removing them. However, it could be that it appeared like the tabs were being duplicated, since the bug did not actually remove the pinned tabs, but gave the appearance of it.

Users may have been re-pinning similar tabs, and then when launching a new window, find that the previously pinned tabs are restored, as well as the re-pinned tabs.

@melvinstubbe
Copy link

melvinstubbe commented Aug 18, 2017

You're right, I just unpinned one of my twitter tabs, both pinned tabs disappeared and one regular twitter tab showed up. Then relaunched the browser and now I have no more duplicate pinned twitter tabs. Apparently I have been trying to re-pin my tabs when they started disappearing.

(this was before today's update, after updating everything is still working fine)

@NejcZdovc
Copy link
Contributor

@brave/qa can you please retest this one with the latest master?

@bsclifton
Copy link
Member

I'll mark this as closed (should have been fixed with #10531) and we can retest with 0.21.x. As-is, I'm not 100% sure there are enough steps here to verify (I personally had not seen the issue- we may need better steps)

@diracdeltas
Copy link
Member Author

i did test this with #10531 and it was still an issue

@diracdeltas diracdeltas reopened this Aug 29, 2017
@bsclifton
Copy link
Member

@diracdeltas awesome, thanks for catching this and reopening 😄 Do you have more specific steps I can retry? I may have missed something

@diracdeltas
Copy link
Member Author

updated STR; just tested on master

@alexwykoff alexwykoff added the priority/P2 Crashes. Loss of data. Severe memory leak. label Sep 12, 2017
@bsclifton
Copy link
Member

I had this happen somewhat recently- I did a fresh install of 0.18.23 and ran into this. Another symptom in my case: bookmarks from the bookmarks toolbar were ordered incorrectly. I did an uninstall / reinstall and that fixed the issue for me.

I wonder if this is related to session being upgraded/downgraded when testing releases further out (0.21, 0.20, etc) and then going back to release

@diracdeltas
Copy link
Member Author

I wonder if this is related to session being upgraded/downgraded when testing releases further out (0.21, 0.20, etc) and then going back to release

i've only stayed on the latest stable release with my release profile

@iamhenry
Copy link

iamhenry commented Oct 18, 2017

I'm still having the issue explained above. I'm using Brave 0.19.54 (Mac OSX)

I have gmail and gcal pinned but when I relaunch the browser those disappear and I have to create new tabs and pin them again.

@petemill petemill self-assigned this Oct 24, 2017
@bbondy bbondy modified the milestones: 0.21.x (Developer Channel), 0.20.x (Beta Channel), Backlog Oct 25, 2017
@hugobuddel
Copy link
Contributor

Perhaps this is the same issue as I just raised, #11694? Not sure though, e.g. that requires 2 restarts or 1 restart + 1 new window to remove the pins.

About profiles: For me it is happening on two different machines with fairly normal profiles I think. One linux machine where brave is installed and updated with apt-get, the first version already being 0.19 IIRC. One windows machine where a 0.19 version was explicitly downloaded when 0.18 was still the stable one. Never downloaded 0.20 or 0.21.

@bbondy bbondy modified the milestones: Triage Backlog, Prioritized Backlog Nov 2, 2017
@modusops
Copy link

modusops commented Jan 6, 2018

I have a similar issue. Initially the pinned Google Calendar favicon wouldn't load. After I did a refresh, the Google Calendar favicon appeared, but only after a minor delay. Tried out other sites, with similar results.

Brave: 0.19.123
rev: f45cb3a
Muon: 4.5.31
libchromiumcontent: 63.0.3239.108
V8: 6.3.292.48
Node.js: 7.9.0
Update Channel: Release
OS Platform: macOS
OS Release: 17.3.0
OS Architecture: x64

brave gcal pinned tab

@moneytrain81
Copy link

This just started happening to me. Brave was using 3.6GB of RAM with 5 tabs open (normal websites, no videos or anything else going on) and 19 processes running, so I shut it down.

Reopened and my 3 pinned tabs had all moved around and one had been replicated as a new pinned tab. I tried to close/unpin the new pinned tab and it did close, for a second, then it repinned itself and also opened another new tab for the site. This tab is not a pinned version. So I now have 3 tabs for the same site. Trying to unpin from here just results in more tabs being pinned again.

Received an update during all of this, applied it and the problem is still here.

I really want to like and use Brave, but things like the memory and processor usage as well as this pin stuff is making it hard to justify.

@bsclifton
Copy link
Member

bsclifton commented Feb 27, 2018

@petemill you've made some great changes with regards to pinned tabs recently and I believe you may have fixed this (in 0.22.x). Can you confirm?

@bsclifton
Copy link
Member

Specifically, I think this may be fixed with #13271

@bsclifton
Copy link
Member

@JFrankfurt you reported this issue with #12995. If you're still using Brave, have you experienced this issue?

@hugobuddel
Copy link
Contributor

I still have two duplicated pins that I can't remove. I unpinned one of the two, and suddenly three other pins disappeared as well (this never happened before). I tried to make a movie because pins kept appearing and disappearing for about 20 seconds before stabilizing but I was too late. Then I opened two new windows and the duplicate pin is back, but the other three are gone.

Using Brave 0.22.669 on Linux. Perhaps I should recreate my profile once more? But this is quite a hassle.

@hugobuddel
Copy link
Contributor

A thought: maybe there is an interplay between multiple windows. As in, I had like 5 or 6 windows open when the weird dance of disappearing pins happened. Maybe some kind of race condition can occur when multiple windows are open. Just speculating here; the pinned-tab behavior can be very inconsistent and unpredictable.

@hugobuddel
Copy link
Contributor

hugobuddel commented May 1, 2018

Here a screencap where I

  1. Unpin the duplicate pinned tabs (from pinboard),
  2. Open a new window with ctrl-n, one of the pinned tabs is back,
  3. Close the newly pinned tab, the pin reappears,
  4. Goto 3

output_optimized2

(Also note that the menu bar appears in the new window, #5086 probably. And also #11109 for the color.)

(First uploaded to #13271, wanted to upload it here.)

@petemill
Copy link
Member

petemill commented May 1, 2018

Duplication can happen as a result of the tactic of storing the originally pinned URL but comparing the pinned tabs to the current URL. I have this same problem with Google Inbox, since it always redirects when loaded, yet somehow in my session-store-1 has the un-redirected URL pinned. The code should compare like-for-like for reliability.

@bsclifton bsclifton modified the milestones: Backlog (Prioritized), Triage Backlog Sep 18, 2018
@bsclifton bsclifton removed this from the Triage Backlog milestone Oct 1, 2018
@bsclifton bsclifton added wontfix fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. labels Oct 1, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug feature/pinned-tabs fixed-with-brave-core This issue will automatically resolved with the replacement of Muon with Brave Core. priority/P2 Crashes. Loss of data. Severe memory leak. wontfix
Projects
None yet
Development

No branches or pull requests