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

Side by Side Diff: third_party/WebKit/Source/platform/scroll/ScrollbarThemeMacNonOverlayAPI.h

Issue 1511003003: Use refs for non-null GraphicsContext, Scrollbar, etc. in scrollbar related code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@ScrollbarRemove
Patch Set: yet another mac fix Created 5 years 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) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google 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 are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 25 matching lines...) Expand all
36 namespace blink { 36 namespace blink {
37 37
38 class CullRect; 38 class CullRect;
39 39
40 class PLATFORM_EXPORT ScrollbarThemeMacNonOverlayAPI : public ScrollbarThemeMacC ommon { 40 class PLATFORM_EXPORT ScrollbarThemeMacNonOverlayAPI : public ScrollbarThemeMacC ommon {
41 public: 41 public:
42 int scrollbarThickness(ScrollbarControlSize = RegularScrollbar) override; 42 int scrollbarThickness(ScrollbarControlSize = RegularScrollbar) override;
43 bool usesOverlayScrollbars() const override { return false; } 43 bool usesOverlayScrollbars() const override { return false; }
44 ScrollbarButtonsPlacement buttonsPlacement() const override; 44 ScrollbarButtonsPlacement buttonsPlacement() const override;
45 45
46 bool paint(const ScrollbarThemeClient*, GraphicsContext*, const CullRect&) o verride; 46 bool paint(const ScrollbarThemeClient&, GraphicsContext&, const CullRect&) o verride;
47 bool invalidateOnWindowActiveChange() const override { return true; } 47 bool invalidateOnWindowActiveChange() const override { return true; }
48 48
49 protected: 49 protected:
50 IntRect trackRect(const ScrollbarThemeClient*, bool painting = false) overri de; 50 IntRect trackRect(const ScrollbarThemeClient&, bool painting = false) overri de;
51 IntRect backButtonRect(const ScrollbarThemeClient*, ScrollbarPart, bool pain ting = false) override; 51 IntRect backButtonRect(const ScrollbarThemeClient&, ScrollbarPart, bool pain ting = false) override;
52 IntRect forwardButtonRect(const ScrollbarThemeClient*, ScrollbarPart, bool p ainting = false) override; 52 IntRect forwardButtonRect(const ScrollbarThemeClient&, ScrollbarPart, bool p ainting = false) override;
53 53
54 void updateButtonPlacement(ScrollbarButtonsPlacement) override; 54 void updateButtonPlacement(ScrollbarButtonsPlacement) override;
55 55
56 bool hasButtons(const ScrollbarThemeClient*) override; 56 bool hasButtons(const ScrollbarThemeClient&) override;
57 bool hasThumb(const ScrollbarThemeClient*) override; 57 bool hasThumb(const ScrollbarThemeClient&) override;
58 58
59 int minimumThumbLength(const ScrollbarThemeClient*) override; 59 int minimumThumbLength(const ScrollbarThemeClient&) override;
60 }; 60 };
61 61
62 } 62 }
63 63
64 #endif 64 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698