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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java

Issue 2392763002: Sanitize unparcable intents (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 2 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: chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java b/chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java
index 04fcc20a57f935529e4efe45fab93877d575006b..15fecc16943de05d6d7c55a599bf4e7a30401216 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ChromeApplication.java
@@ -89,7 +89,10 @@ public class ChromeApplication extends ContentApplication {
public ChromeApplication() {
super();
- ContextUtils.initApplicationContext(this);
+ // Do not try to reset a test context.
+ if (ContextUtils.getApplicationContext() == null) {
Ted C 2016/10/04 20:19:23 why do we need to add this? IMO, the test should
kraush 2016/10/04 21:36:36 This was required because the test framework spawn
+ ContextUtils.initApplicationContext(this);
+ }
}
/**

Powered by Google App Engine
This is Rietveld 408576698