Make activities have a thick border when in overview mode part 1

athena/activity/activity_frame_view.cc

 // Copyright 2014 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.
 
 #include "athena/activity/activity_frame_view.h"
 
-#include <algorithm>
-#include <vector>
-
 #include "athena/activity/public/activity_view_model.h"
+#include "athena/wm/public/window_manager.h"
+#include "third_party/skia/include/core/SkBitmap.h"
 #include "ui/base/hit_test.h"
+#include "ui/gfx/canvas.h"
+#include "ui/gfx/image/image_skia.h"
 #include "ui/views/background.h"
-#include "ui/views/border.h"
+#include "ui/views/controls/image_view.h"
 #include "ui/views/controls/label.h"
 #include "ui/views/view.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/widget/widget_delegate.h"
+#include "ui/views/window/client_view.h"
 
 namespace athena {
+namespace {
 
-////////////////////////////////////////////////////////////////////////////////
-// FrameViewAthena, public:
+// The icon size.
+const int kIconSize = 32;
+
+// The distance between the icon and the title when the icon is visible.
+const int kIconTitleSpacing = 5;
+
+// The height of the top border necessary to display the title without the icon.
+const int kDefaultTitleHeight = 25;
+
+// The height of the top border in overview mode.
+const int kOverviewTitleHeight = 55;
+
+// The height of the top border for fullscreen and frameless activities in
+// overview mode.
+const int kOverviewShortTitleHeight = 30;
+
+// The thickness of the left, right and bottom borders in overview mode.
+const int kOverviewBorderThickness = 5;
+
+}  // namespace
 
 // static
 const char ActivityFrameView::kViewClassName[] = "ActivityFrameView";
 
 ActivityFrameView::ActivityFrameView(views::Widget* frame,
                                      ActivityViewModel* view_model)
-    : frame_(frame), view_model_(view_model), title_(new views::Label) {
-  title_->SetHorizontalAlignment(gfx::ALIGN_CENTER);
+    : frame_(frame),
+      view_model_(view_model),
+      title_(new views::Label),
+      icon_(new views::ImageView),
+      in_overview_(false) {
   title_->SetEnabledColor(SkColorSetA(SK_ColorBLACK, 0xe5));
-  title_->SetBorder(views::Border::CreateSolidSidedBorder(0, 0, 1, 0,
-      SkColorSetA(SK_ColorGRAY, 0x7f)));
+
+  SkBitmap bitmap;
+  bitmap.allocN32Pixels(kIconSize, kIconSize);
+  bitmap.eraseARGB(255, 0, 255, 0);
+  icon_->SetImage(gfx::ImageSkia::CreateFrom1xBitmap(bitmap));
+
   AddChildView(title_);
+  AddChildView(icon_);

[sadrul, 2014/09/03 16:10:18]

Add |icon_| only when entering the overview mode,

[sadrul, 2014/09/03 20:05:07]

ping?

[pkotwicz, 2014/09/03 20:15:45]

Sorry for missing your comment

I believe that your initial concern was "Instead of always having |icon_| with
zero opacity, it'd be better to add it
only when in overview mode."
I believe I have addressed this concern by toggling |icon_|'s visibility based
on overview mode. I think that it is ok to always keep |icon_| in the view
hierarchy.

[sadrul, 2014/09/03 20:28:43]

Sounds good.

+
+  SkColor bgcolor = view_model_->GetRepresentativeColor();
+  set_background(views::Background::CreateSolidBackground(bgcolor));
   UpdateWindowTitle();
+
+  WindowManager::GetInstance()->AddObserver(this);
 }
 
 ActivityFrameView::~ActivityFrameView() {
+  WindowManager::GetInstance()->RemoveObserver(this);
 }
 
-////////////////////////////////////////////////////////////////////////////////
-// ActivityFrameView, views::NonClientFrameView overrides:
-
 gfx::Rect ActivityFrameView::GetBoundsForClientView() const {
   gfx::Rect client_bounds = bounds();
-  if (view_model_->UsesFrame())
-    client_bounds.Inset(0, NonClientTopBorderHeight(), 0, 0);
+  client_bounds.Inset(NonClientBorderInsets());
   return client_bounds;
 }
 
 gfx::Rect ActivityFrameView::GetWindowBoundsForClientBounds(
     const gfx::Rect& client_bounds) const {
   gfx::Rect window_bounds = client_bounds;
-  if (view_model_->UsesFrame())
-    window_bounds.Inset(0, -NonClientTopBorderHeight(), 0, 0);
+  window_bounds.Inset(-NonClientBorderInsets());
   return window_bounds;
 }
 
 int ActivityFrameView::NonClientHitTest(const gfx::Point& point) {
-  if (frame_->IsFullscreen())
-    return 0;
-  if (title_->bounds().Contains(point))
-    return HTCAPTION;
-  return 0;
+  if (!bounds().Contains(point))
+    return HTNOWHERE;
+  int client_hit_test = frame_->client_view()->NonClientHitTest(point);
+  return (client_hit_test == HTNOWHERE) ?
+      HTCAPTION : client_hit_test;

[sadrul, 2014/09/03 16:10:18]

This still doesn't look right. We should return HTCAPTION only for the window
title (and maybe for the icon, but we don't use hit-test in overview mode)

[pkotwicz, 2014/09/03 19:40:59]

You're right, I did not take into account the left, right and bottom borders in
overview mode.

I am going to keep returning HCAPTION for the window icon. This is similar to
what we do with BrowserNonClientFrameViewAsh::window_icon_

 }
 
 void ActivityFrameView::GetWindowMask(const gfx::Size& size,
                                       gfx::Path* window_mask) {
 }
 
 void ActivityFrameView::ResetWindowControls() {
 }
 
 void ActivityFrameView::UpdateWindowIcon() {

[sadrul, 2014/09/03 16:10:18]

Update icon_ here?
(https://codereview.chromium.org/479533003/diff/340001/athena/activity/activit...)

[pkotwicz, 2014/09/03 19:40:59]

Yes, when we have a real icon. Currently, |icon_| is a blue bitmap.

Aside: It may be possible for the activity's representative color to change
independant of either the title or the icon changing so we will have to deal
with that.

[sadrul, 2014/09/03 20:05:07]

Ah, I see. It would be a good idea to plumb the icon from FaviconStatus through
the AcitivityViewModel (so we would get the icon when running from chrome?). But
we can do that in a separate CL.

 }
 
 void ActivityFrameView::UpdateWindowTitle() {
   if (!view_model_->UsesFrame())
     return;
 
   SkColor bgcolor = view_model_->GetRepresentativeColor();
-  title_->set_background(views::Background::CreateSolidBackground(bgcolor));
+  set_background(views::Background::CreateSolidBackground(bgcolor));
   title_->SetBackgroundColor(bgcolor);
   title_->SetText(frame_->widget_delegate()->GetWindowTitle());
+  Layout();
 }
 
-////////////////////////////////////////////////////////////////////////////////
-// ActivityFrameView, views::View overrides:
-
 gfx::Size ActivityFrameView::GetPreferredSize() const {
   gfx::Size pref = frame_->client_view()->GetPreferredSize();
   gfx::Rect bounds(0, 0, pref.width(), pref.height());
   return frame_->non_client_view()
       ->GetWindowBoundsForClientBounds(bounds)
       .size();
 }
 
 const char* ActivityFrameView::GetClassName() const {
   return kViewClassName;
 }
 
 void ActivityFrameView::Layout() {
-  title_->SetBounds(0, 0, width(), NonClientTopBorderHeight());
+  if (frame_->IsFullscreen() || !view_model_->UsesFrame()) {
+    title_->SetVisible(false);
+    icon_->SetVisible(false);
+    return;
+  }
+
+  title_->SetVisible(true);
+  icon_->SetVisible(in_overview_);
+
+  gfx::Size preferred_title_size = title_->GetPreferredSize();
+  int top_height = NonClientTopBorderHeight();
+  int title_x = 0;
+  if (in_overview_) {
+    int edge = (top_height - kIconSize) / 2;
+    icon_->SetBounds(edge, edge, kIconSize, kIconSize);
+
+    title_x = icon_->bounds().right() + kIconTitleSpacing;
+  } else {
+    title_x = (width() - preferred_title_size.width()) / 2;
+  }
+
+  title_->SetBounds(title_x,
+                    (top_height - preferred_title_size.height()) / 2,
+                    preferred_title_size.width(),
+                    preferred_title_size.height());

[sadrul, 2014/09/03 16:10:17]

Consider using a BoxLayout instead. (in the box layout, set flex = 1 for title_,
and 0 for icon_). When going into overview mode, left-align title_, and reset to
center-align when going out of overview mode. Would that work?

[pkotwicz, 2014/09/03 19:40:59]

I think that |icon_| and |title_| would need to be children of a container view
which covers the header portion (as opposed to the entire NonClientFrameView)
for using BoxLayout to work. I doubt that using BoxLayout will decrease line
count / complexity in this case.

Thanks for suggesting BoxLayout though.

 }
 
-////////////////////////////////////////////////////////////////////////////////
-// ActivityFrameView, private:
+void ActivityFrameView::OnPaintBackground(gfx::Canvas* canvas) {
+  View::OnPaintBackground(canvas);
+
+  gfx::Rect border_bounds = GetLocalBounds();
+  border_bounds.Inset(NonClientBorderInsets());
+  border_bounds.Inset(-1, -1, 0, 0);
+  canvas->DrawRect(border_bounds, SkColorSetA(SK_ColorGRAY, 0x7f));
+}
+
+void ActivityFrameView::OnOverviewModeEnter() {
+  view_model_->PrepareContentsForOverview();
+  in_overview_ = true;
+  InvalidateLayout();
+  frame_->client_view()->InvalidateLayout();
+  frame_->GetRootView()->Layout();
+  SchedulePaint();
+}
+
+void ActivityFrameView::OnOverviewModeExit() {
+  in_overview_ = false;
+  InvalidateLayout();
+  frame_->client_view()->InvalidateLayout();
+  frame_->GetRootView()->Layout();
+  SchedulePaint();
+  view_model_->ResetContentsView();
+}
+
+void ActivityFrameView::OnActivityOrderHasChanged() {
+}
+
+gfx::Insets ActivityFrameView::NonClientBorderInsets() const {
+  int edge_width = in_overview_ ? kOverviewBorderThickness : 0;
+  return gfx::Insets(NonClientTopBorderHeight(),
+                     edge_width,
+                     edge_width,
+                     edge_width);
+}
 
 int ActivityFrameView::NonClientTopBorderHeight() const {
-  const int kDefaultTitleHeight = 25;
-  return frame_->IsFullscreen() ? 0 : kDefaultTitleHeight;
+  if (frame_->IsFullscreen() || !view_model_->UsesFrame())
+    return in_overview_ ? kOverviewShortTitleHeight : 0;
+  return in_overview_ ? kOverviewTitleHeight : kDefaultTitleHeight;
 }
 
-}  // namespace ash
+}  // namespace athena