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"
+#import "base/mac/sdk_forward_declarations.h"
+
+@interface ViewsScrollbarBridge ()
+
+// Called when we receive a NSPreferredScrollerStyleDidChangeNotification.
+- (void)onScrollerStyleChanged:(NSNotification*)notification;
+
+@end
+
+@implementation ViewsScrollbarBridge
+
+- (id)initWithDelegate:(ViewsScrollbarBridgeDelegate*)delegate {
+  if ((self = [super init])) {
+    delegate_ = delegate;
+    [[NSNotificationCenter defaultCenter]
+        addObserver:self
+           selector:@selector(onScrollerStyleChanged:)
+               name:NSPreferredScrollerStyleDidChangeNotification
+             object:nil];
+  }
+  return self;
+}
+
+-(void)clearDelegate {

[tapted, 2016/02/22 23:41:55]

nit: space after `-`

[spqchan, 2016/02/23 17:56:27]

Done.

+  delegate_ = nullptr;
+  [[NSNotificationCenter defaultCenter] removeObserver:self];
+}
+
+- (void)dealloc {

[tapted, 2016/02/22 23:41:55]

nit: move dealloc up -- should always come after initializers

[spqchan, 2016/02/23 17:56:27]

Done.

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