Syntax error in onTabReplaced() about tabfern HOT 8 CLOSED

Thanks for reporting! Would you please try branch issue304 @ 5f62e0a and confirm the issue is resolved? I don't have a good repro for onTabReplaced handy :) . Much appreciated!

from tabfern.

I'm now running 0.3.1.1 in parallel. I'll know if it works properly in a few hours.

Edit: One of the computers I'm using has only 2G of ram. It's not a problem with the old win7. And with ublock running in my browsers the webpages are much lighter. So the browser is still perfectly functionnal. It starts swapping when I reach 15-20 tabs (or less if they are big pages).

from tabfern.

The fix works. I don't have orphaned tabs left in TF list.
But there is still the second error. I've launched chrome with TF 0.3.0 and 0.3.1.1 and looked at the dev tools. Then I opened as many sites as needed to trigger a tabReplace. What I could see is that for every error in 0.3.0 I can see an error in 0.3.1.1

in 0.3.0

Error in event handler: ReferenceError: addedTabID is not defined
    at onTabReplaced (chrome-extension://oflhofndbpjhlfnkbclmfkchpnkghflg/win/main.js:37839:42)

in 0.3.1.1:

Error in event handler: ReferenceError: tab_val is not defined
    at onTabReplaced (chrome-extension://oflhofndbpjhlfnkbclmfkchpnkghflg/win/main.js:37844:75)

from tabfern.

Sorry --- missed that second item (now marked "Item 2") when I read your initial report! Fixed in force-pushed issue304, now 0.3.1.2. Please let me know if the issue is fixed at 98e8924. I tested the changes by manually invoking the onTabReplaced() function and it at least runs to completion :) . Thanks!

from tabfern.

I can confirm. Comparing side by side:
No more errors. No orphaned tabs in the list. Looks good. :)

from tabfern.

@Procyon-b Much appreciated! Will release shortly.

from tabfern.

0.3.1 submitted for Google review --- will tag when it is released.

from tabfern.

It's released. :)

from tabfern.