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

Unified Diff: chrome/android/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java

Issue 2935783002: Send microsecond resolution timestamps to Java history page so they can be sent back. (Closed)
Patch Set: Updates for twellington@ Created 3 years, 6 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/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java
diff --git a/chrome/android/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java b/chrome/android/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java
index 6feba1ac78b0e6e5369b29a52b02fbd109da9817..6f5fb8144c934bc82cc626d24937ca93571d2375 100644
--- a/chrome/android/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java
+++ b/chrome/android/javatests/src/org/chromium/chrome/browser/history/StubbedHistoryProvider.java
@@ -99,22 +99,26 @@ public class StubbedHistoryProvider implements HistoryProvider {
mItems.remove(item);
}
- public static HistoryItem createHistoryItem(int which, long[] timestamps) {
+ public static HistoryItem createHistoryItem(int which, long timestamp) {
+ long[] nativeTimestamps = {timestamp * 1000};
if (which == 0) {
- return new HistoryItem("http://google.com/", "www.google.com", "Google", timestamps,
- false);
+ return new HistoryItem("http://google.com/", "www.google.com", "Google", timestamp,
+ nativeTimestamps, false);
} else if (which == 1) {
- return new HistoryItem("http://foo.com/", "www.foo.com", "Foo", timestamps, false);
+ return new HistoryItem(
+ "http://foo.com/", "www.foo.com", "Foo", timestamp, nativeTimestamps, false);
} else if (which == 2) {
- return new HistoryItem("http://bar.com/", "www.bar.com", "Bar", timestamps, false);
+ return new HistoryItem(
+ "http://bar.com/", "www.bar.com", "Bar", timestamp, nativeTimestamps, false);
} else if (which == 3) {
- return new HistoryItem("http://news.com/", "www.news.com", "News", timestamps, false);
+ return new HistoryItem(
+ "http://news.com/", "www.news.com", "News", timestamp, nativeTimestamps, false);
} else if (which == 4) {
- return new HistoryItem("http://eng.com/", "www.eng.com", "Engineering", timestamps,
- false);
+ return new HistoryItem("http://eng.com/", "www.eng.com", "Engineering", timestamp,
+ nativeTimestamps, false);
} else if (which == 5) {
return new HistoryItem("http://blocked.com/", "www.blocked.com", "Cannot Visit",
- timestamps, true);
+ timestamp, nativeTimestamps, true);
} else {
return null;
}

Powered by Google App Engine
This is Rietveld 408576698