[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