210512 2020-04-14 13:16:18 -0700 Storage events are fired for the same tab 2020-11-23 10:02:26 -0800 1 1 1 Unclassified WebKit New Bugs Safari 13 Unspecified Unspecified RESOLVED DUPLICATE 211503 https://bugs.webkit.org/show_bug.cgi?id=211503 InRadar P2 Normal --- 1 merihakar webkit-unassigned bdrasin beidson cdumez jimmy rkit.ru sihui_liu webkit-bug-importer oldest_to_newest 1641518 0 396456 merihakar 2020-04-14 13:16:18 -0700 Created attachment 396456 testcase As per specification storage events should be fired for the tabs other than the tab that's changing the storage area, however it's also fired on the same tab in Safari. Note that the issue happens when the testcase file is served via a server, doesn't happen when you open it directly in the browser. 1641624 1 webkit-bug-importer 2020-04-14 17:47:07 -0700 <rdar://problem/61799443> 1709846 2 jimmy 2020-11-22 13:26:19 -0800 I can confirm this with Safari 13.1 (14609.1.20.111.8) on macOS 10.14.6 (18G4032), as well as with Safari on iOS (iPhone) 13.7. Work around: Only execute OnStorage logic if document is not focused: if (!document.hasFocus()) { // Execute OnStorage logic - tab does not have focus } Notice though that hasFocus() will return false in the active tab if Developer Tools is open and has focus. 1709847 3 jimmy 2020-11-22 13:35:32 -0800 Work around suggestion only makes sense if OnStorage events are triggered by user interaction, of course. Also, it seems Safari Developer Tools does not steal focus from the document - Chrome Developer Tools does. 1709849 4 cdumez 2020-11-22 16:17:38 -0800 This rings a bell and I thought I had fixed this. Are you able to try in Safari Technology Preview? 1709850 5 cdumez 2020-11-22 16:21:34 -0800 This looks a lot like 1709851 6 cdumez 2020-11-22 16:22:08 -0800 (In reply to Chris Dumez from comment #5) > This looks a lot like This looks a lot like https://bugs.webkit.org/show_bug.cgi?id=211503 which got fixed. 1709896 7 jimmy 2020-11-23 08:21:11 -0800 (In reply to Chris Dumez from comment #6) > (In reply to Chris Dumez from comment #5) > > This looks a lot like > > This looks a lot like https://bugs.webkit.org/show_bug.cgi?id=211503 which > got fixed. Chris Dumez: It sure does sound like the same thing. Safari Version 14.0.1 (14610.2.11.51.9) works as expected. I can no longer reproduce the problem with localStorage. I think its safe to close this bug report, although I'm not the one who initially reported the bug - I just confirmed it. - Thank you Jimmy 1709912 8 cdumez 2020-11-23 10:02:26 -0800 (In reply to Jimmy Thomsen from comment #7) > (In reply to Chris Dumez from comment #6) > > (In reply to Chris Dumez from comment #5) > > > This looks a lot like > > > > This looks a lot like https://bugs.webkit.org/show_bug.cgi?id=211503 which > > got fixed. > > Chris Dumez: It sure does sound like the same thing. Safari Version 14.0.1 > (14610.2.11.51.9) works as expected. I can no longer reproduce the problem > with localStorage. I think its safe to close this bug report, although I'm > not the one who initially reported the bug - I just confirmed it. > > - Thank you > > Jimmy Awesome. Thank you for confirming! *** This bug has been marked as a duplicate of bug 211503 *** 396456 2020-04-14 13:16:18 -0700 2020-04-14 13:16:18 -0700 testcase storage_bug.html text/html 622 merihakar PCFkb2N0eXBlIGh0bWw+CjxodG1sPgogICAgPGhlYWQ+CiAgICAgICAgPG1ldGEgY2hhcnNldD0i dXRmLTgiPgogICAgICAgIDxzY3JpcHQ+CiAgICAgICAgICAgIHdpbmRvdy5hZGRFdmVudExpc3Rl bmVyKCJsb2FkIiwgKCkgPT4gewogICAgICAgICAgICAgICAgd2luZG93LmFkZEV2ZW50TGlzdGVu ZXIoInN0b3JhZ2UiLCBldiA9PiB7CiAgICAgICAgICAgICAgICAgICAgY29uc29sZS5sb2coZXYu c3RvcmFnZUFyZWEuZ2V0SXRlbShldi5rZXkpKTsKICAgICAgICAgICAgICAgICAgICBldi5zdG9y YWdlQXJlYS5yZW1vdmVJdGVtKGV2LmtleSkKICAgICAgICAgICAgICAgIH0pOwoKICAgICAgICAg ICAgICAgIGRvY3VtZW50LnF1ZXJ5U2VsZWN0b3IoImJ1dHRvbiIpLmFkZEV2ZW50TGlzdGVuZXIo ImNsaWNrIiwgKCkgPT4gewogICAgICAgICAgICAgICAgICAgIGxvY2FsU3RvcmFnZS5zZXRJdGVt KCJrZXkiLCAidmFsdWUiKTsKICAgICAgICAgICAgICAgIH0pOwogICAgICAgICAgICB9KTsKICAg ICAgICA8L3NjcmlwdD4KICAgIDwvaGVhZD4KICAgIDxib2R5PgogICAgICAgIDxidXR0b24gdHlw ZT0iYnV0dG9uIj5TZXQgc3RvcmFnZTwvYnV0dG9uPgogICAgPC9ib2R5Pgo8L2h0bWw+Cg==