Credential saving clank infobar for Smart Lock.

chrome/browser/ui/android/infobars/save_password_infobar.h

+// 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.
+
+#ifndef CHROME_BROWSER_UI_ANDROID_INFOBARS_SAVE_PASSWORD_INFOBAR_H_
+#define CHROME_BROWSER_UI_ANDROID_INFOBARS_SAVE_PASSWORD_INFOBAR_H_
+
+#include "chrome/browser/password_manager/save_password_infobar_delegate.h"
+#include "chrome/browser/ui/android/infobars/confirm_infobar.h"
+
+class SavePasswordInfoBar : public ConfirmInfoBar {

[gone, 2015/03/02 17:39:38]

add a comment describing the class

[melandory, 2015/03/03 11:55:47]

Done.

+ public:
+  explicit SavePasswordInfoBar(
+      scoped_ptr<SavePasswordInfoBarDelegate> delegate);
+
+  ~SavePasswordInfoBar() override;
+
+ private:
+  // ConfirmInfoBar:
+  base::android::ScopedJavaLocalRef<jobject> CreateRenderInfoBar(
+      JNIEnv* env) override;
+
+  SavePasswordInfoBarDelegate* GetDelegate();
+
+  DISALLOW_COPY_AND_ASSIGN(SavePasswordInfoBar);
+};
+
+bool RegisterSavePasswordInfoBar(JNIEnv* env);

[Bernhard Bauer, 2015/03/02 10:28:09]

You could move this into the class as a static method, to avoid polluting the
global namespace.

Also, is there ever called somewhere?

[gone, 2015/03/02 17:39:38]

Supposed to be called from chrome_jni_registrar.  It'll crash when the infobar
appears, otherwise, since the methods aren't registered.

[melandory, 2015/03/03 11:55:47]

Done.

+
+#endif  // CHROME_BROWSER_UI_ANDROID_INFOBARS_SAVE_PASSWORD_INFOBAR_H_