MacViews: Overlay Scrollbars with Show/Hide Animations

ui/views/cocoa/views_scrollbar_bridge.mm

+// Copyright 2016 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/views_scrollbar_bridge.h"
+
+#include "base/mac/mac_util.h"
+#include "base/mac/sdk_forward_declarations.h"
+
+@interface ViewsScrollbarBridge ()
+
+// Called when we receive a NSPreferredScrollerStyleDidChangeNotification.
+- (void)onScrollerStyleChanged:(NSNotification*)notification;
+
+@end
+
+@implementation ViewsScrollbarBridge
+
+- (void)setDelegate:(ViewsScrollbarBridgeDelegate*)delegate {
+  DCHECK_NE(delegate, delegate_);
+  if (!delegate_) {
+    [[NSNotificationCenter defaultCenter]
+        removeObserver:self
+                  name:NSPreferredScrollerStyleDidChangeNotification
+                object:nil];
+    return;
+  }
+  [[NSNotificationCenter defaultCenter]
+      addObserver:self
+         selector:@selector(onScrollerStyleChanged:)
+             name:NSPreferredScrollerStyleDidChangeNotification
+             object:nil];
+}
+
+- (void)dealloc {
+  DCHECK(!delegate_);
+  [[NSNotificationCenter defaultCenter] removeObserver:self];

[spqchan, 2016/02/09 21:21:26]

This might be unnecessary? Especially since we're removing it in setDelegate

[tapted, 2016/02/09 23:51:48]

Yeah - I think it's redundant with the DCHECK above. But I think we can use this
simpler form of removeObserver in setDelegate.

[spqchan, 2016/02/10 00:35:16]

Gotcha, removed it

+  [super dealloc];
+}
+
+- (void)onScrollerStyleChanged:(NSNotification*)notification {
+  delegate_->OnScrollerStyleChanged();
+}
+
+- (NSScrollerStyle)getPreferredScrollerStyle {
+  if (![NSScroller respondsToSelector:@selector(preferredScrollerStyle)]) {
+    DCHECK(base::mac::IsOSSnowLeopard());
+    return NSScrollerStyleLegacy;
+  }
+  return [NSScroller preferredScrollerStyle];
+}
+
+@end