Fix Physical Web WebUI broken favicon

chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java

Index: chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java
index 662d8bdb0b70a2746550c85e47145a91b9094c78..c12ed4e81697e37e3f549197597e6d3c5b9217bf 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/physicalweb/UrlManager.java
@@ -436,7 +436,9 @@ class UrlManager {
             currentUrlInfo = urlInfo;
         } else {
             mUrlsSortedByTimestamp.remove(urlInfo.getUrl());
-            currentUrlInfo.setScanTimestamp(urlInfo.getScanTimestamp());
+
+            // TODO(ranj): Update scantimestamp when metadata is refreshed appropriately.
+            // currentUrlInfo.setScanTimestamp(urlInfo.getScanTimestamp());

[mmocny, 2017/02/22 19:36:52]

If we don't update this here, won't these results get garbage collected too
early?

[cco3, 2017/02/22 20:21:47]

Yes, but that's OK.  time-based garbage collection had mostly to do with
notification frequency (so that after 24 hours, swipes would be reset).  In
fact, now that we don't have notifications, it might be worth reconsidering why
we have a cache for anything other than PwsResults

             if (urlInfo.getDistance() > 0.0) {
                 currentUrlInfo.setDistance(urlInfo.getDistance());
             }