Remove nearby URL cache from the PW UrlManager

Remove nearby URL cache from the PW UrlManager

Given that we can't inspect the status of our Nearby subscription, we
have to resubscribe on every startup.  That means that there is no
point in caching nearby URLs when we will have to repopulate those
through the new subscription anyway.  Additionally, our current
practice is sending noise to listeners (onFound at startup, onLost at
cache deletion, onFound at resubscription).  This change just removes
that cache.

BUG=667754
Review-Url: https://codereview.chromium.org/2606933003
Cr-Commit-Position: refs/heads/master@{#442108}
Committed: https://chromium.googlesource.com/chromium/src/+/91f5b8d2f34faf9c21fb4c08146cb1a7d9ad2144

[cco3, 2016-12-29 00:40:46]

cco3@chromium.org changed reviewers:
+ mattreynolds@chromium.org

[mattreynolds, 2017-01-04 01:25:22]

lgtm

https://codereview.chromium.org/2606933003/diff/1/chrome/android/java/src/org...
File
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWebBleClient.java
(right):

https://codereview.chromium.org/2606933003/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/physicalweb/PhysicalWebBleClient.java:41:
Log.d(TAG, "cco3 onFound: " + url);
remove debug logging (here and below)

[cco3, 2017-01-05 22:49:24]

cco3@chromium.org changed reviewers:
+ miguelg@chromium.org

[cco3, 2017-01-05 22:49:24]

Hi Miguel, would you be able to review this change?

[cco3, 2017-01-06 18:39:34]

cco3@chromium.org changed reviewers:
+ dtrainor@chromium.org

[cco3, 2017-01-06 18:39:35]

Hi David, would you be able to review this change?

[David Trainor- moved to gerrit, 2017-01-06 23:32:00]

owners lgtm.  thanks!

[cco3, 2017-01-06 23:36:39]

The CQ bit was checked by cco3@chromium.org

[cco3, 2017-01-06 23:36:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2606933003/#ps20001
(title: "Remove unnecessary logging")

[commit-bot: I haz the power, 2017-01-06 23:37:07]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-06 23:38:51]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-06 23:38:52]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-device-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device-xcode-...)
  ios-simulator-xcode-clang on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator-xco...)

[cco3, 2017-01-06 23:39:20]

The CQ bit was unchecked by cco3@chromium.org

[cco3, 2017-01-06 23:51:10]

The CQ bit was checked by cco3@chromium.org

[cco3, 2017-01-06 23:51:11]

The patchset sent to the CQ was uploaded after l-g-t-m from
dtrainor@chromium.org, mattreynolds@chromium.org
Link to the patchset: https://codereview.chromium.org/2606933003/#ps40001
(title: "Rebase")

[commit-bot: I haz the power, 2017-01-06 23:51:42]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-07 00:24:17]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1483746670635170,
"parent_rev": "516ef6d9c0c19352fb80ecaa4e635e60501125fb", "commit_rev":
"91f5b8d2f34faf9c21fb4c08146cb1a7d9ad2144"}

[commit-bot: I haz the power, 2017-01-07 00:24:44]

Description was changed from

==========
Remove nearby URL cache from the PW UrlManager

Given that we can't inspect the status of our Nearby subscription, we
have to resubscribe on every startup.  That means that there is no
point in caching nearby URLs when we will have to repopulate those
through the new subscription anyway.  Additionally, our current
practice is sending noise to listeners (onFound at startup, onLost at
cache deletion, onFound at resubscription).  This change just removes
that cache.

BUG=667754
==========

to

==========
Remove nearby URL cache from the PW UrlManager

Given that we can't inspect the status of our Nearby subscription, we
have to resubscribe on every startup.  That means that there is no
point in caching nearby URLs when we will have to repopulate those
through the new subscription anyway.  Additionally, our current
practice is sending noise to listeners (onFound at startup, onLost at
cache deletion, onFound at resubscription).  This change just removes
that cache.

BUG=667754

Review-Url: https://codereview.chromium.org/2606933003
Cr-Commit-Position: refs/heads/master@{#442108}
Committed:
https://chromium.googlesource.com/chromium/src/+/91f5b8d2f34faf9c21fb4c08146c...
==========

[commit-bot: I haz the power, 2017-01-07 00:24:45]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/91f5b8d2f34faf9c21fb4c08146c...