Bug 103585

Summary: run-perf-tests --chromium-android fails due to Skia INFO logging
Product: WebKit Reporter: Eric Seidel (no email) <eric>
Component: New BugsAssignee: Eric Seidel (no email) <eric>
Status: RESOLVED FIXED    
Severity: Normal CC: abarth, dpranke, peter, rniwa, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Eric Seidel (no email)
Reported 2012-11-28 17:44:45 PST
run-perf-tests --chromium-android fails due to Skia INFO logging
Attachments
Patch (3.04 KB, patch)
2012-11-28 23:41 PST, Eric Seidel (no email) 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Eric Seidel (no email)
Comment 1 2012-11-28 17:46:21 PST
[INFO:SkFontHost_android.cpp(1158)] Use Test Config File Main /data/local/tmp/drt/android_main_fonts.xml, Fallback /data/local/tmp/drt/android_fallback_fonts.xml, Font Dir /data/local/tmp/drt/fonts/ is the log which was causing the failure.
Ryosuke Niwa
Comment 2 2012-11-28 17:54:47 PST
We can add a regex to _lines_to_ignore_in_stderr to ignore this.
Adam Barth
Comment 3 2012-11-28 18:19:07 PST
Yeah, I showed him how we handled the other cases. I think he's got a patch locally.
Eric Seidel (no email)
Comment 4 2012-11-28 22:57:16 PST
Oh, I thought I uploaded this. I think I may have reverted the patch. I'll just re-write it.
Eric Seidel (no email)
Comment 5 2012-11-28 23:41:23 PST
Created attachment 176658 [details] Patch
Adam Barth
Comment 6 2012-11-29 08:45:47 PST
Comment on attachment 176658 [details] Patch rniwa didn't want us to skip all the WARNINGs, but skipping all the INFOs seems fine.
WebKit Review Bot
Comment 7 2012-11-29 08:51:36 PST
Comment on attachment 176658 [details] Patch Clearing flags on attachment: 176658 Committed r136134: <http://trac.webkit.org/changeset/136134>
WebKit Review Bot
Comment 8 2012-11-29 08:51:39 PST
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.