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

Unified Diff: third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp

Issue 1560403002: Scale scrollbar in use-zoom-for-dsf mode (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 11 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp
diff --git a/third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp b/third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp
index 837aece81427cb9623f5e964f16c865ec811550f..79018ed16d108ccd9db8ebd8e0529533a4618331 100644
--- a/third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp
+++ b/third_party/WebKit/Source/platform/scroll/ScrollbarThemeNonMacCommon.cpp
@@ -30,7 +30,6 @@
#include "platform/graphics/GraphicsContextStateSaver.h"
#include "platform/graphics/paint/DrawingRecorder.h"
#include "platform/scroll/ScrollableArea.h"
-#include "platform/scroll/ScrollbarThemeClient.h"
namespace blink {
@@ -73,11 +72,14 @@ IntRect ScrollbarThemeNonMacCommon::trackRect(const ScrollbarThemeClient& scroll
{
// The track occupies all space between the two buttons.
IntSize bs = buttonSize(scrollbar);
- int thickness = scrollbarThickness(scrollbar.controlSize());
skobes 2016/01/07 21:49:10 The other caller of ScrollbarThemeClient::controlS
oshima 2016/01/09 00:58:39 Other uses seems to be for custom scrollbars, whic
+ int thickness;
if (scrollbar.orientation() == HorizontalScrollbar) {
+ thickness = scrollbar.height();
if (scrollbar.width() <= 2 * bs.width())
return IntRect();
return IntRect(scrollbar.x() + bs.width(), scrollbar.y(), scrollbar.width() - 2 * bs.width(), thickness);
+ } else {
skobes 2016/01/07 21:49:10 Remove the "else" since we return in the if block.
oshima 2016/01/09 00:58:39 Done.
+ thickness = scrollbar.width();
}
if (scrollbar.height() <= 2 * bs.height())
return IntRect();

Powered by Google App Engine
This is Rietveld 408576698