Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(47)

Side by Side Diff: third_party/WebKit/Source/platform/scroll/ScrollbarThemeMac.mm

Issue 1863513002: Mac: Remove some remaining 10.6 logic for absent overlay scrollbar APIs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2008, 2011 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008, 2011 Apple Inc. All Rights Reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 204 matching lines...) Expand 10 before | Expand all | Expand 10 after
215 gPreferredScrollerStyle = preferredScrollerStyle; 215 gPreferredScrollerStyle = preferredScrollerStyle;
216 if (redraw && !scrollbarSet().isEmpty()) { 216 if (redraw && !scrollbarSet().isEmpty()) {
217 ScrollbarSet::iterator end = scrollbarSet().end(); 217 ScrollbarSet::iterator end = scrollbarSet().end();
218 for (ScrollbarSet::iterator it = scrollbarSet().begin(); it != end; ++it ) { 218 for (ScrollbarSet::iterator it = scrollbarSet().begin(); it != end; ++it ) {
219 (*it)->styleChanged(); 219 (*it)->styleChanged();
220 (*it)->invalidate(); 220 (*it)->invalidate();
221 } 221 }
222 } 222 }
223 if (sendScrollerStyleNotification) { 223 if (sendScrollerStyleNotification) {
224 [[NSNotificationCenter defaultCenter] 224 [[NSNotificationCenter defaultCenter]
225 postNotificationName:@"NSPreferredScrollerStyleDidChangeNotification " 225 postNotificationName:NSPreferredScrollerStyleDidChangeNotification
tapted 2016/04/05 09:06:02 nothing in blink seems to listen for this - maybe
Avi (use Gerrit) 2016/04/05 11:26:32 delete away.
Nico 2016/04/05 12:30:30 Does chrome do the right thing if you change the p
tapted 2016/04/05 12:35:17 Yup - the listener is at content/browser/theme_hel
tapted 2016/04/05 12:53:13 Done.
226 object:nil 226 object:nil
227 userInfo:@{ @"NSScrollerStyle" : @(gPreferredScrollerSty le) }]; 227 userInfo:@{ @"NSScrollerStyle" : @(gPreferredScrollerSty le) }];
228 } 228 }
229 } 229 }
230 230
231 double ScrollbarThemeMac::initialAutoscrollTimerDelay() 231 double ScrollbarThemeMac::initialAutoscrollTimerDelay()
232 { 232 {
233 return gInitialButtonDelay; 233 return gInitialButtonDelay;
234 } 234 }
235 235
(...skipping 199 matching lines...) Expand 10 before | Expand all | Expand 10 after
435 435
436 // static 436 // static
437 NSScrollerStyle ScrollbarThemeMac::recommendedScrollerStyle() 437 NSScrollerStyle ScrollbarThemeMac::recommendedScrollerStyle()
438 { 438 {
439 if (RuntimeEnabledFeatures::overlayScrollbarsEnabled()) 439 if (RuntimeEnabledFeatures::overlayScrollbarsEnabled())
440 return NSScrollerStyleOverlay; 440 return NSScrollerStyleOverlay;
441 return gPreferredScrollerStyle; 441 return gPreferredScrollerStyle;
442 } 442 }
443 443
444 } // namespace blink 444 } // namespace blink
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/platform/mac/NSScrollerImpDetails.h ('k') | ui/base/test/scoped_preferred_scroller_style_mac.mm » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698