AdguardBrowserExtension icon indicating copy to clipboard operation
AdguardBrowserExtension copied to clipboard

Remove usage of deprecated navigator api

Open scrutinizer11 opened this issue 2 years ago • 17 comments

Issue Details

  • AdGuard version:

    • AdGuard Extension v4.1.xx
  • Browser and version:

    • Chromium 105.0.5195.52 (Official Build)
  • Operating system and version: Mac OSX 10.7, OSX 10.9, macOS 10.14

  • Filters you use in AdGuard:

    • AdGuard Annoyances filter
    • Adblock Warning Removal List
    • Web Annoyances Ultralist
    • I don't care about cookies
    • EasyList Cookie List
    • AdGuard URL Tracking filter
    • AdGuard Tracking Protection filter
    • AdGuard Base filter
    • EasyList
    • NoCoin Filter List
    • AdGuard Russian filter
    • EasyList Italy
    • EasyList Lithuania
    • RU AdList: Counters
  • Stealth Mode settings: On

    • Block Trackers
    • Remove tracking parameters
    • Hide your search query
    • Send websites signals not to track you
    • Hide referrer from third parties
    • Remove X-Client-Data header
  • Other browser extensions:

    • Stylus
    • DNS Checker
    • File Converter - By Online-Convert.com
    • iCloud Dashboard
    • Grammarly
    • Mate Translate
    • Wayback Machine
    • Dark Reader
    • Youtube Enhancer
    • Wikiwand

Additional Information

Sometimes, but often enough to get a noticeable effect, Adguard Extension crashes when I launch Chromium. Chromium sends a notification about that telling me to reload it. Very often the setup is multiple open tabs with resource-intensive content such as several tabs with media sites (YouTube, Netflix etc.).

scrutinizer11 avatar Aug 10 '22 04:08 scrutinizer11

@scrutinizer11 Hi, I wasn't able to reproduce the issue with the same extensions in chrome 104. I suspect some of these extensions might affect the work of AdGuard. Could you try disabling some of them and monitor the issue? Also a screenshot of the notification would be helpful.

Birbber avatar Aug 12 '22 11:08 Birbber

@scrutinizer11 any news?

zzebrum avatar Aug 26 '22 15:08 zzebrum

No news. Still crashes. It's nothing special to show other than the notification Adguard extension has crashed. Click this balloon to reload it. That's it. It can happen several times in a row during a short interval.

scrutinizer11 avatar Aug 27 '22 00:08 scrutinizer11

One tab was playing YouTube content, another one was an online forum. The extension crashed more than once and the active tab (the forum) got rid of its contents, so the tab became completely blank. Facebook is susceptible to this misbehaviour too. The extension's management pane displayed the error

Uncaught (in promise) Error: Could not establish connection. Receiving end does not exist.

The error pointed to this code:

Adguard-error.log

scrutinizer11 avatar Aug 27 '22 04:08 scrutinizer11

@scrutinizer11 I didn't find any errors in the code you sent. The error you are referring can't be the cause of crashing. Can you share your extension settings and the websites open in the tabs?

maximtop avatar Aug 29 '22 04:08 maximtop

I didn't say the code contained the error. I said that the quoted error pointed to the code snippet that showed up on the extension's management pane.

I enclose a JSON file with the settings, as you requested. As to the tab configuration, I have it varying frequently so it's difficult to recall the exact pattern at which the tabs turn up at the time of the bug cropping up because the crashes occur out of the blue. I did notice though, that for the most part, they happen when one or several tabs with resource-hogging content are opened. Usually, it's Facebook and Youtube among others. Open any number of tabs above 2 including any demanding site. 20220829_080824_adg_ext_settings_4.1.1.txt

scrutinizer11 avatar Aug 29 '22 05:08 scrutinizer11

Cannot load extension with file or directory name metadata. Filenames starting with "" are reserved for use by the system.

ag-warning-30·08·2022 17·05.txt

scrutinizer11 avatar Aug 30 '22 14:08 scrutinizer11

Now it crashes like crazy on every Chromium launch, every tab opening (even this page). A new error. Screen Shot 2022-09-04 at 15 59 31

scrutinizer11 avatar Sep 04 '22 12:09 scrutinizer11

Can you describe the steps to reproduce the error in the last comment?

maximtop avatar Sep 05 '22 09:09 maximtop

Just launch the browser with several tabs or open them after the launch. Some of those have to be opened on YouTube, Facebook or a similar site. It's a good chance it may then kick off the trigger setting a crash cycle in motion. In some cases, even opening such sites as GitHub leads to a crash as well. Better yet, use a machine from 2012 with any Mac operating system.

scrutinizer11 avatar Sep 05 '22 09:09 scrutinizer11

Are the settings the same as you've sent before?

maximtop avatar Sep 05 '22 09:09 maximtop

@scrutinizer11 I've asked QA to test on Mohave 10.14, but they could not reproduce crashes.

maximtop avatar Sep 05 '22 11:09 maximtop

I have discovered something accidentally but am not sure if this is related and what it means. I clicked on Inspect Views background.js:1 just upon reloading the extension after it crashed: it opened the referred "view" and under Issues I saw this:

Screen Shot 2022-09-14 at 02 41 00

Is it in any way helpful?

scrutinizer11 avatar Sep 13 '22 23:09 scrutinizer11

@scrutinizer11 we will remove usage of described api, but it is unlikely to be the cause of the crashes

maximtop avatar Sep 14 '22 05:09 maximtop

ADG-error_2022-09-25_02-41-46-660

scrutinizer11 avatar Sep 24 '22 23:09 scrutinizer11

ADG-error_2022-09-25_02-41-46-660

Have you reloaded the extension? This error may occur if the extension background page was reloaded while the content script was sending messages.

maximtop avatar Sep 26 '22 08:09 maximtop

Yes, I reloaded it after ~~it crashed~~ the page in question cleared its contents. The thing crashes a hell of a lot, coinciding with the page blanking, at the beginning and the middle of the browsing session.

scrutinizer11 avatar Sep 26 '22 23:09 scrutinizer11

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Nov 26 '22 00:11 github-actions[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] avatar Jan 28 '23 11:01 github-actions[bot]