Do not require explicit initialization of ResourceExtractor

Do not require explicit initialization of ResourceExtractor

It's used only for locale pack files now, so shouldn't require so much
attention.

This also removes the extraction args from
ComponentsBrowserTestsApplication.java, which don't appear to be
necessary

BUG=

Committed: https://crrev.com/5f042d748484efcdac252f44469cb764c4decd3e
Cr-Commit-Position: refs/heads/master@{#336020}

[agrieve, 2015-06-17 15:32:27]

agrieve@chromium.org changed reviewers:
+ gunsch@chromium.org, jaekyun@chromium.org, yfriedman@chromium.org

[agrieve, 2015-06-17 15:32:27]

jaekyun@chromium.org: Please review changes in components/

gunsch@chromium.org: Please review changes in chromecast/

yfriedman@chromium.org: Please review changes in everything else

[gunsch, 2015-06-17 15:58:49]

chromecast lgtm

[agrieve, 2015-06-17 17:33:26]

agrieve@chromium.org changed reviewers:
+ torne@chromium.org

[agrieve, 2015-06-17 17:33:26]

torne@chromium.org: Please review changes in

[Yaron, 2015-06-17 19:09:54]

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
File
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java
(right):

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java:239:
PathService.override(DIR_RESOURCE_PAKS_ANDROID,
"/system/framework/webview/paks");
If we haven't extracted any paks, why is this needed?

[agrieve, 2015-06-17 19:23:12]

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
File
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java
(right):

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java:239:
PathService.override(DIR_RESOURCE_PAKS_ANDROID,
"/system/framework/webview/paks");
On 2015/06/17 19:09:54, Yaron wrote:
> If we haven't extracted any paks, why is this needed? 

Educated guess: WebView tests don't bundle assets, but use .isolate instead. So
this is needed for tests.

[Yaron, 2015-06-17 20:07:48]

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
File
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java
(right):

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java:239:
PathService.override(DIR_RESOURCE_PAKS_ANDROID,
"/system/framework/webview/paks");
On 2015/06/17 19:23:12, agrieve wrote:
> On 2015/06/17 19:09:54, Yaron wrote:
> > If we haven't extracted any paks, why is this needed? 
> 
> Educated guess: WebView tests don't bundle assets, but use .isolate instead.
So
> this is needed for tests.

Can we move this to the test harness then?

[agrieve, 2015-06-17 20:16:29]

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
File
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java
(right):

https://codereview.chromium.org/1181733004/diff/40001/android_webview/glue/ja...
android_webview/glue/java/src/com/android/webview/chromium/WebViewChromiumFactoryProvider.java:239:
PathService.override(DIR_RESOURCE_PAKS_ANDROID,
"/system/framework/webview/paks");
On 2015/06/17 20:07:48, Yaron wrote:
> On 2015/06/17 19:23:12, agrieve wrote:
> > On 2015/06/17 19:09:54, Yaron wrote:
> > > If we haven't extracted any paks, why is this needed? 
> > 
> > Educated guess: WebView tests don't bundle assets, but use .isolate instead.
> So
> > this is needed for tests.
> 
> Can we move this to the test harness then?

torne@ - leaving this question for you. Not really relevant to this review
though.

[Jaekyun Seok (inactive), 2015-06-17 21:16:49]

lgtm for components/

[Torne, 2015-06-18 16:30:10]

android_webview LGTM

[Yaron, 2015-06-18 17:20:33]

On 2015/06/18 16:30:10, Torne wrote:
> android_webview LGTM

lgtm

[agrieve, 2015-06-24 21:26:14]

The CQ bit was checked by agrieve@chromium.org

[agrieve, 2015-06-24 21:26:14]

The patchset sent to the CQ was uploaded after l-g-t-m from gunsch@chromium.org,
yfriedman@chromium.org, torne@chromium.org, jaekyun@chromium.org
Link to the patchset: https://codereview.chromium.org/1181733004/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2015-06-24 21:26:59]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1181733004/80001

[commit-bot: I haz the power, 2015-06-24 22:57:27]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2015-06-24 22:58:37]

Patchset 5 (id:??) landed as
https://crrev.com/5f042d748484efcdac252f44469cb764c4decd3e
Cr-Commit-Position: refs/heads/master@{#336020}