Omnibox UI Experiments: Simple border for narrow mode

chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.cc

 // Copyright (c) 2012 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 "chrome/browser/ui/views/omnibox/omnibox_popup_contents_view.h"
 
 #include <algorithm>
 
+#include "base/feature_list.h"
 #include "base/lazy_instance.h"
 #include "base/macros.h"
 #include "build/build_config.h"
 #include "chrome/browser/search/search.h"
 #include "chrome/browser/themes/theme_properties.h"
 #include "chrome/browser/ui/views/location_bar/location_bar_view.h"
 #include "chrome/browser/ui/views/omnibox/omnibox_result_view.h"
 #include "chrome/browser/ui/views/theme_copying_widget.h"
+#include "components/omnibox/browser/omnibox_field_trial.h"
 #include "components/omnibox/browser/omnibox_view.h"
 #include "third_party/skia/include/core/SkDrawLooper.h"
 #include "ui/base/theme_provider.h"
 #include "ui/compositor/clip_recorder.h"
 #include "ui/compositor/paint_recorder.h"
 #include "ui/gfx/canvas.h"
+#include "ui/gfx/color_palette.h"
 #include "ui/gfx/geometry/safe_integer_conversions.h"
 #include "ui/gfx/image/image.h"
 #include "ui/gfx/image/image_skia_operations.h"
 #include "ui/gfx/path.h"
 #include "ui/gfx/shadow_value.h"
