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

Side by Side Diff: third_party/WebKit/public/platform/WebThemeEngine.h

Issue 2467693002: Implement overlay scrollbar fade out for non-composited scrollers. (Closed)
Patch Set: overlay-scrollbar-mouse-capture now works on Mac Created 4 years, 1 month 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) 2010 Google Inc. All rights reserved. 2 * Copyright (C) 2010 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 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 // like vertical scrollbar thumbs, the width will be the required width of 158 // like vertical scrollbar thumbs, the width will be the required width of
159 // the track while the height will be the minimum height. 159 // the track while the height will be the minimum height.
160 virtual WebSize getSize(Part) { return WebSize(); } 160 virtual WebSize getSize(Part) { return WebSize(); }
161 161
162 struct ScrollbarStyle { 162 struct ScrollbarStyle {
163 int thumbThickness; 163 int thumbThickness;
164 int scrollbarMargin; 164 int scrollbarMargin;
165 WebColor color; 165 WebColor color;
166 }; 166 };
167 167
168 virtual double getOverlayScrollbarFadeOutDelaySeconds() { return 0.0; }
skobes 2016/11/02 21:02:43 Could these just be fields in ScrollbarStyle and r
bokan 2016/11/02 22:41:57 Yup, done. I also added a TODO to set those member
169 virtual double getOverlayScrollbarFadeOutDurationSeconds() { return 0.0; }
170
168 // Gets the overlay scrollbar style. Used for mobile theme. 171 // Gets the overlay scrollbar style. Used for mobile theme.
169 virtual void getOverlayScrollbarStyle(ScrollbarStyle*) {} 172 virtual void getOverlayScrollbarStyle(ScrollbarStyle*) {}
170 173
171 // Paint the given the given theme part. 174 // Paint the given the given theme part.
172 virtual void paint(WebCanvas*, 175 virtual void paint(WebCanvas*,
173 Part, 176 Part,
174 State, 177 State,
175 const WebRect&, 178 const WebRect&,
176 const ExtraParams*) {} 179 const ExtraParams*) {}
177 }; 180 };
178 181
179 } // namespace blink 182 } // namespace blink
180 183
181 #endif 184 #endif
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698