| commit | 4e6b80dc770f3afc6574c6509876a3616616c2ea | |
| author | dglazkov <dglazkov@chromium.org> | |
| Mon, 14 Sep 2015 04:12:27 +0000 (13 21:12 -0700) | ||
| committer | Commit bot <commit-bot@chromium.org> | |
| Mon, 14 Sep 2015 04:12:56 +0000 (14 04:12 +0000) | ||
| tree | c407e255b9fd75efa6cd41ad8072679153aa69ae | treesnapshot (tar.gz zip) |
| parent | a2fb178e8ec77bee156bc3c035222018fb1bc928 | commitdiff |
Change ChromeRenderViewObserver to use DidFinishLoad.
Currently, it uses DidStopLoading, but it really needs to use
DidFinishLoad instead.
The DidStopLoading is invoked in more cases (JS URLs, frame being detached, etc.), and we need none of those for the purposes of ChromeRenderViewObserver.
BUG=521166
R=thakis
Review URL: https://codereview.chromium.org/1315233004
Cr-Commit-Position: refs/heads/master@{#348551}
Currently, it uses DidStopLoading, but it really needs to use
DidFinishLoad instead.
The DidStopLoading is invoked in more cases (JS URLs, frame being detached, etc.), and we need none of those for the purposes of ChromeRenderViewObserver.
BUG=521166
R=thakis
Review URL: https://codereview.chromium.org/1315233004
Cr-Commit-Position: refs/heads/master@{#348551}
| chrome/renderer/chrome_render_view_observer.cc | diffblobblamehistory | |
| chrome/renderer/chrome_render_view_observer.h | diffblobblamehistory |