+#include "ui/views/bubble/bubble_border.h"
 #include "ui/views/controls/image_view.h"
 #include "ui/views/view_targeter.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/window/non_client_view.h"
 
 namespace {
 
 // Cache the shadow images so that potentially expensive shadow drawing isn't
 // repeated.
 base::LazyInstance<gfx::ImageSkia>::DestructorAtExit g_top_shadow =
@@ skipping 41 matching lines @@
       omnibox_view_(omnibox_view),
       location_bar_view_(location_bar_view),
       font_list_(font_list),
       ignore_mouse_drag_(false),
       size_animation_(this),
       start_margin_(0),
       end_margin_(0) {
   // The contents is owned by the LocationBarView.
   set_owned_by_client();
 
-  if (g_top_shadow.Get().isNull()) {
+  if (g_top_shadow.Get().isNull() &&
+      !base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {

[Peter Kasting, 2017/06/09 19:49:30]

Nit: Pull out to a temp

[tommycli, 2017/06/09 21:12:54]

Done.

     std::vector<gfx::ShadowValue> shadows;
     // Blur by 1dp. See comment below about blur accounting.
     shadows.emplace_back(gfx::Vector2d(), 2, SK_ColorBLACK);
     g_top_shadow.Get() =
         gfx::ImageSkiaOperations::CreateHorizontalShadow(shadows, false);
   }
-  if (g_bottom_shadow.Get().isNull()) {
+  if (g_bottom_shadow.Get().isNull() &&
+      !base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {
     const int kSmallShadowBlur = 3;
     const int kLargeShadowBlur = 8;
     const int kLargeShadowYOffset = 3;
 
     std::vector<gfx::ShadowValue> shadows;
     // gfx::ShadowValue counts blur pixels both inside and outside the shape,
     // whereas these blur values only describe the outside portion, hence they
     // must be doubled.
     shadows.emplace_back(gfx::Vector2d(), 2 * kSmallShadowBlur,
                          SK_ColorBLACK);
@@ skipping 112 matching lines @@
     }
     if (match.type == AutocompleteMatchType::SEARCH_SUGGEST_TAIL) {
       max_match_contents_width_ = std::max(
           max_match_contents_width_, view->GetMatchContentsWidth());
     }
   }
 
   for (size_t i = result_size; i < AutocompleteResult::GetMaxMatches(); ++i)
     child_at(i)->SetVisible(false);
 
-  // We want the popup to appear to overlay the bottom of the toolbar. So we
-  // shift the popup to completely cover the client edge, and then draw an
-  // additional semitransparent shadow above that.
-  int top_edge_overlap = views::NonClientFrameView::kClientEdgeThickness +
-                         g_top_shadow.Get().height();
+  int top_edge_overlap = 0;
+  if (!base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {

[Peter Kasting, 2017/06/09 19:49:30]

Nit: Pull out to a temp

[tommycli, 2017/06/09 21:12:54]

Done.

+    // We want the popup to appear to overlay the bottom of the toolbar. So we
+    // shift the popup to completely cover the client edge, and then draw an
+    // additional semitransparent shadow above that.
+    top_edge_overlap = g_top_shadow.Get().height() +
+                       views::NonClientFrameView::kClientEdgeThickness;
+  }
 
   gfx::Point top_left_screen_coord;
   int width;
   location_bar_view_->GetOmniboxPopupPositioningInfo(
       &top_left_screen_coord, &width, &start_margin_,
       &end_margin_, top_edge_overlap);
   gfx::Rect new_target_bounds(top_left_screen_coord,
                               gfx::Size(width, CalculatePopupHeight()));
 
+  if (base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {
+    SkColor background_color =
+        GetNativeTheme()

[Peter Kasting, 2017/06/09 19:49:30]

When can this fail?

[tommycli, 2017/06/09 21:12:54]

Done. Digging into it, I don't see a failure state... and in fact most of the
callers use it as if it can never fail.

+            ? GetNativeTheme()->GetSystemColor(
+                  ui::NativeTheme::kColorId_ResultsTableNormalBackground)
+            : gfx::kPlaceholderColor;
+    auto border = base::MakeUnique<views::BubbleBorder>(
+        views::BubbleBorder::NONE, views::BubbleBorder::SMALL_SHADOW,
+        background_color);
+
+    // Reposition the popup to factor in for the shadow size.

[Peter Kasting, 2017/06/09 19:49:30]

Nit: "Reposition" -> "Outdent"; Remove "for"

[tommycli, 2017/06/09 21:12:54]

Done.

+    int border_thickness = border->GetBorderThickness();
+    new_target_bounds.Inset(-border_thickness, -border_thickness,
+                            -border_thickness, -border_thickness);
+
+    SetBackground(base::MakeUnique<views::BubbleBackground>(border.get()));
+    SetBorder(std::move(border));
+  }
+
   // If we're animating and our target height changes, reset the animation.
   // NOTE: If we just reset blindly on _every_ update, then when the user types
   // rapidly we could get "stuck" trying repeatedly to animate shrinking by the
   // last few pixels to get to one visible result.
   if (new_target_bounds.height() != target_bounds_.height())
     size_animation_.Reset();
   target_bounds_ = new_target_bounds;
 
   if (!popup_) {
     views::Widget* popup_parent = location_bar_view_->GetWidget();
@@ skipping 195 matching lines @@
 }
 
 ////////////////////////////////////////////////////////////////////////////////
 // OmniboxPopupContentsView, views::View overrides, private:
 
 const char* OmniboxPopupContentsView::GetClassName() const {
   return "OmniboxPopupContentsView";
 }
 
 void OmniboxPopupContentsView::OnPaint(gfx::Canvas* canvas) {
+  if (base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {
+    OnPaintBackground(canvas);
+    OnPaintBorder(canvas);

[Peter Kasting, 2017/06/09 19:49:30]

Just call View::OnPaint() instead of manually calling these.

[tommycli, 2017/06/09 21:12:54]

Done.

+    return;
+  }
+
   canvas->TileImageInt(g_top_shadow.Get(), 0, 0, width(),
                        g_top_shadow.Get().height());
   canvas->TileImageInt(g_bottom_shadow.Get(), 0,
                        height() - g_bottom_shadow.Get().height(), width(),
                        g_bottom_shadow.Get().height());
 }
 
 void OmniboxPopupContentsView::PaintChildren(const ui::PaintContext& context) {
+  if (base::FeatureList::IsEnabled(omnibox::kUIExperimentNarrowDropdown)) {
+    View::PaintChildren(context);
+    return;
+  }
+
   gfx::Rect contents_bounds = GetContentsBounds();
   contents_bounds.Inset(0, g_top_shadow.Get().height(), 0,
                         g_bottom_shadow.Get().height());
 
   ui::ClipRecorder clip_recorder(context);
   clip_recorder.ClipRect(contents_bounds);
   {
     ui::PaintRecorder recorder(context, size());
     SkColor background_color = result_view_at(0)->GetColor(
         OmniboxResultView::NORMAL, OmniboxResultView::BACKGROUND);
@@ skipping 52 matching lines @@
   size_t index = GetIndexForPoint(event.location());
   if (!HasMatchAt(index))
     return;
   omnibox_view_->OpenMatch(model_->result().match_at(index), disposition,
                            GURL(), base::string16(), index);
 }
 
 OmniboxResultView* OmniboxPopupContentsView::result_view_at(size_t i) {
   return static_cast<OmniboxResultView*>(child_at(static_cast<int>(i)));
 }