chrome.tabs.onUpdated doesn't called with 'loading' twice.

chrome/browser/extensions/api/tabs/tabs_event_router.h

 // Copyright 2013 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.
 
 #ifndef CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
 #define CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_
 
 #include <map>
 #include <string>
 
@@ skipping 130 matching lines @@
   //  - distinguish between tab creation and tab insertion
   //  - not send tab-detached after tab-removed
   //  - reduce the "noise" of TabChangedAt() when sending events to extensions
   //  - remember last muted and audible states to know if there was a change
   class TabEntry {
    public:
     // Create a TabEntry associated with, and tracking state changes to,
     // |contents|.
     explicit TabEntry(content::WebContents* contents);
 
-    // Indicate via a list of key/value pairs if a tab is loading based on its
-    // WebContents. Whether the state has changed or not is used to determine
-    // if events needs to be sent to extensions during processing of
-    // TabChangedAt(). If this method indicates that a tab should "hold" a
-    // state-change to "loading", the DidNavigate() method should eventually
-    // send a similar message to undo it. If false, the returned key/value
-    // pairs list is empty.
-    scoped_ptr<base::DictionaryValue> UpdateLoadState();
-
-    // Indicate via a list of key/value pairs that a tab load has resulted in a
-    // navigation and the destination url is available for inspection. The list
-    // is empty if no updates should be sent.
-    scoped_ptr<base::DictionaryValue> DidNavigate();
+    // Indicate via a list of key/value pairs that being changed when called.
+    scoped_ptr<base::DictionaryValue> Update();
 
     // Update the audible and muted states and return whether they were changed
     bool SetAudible(bool new_val);
     bool SetMuted(bool new_val);
 
     content::WebContents* web_contents() { return contents_; }
 
    private:
     content::WebContents* contents_;
 
-    // Whether we are waiting to fire the 'complete' status change. This will
-    // occur the first time the WebContents stops loading after the
-    // NAV_ENTRY_COMMITTED was fired. The tab may go back into and out of the
-    // loading state subsequently, but we will ignore those changes.
-    bool complete_waiting_on_load_;
-
     // Previous audible and muted states
     bool was_audible_;
     bool was_muted_;
 
     GURL url_;
+
+    // Indicate the last 'status' value.
+    std::string status_;
   };
 
   // Gets the TabEntry for the given |contents|. Returns linked_ptr<TabEntry>
   // if found, NULL if not.
   linked_ptr<TabEntry> GetTabEntry(content::WebContents* contents);
 
   using TabEntryMap = std::map<int, linked_ptr<TabEntry>>;
   TabEntryMap tab_entries_;
 
   // The main profile that owns this event router.
   Profile* profile_;
 
   ScopedObserver<favicon::FaviconDriver, TabsEventRouter>
       favicon_scoped_observer_;
 
   DISALLOW_COPY_AND_ASSIGN(TabsEventRouter);
 };
 
 }  // namespace extensions
 
 #endif  // CHROME_BROWSER_EXTENSIONS_API_TABS_TABS_EVENT_ROUTER_H_