Update the tracked URL of a frame if it changes during a navigation

chrome/browser/extensions/extension_webnavigation_api.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 // Implements the Chrome Extensions WebNavigation API.
 
 #include "chrome/browser/extensions/extension_webnavigation_api.h"
 
 #include "base/json/json_writer.h"
 #include "base/string_number_conversions.h"
@@ skipping 302 matching lines @@
       tab_contents(), frame_id, is_main_frame, validated_url);
 }
 
 void ExtensionWebNavigationTabObserver::DidCommitProvisionalLoadForFrame(
     int64 frame_id,
     bool is_main_frame,
     const GURL& url,
     PageTransition::Type transition_type) {
   if (!navigation_state_.CanSendEvents(frame_id))
     return;
+
+  bool is_reference_fragment_navigation =
+      IsReferenceFragmentNavigation(frame_id, url);
+
+  // Update the URL as it might have changed.
+  navigation_state_.TrackFrame(frame_id,
+                               url,
+                               is_main_frame,
+                               false,
+                               tab_contents());
+
   // On reference fragment navigations, only a new navigation state is
   // committed. We need to catch this case and generate a full sequence
   // of events.
-  if (IsReferenceFragmentNavigation(frame_id, url)) {
+  if (is_reference_fragment_navigation) {
     NavigatedReferenceFragment(frame_id, is_main_frame, url, transition_type);
     return;
   }
   DispatchOnCommitted(
       tab_contents(), frame_id, is_main_frame, url, transition_type);
 }
 
 void ExtensionWebNavigationTabObserver::DidFailProvisionalLoad(
     int64 frame_id,
     bool is_main_frame,
@@ skipping 80 matching lines @@
   replacements.ClearRef();
   return existing_url.ReplaceComponents(replacements) ==
       url.ReplaceComponents(replacements);
 }
 
 void ExtensionWebNavigationTabObserver::NavigatedReferenceFragment(
     int64 frame_id,
     bool is_main_frame,
     const GURL& url,
     PageTransition::Type transition_type) {
-  navigation_state_.TrackFrame(frame_id,
-                               url,
-                               is_main_frame,
-                               false,
-                               tab_contents());
-
   DispatchOnBeforeNavigate(tab_contents(),
                            frame_id,
                            is_main_frame,
                            url);
   DispatchOnCommitted(tab_contents(),
                       frame_id,
                       is_main_frame,
                       url,
                       transition_type);
   DispatchOnDOMContentLoaded(tab_contents(),
                              url,
                              is_main_frame,
                              frame_id);
   DispatchOnCompleted(tab_contents(),
                       url,
                       is_main_frame,
                       frame_id);
 }