[SafeBrowsing] Reset malware indicator on page nav start.

chrome/browser/safe_browsing/client_side_detection_host.h

Index: chrome/browser/safe_browsing/client_side_detection_host.h
diff --git a/chrome/browser/safe_browsing/client_side_detection_host.h b/chrome/browser/safe_browsing/client_side_detection_host.h
index c24554f6b19faae3c1f40a23c1590c7f013dd62b..6557fdca004d7b6eb444d593566631d10b4b98fc 100644
--- a/chrome/browser/safe_browsing/client_side_detection_host.h
+++ b/chrome/browser/safe_browsing/client_side_detection_host.h
@@ -14,6 +14,7 @@
 #include "chrome/browser/safe_browsing/browser_feature_extractor.h"
 #include "chrome/browser/safe_browsing/database_manager.h"
 #include "chrome/browser/safe_browsing/ui_manager.h"
+#include "content/public/browser/navigation_controller.h"

[Scott Hess - ex-Googler, 2013/12/12 21:28:44]

I can see IWYU arguments for this, but do those apply when the class you're
subclassing should have provided everything necessary to declare your override? 
[I honestly don't know, I could see it going both ways.]

[Greg Billock, 2013/12/12 22:14:44]

I'm flexible. Agreed that the super-type has to include it.

 #include "content/public/browser/notification_registrar.h"
 #include "content/public/browser/resource_request_details.h"
 #include "content/public/browser/web_contents_observer.h"
@@ -47,6 +48,12 @@ class ClientSideDetectionHost : public content::WebContentsObserver,
       const content::LoadCommittedDetails& details,
       const content::FrameNavigateParams& params) OVERRIDE;
 
+  // From content::WebContentsObserver. Causes a reset to the
+  // malware_or_phishing_match_ indicator.
+  virtual void DidStartNavigationToPendingEntry(
+      const GURL& url,
+      content::NavigationController::ReloadType reload_type) OVERRIDE;
+
   // Called when the SafeBrowsingService found a hit with one of the
   // SafeBrowsing lists.  This method is called on the UI thread.
   virtual void OnSafeBrowsingHit(