Add Physical Web WebUI Default Url Icon

components/physical_web/webui/resources/physical_web.js

Index: components/physical_web/webui/resources/physical_web.js
diff --git a/components/physical_web/webui/resources/physical_web.js b/components/physical_web/webui/resources/physical_web.js
index 418dc1d6e90c1b5e8daffb35b8fbd841900eae08..e9f596e8730aeefc132aab122e57366996918a98 100644
--- a/components/physical_web/webui/resources/physical_web.js
+++ b/components/physical_web/webui/resources/physical_web.js
@@ -24,8 +24,18 @@ function physicalWebItemClicked(index) {
 function returnNearbyURLs(nearbyUrlsData) {
   var bodyContainer = $('body-container');
   renderTemplate(nearbyUrlsData);
-
   bodyContainer.style.visibility = 'visible';

[mmocny, 2017/03/02 17:13:35]

Nit: could you create a CSS class for hidden items, and then add/remove the
class instead of manually adjusting styles like this?

You can use Element.classList.add/remove to this:
https://developer.mozilla.org/en/docs/Web/API/Element/classList

[Ran, 2017/03/23 16:56:26]

Done.

+  addErrorHandler();
+}
+
+function addErrorHandler() {

[mmocny, 2017/03/02 17:13:35]

Please rename: `assignImageLoadErrorHandlers`

[Ran, 2017/03/23 16:56:26]

Done.

+  var images = document.getElementsByTagName('img');

[Matt Reynolds, 2017/03/02 01:02:39]

Would getElementsByName work here to get only the favicons?

[mmocny, 2017/03/02 17:13:35]

Nit: chrome support `let` keyword now, which is more sane.

It doesn't matter much for `images` variable, but your use of `i` and `img`
later on are not scoped the way you expect.

[mmocny, 2017/03/02 17:13:35]

Personally I like using document.querySelector() (or document.querySelectorAll()
in this case) since it uses the same syntax as your CSS selectors.  Its
trivially less efficient (for this size UI).

https://developer.mozilla.org/en-US/docs/Web/API/Element/querySelector
https://developer.mozilla.org/en-US/docs/Web/API/Document/querySelector

[Ran, 2017/03/23 16:56:26]

Done.

[Ran, 2017/03/23 16:56:26]

Done.

[Ran, 2017/03/23 16:56:26]

Done.

+  for (var i = 0; i < images.length; i++) {
+    var img = images[i];
+    img.addEventListener('error', function() {
+      this.src = "chrome://physical-web/link.png";

[Matt Reynolds, 2017/03/02 01:02:39]

Use a relative path here (relative to the webui URL chrome://physical-web/)

Also, shouldn't this be ic_link_grey600_36dp.png?

[Ran, 2017/03/23 16:56:25]

Done.

+    });
+  }
 }
 
 document.addEventListener('DOMContentLoaded', requestNearbyURLs);