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

Unified Diff: chrome/browser/ui/views/frame/browser_non_client_frame_view.cc

Issue 1614523002: Remove old-avatar supervised user labels on non-Mac. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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/frame/browser_non_client_frame_view.cc
diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc b/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc
index 6b0ed393a50398b8f523b84b3b359e6db3983d7f..c12e74de530828e182cae7091b97a05c5b46f135 100644
--- a/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc
+++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view.cc
@@ -30,17 +30,10 @@
#include "ui/views/background.h"
#include "ui/views/resources/grit/views_resources.h"
-#if defined(ENABLE_SUPERVISED_USERS)
-#include "chrome/browser/ui/views/profiles/supervised_user_avatar_label.h"
-#endif
-
BrowserNonClientFrameView::BrowserNonClientFrameView(BrowserFrame* frame,
BrowserView* browser_view)
: frame_(frame),
browser_view_(browser_view),
-#if defined(ENABLE_SUPERVISED_USERS)
- supervised_user_avatar_label_(nullptr),
-#endif
#if defined(FRAME_AVATAR_BUTTON)
new_avatar_button_(nullptr),
#endif
@@ -109,13 +102,6 @@ void BrowserNonClientFrameView::ChildPreferredSizeChanged(View* child) {
#endif
}
-#if defined(ENABLE_SUPERVISED_USERS)
-void BrowserNonClientFrameView::OnThemeChanged() {
- if (supervised_user_avatar_label_)
- supervised_user_avatar_label_->UpdateLabelStyle();
-}
-#endif
-
bool BrowserNonClientFrameView::ShouldPaintAsThemed() const {
return browser_view_->IsBrowserTypeNormal();
}
@@ -196,16 +182,6 @@ void BrowserNonClientFrameView::UpdateAvatar() {
void BrowserNonClientFrameView::UpdateOldAvatarButton() {
if (browser_view_->ShouldShowAvatar()) {
if (!avatar_button_) {
-#if defined(ENABLE_SUPERVISED_USERS)
- Profile* profile = browser_view_->browser()->profile();
- if (profile->IsSupervised() && !supervised_user_avatar_label_) {
- supervised_user_avatar_label_ =
- new SupervisedUserAvatarLabel(browser_view_);
- supervised_user_avatar_label_->set_id(
- VIEW_ID_SUPERVISED_USER_AVATAR_LABEL);
- AddChildView(supervised_user_avatar_label_);
- }
-#endif
avatar_button_ = new AvatarMenuButton(browser_view_);
avatar_button_->set_id(VIEW_ID_AVATAR_BUTTON);
AddChildView(avatar_button_);
@@ -214,14 +190,6 @@ void BrowserNonClientFrameView::UpdateOldAvatarButton() {
frame_->GetRootView()->Layout();
}
} else if (avatar_button_) {
-#if defined(ENABLE_SUPERVISED_USERS)
- // The avatar label can just be there if there is also an avatar button.
- if (supervised_user_avatar_label_) {
- RemoveChildView(supervised_user_avatar_label_);
- delete supervised_user_avatar_label_;
- supervised_user_avatar_label_ = nullptr;
- }
-#endif
RemoveChildView(avatar_button_);
delete avatar_button_;
avatar_button_ = nullptr;

Powered by Google App Engine
This is Rietveld 408576698