MacViews: Clip contents for non-rectangular windows.

ui/views/cocoa/bridged_native_widget.mm

 // 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.
 
 #import "ui/views/cocoa/bridged_native_widget.h"
 
+#import <Quartz/Quartz.h>

[tapted, 2016/02/24 05:05:03]

this can be grouped with the other <> includes, since they are all "operating
system headers" rather than STL/standard-library headers

[karandeepb, 2016/02/25 03:31:06]

Done. Aren't stdint.h and stddef.h standard library headers?

[tapted, 2016/02/25 08:07:21]

I guess it's a bit arbitrary, but I generally see it applied as "group stuff
with a '.' in the header, and group stuff like <vector><string>,etc."

+
 #import <objc/runtime.h>
 #include <stddef.h>
 #include <stdint.h>
 
 #include "base/logging.h"
 #import "base/mac/foundation_util.h"
 #include "base/mac/mac_util.h"
 #import "base/mac/sdk_forward_declarations.h"
 #include "base/thread_task_runner_handle.h"
 #include "ui/accelerated_widget_mac/window_resize_helper_mac.h"
 #import "ui/base/cocoa/constrained_window/constrained_window_animation.h"
 #include "ui/base/hit_test.h"
 #include "ui/base/ime/input_method.h"
 #include "ui/base/ime/input_method_factory.h"
 #include "ui/gfx/display.h"
 #include "ui/gfx/geometry/dip_util.h"
 #import "ui/gfx/mac/coordinate_conversion.h"
 #import "ui/gfx/mac/nswindow_frame_controls.h"
+#import "ui/gfx/path_mac.h"
 #include "ui/gfx/screen.h"
 #import "ui/views/cocoa/bridged_content_view.h"
 #import "ui/views/cocoa/cocoa_mouse_capture.h"
 #include "ui/views/cocoa/tooltip_manager_mac.h"
 #import "ui/views/cocoa/views_nswindow_delegate.h"
 #import "ui/views/cocoa/widget_owner_nswindow_adapter.h"
 #include "ui/views/view.h"
 #include "ui/views/views_delegate.h"
 #include "ui/views/widget/native_widget_mac.h"
 #include "ui/views/widget/widget.h"
@@ skipping 641 matching lines @@
 
 void BridgedNativeWidget::OnSizeChanged() {
   gfx::Size new_size = GetClientAreaSize();
   native_widget_mac_->GetWidget()->OnNativeWidgetSizeChanged(new_size);
   if (layer()) {
     UpdateLayerProperties();
     if ([window_ inLiveResize])
       MaybeWaitForFrame(new_size);
   }
 
-  // 10.9 is unable to generate a window shadow from the composited CALayer, so
-  // use Quartz.
-  // We don't update the window mask during a live resize, instead it is done
-  // after the resize is completed in viewDidEndLiveResize: in
-  // BridgedContentView.
-  if (base::mac::IsOSMavericksOrEarlier() && ![window_ inLiveResize])
-    [bridged_view_ updateWindowMask];
+  [bridged_view_ onSizeChanged];

[tapted, 2016/02/24 05:05:03]

Yeah, I'd call the method updateWindowMask and invoke it from UpdateMaskPath()
instead of here

[karandeepb, 2016/02/25 03:31:06]

Done.

+
+  // Update mask path to match the bridged view's new window mask.
+  UpdateMaskPath();
 }
 
 void BridgedNativeWidget::OnVisibilityChanged() {
   OnVisibilityChangedTo([window_ isVisible]);
 }
 
 void BridgedNativeWidget::OnVisibilityChangedTo(bool new_visibility) {
   if (window_visible_ == new_visibility)
     return;
 
@@ skipping 534 matching lines @@
 void BridgedNativeWidget::SetDraggable(bool draggable) {
   [bridged_view_ setMouseDownCanMoveWindow:draggable];
   // AppKit will not update its cache of mouseDownCanMoveWindow unless something
   // changes. Previously we tried adding an NSView and removing it, but for some
   // reason it required reposting the mouse-down event, and didn't always work.
   // Calling the below seems to be an effective solution.
   [window_ setMovableByWindowBackground:NO];
   [window_ setMovableByWindowBackground:YES];
 }
 
+void BridgedNativeWidget::UpdateMaskPath() {
+  // We only set the mask layer for windows with non-rectangular masks.
+  if (![bridged_view_ windowMask])

[tapted, 2016/02/24 05:05:03]

perhaps after this, do

{
  // Removing a mask after adding one isn't currently supported.
  DCHECK(![[compositor_superview_ layer] mask]);
  return;
}

[karandeepb, 2016/02/25 03:31:06]

Done.

+    return;
+
+  CAShapeLayer* mask_layer = base::mac::ObjCCastStrict<CAShapeLayer>(
+      [[compositor_superview_ layer] mask]);
+  if (mask_layer) {
+    [mask_layer
+        setPath:gfx::CreateCGPathFromNSBezierPath([bridged_view_ windowMask])];
+    return;
+  }
+
+  // If mask layer does not exist yet, create one and set it as the background

[karandeepb, 2016/02/24 04:27:18]

Another place to initialize the mask layer could be through
NativeWidgetMac::OnWidgetInitDone since by then the non client frame view would
have been attached to the widget.

[tapted, 2016/02/24 05:05:03]

I think this here is OK. E.g. it's feasible that we may want to support removing
the mask later on if the client frame view changes its mind about things after
it gets shown.

[karandeepb, 2016/02/25 03:31:06]

Done.

+  // layer's mask. Clipping will be done based on the alpha channel of the mask
+  // layer.
+  base::scoped_nsobject<CAShapeLayer> mask([[CAShapeLayer alloc] init]);
+  [mask setPath:gfx::CreateCGPathFromNSBezierPath([bridged_view_ windowMask])];
+  [mask setBackgroundColor:[[NSColor clearColor] CGColor]];

[tapted, 2016/02/24 05:05:03]

There's a CGColor constant for this (also I think this is default? So might not
need to be specified)

[karandeepb, 2016/02/25 03:31:06]

Done.

+
+  CALayer* background_layer = [compositor_superview_ layer];
+  [background_layer setMask:mask];
+  [background_layer setMasksToBounds:YES];
+}
+
 }  // namespace views