Add notification of data reduction proxy field trial

chrome/browser/ui/android/infobars/data_reduction_proxy_infobar.cc

Index: chrome/browser/ui/android/infobars/data_reduction_proxy_infobar.cc
diff --git a/chrome/browser/ui/android/infobars/data_reduction_proxy_infobar.cc b/chrome/browser/ui/android/infobars/data_reduction_proxy_infobar.cc
new file mode 100644
index 0000000000000000000000000000000000000000..b86146e049528b827857af355ae3fa215de12a97
--- /dev/null
+++ b/chrome/browser/ui/android/infobars/data_reduction_proxy_infobar.cc
@@ -0,0 +1,62 @@
+// 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/ui/android/infobars/data_reduction_proxy_infobar.h"
+
+#include "base/android/jni_android.h"
+#include "base/android/jni_string.h"
+#include "base/logging.h"
+#include "chrome/browser/android/resource_mapper.h"
+#include "chrome/browser/net/spdyproxy/data_reduction_proxy_infobar_delegate.h"
+#include "content/public/browser/web_contents.h"
+#include "jni/DataReductionProxyInfoBarDelegate_jni.h"
+
+void Launch(JNIEnv* env, jclass clazz, jobject jweb_contents) {

[aruslan, 2014/07/15 23:14:28]

nit: add some comment like // JNI for DataReductionProxyInfoBarDelegate

[bengr, 2014/07/16 00:43:25]

Done.

+  DataReductionProxyInfoBar::Launch(env, clazz, jweb_contents);
+}
+
+// static
+void DataReductionProxyInfoBar::Launch(
+    JNIEnv* env, jclass, jobject jweb_contents) {
+  content::WebContents* web_contents =
+      content::WebContents::FromJavaWebContents(jweb_contents);
+  DCHECK(web_contents);
+  DataReductionProxyInfoBarDelegate::Create(web_contents);
+}
+
+// static
+scoped_ptr<infobars::InfoBar> DataReductionProxyInfoBarDelegate::CreateInfoBar(

[aruslan, 2014/07/15 23:14:28]

nit: This file combines implementations from different classes, so some comments
like // DataReductionProxyInfoBarDelegate: might be helpful.

[aruslan, 2014/07/15 23:14:28]

nit: It would be great if you could order the methods as in corresponding header
files.  This way you would have only three sections: implementations for
DataReductionProxyInfoBar, implementations for DataReductionProxyInfoBarDelegate
and JNI-related stuff.

[bengr, 2014/07/16 00:43:25]

Done.

[bengr, 2014/07/16 00:43:25]

Done.

+    scoped_ptr<DataReductionProxyInfoBarDelegate> delegate) {
+  return scoped_ptr<infobars::InfoBar>(
+      new DataReductionProxyInfoBar(delegate.Pass()));
+}
+
+DataReductionProxyInfoBar::DataReductionProxyInfoBar(
+    scoped_ptr<DataReductionProxyInfoBarDelegate> delegate)
+    : ConfirmInfoBar(delegate.PassAs<ConfirmInfoBarDelegate>()),
+      java_data_reduction_proxy_delegate_() {
+}
+
+DataReductionProxyInfoBar::~DataReductionProxyInfoBar() {
+}
+
+base::android::ScopedJavaLocalRef<jobject>
+DataReductionProxyInfoBar::CreateRenderInfoBar(JNIEnv* env) {
+  java_data_reduction_proxy_delegate_.Reset(
+      Java_DataReductionProxyInfoBarDelegate_create(env));
+
+  return Java_DataReductionProxyInfoBarDelegate_showDataReductionProxyInfoBar(
+      env,
+      java_data_reduction_proxy_delegate_.obj(),
+      reinterpret_cast<intptr_t>(this),
+      GetEnumeratedIconId());
+}
+
+DataReductionProxyInfoBarDelegate* DataReductionProxyInfoBar::GetDelegate() {
+  return static_cast<DataReductionProxyInfoBarDelegate*>(delegate());
+}
+
+bool DataReductionProxyInfoBar::Register(JNIEnv* env) {
+  return RegisterNativesImpl(env);
+}