[webkit-reviews] review denied: [Bug 40848] [Chromium] Show scrollbar steppers on Linux : [Attachment 59260] Fix code style.
bugzilla-daemon at webkit.org
bugzilla-daemon at webkit.org
Tue Jun 22 00:00:56 PDT 2010
Kent Tamura <tkent at chromium.org> has denied xiyuan <xiyuan at chromium.org>'s
request for review:
Bug 40848: [Chromium] Show scrollbar steppers on Linux
https://bugs.webkit.org/show_bug.cgi?id=40848
Attachment 59260: Fix code style.
https://bugs.webkit.org/attachment.cgi?id=59260&action=review
------- Additional Comments from Kent Tamura <tkent at chromium.org>
WebCore/platform/chromium/ScrollbarThemeChromiumLinux.cpp:86
+ drawHorizLine(canvas, rect.x(), right, rect.y(), paint);
Indentatio is incorrect.
WebCore/platform/chromium/ScrollbarThemeChromiumLinux.cpp:88
+ drawVertLine(canvas, right, rect.y(), bottom, paint);
ditto.
WebCore/platform/chromium/ScrollbarThemeChromiumLinux.cpp:90
+ drawHorizLine(canvas, rect.x(), right, bottom, paint);
ditto.
WebCore/platform/chromium/ScrollbarThemeChromiumLinux.cpp:92
+ drawVertLine(canvas, rect.x(), rect.y(), bottom, paint);
ditto.
I also think a screenshot is helpful.
More information about the webkit-reviews
mailing list