MacViews: Add native drop shadow to dialogs on OSX < 10.10.

ui/views/bubble/bubble_frame_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 "ui/views/bubble/bubble_frame_view.h"
 
 #include <algorithm>
 #include <utility>
 
 #include "build/build_config.h"
@@ skipping 137 matching lines @@
       return HTSYSMENU;
     if (point.y() < title_->bounds().bottom())
       return HTCAPTION;
   }
 
   return GetWidget()->client_view()->NonClientHitTest(point);
 }
 
 void BubbleFrameView::GetWindowMask(const gfx::Size& size,
                                     gfx::Path* window_mask) {
-  // NOTE: this only provides implementations for the types used by dialogs.
-  if ((bubble_border_->arrow() != BubbleBorder::NONE &&
-       bubble_border_->arrow() != BubbleBorder::FLOAT) ||
-      (bubble_border_->shadow() != BubbleBorder::SMALL_SHADOW &&
-       bubble_border_->shadow() != BubbleBorder::NO_SHADOW_OPAQUE_BORDER))
+  if (bubble_border_->shadow() != BubbleBorder::SMALL_SHADOW &&
+      bubble_border_->shadow() != BubbleBorder::NO_SHADOW_OPAQUE_BORDER &&
+      bubble_border_->shadow() != BubbleBorder::NO_ASSETS)
+    return;
+
+  // We don't return a mask for windows with arrows unless they use
+  // BubbleBorder::NO_ASSETS.
+  if (bubble_border_->shadow() != BubbleBorder::NO_ASSETS &&
+      bubble_border_->arrow() != BubbleBorder::NONE &&
+      bubble_border_->arrow() != BubbleBorder::FLOAT)
     return;
 
   // Use a window mask roughly matching the border in the image assets.
-  static const int kBorderStrokeSize = 1;
-  static const SkScalar kCornerRadius = SkIntToScalar(6);

[msw, 2016/02/11 23:57:38]

Odd, the corner radii are 4 and 2; I wonder why this was 6?
https://code.google.com/p/chromium/codesearch#chromium/src/ui/views/bubble/bu...

[karandeepb, 2016/02/12 03:45:52]

Not sure about it either. 

+  const int kBorderStrokeSize =
+      bubble_border_->shadow() == BubbleBorder::NO_ASSETS ? 0 : 1;
+  const SkScalar kCornerRadius =
+      SkIntToScalar(bubble_border_->GetBorderCornerRadius());
   const gfx::Insets border_insets = bubble_border_->GetInsets();
-  SkRect rect = { SkIntToScalar(border_insets.left() - kBorderStrokeSize),
-                  SkIntToScalar(border_insets.top() - kBorderStrokeSize),
-                  SkIntToScalar(size.width() - border_insets.right() +
-                                kBorderStrokeSize),
-                  SkIntToScalar(size.height() - border_insets.bottom() +
-                                kBorderStrokeSize) };
-  if (bubble_border_->shadow() == BubbleBorder::NO_SHADOW_OPAQUE_BORDER) {
+  SkRect rect = {
+      SkIntToScalar(border_insets.left() - kBorderStrokeSize),
+      SkIntToScalar(border_insets.top() - kBorderStrokeSize),
+      SkIntToScalar(size.width() - border_insets.right() + kBorderStrokeSize),
+      SkIntToScalar(size.height() - border_insets.bottom() +
+                    kBorderStrokeSize)};
+
+  if (bubble_border_->shadow() == BubbleBorder::NO_SHADOW_OPAQUE_BORDER ||
+      bubble_border_->shadow() == BubbleBorder::NO_ASSETS) {
     window_mask->addRoundRect(rect, kCornerRadius, kCornerRadius);
   } else {
     static const int kBottomBorderShadowSize = 2;
     rect.fBottom += SkIntToScalar(kBottomBorderShadowSize);
     window_mask->addRect(rect);
   }
+  gfx::Path arrow_path;
+  if (bubble_border_->GetArrowPath(gfx::Rect(size), &arrow_path))
+    window_mask->addPath(arrow_path, 0, 0);
 }
 
 void BubbleFrameView::ResetWindowControls() {
   close_->SetVisible(GetWidget()->widget_delegate()->ShouldShowCloseButton());
 }
 
 void BubbleFrameView::UpdateWindowIcon() {
   gfx::ImageSkia image;
   if (GetWidget()->widget_delegate()->ShouldShowWindowIcon())
     image = GetWidget()->widget_delegate()->GetWindowIcon();
@@ skipping 314 matching lines @@
   size.Enlarge(client_insets.width(), client_insets.height());
   size.SetToMax(gfx::Size(title_bar_width, 0));
 
   if (footnote_container_)
     size.Enlarge(0, footnote_container_->GetHeightForWidth(size.width()));
 
   return size;
 }
 
 }  // namespace views