Stop using ScrollbarThemeClient and just use Scrollbar directly
Created attachment 255567 [details] Patch
Attachment 255567 [details] did not pass style-queue: ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:110: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:111: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:114: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:110: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:111: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:117: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:118: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:117: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:118: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:279: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:280: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:281: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:293: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:294: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:293: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:152: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:158: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/Scrollbar.cpp:342: Wrong number of spaces before statement. (expected: 26) [whitespace/indent] [4] ERROR: Source/WebCore/platform/Scrollbar.cpp:350: Wrong number of spaces before statement. (expected: 26) [whitespace/indent] [4] ERROR: Source/WebCore/platform/Scrollbar.cpp:342: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/Scrollbar.cpp:410: An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:149: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:151: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:153: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:234: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:234: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:235: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:290: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:291: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Total errors found: 30 in 25 files If any of these errors are false positives, please file a bug against check-webkit-style.
Created attachment 255568 [details] Patch
Attachment 255568 [details] did not pass style-queue: ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:110: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:111: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:114: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:110: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:111: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:117: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:118: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:117: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeSafari.cpp:118: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:279: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:280: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:281: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:293: Should be indented on a separate line, with the colon or comma first on that line. [whitespace/indent] [4] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:294: Wrong number of spaces before statement. (expected: 17) [whitespace/indent] [4] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:293: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/mac/ScrollbarThemeMac.mm:294: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:152: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/gtk/ScrollbarThemeGtk.cpp:158: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/Scrollbar.cpp:342: Wrong number of spaces before statement. (expected: 26) [whitespace/indent] [4] ERROR: Source/WebCore/platform/Scrollbar.cpp:350: Wrong number of spaces before statement. (expected: 26) [whitespace/indent] [4] ERROR: Source/WebCore/platform/Scrollbar.cpp:342: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/Scrollbar.cpp:410: An else if statement should be written as an if statement when the prior "if" concludes with a return, break, continue or goto statement. [readability/control_flow] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:149: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:151: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:153: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:234: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:234: Multi line control clauses should use braces. [whitespace/braces] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:235: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:290: Boolean expressions that span multiple lines should have their operators on the left side of the line instead of the right side. [whitespace/operators] [4] ERROR: Source/WebCore/platform/win/ScrollbarThemeWin.cpp:291: Weird number of spaces at line-start. Are you using a 4-space indent? [whitespace/indent] [3] Total errors found: 30 in 25 files If any of these errors are false positives, please file a bug against check-webkit-style.
Committed r185964: <http://trac.webkit.org/changeset/185964>
This broke the windows build.
(In reply to comment #6) > This broke the windows build. Brent landed the fix in https://trac.webkit.org/changeset/185975. One more reason why we need a working Windows EWS. But we don't have it 2 weeks ago. It seems solving a network issue needs at least a months for Apple. :)
(In reply to comment #7) > But we don't have it 2 weeks ago. It seems solving > a network issue needs at least a months for Apple. :) That's so much for your always helpful comments, Csaba.
(In reply to comment #8) > (In reply to comment #7) > > > But we don't have it 2 weeks ago. It seems solving > > a network issue needs at least a months for Apple. :) > > That's so much for your always helpful comments, Csaba. I didn't want to add a helpful comment here, but a strong note. It's simply ridiculous that 2 weeks weren't enough to make developer.apple.com available for Apple Windows EWS bots ...
(In reply to comment #9) > (In reply to comment #8) > > (In reply to comment #7) > > > > > But we don't have it 2 weeks ago. It seems solving > > > a network issue needs at least a months for Apple. :) > > > > That's so much for your always helpful comments, Csaba. > > I didn't want to add a helpful comment here, but a strong note. > It's simply ridiculous that 2 weeks weren't enough to make > developer.apple.com available for Apple Windows EWS bots ... Don't you think it's better to bring that up on one of our mailing lists instead of in a random bugzilla bug?
(In reply to comment #10) > Don't you think it's better to bring that up on one of our mailing lists > instead of in a random bugzilla bug? I did write private mails to the bot maintainers twice in the latest two weeks, but nothing happened only a promise that it will be fixed in the (far) future. What kind of random bug do you talk? I commented this bug, because this patch broke the Windows build because of the lack of working Windows EWS.
(In reply to comment #11) > (In reply to comment #10) > > Don't you think it's better to bring that up on one of our mailing lists > > instead of in a random bugzilla bug? > > I did write private mails to the bot maintainers twice in the latest two > weeks, > but nothing happened only a promise that it will be fixed in the (far) > future. > > What kind of random bug do you talk? I commented this bug, because this > patch broke the Windows build because of the lack of working Windows EWS. And you're just wasting your (as well as my) time because I can't do anything about it.