| Index: components/favicon/content/content_favicon_driver_unittest.cc
|
| diff --git a/components/favicon/content/content_favicon_driver_unittest.cc b/components/favicon/content/content_favicon_driver_unittest.cc
|
| index 85435be32e3bfafc730dadbf693d25f602422f40..4a4e7523dccc8b50468d8259d27f2b9fca992535 100644
|
| --- a/components/favicon/content/content_favicon_driver_unittest.cc
|
| +++ b/components/favicon/content/content_favicon_driver_unittest.cc
|
| @@ -106,11 +106,11 @@ TEST_F(ContentFaviconDriverTest, UnableToDownloadFavicon) {
|
| favicon_service()->Shutdown();
|
| }
|
|
|
| -// Test that ContentFaviconDriver ignores updated favicon URLs if there is no
|
| -// last committed entry. This occurs when script is injected in about:blank.
|
| -// See crbug.com/520759 for more details
|
| -TEST_F(ContentFaviconDriverTest, FaviconUpdateNoLastCommittedEntry) {
|
| - ASSERT_EQ(nullptr, web_contents()->GetController().GetLastCommittedEntry());
|
| +// Test that ContentFaviconDriver handles updated favicon URLs for the initial
|
| +// navigation. This occurs when script is injected in about:blank.
|
| +// See crbug.com/520759 for more details.
|
| +TEST_F(ContentFaviconDriverTest, FaviconUpdateForInitialNavigation) {
|
| + ASSERT_TRUE(web_contents()->GetController().IsInitialNavigation());
|
|
|
| std::vector<content::FaviconURL> favicon_urls;
|
| favicon_urls.push_back(content::FaviconURL(
|
| @@ -122,7 +122,7 @@ TEST_F(ContentFaviconDriverTest, FaviconUpdateNoLastCommittedEntry) {
|
| ->DidUpdateFaviconURL(favicon_urls);
|
|
|
| // Test that ContentFaviconDriver ignored the favicon url update.
|
| - EXPECT_TRUE(driver->favicon_urls().empty());
|
| + EXPECT_FALSE(driver->favicon_urls().empty());
|
| }
|
|
|
| } // namespace
|
|
|