Chromium Code Reviews| Index: chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc |
| diff --git a/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc b/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc |
| index 1543bf38ae423f607f401202682934f1e0f5fbf6..0125b5b9907f1c338511a686212c5bff0f0d227b 100644 |
| --- a/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc |
| +++ b/chrome/browser/ui/views/frame/browser_non_client_frame_view_mus.cc |
| @@ -22,7 +22,6 @@ |
| #include "chrome/browser/web_applications/web_app.h" |
| #include "chrome/grit/theme_resources.h" |
| #include "content/public/browser/web_contents.h" |
| -#include "services/ui/public/cpp/window.h" |
| #include "ui/accessibility/ax_node_data.h" |
| #include "ui/aura/client/aura_constants.h" |
| #include "ui/aura/window.h" |
| @@ -36,6 +35,7 @@ |
| #include "ui/gfx/scoped_canvas.h" |
| #include "ui/views/controls/label.h" |
| #include "ui/views/layout/layout_constants.h" |
| +#include "ui/views/mus/desktop_window_tree_host_mus.h" |
| #include "ui/views/mus/window_manager_frame_values.h" |
| #include "ui/views/widget/widget.h" |
| #include "ui/views/widget/widget_delegate.h" |
| @@ -333,20 +333,17 @@ gfx::ImageSkia BrowserNonClientFrameViewMus::GetFaviconForTabIconView() { |
| // BrowserNonClientFrameView: |
| void BrowserNonClientFrameViewMus::UpdateProfileIcons() { |
| #if defined(FRAME_AVATAR_BUTTON) |
| - if (browser_view()->IsRegularOrGuestSession()) |
| + if (browser_view()->IsRegularOrGuestSession()) { |
| profile_switcher_.Update(AvatarButtonStyle::NATIVE); |
| - else |
| + return; |
| + } |
| #endif |
| - UpdateProfileIndicatorIcon(); |
| + UpdateProfileIndicatorIcon(); |
| } |
| /////////////////////////////////////////////////////////////////////////////// |
| // BrowserNonClientFrameViewMus, private: |
| -ui::Window* BrowserNonClientFrameViewMus::mus_window() { |
| - return static_cast<BrowserFrameMus*>(frame()->native_widget())->window(); |
| -} |
| - |
| void BrowserNonClientFrameViewMus::UpdateClientArea() { |
| std::vector<gfx::Rect> additional_client_area; |
| if (tab_strip_) { |
| @@ -356,9 +353,10 @@ void BrowserNonClientFrameViewMus::UpdateClientArea() { |
| additional_client_area.push_back(tab_strip_bounds); |
| } |
| } |
| - mus_window()->SetClientArea( |
| - views::WindowManagerFrameValues::instance().normal_insets, |
| - additional_client_area); |
| + static_cast<aura::WindowTreeHostMus*>( |
| + GetWidget()->GetNativeWindow()->GetHost()) |
|
sadrul
2016/12/16 02:17:48
We are using this static-cast in a few places. May
sky
2016/12/16 03:51:50
I'm hesitant to do that because it's generally not
|
| + ->SetClientArea(views::WindowManagerFrameValues::instance().normal_insets, |
| + additional_client_area); |
| } |
| void BrowserNonClientFrameViewMus::TabStripMaxXChanged(TabStrip* tab_strip) { |