Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(68)

Side by Side Diff: chrome/browser/ui/android/infobars/update_password_infobar.h

Issue 1490193003: [Password Manager] Update Confirmation UI for saved password change for Chrome on Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 // Copyright 2016 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_
6 #define CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_
7
8 #include "base/android/scoped_java_ref.h"
9 #include "base/macros.h"
10 #include "chrome/browser/password_manager/update_password_infobar_delegate.h"
Peter Kasting 2016/02/13 01:44:52 Nit: I think you can forward-declare UpdatePasswor
melandory 2016/02/22 12:41:09 Done.
11 #include "chrome/browser/ui/android/infobars/confirm_infobar.h"
12
13 // The Android infobar that offers the user the ability to update a password
14 // for the site.
Peter Kasting 2016/02/13 01:44:52 Nit: How about: The infobar to be used with Updat
melandory 2016/02/22 12:41:09 Done.
15 class UpdatePasswordInfoBar : public ConfirmInfoBar {
16 public:
17 static bool Register(JNIEnv* env);
Peter Kasting 2016/02/13 01:44:52 Nit: Google style guide says static methods go bel
melandory 2016/02/22 12:41:09 Done.
18
19 explicit UpdatePasswordInfoBar(
20 scoped_ptr<UpdatePasswordInfoBarDelegate> delegate);
21 ~UpdatePasswordInfoBar() override;
22
23 int GetIdOfSelectedUsername();
Peter Kasting 2016/02/13 01:44:52 Nit: const?
melandory 2016/02/22 12:41:09 Done.
24
25 private:
26 // ConfirmInfoBar:
27 base::android::ScopedJavaLocalRef<jobject> CreateRenderInfoBar(
28 JNIEnv* env) override;
29 void OnLinkClicked(JNIEnv* env,
30 const base::android::JavaParamRef<jobject>& obj) override;
31
32 base::android::ScopedJavaGlobalRef<jobject> java_infobar_;
33
34 DISALLOW_COPY_AND_ASSIGN(UpdatePasswordInfoBar);
35 };
36
37 #endif // CHROME_BROWSER_UI_ANDROID_INFOBARS_UPDATE_PASSWORD_INFOBAR_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698