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

Unified Diff: chrome/android/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java

Issue 2341163002: Canonicalize URLs prior to comparing in ManifestUpgradeDetector (Closed)
Patch Set: Merge branch 'master' into url_canonicalize Created 4 years, 3 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/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java
diff --git a/chrome/android/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java b/chrome/android/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java
index 1c40d0caff632cc5e533daac664878a6e10074bc..d23baa3a720e5c80b02455025717c26b3ad64bd4 100644
--- a/chrome/android/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java
+++ b/chrome/android/junit/src/org/chromium/chrome/browser/webapps/ManifestUpgradeDetectorTest.java
@@ -8,6 +8,7 @@ import android.content.Context;
import android.graphics.Bitmap;
import android.graphics.Color;
import android.os.Bundle;
+import android.text.TextUtils;
import org.chromium.base.ContextUtils;
import org.chromium.blink_public.platform.WebDisplayMode;
@@ -105,6 +106,12 @@ public class ManifestUpgradeDetectorTest {
Mockito.any(ManifestUpgradeDetectorFetcher.Callback.class));
return fetcher;
}
+
+ // Stubbed out because real implementation uses native.
+ @Override
+ protected boolean urlsMatchIgnoringFragments(String url1, String url2) {
+ return TextUtils.equals(url1, url2);
+ }
}
/**

Powered by Google App Engine
This is Rietveld 408576698