Bug 88610

Summary: include base/net jarball in the DumpRenderTree apk building
Product: WebKit Reporter: Wei James (wistoch) <james.wei>
Component: New BugsAssignee: Wei James (wistoch) <james.wei>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: jrg, wangxianzhu
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Android   
OS: Android   
Attachments:
Description Flags
Patch none

Wei James (wistoch)
Reported 2012-06-07 20:21:12 PDT
include base/net jarball in the DumpRenderTree apk building
Attachments
Patch (2.31 KB, patch)
2012-06-07 20:21 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-06-07 20:21:34 PDT
Created attachment 146464 [details] Patch
Wei James (wistoch)
Comment 2 2012-06-07 20:22:53 PDT
without these jarballs, there are some errors like: F/chromium( 9374): [0608/030727:FATAL:jni_android.cc(140)] Check failed: clazz && !ClearException(env). Failed to find class org/chromium/base/LocaleUtils
Xianzhu Wang
Comment 3 2012-06-07 20:37:50 PDT
Thanks very much for filing this and the patch. The patch looks good to me. However I also filed bug 88598 just at the same time as this one (I should have cc'ed you) and am about to upload a patch. As my patch will match our downstream change, would you mind mark this bug duplication of bug 88598?
Wei James (wistoch)
Comment 4 2012-06-07 22:08:23 PDT
(In reply to comment #3) > Thanks very much for filing this and the patch. The patch looks good to me. However I also filed bug 88598 just at the same time as this one (I should have cc'ed you) and am about to upload a patch. As my patch will match our downstream change, would you mind mark this bug duplication of bug 88598? sure. thanks for the information.
Wei James (wistoch)
Comment 5 2012-06-07 22:10:15 PDT
*** This bug has been marked as a duplicate of bug 88598 ***
Laszlo Gombos
Comment 6 2012-07-08 19:52:25 PDT
Comment on attachment 146464 [details] Patch Marking the patch obsolete as a duplicated patch is landed as r119888; this will take the patch out from the review queue.
Note You need to log in before you can comment on or make changes to this bug.