From 02dd270ca6937a038ffd3b3a03bd44cce03f963a Mon Sep 17 00:00:00 2001 From: Pavel Beloborodov <73575789+boocmp@users.noreply.github.com> Date: Tue, 6 Feb 2024 17:48:26 +0700 Subject: [PATCH] Scripts removed to prevent a possible noise in the console. (#21900) * Removed scripts from test page to prevent the possible noise in the console. --- .../speedreader/speedreader_browsertest.cc | 8 ++++---- .../article}/guardian.html | 19 ------------------- .../{articles => speedreader/article}/rss.xml | 0 .../article}/rss.xml.mock-http-headers | 0 4 files changed, 4 insertions(+), 23 deletions(-) rename test/data/{articles => speedreader/article}/guardian.html (89%) rename test/data/{articles => speedreader/article}/rss.xml (100%) rename test/data/{articles => speedreader/article}/rss.xml.mock-http-headers (100%) diff --git a/browser/speedreader/speedreader_browsertest.cc b/browser/speedreader/speedreader_browsertest.cc index 195cdac4278..527c9778c34 100644 --- a/browser/speedreader/speedreader_browsertest.cc +++ b/browser/speedreader/speedreader_browsertest.cc @@ -66,12 +66,12 @@ const char kTestHost[] = "a.test"; const char kTestPageSimple[] = "/simple.html"; -const char kTestPageReadable[] = "/articles/guardian.html"; +const char kTestPageReadable[] = "/speedreader/article/guardian.html"; const char kTestEsPageReadable[] = "/speedreader/article/es.html"; const char kTestPageReadableOnUnreadablePath[] = "/speedreader/rewriter/pages/news_pages/abcnews.com/distilled.html"; const char kTestPageRedirect[] = "/articles/redirect_me.html"; -const char kTestXml[] = "/articles/rss.xml"; +const char kTestXml[] = "/speedreader/article/rss.xml"; const char kTestTtsSimple[] = "/speedreader/article/simple.html"; const char kTestTtsTags[] = "/speedreader/article/tags.html"; const char kTestTtsStructure[] = "/speedreader/article/structure.html"; @@ -301,7 +301,7 @@ IN_PROC_BROWSER_TEST_F(SpeedReaderBrowserTest, SmokeTest) { content::EXECUTE_SCRIPT_DEFAULT_OPTIONS, ISOLATED_WORLD_ID_BRAVE_INTERNAL) .ExtractInt(); - EXPECT_LT(106000, first_load_page_length); + EXPECT_LT(83000, first_load_page_length); ToggleSpeedreader(); EXPECT_TRUE(speedreader_service()->IsEnabledForAllSites()); @@ -359,7 +359,7 @@ IN_PROC_BROWSER_TEST_F(SpeedReaderBrowserTest, SmokeTest) { content::EXECUTE_SCRIPT_DEFAULT_OPTIONS, ISOLATED_WORLD_ID_BRAVE_INTERNAL) .ExtractInt(); - EXPECT_LT(106000, second_load_page_length) + EXPECT_LT(83000, second_load_page_length) << " First load length: " << first_load_page_length << " speedreaded length: " << speedreaded_length << " Second load length: " << second_load_page_length diff --git a/test/data/articles/guardian.html b/test/data/speedreader/article/guardian.html similarity index 89% rename from test/data/articles/guardian.html rename to test/data/speedreader/article/guardian.html index ce4b8cf519e..0e2c703e5b4 100644 --- a/test/data/articles/guardian.html +++ b/test/data/speedreader/article/guardian.html @@ -182,20 +182,6 @@ - - -
Skip to main content @@ -1618,14 +1604,9 @@ Subscribe - - - diff --git a/test/data/articles/rss.xml b/test/data/speedreader/article/rss.xml similarity index 100% rename from test/data/articles/rss.xml rename to test/data/speedreader/article/rss.xml diff --git a/test/data/articles/rss.xml.mock-http-headers b/test/data/speedreader/article/rss.xml.mock-http-headers similarity index 100% rename from test/data/articles/rss.xml.mock-http-headers rename to test/data/speedreader/article/rss.xml.mock-http-headers