Summary: | Upstreaming gtests from chromium: UniscribeHelper and TransparencyWin | ||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Yaar Schnitman <yaar> | ||||||||||
Component: | New Bugs | Assignee: | Yaar Schnitman <yaar> | ||||||||||
Status: | RESOLVED FIXED | ||||||||||||
Severity: | Normal | CC: | commit-queue, fishd, webkit.review.bot, yurys | ||||||||||
Priority: | P2 | ||||||||||||
Version: | 528+ (Nightly build) | ||||||||||||
Hardware: | Other | ||||||||||||
OS: | OS X 10.5 | ||||||||||||
Attachments: |
|
Description
Yaar Schnitman
2010-02-02 18:33:50 PST
Created attachment 47985 [details]
Patch
Created attachment 47986 [details]
Patch 2
Attachment 47985 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/chromium/tests/TransparencyWinTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:36: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:38: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:35: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:36: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
Total errors found: 6
If any of these errors are false positives, please file a bug against check-webkit-style.
Attachment 47986 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/chromium/tests/TransparencyWinTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:36: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:38: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:35: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:36: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
Total errors found: 6
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 47986 [details] Patch 2 > +2010-02-02 Yaar Schnitman <yaar@chromium.org> > + > + Reviewed by NOBODY (OOPS!). > + > + Need a short description and bug URL (OOPS!) > + ^^^ please fill out the change log appropriately. otherwise, LGTM Created attachment 48080 [details]
Patch
Attachment 48080 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/chromium/tests/TransparencyWinTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:36: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:38: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:35: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:36: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
Total errors found: 6
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 48080 [details] Patch Clearing flags on attachment: 48080 Committed r54333: <http://trac.webkit.org/changeset/54333> All reviewed patches have been landed. Closing bug. (In reply to comment #9) > All reviewed patches have been landed. Closing bug. I reverted the change since it didn't compile on Chromium Win bot: http://build.webkit.org/builders/Chromium%20Win%20Release/builds/1862 Created attachment 48178 [details]
Patch
Attachment 48178 [details] did not pass style-queue:
Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebKit/chromium/tests/TransparencyWinTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:36: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/TransparencyWinTest.cpp:38: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:33: Found other header before a header this file implements. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:35: Alphabetical sorting problem. [build/include_order] [4]
WebKit/chromium/tests/UniscribeHelperTest.cpp:36: Found header this file implements after other header. Should be: config.h, primary header, blank line, and then alphabetically sorted. [build/include_order] [4]
Total errors found: 6
If any of these errors are false positives, please file a bug against check-webkit-style.
Comment on attachment 48178 [details] Patch Clearing flags on attachment: 48178 Committed r54410: <http://trac.webkit.org/changeset/54410> All reviewed patches have been landed. Closing bug. |