Bug 80852

Summary: [Chromium-android] Add unit test for Android layout test script (chromium_android.py)
Product: WebKit Reporter: Xianzhu Wang <wangxianzhu>
Component: Tools / TestsAssignee: Hao Zheng <zhenghao>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dpranke, jnd, ojan, tony, webkit.review.bot, zhenghao
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 66687    
Attachments:
Description Flags
Patch none

Xianzhu Wang
Reported 2012-03-12 09:42:04 PDT
<EOM>
Attachments
Patch (7.71 KB, patch)
2012-05-09 01:06 PDT, Hao Zheng 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Hao Zheng
Comment 1 2012-05-09 01:06:42 PDT
Created attachment 140882 [details] Patch
Adam Barth
Comment 2 2012-05-09 08:17:25 PDT
Comment on attachment 140882 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=140882&action=review Thanks for the tests! > Tools/Scripts/webkitpy/layout_tests/port/chromium_android_unittest.py:121 > +if __name__ == '__main__': > + unittest.main() We usually skip these in new unit test files.
WebKit Review Bot
Comment 3 2012-05-09 08:27:16 PDT
Comment on attachment 140882 [details] Patch Clearing flags on attachment: 140882 Committed r116526: <http://trac.webkit.org/changeset/116526>
WebKit Review Bot
Comment 4 2012-05-09 08:27:21 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.