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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java

Issue 2619093002: Remove Context params from GeolocationHeader (Closed)
Patch Set: ebase Created 3 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
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java b/chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java
index 0e64155338abb55ecb4be26d69d8ec5dd0b7d9ab..c8097038659c11812a6fc307bb2628ee8efd383b 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/geo/GeolocationHeaderTest.java
@@ -10,7 +10,6 @@ import android.os.Build;
import android.os.SystemClock;
import android.support.test.filters.SmallTest;
-import org.chromium.base.ContextUtils;
import org.chromium.base.ThreadUtils;
import org.chromium.base.library_loader.ProcessInitException;
import org.chromium.base.test.util.Feature;
@@ -86,7 +85,7 @@ public class GeolocationHeaderTest extends ChromeActivityTestCaseBase<ChromeActi
GeolocationInfo infoHttp = new GeolocationInfo("http://www.google.de", null, false);
infoHttps.setContentSetting(httpsPermission);
infoHttp.setContentSetting(httpPermission);
- String header = GeolocationHeader.getGeoHeader(ContextUtils.getApplicationContext(),
+ String header = GeolocationHeader.getGeoHeader(
"https://www.google.de/search?q=kartoffelsalat",
getActivity().getActivityTab());
assertHeaderState(header, shouldBeNull);
@@ -100,8 +99,8 @@ public class GeolocationHeaderTest extends ChromeActivityTestCaseBase<ChromeActi
@Override
public void run() {
setMockLocation(latitute, longitude, time);
- String header = GeolocationHeader.getGeoHeader(ContextUtils.getApplicationContext(),
- SEARCH_URL_1, getActivity().getActivityTab());
+ String header = GeolocationHeader.getGeoHeader(SEARCH_URL_1,
+ getActivity().getActivityTab());
assertHeaderState(header, shouldBeNull);
}
});
@@ -134,8 +133,7 @@ public class GeolocationHeaderTest extends ChromeActivityTestCaseBase<ChromeActi
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- assertNull(GeolocationHeader.getGeoHeader(
- ContextUtils.getApplicationContext(), url, tab));
+ assertNull(GeolocationHeader.getGeoHeader(url, tab));
}
});
} catch (InterruptedException e) {
@@ -149,8 +147,7 @@ public class GeolocationHeaderTest extends ChromeActivityTestCaseBase<ChromeActi
ThreadUtils.runOnUiThreadBlocking(new Runnable() {
@Override
public void run() {
- assertNotNull(GeolocationHeader.getGeoHeader(
- ContextUtils.getApplicationContext(), url, tab));
+ assertNotNull(GeolocationHeader.getGeoHeader(url, tab));
}
});
} catch (InterruptedException e) {
« no previous file with comments | « chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleWebsitePreferences.java ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698