Bug 90438

Summary: nested fragment parser test slow on Android
Product: WebKit Reporter: Wei James (wistoch) <james.wei>
Component: Tools / TestsAssignee: Wei James (wistoch) <james.wei>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, wangxianzhu, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Android   
OS: Android   
Attachments:
Description Flags
Patch
none
rebase for landing none

Wei James (wistoch)
Reported 2012-07-03 01:20:42 PDT
nested fragment parser test slow on Android
Attachments
Patch (1.54 KB, patch)
2012-07-03 01:21 PDT, Wei James (wistoch) 		no flags
DetailsFormatted DiffDiff
rebase for landing (1.54 KB, patch)
2012-08-08 18:36 PDT, Wei James (wistoch) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Wei James (wistoch)
Comment 1 2012-07-03 01:21:11 PDT
Created attachment 150555 [details] Patch
Wei James (wistoch)
Comment 2 2012-07-03 01:24:50 PDT
when running layout test for chromium android, I found fast/parser/nested-fragment-parser-crash.html TIMEOUT. after invesitigation, I found the root cause should be the running time on Android is too long. It will need about 10 seconds to complete the test. adding this line in the TestExpectations works for this case.
WebKit Review Bot
Comment 3 2012-08-08 08:12:06 PDT
Comment on attachment 150555 [details] Patch Rejecting attachment 150555 [details] from commit-queue. Failed to run "['/mnt/git/webkit-commit-queue/Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '-..." exit_code: 2 Last 500 characters of output: code: 1 cwd: /mnt/git/webkit-commit-queue/ Parsed 2 diffs from patch file(s). patching file LayoutTests/ChangeLog Hunk #1 succeeded at 1 with fuzz 3. patching file LayoutTests/platform/chromium/TestExpectations Hunk #1 FAILED at 11. 1 out of 1 hunk FAILED -- saving rejects to file LayoutTests/platform/chromium/TestExpectations.rej Failed to run "[u'/mnt/git/webkit-commit-queue/Tools/Scripts/svn-apply', u'--force', u'--reviewer', u'Ryosuke Ni..." exit_code: 1 cwd: /mnt/git/webkit-commit-queue/ Full output: http://queues.webkit.org/results/13456508
Wei James (wistoch)
Comment 4 2012-08-08 18:36:55 PDT
Created attachment 157361 [details] rebase for landing
WebKit Review Bot
Comment 5 2012-08-08 21:26:59 PDT
Comment on attachment 157361 [details] rebase for landing Clearing flags on attachment: 157361 Committed r125148: <http://trac.webkit.org/changeset/125148>
WebKit Review Bot
Comment 6 2012-08-08 21:27:03 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.