Skip installation process if WebAPK is already installed.

chrome/browser/android/webapps/add_to_homescreen_manager.h

 // Copyright 2016 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_ANDROID_WEBAPPS_ADD_TO_HOMESCREEN_MANAGER_H_
 #define CHROME_BROWSER_ANDROID_WEBAPPS_ADD_TO_HOMESCREEN_MANAGER_H_
 
 #include "base/android/jni_android.h"
 #include "base/android/scoped_java_ref.h"
 #include "base/macros.h"
@@ skipping 38 matching lines @@
 
   // Shows alert to prompt user for name of home screen shortcut.
   void ShowDialog();
 
   // Called only when the AddToHomescreenDataFetcher has retrieved all of the
   // data needed to add the shortcut.
   void AddShortcut(const ShortcutInfo& info, const SkBitmap& icon);
 
   // Called only when the AddToHomescreenDataFetcher has retrieved all of the
   // data needed to install a WebAPK.
-  void CreateInfoBarForWebAPK(const ShortcutInfo& info, const SkBitmap& icon);
+  void CreateInfoBarForWebApk(const ShortcutInfo& info, const SkBitmap& icon);
 
   void RecordAddToHomescreen();
 
   // AddToHomescreenDataFetcher::Observer:
   void OnDidDetermineWebApkCompatibility(bool is_webapk_compatible) override;
   void OnUserTitleAvailable(const base::string16& user_title) override;
   void OnDataAvailable(const ShortcutInfo& info, const SkBitmap& icon) override;
   SkBitmap FinalizeLauncherIconInBackground(const SkBitmap& icon,
                                             const GURL& url,
                                             bool* is_generated) override;
 
   // Points to the Java object.
   base::android::ScopedJavaGlobalRef<jobject> java_ref_;
 

[pkotwicz, 2016/09/29 14:18:54]

Everyone has a slightly different commenting style. It is unclear to me that
your version of the comments is better than the existing comments. Your version
does not seem to be worse than the existing comments either.

I would advise not to modify the existing comments because:
- it is unclear that your version is better.
- you do not have changes in this file. If you had added a member variable in
this CL you could probably get away with changing the comments for an unrelated
member variable.

[F, 2016/09/30 18:02:02]

Done.

-  // Whether the user has requested that a shortcut be added while a fetch was
-  // in progress.
+  // Indicates whether the user has requested that a shortcut be added while a
+  // fetch was in progress.
   bool add_shortcut_pending_;
 
-  // Whether the site is WebAPK-compatible.
+  // Indicates whether the site is WebAPK-compatible.
   bool is_webapk_compatible_;
 
   // Fetches data required to add a shortcut.
   scoped_refptr<AddToHomescreenDataFetcher> data_fetcher_;
 
   DISALLOW_COPY_AND_ASSIGN(AddToHomescreenManager);
 };
 
 #endif  // CHROME_BROWSER_ANDROID_WEBAPPS_ADD_TO_HOMESCREEN_MANAGER_H_