diff options
| author | jochen@chromium.org <jochen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-17 12:34:51 +0000 |
|---|---|---|
| committer | jochen@chromium.org <jochen@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-09-17 12:34:51 +0000 |
| commit | 1717f29fbd361acecd5bf38132d42ed2a00afe47 (patch) | |
| tree | 38f663e55b2708d9f43981150fc3af624a8a0aa9 /chrome/browser/extensions/api/web_navigation | |
| parent | c9fa8f31d868cce1592bca5c9989d4872ecfe6fb (diff) | |
| download | chromium_src-1717f29fbd361acecd5bf38132d42ed2a00afe47.zip chromium_src-1717f29fbd361acecd5bf38132d42ed2a00afe47.tar.gz chromium_src-1717f29fbd361acecd5bf38132d42ed2a00afe47.tar.bz2 | |
Don't send the oncompleted signal twice in the webnavigation API
We shouldn't preemptivly mark a navigation as completed if it's a
history or reference fragment navigation, as such a navigation might
happen in parallel to an ongoing navigation
BUG=279483
R=marja@chromium.org
Review URL: https://chromiumcodereview.appspot.com/24197002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@223596 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/extensions/api/web_navigation')
| -rw-r--r-- | chrome/browser/extensions/api/web_navigation/web_navigation_api.cc | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc index ed2e61c..cdf0554 100644 --- a/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc +++ b/chrome/browser/extensions/api/web_navigation/web_navigation_api.cc @@ -447,9 +447,6 @@ void WebNavigationTabObserver::DidCommitProvisionalLoadForFrame( navigation_state_.UpdateFrame(frame_id, url); navigation_state_.SetNavigationCommitted(frame_id); - if (is_reference_fragment_navigation || is_history_state_modification) - navigation_state_.SetNavigationCompleted(frame_id); - if (!navigation_state_.CanSendEvents(frame_id)) return; |