eWEEK news page is Speedreader-enabled but probably shouldn't be

[stephendonner]

Description

eWEEK news page is Speedreader-enabled but probably shouldn't be

Steps to Reproduce

1. new profile
2. launch Brave
3. load https://www.eweek.com/news/
4. click on the Speedreader icon in the URL bar
5. after it reloads, look at the resulting page

Actual result:

Speedreader enabled	Speedreader disabled

Expected result:

Not sure; the page doesn't appear to be a good candidate for Speedreader, though. It looks pretty close, if not identical, to non-Speedreader

Reproduces how often:

100%

Brave version (brave://version info)

Brave	1.32.4 Chromium: 94.0.4606.50 (Official Build) nightly (x86_64)
Revision	0c1ac2c4842a4746c27c937c1a0453f98da1a972-refs/branch-heads/4606@{#1049}
OS	macOS Version 11.6 (Build 20G165)

/cc @keur

[kkuehlz]

This should have been fixed by #17355 and #18151. I can repro on Linux for release but not 1.32.x. Does it show the icon for https://cnet.com/news also?

[kkuehlz]

was able to repro on linux too

[stephendonner]

This should have been fixed by #17355 and #18151. I can repro on Linux for release but not 1.32.x. Does it show the icon for https://cnet.com/news also?

Yup!

[ryanbr]

Just to clean up the blank space on eweek. Landed: easylist/easylist@1bf2a3c

[kkuehlz]

@stephendonner For the test plan just verify those two sites don't show the icon any longer and this all works as expected. Some things to note:

1. You will need to hard refresh on those sites that are being incorrectly shown as readable, since the browser cached that incorrect data. Once you do that it should never happen again.
2. I added some regression tests for the icon. Should help for future rebases.