Suppress native app install banner if the site has a manifest.

chrome/browser/android/banners/app_banner_manager.cc

 // Copyright 2014 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.
 
 #include "chrome/browser/android/banners/app_banner_manager.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
+#include "base/bind.h"
 #include "base/command_line.h"
 #include "base/metrics/histogram.h"
 #include "chrome/browser/android/banners/app_banner_metrics_ids.h"
 #include "chrome/browser/android/banners/app_banner_settings_helper.h"
 #include "chrome/browser/android/banners/app_banner_utilities.h"
 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h"
 #include "chrome/browser/profiles/profile.h"
+#include "chrome/common/chrome_constants.h"
 #include "chrome/common/chrome_switches.h"
+#include "chrome/common/render_messages.h"
 #include "content/public/browser/android/content_view_core.h"
 #include "content/public/browser/navigation_details.h"
+#include "content/public/browser/render_frame_host.h"
 #include "content/public/browser/web_contents.h"
 #include "content/public/common/frame_navigate_params.h"
+#include "content/public/common/manifest.h"
 #include "jni/AppBannerManager_jni.h"
 #include "net/base/load_flags.h"
 #include "ui/gfx/android/java_bitmap.h"
 
 using base::android::ConvertJavaStringToUTF8;
 using base::android::ConvertUTF8ToJavaString;
 
 namespace {
 const char kBannerTag[] = "google-play-id";
 }
 
 namespace banners {
 
 AppBannerManager::AppBannerManager(JNIEnv* env, jobject obj)
-    : MetaTagObserver(kBannerTag),
-      weak_java_banner_view_manager_(env, obj) {}
+    : weak_java_banner_view_manager_(env, obj) {}
 
 AppBannerManager::~AppBannerManager() {
 }
 
 void AppBannerManager::Destroy(JNIEnv* env, jobject obj) {
   delete this;
 }
 
 void AppBannerManager::BlockBanner(JNIEnv* env,
                                    jobject obj,
@@ skipping 19 matching lines @@
     const content::LoadCommittedDetails& details,
     const content::FrameNavigateParams& params) {
   // Get rid of the current banner.
   JNIEnv* env = base::android::AttachCurrentThread();
   ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);
   if (jobj.is_null())
     return;
   Java_AppBannerManager_dismissCurrentBanner(env, jobj.obj(), DISMISS_NAVIGATE);
 }
 
+void AppBannerManager::DidFinishLoad(
+    content::RenderFrameHost* render_frame_host,
+    const GURL& validated_url) {
+  if (render_frame_host->GetParent())
+    return;
+  validated_url_ = validated_url;
+
+  // See if the page has a manifest. Using Unretained(this) here is safe as the
+  // lifetime of this object extends beyond the lifetime of the web_contents(),

[benwells, 2015/01/28 09:24:23]

I'm not absolutely sure of this and didn't get around to checking - is it true?
I think this is owned by indirectly the java Tab object so I imagine it will
live longer than the web_contents.

[gone, 2015/01/28 18:44:13]

Yeah, should be fine.  The native AppBannerManager is owned by the Java version,
which lives as long as the Tab does.

+  // and when web_contents() is destroyed it will call OnDidGetManifest.
+  web_contents()->GetManifest(base::Bind(&AppBannerManager::OnDidGetManifest,
+                                         base::Unretained(this)));
+}
+
+void AppBannerManager::OnDidGetManifest(const content::Manifest& manifest) {
+  if (web_contents()->IsBeingDestroyed())
+    return;
+
+  if (manifest.IsEmpty()) {
+    // No manifest, see if there is a play store meta tag.
+    Send(new ChromeViewMsg_RetrieveMetaTagContent(routing_id(),
+                                                  validated_url_,
+                                                  kBannerTag));
+    return;
+  }
+
+  // TODO(benwells): Check triggering parameters here and if there is a meta
+  // tag.
+  // TODO(dfalcantara): Show banner for web site with manifest.

[benwells, 2015/01/28 09:24:23]

I expect this TODO to move around.

+}
+
+bool AppBannerManager::OnMessageReceived(const IPC::Message& message) {
+  bool handled = true;
+  IPC_BEGIN_MESSAGE_MAP(AppBannerManager, message)
+    IPC_MESSAGE_HANDLER(ChromeViewHostMsg_DidRetrieveMetaTagContent,
+                        OnDidRetrieveMetaTagContent)
+    IPC_MESSAGE_UNHANDLED(handled = false)
+  IPC_END_MESSAGE_MAP()
+  return handled;
+}
+
 void AppBannerManager::OnFetchComplete(const GURL url, const SkBitmap* bitmap) {
   if (bitmap) {
     JNIEnv* env = base::android::AttachCurrentThread();
 
     ScopedJavaLocalRef<jobject> jobj = weak_java_banner_view_manager_.get(env);
     if (jobj.is_null())
       return;
 
     ScopedJavaLocalRef<jstring> jimage_url(
         ConvertUTF8ToJavaString(env, url.spec()));
     ScopedJavaLocalRef<jobject> jimage = gfx::ConvertToJavaBitmap(bitmap);
     bool displayed = Java_AppBannerManager_createBanner(env,
                                                         jobj.obj(),
                                                         jimage_url.obj(),
                                                         jimage.obj());
     if (displayed)
       banners::TrackDisplayEvent(DISPLAY_CREATED);
   } else {
     DVLOG(1) << "Failed to retrieve image: " << url;
   }
 
   fetcher_.reset();
 }
 
-void AppBannerManager::HandleMetaTagContent(const std::string& tag_content,
-                                            const GURL& expected_url) {
+void AppBannerManager::OnDidRetrieveMetaTagContent(
+    bool success,
+    const std::string& tag_name,
+    const std::string& tag_content,
+    const GURL& expected_url) {
   DCHECK(web_contents());
+  if (!success || tag_name != kBannerTag || validated_url_ != expected_url ||
+      tag_content.size() >= chrome::kMaxMetaTagAttributeLength) {
+    return;
+  }
+
   banners::TrackDisplayEvent(DISPLAY_BANNER_REQUESTED);
 
   if (!AppBannerSettingsHelper::IsAllowed(web_contents(),
                                           expected_url,
                                           tag_content)) {
     return;
   }
 
   // Send the info to the Java side to get info about the app.
   JNIEnv* env = base::android::AttachCurrentThread();
@@ skipping 47 matching lines @@
   return base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kEnableAppInstallAlerts);
 }
 
 // Register native methods
 bool RegisterAppBannerManager(JNIEnv* env) {
   return RegisterNativesImpl(env);
 }
 
 }  // namespace banners