Webapps: Clear AppBannerManagerAndroid::native_app_data_ prior to native app check

chrome/browser/banners/app_banner_manager.h

Index: chrome/browser/banners/app_banner_manager.h
diff --git a/chrome/browser/banners/app_banner_manager.h b/chrome/browser/banners/app_banner_manager.h
index be43a915fbb21c84a08228aa77b0bf712d7976d4..ab7e49466d6a88a4a00c6fbff6628f851c18c7d6 100644
--- a/chrome/browser/banners/app_banner_manager.h
+++ b/chrome/browser/banners/app_banner_manager.h
@@ -161,6 +161,9 @@ class AppBannerManager : public content::WebContentsObserver,
   void ReportStatus(content::WebContents* web_contents,
                     InstallableStatusCode code);
 
+  // Resets the manager's state.

[dominickn, 2017/02/06 02:16:53]

The comment should be:

"Resets all fetched data for the current page".

Since we actually remember some of the manager's state for metrics purposes.

[pkotwicz, 2017/02/06 03:00:11]

Done. I have also renamed the method to ResetCurrentPageData()

+  virtual void ResetData();
+
   // Stops the banner pipeline, preventing any outstanding callbacks from
   // running and resetting the manager state. This method is virtual to allow
   // tests to intercept it and verify correct behaviour.