Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(346)

Unified Diff: chrome/browser/android/banners/app_banner_manager_android.h

Issue 2393513004: Convert app banners to use Mojo. (Closed)
Patch Set: Add TODO Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/browser/android/banners/app_banner_manager_android.h
diff --git a/chrome/browser/android/banners/app_banner_manager_android.h b/chrome/browser/android/banners/app_banner_manager_android.h
index 541501aa9b7d7856e11c54aa11d9a64bc7851d1c..29750ed030f6765b1efad5d436068874ef5081c2 100644
--- a/chrome/browser/android/banners/app_banner_manager_android.h
+++ b/chrome/browser/android/banners/app_banner_manager_android.h
@@ -5,6 +5,8 @@
#ifndef CHROME_BROWSER_ANDROID_BANNERS_APP_BANNER_MANAGER_ANDROID_H_
#define CHROME_BROWSER_ANDROID_BANNERS_APP_BANNER_MANAGER_ANDROID_H_
+#include <string>
+
#include "base/android/scoped_java_ref.h"
#include "base/callback_forward.h"
#include "base/macros.h"
@@ -13,13 +15,14 @@
namespace banners {
-// Extends the AppBannerManager to support native Android apps.
-// This class owns a Java-side AppBannerManager which is used to interface with
-// the Java runtime for fetching native app data and installing them when
-// requested.
+// Extends the AppBannerManager to support native Android apps. This class owns
+// a Java-side AppBannerManager which interfaces with the Java runtime to fetch
+// native app data and install them when requested.
+//
// A site requests a native app banner by setting "prefer_related_applications"
// to true in its manifest, and providing at least one related application for
// the "play" platform with a Play Store ID.
+//
// This class uses that information to request the app's metadata, including an
// icon. If successful, the icon is downloaded and the native app banner shown.
// Otherwise, if no related applications were detected, or their manifest

Powered by Google App Engine
This is Rietveld 408576698