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

Unified Diff: chrome/browser/ui/views/passwords/credentials_item_view.h

Issue 1701363002: Change the account chooser on Win to look like in the mocks. (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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/ui/views/passwords/credentials_item_view.h
diff --git a/chrome/browser/ui/views/passwords/credentials_item_view.h b/chrome/browser/ui/views/passwords/credentials_item_view.h
index c9f98deb8255af863c079ed07381118975c2cfc4..a0cb4fd555300ad68b36ba5a0d3e979c48d3f613 100644
--- a/chrome/browser/ui/views/passwords/credentials_item_view.h
+++ b/chrome/browser/ui/views/passwords/credentials_item_view.h
@@ -8,7 +8,6 @@
#include "base/macros.h"
#include "base/memory/weak_ptr.h"
#include "chrome/browser/ui/passwords/account_avatar_fetcher.h"
-#include "components/password_manager/core/common/credential_manager_types.h"
#include "ui/views/controls/button/label_button.h"
namespace autofill {
@@ -34,34 +33,36 @@ class CredentialsItemView : public AccountAvatarFetcherDelegate,
public views::LabelButton {
public:
CredentialsItemView(views::ButtonListener* button_listener,
- const autofill::PasswordForm* form,
- password_manager::CredentialType credential_type,
const base::string16& upper_text,
const base::string16& lower_text,
+ SkColor hover_color,
+ const autofill::PasswordForm* form,
net::URLRequestContextGetter* request_context);
~CredentialsItemView() override;
const autofill::PasswordForm* form() const { return form_; }
- password_manager::CredentialType credential_type() const {
- return credential_type_;
- }
// AccountAvatarFetcherDelegate:
void UpdateAvatar(const gfx::ImageSkia& image) override;
+ void SetLowerLabelColor(SkColor color);
+ void SetHoverColor(SkColor color);
+
private:
// views::LabelButton:
gfx::Size GetPreferredSize() const override;
int GetHeightForWidth(int w) const override;
void Layout() override;
+ void OnPaint(gfx::Canvas* canvas) override;
const autofill::PasswordForm* form_;
- const password_manager::CredentialType credential_type_;
views::ImageView* image_view_;
views::Label* upper_label_;
views::Label* lower_label_;
+ SkColor hover_color_;
+
base::WeakPtrFactory<CredentialsItemView> weak_ptr_factory_;
DISALLOW_COPY_AND_ASSIGN(CredentialsItemView);

Powered by Google App Engine
This is Rietveld 408576698