WebCore/platform/qt/ScrollbarThemeQt.cpp:35: Alphabetical sorting problem. [build/include_order] [4] WebCore/platform/qt/ScrollbarThemeQt.cpp:42: Alphabetical sorting problem. [build/include_order] [4] WebCore/platform/qt/ScrollbarThemeQt.cpp:59: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/platform/qt/ScrollbarThemeQt.cpp:81: A case label should not be indented, but line up with its switch statement. [whitespace/indent] [4] WebCore/platform/qt/ScrollbarThemeQt.cpp:125: 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] WebCore/platform/qt/ScrollbarThemeQt.cpp:126: 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] Total errors found: 6 in 1 files
Created attachment 57144 [details] Patch that fixes the coding style issues
QtTriaged
Comment on attachment 57144 [details] Patch that fixes the coding style issues > No new tests. (OOPS!) Ditto.
Created attachment 57939 [details] Patch
Comment on attachment 57939 [details] Patch LGTM.
Comment on attachment 57939 [details] Patch Clearing flags on attachment: 57939 Committed r60737: <http://trac.webkit.org/changeset/60737>
All reviewed patches have been landed. Closing bug.