Implement "appinstalled" event on Android.

chrome/browser/android/banners/app_banner_infobar_delegate_android.cc

 // 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.
 
 #include "chrome/browser/android/banners/app_banner_infobar_delegate_android.h"
 
 #include "base/android/jni_android.h"
 #include "base/android/jni_string.h"
 #include "base/guid.h"
 #include "base/location.h"
@@ skipping 253 matching lines @@
   DCHECK(!native_app_data_.is_null());
   CreateJavaDelegate();
 }
 
 void AppBannerInfoBarDelegateAndroid::CreateJavaDelegate() {
   java_delegate_.Reset(Java_AppBannerInfoBarDelegateAndroid_create(
       base::android::AttachCurrentThread(),
       reinterpret_cast<intptr_t>(this)));
 }
 
 void AppBannerInfoBarDelegateAndroid::SendBannerAccepted(

[dominickn, 2016/10/12 22:35:09]

What about AppBannerInfoBarDelegateDesktop? Shouldn't that also call OnInstall?

[Matt Giuca, 2016/12/09 04:36:48]

I think desktop support for this can go in another CL (even if trivial). Testing
on desktop, at least, will be non-trivial.

     content::WebContents* web_contents,
     const std::string& platform) {
   web_contents->GetMainFrame()->Send(
       new ChromeViewMsg_AppBannerAccepted(
           web_contents->GetMainFrame()->GetRoutingID(),
           event_request_id_,
           platform));
+  if (weak_manager_)
+    weak_manager_->OnInstall();

[dominickn, 2016/10/12 22:35:09]

We're already sending a message to the renderer here. It would be nice if we
could combine them. My patch makes the first message go straight to the
BeforeInstallPromptEvent; perhaps that can then ask the Frame to call OnInstall?

Also, my CL moves the IPC call from here to AppBannerManager so you probably
won't need to have this call in here at all - it'll be in AppBannerManager.

[Matt Giuca, 2016/12/09 04:36:48]

Hmmm yeah your CL did make this easier but it's still a bit messy to do that
because OnInstall can get called from not the banner. I prefer not to go through
my own mojo message for some cases but hitch-hike in your mojo message for this
one. Hard to understand.

Not worth it given how rare this message is sent.

 }
 
 infobars::InfoBarDelegate::InfoBarIdentifier
 AppBannerInfoBarDelegateAndroid::GetIdentifier() const {
   return APP_BANNER_INFOBAR_DELEGATE_ANDROID;
 }
 
 gfx::Image AppBannerInfoBarDelegateAndroid::GetIcon() const {
   return gfx::Image::CreateFrom1xBitmap(*icon_.get());
 }
@@ skipping 149 matching lines @@
 
   TrackDismissEvent(DISMISS_EVENT_BANNER_CLICK);
   return true;
 }
 
 bool RegisterAppBannerInfoBarDelegateAndroid(JNIEnv* env) {
  return RegisterNativesImpl(env);
 }
 
 }  // namespace banners