Use OneShotTimer in AddToHomescreenDataFetcher

chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc

Index: chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
diff --git a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
index a05a2037283710922947277e90a032e950490e79..5bea4a450c69518e3db4b046377120c4e8c7bbe0 100644
--- a/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
+++ b/chrome/browser/android/webapps/add_to_homescreen_data_fetcher.cc
@@ -81,7 +81,6 @@ AddToHomescreenDataFetcher::AddToHomescreenDataFetcher(
       weak_observer_(observer),
       shortcut_info_(GetShortcutUrl(web_contents->GetBrowserContext(),
                                     web_contents->GetLastCommittedURL())),
-      data_timeout_timer_(false, false),
       ideal_icon_size_in_px_(ideal_icon_size_in_px),
       minimum_icon_size_in_px_(minimum_icon_size_in_px),
       ideal_splash_image_size_in_px_(ideal_splash_image_size_in_px),
@@ -206,6 +205,8 @@ void AddToHomescreenDataFetcher::OnDidPerformInstallableCheck(
 
   is_installable_check_complete_ = true;
 
+  data_timeout_timer_.Stop();

[dominickn, 2017/03/23 22:37:35]

Move this to be the first thing in this method (otherwise we may return from
this method due to the web_contents being dead, then the timer fires, and we
have to do the same thing in OnDataTimedout. May as well stop early).

[F, 2017/03/24 02:27:11]

Done.

+
   bool webapk_compatible = false;
   if (check_webapk_compatibility_) {
     webapk_compatible = (data.error_code == NO_ERROR_DETECTED &&