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

Side by Side Diff: Source/core/rendering/RenderScrollbarTheme.cpp

Issue 51673005: Remove unnecessary dependency upon ScrollView from the ScrollbarTheme classes. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink@master
Patch Set: Remove redefinition of scrollOffset Created 7 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
« no previous file with comments | « Source/core/rendering/RenderScrollbarTheme.h ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2008 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008 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 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
100 if (scrollbar->orientation() == HorizontalScrollbar) { 100 if (scrollbar->orientation() == HorizontalScrollbar) {
101 result.setX(backRect.x()); 101 result.setX(backRect.x());
102 result.setWidth(forwardRect.maxX() - backRect.x()); 102 result.setWidth(forwardRect.maxX() - backRect.x());
103 } else { 103 } else {
104 result.setY(backRect.y()); 104 result.setY(backRect.y());
105 result.setHeight(forwardRect.maxY() - backRect.y()); 105 result.setHeight(forwardRect.maxY() - backRect.y());
106 } 106 }
107 return result; 107 return result;
108 } 108 }
109 109
110 void RenderScrollbarTheme::paintScrollCorner(ScrollView*, GraphicsContext* conte xt, const IntRect& cornerRect) 110 void RenderScrollbarTheme::paintScrollCorner(GraphicsContext* context, const Int Rect& cornerRect)
111 { 111 {
112 // FIXME: Implement. 112 // FIXME: Implement.
113 context->fillRect(cornerRect, Color::white); 113 context->fillRect(cornerRect, Color::white);
114 } 114 }
115 115
116 void RenderScrollbarTheme::paintScrollbarBackground(GraphicsContext* context, Sc rollbarThemeClient* scrollbar) 116 void RenderScrollbarTheme::paintScrollbarBackground(GraphicsContext* context, Sc rollbarThemeClient* scrollbar)
117 { 117 {
118 toRenderScrollbar(scrollbar)->paintPart(context, ScrollbarBGPart, scrollbar- >frameRect()); 118 toRenderScrollbar(scrollbar)->paintPart(context, ScrollbarBGPart, scrollbar- >frameRect());
119 } 119 }
120 120
(...skipping 16 matching lines...) Expand all
137 { 137 {
138 toRenderScrollbar(scrollbar)->paintPart(context, ThumbPart, rect); 138 toRenderScrollbar(scrollbar)->paintPart(context, ThumbPart, rect);
139 } 139 }
140 140
141 void RenderScrollbarTheme::paintTickmarks(GraphicsContext* context, ScrollbarThe meClient* scrollbar, const IntRect& rect) 141 void RenderScrollbarTheme::paintTickmarks(GraphicsContext* context, ScrollbarThe meClient* scrollbar, const IntRect& rect)
142 { 142 {
143 ScrollbarTheme::theme()->paintTickmarks(context, scrollbar, rect); 143 ScrollbarTheme::theme()->paintTickmarks(context, scrollbar, rect);
144 } 144 }
145 145
146 } 146 }
OLDNEW
« no previous file with comments | « Source/core/rendering/RenderScrollbarTheme.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698