Cache results from respondsToSelector.

Source/core/platform/mac/NSScrollerImpDetails.mm

 /*
  * Copyright (C) 2011 Apple Inc. All Rights Reserved.
  * Copyright (C) 2011 Google Inc. All Rights Reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 11 matching lines @@
  * OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 
 #include "config.h"
 #include "RuntimeEnabledFeatures.h"
 #include "core/platform/mac/NSScrollerImpDetails.h"
 
+namespace {
+
+// Declare notification names from the 10.7 SDK.
+#if !defined(MAC_OS_X_VERSION_10_7) || MAC_OS_X_VERSION_MAX_ALLOWED < MAC_OS_X_VERSION_10_7
+NSString* NSPreferredScrollerStyleDidChangeNotification = @"NSPreferredScrollerStyleDidChangeNotification";
+#endif
+
+// Storing the current NSScrollerStyle as a global is appreciably faster than
+// having it be a property of ScrollerStylerObserver.
+NSScrollerStyle g_scrollerStyle = NSScrollerStyleLegacy;
+
+}  // anonymous namespace
+
+@interface ScrollerStyleObserver : NSObject
+- (id)init;
+- (void)preferredScrollerStyleDidChange:(NSNotification*)notification;
+@end
+
+@implementation ScrollerStyleObserver
+- (id)init
+{
+    if ((self = [super init])) {
+        if ([NSScroller respondsToSelector:@selector(preferredScrollerStyle)]) {
+            g_scrollerStyle = [NSScroller preferredScrollerStyle];
+            NSNotificationCenter* center = [NSNotificationCenter defaultCenter];
+            [center addObserver:self
+                       selector:@selector(preferredScrollerStyleDidChange:)
+                           name:NSPreferredScrollerStyleDidChangeNotification
+                         object:nil];
+        }
+    }
+    return self;
+}
+
+- (void)preferredScrollerStyleDidChange:(NSNotification*)notification
+{
+    g_scrollerStyle = [NSScroller preferredScrollerStyle];
+}
+@end
+
 namespace WebCore {
 
 bool isScrollbarOverlayAPIAvailable()
 {
-    static bool apiAvailable;
-    static bool shouldInitialize = true;
-    if (shouldInitialize) {
-        shouldInitialize = false;
-        Class scrollerImpClass = NSClassFromString(@"NSScrollerImp");
-        Class scrollerImpPairClass = NSClassFromString(@"NSScrollerImpPair");
-        apiAvailable = [scrollerImpClass respondsToSelector:@selector(scrollerImpWithStyle:controlSize:horizontal:replacingScrollerImp:)]
-                       && [scrollerImpPairClass instancesRespondToSelector:@selector(scrollerStyle)];
-    }
+    static bool apiAvailable =
+        [NSClassFromString(@"NSScrollerImp") respondsToSelector:@selector(scrollerImpWithStyle:controlSize:horizontal:replacingScrollerImp:)]
+        && [NSClassFromString(@"NSScrollerImpPair") instancesRespondToSelector:@selector(scrollerStyle)];
     return apiAvailable;
 }
 
-NSScrollerStyle recommendedScrollerStyle() {
+NSScrollerStyle recommendedScrollerStyle()
+{
     if (RuntimeEnabledFeatures::overlayScrollbarsEnabled())
         return NSScrollerStyleOverlay;
-    if ([NSScroller respondsToSelector:@selector(preferredScrollerStyle)])
-        return [NSScroller preferredScrollerStyle];
-    return NSScrollerStyleLegacy;
+
+    // The ScrollerStyleObserver will update g_scrollerStyle at init and when needed.

[Nico, 2013/10/15 22:20:44]

(might want to add a "this function is hot" comment the next time you're here)

+    static ScrollerStyleObserver* scrollerStyleObserver = [[ScrollerStyleObserver alloc] init];
+    return g_scrollerStyle;
 }
 
 }