UI for the Data Saver InfoBar Promo

chrome/browser/net/spdyproxy/data_reduction_promo_infobar_delegate_android.cc

+// 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.
+
+#include "chrome/browser/net/spdyproxy/data_reduction_promo_infobar_delegate_android.h"
+
+#include "chrome/browser/android/android_theme_resources.h"
+#include "chrome/browser/infobars/infobar_service.h"
+#include "components/infobars/core/infobar.h"
+#include "components/infobars/core/infobar_delegate.h"
+#include "content/public/browser/web_contents.h"
+#include "grit/components_strings.h"
+#include "ui/base/l10n/l10n_util.h"
+
+// static
+void DataReductionPromoInfoBarDelegateAndroid::Create(
+    content::WebContents* web_contents) {
+  InfoBarService* infobar_service =
+      InfoBarService::FromWebContents(web_contents);
+  infobar_service->AddInfoBar(
+      DataReductionPromoInfoBarDelegateAndroid::CreateInfoBar(
+          infobar_service,
+          std::unique_ptr<DataReductionPromoInfoBarDelegateAndroid>(
+              new DataReductionPromoInfoBarDelegateAndroid())));
+}
+
+DataReductionPromoInfoBarDelegateAndroid::
+    ~DataReductionPromoInfoBarDelegateAndroid() {}

[Raj, 2016/06/13 17:52:18]

nit: This could be inline in the header file.

[megjablon, 2016/06/23 23:17:30]

Done.

+
+DataReductionPromoInfoBarDelegateAndroid::
+    DataReductionPromoInfoBarDelegateAndroid()
+    : ConfirmInfoBarDelegate() {}
+
+infobars::InfoBarDelegate::InfoBarIdentifier
+DataReductionPromoInfoBarDelegateAndroid::GetIdentifier() const {
+  return DATA_REDUCTION_PROMO_INFOBAR_DELEGATE_ANDROID;
+}
+
+bool DataReductionPromoInfoBarDelegateAndroid::ShouldExpire(
+    const NavigationDetails& details) const {
+  return false;
+}
+
+base::string16 DataReductionPromoInfoBarDelegateAndroid::GetMessageText()

[Raj, 2016/06/13 17:52:18]

Is this function needed ?

[megjablon, 2016/06/23 23:17:30]

Yes, it is abstract so it needs to be overwritten.

+    const {
+  return base::string16();
+}
+
+int DataReductionPromoInfoBarDelegateAndroid::GetButtons() const {

[Raj, 2016/06/13 17:52:18]

Is this function needed ?

[megjablon, 2016/06/23 23:17:30]

Nope removed.

+  return BUTTON_OK | BUTTON_CANCEL;
+}