commit | d51db23f06cd4da8a968e96cf455f14e42788d38 | [log] [tgz] |
---|---|---|
author | Eugene But <eugenebut@google.com> | Wed Dec 06 23:28:14 2017 |
committer | Eugene But <eugenebut@chromium.org> | Wed Dec 06 23:28:14 2017 |
tree | b176c73dedf4b71120f252de66488fbbf7613788 | |
parent | ea7f1beaa44c73a187665dc17050856706589452 [diff] |
Call WebStateObserver::FaviconUrlUpdated for same-document navigations. WebStateImpl will cache favicon urls in OnFaviconUrlUpdated and will call WebStateObserver::FaviconUrlUpdated in OnNavigationFinished for same document navigations. Also removed favicon urls caching in WebFaviconDriver introduced in crrev.com/c/695761. This CL is not exactly a revert of crrev.com/c/695761, because deprecated NavigationItemCommitted is replaced with DidFinishNavigation. NavigationItemCommitted is not called for push/replace state same-document navigaiton, which may cause other bugs for favicons caching. TBR=eugenebut@google.com (cherry picked from commit 19825f69ff27c5e99fb0a230ce730d8cc0561830) Bug: 789581 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I42922ab4d7812380787e350713640efe4d599680 Reviewed-on: https://chromium-review.googlesource.com/797072 Reviewed-by: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by: Gauthier Ambard <gambard@chromium.org> Commit-Queue: Eugene But <eugenebut@chromium.org> Cr-Original-Commit-Position: refs/heads/master@{#521876} Reviewed-on: https://chromium-review.googlesource.com/812468 Reviewed-by: Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/branch-heads/3282@{#66} Cr-Branched-From: 5fdc0fab22ce7efd32532ee989b223fa12f8171e-refs/heads/master@{#520840}
Chromium is an open-source browser project that aims to build a safer, faster, and more stable way for all users to experience the web.
The project's web site is https://www.chromium.org.
Documentation in the source is rooted in docs/README.md.
Learn how to Get Around the Chromium Source Code Directory Structure .