Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1756)

Unified Diff: android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java

Issue 11567061: Throw exception when initialization failed. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressed comments Created 7 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java
diff --git a/android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java b/android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java
index 682706a2bd35fcf51f3b6220a94a3ca973dec69b..a31d1b41aacba5f7a7cc3c1bbf4a37ea41d73368 100644
--- a/android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java
+++ b/android_webview/java/src/org/chromium/android_webview/AwBrowserProcess.java
@@ -11,6 +11,7 @@ import org.chromium.base.ThreadUtils;
import org.chromium.content.app.LibraryLoader;
import org.chromium.content.browser.AndroidBrowserProcess;
import org.chromium.content.browser.ResourceExtractor;
+import org.chromium.content.common.ProcessInitException;
/**
* Wrapper for the steps needed to initialize the java and native sides of webview chromium.
@@ -46,9 +47,13 @@ public abstract class AwBrowserProcess {
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- LibraryLoader.ensureInitialized();
- AndroidBrowserProcess.initContentViewProcess(context,
- AndroidBrowserProcess.MAX_RENDERERS_SINGLE_PROCESS);
+ try {
+ LibraryLoader.ensureInitialized();
+ AndroidBrowserProcess.initContentViewProcess(context,
+ AndroidBrowserProcess.MAX_RENDERERS_SINGLE_PROCESS);
+ } catch (ProcessInitException e) {
+ throw new Error("Cannot initialize WebView", e);
+ }
}
});
}

Powered by Google App Engine
This is Rietveld 408576698