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

chrome/browser/banners/app_banner_manager.h

 // Copyright 2015 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_BANNERS_APP_BANNER_MANAGER_H_
 #define CHROME_BROWSER_BANNERS_APP_BANNER_MANAGER_H_
 
 #include <memory>
 #include <vector>
 
@@ skipping 143 matching lines @@
 
   // Records that a banner was shown. The |event_name| corresponds to the RAPPOR
   // metric being recorded.
   void RecordDidShowBanner(const std::string& event_name);
 
   // Logs an error message corresponding to |code| to the devtools console
   // attached to |web_contents|. Does nothing if IsDebugMode() returns false.
   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.
   virtual void Stop();
 
   // Sends a message to the renderer that the page has met the requirements to
   // show a banner. The page can respond to cancel the banner (and possibly
   // display it later), or otherwise allow it to be shown.
   void SendBannerPromptRequest();
 
@@ skipping 103 matching lines @@
   // scoped to the WebContents which they are observing. This allows us to use
   // weak pointers for callbacks.
   base::WeakPtrFactory<AppBannerManager> weak_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(AppBannerManager);
 };
 
 }  // namespace banners
 
 #endif  // CHROME_BROWSER_BANNERS_APP_BANNER_MANAGER_H_