Update Physical Web WebUI to show new results automatically

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 70d856fe7db71a2222280e9cda9630a922507d4b..7ed9e577eede6245ea284613ec297ff669ff0900 100644
--- a/components/physical_web/webui/resources/physical_web.js
+++ b/components/physical_web/webui/resources/physical_web.js
@@ -9,12 +9,25 @@
  *      nearby devices
  */
 function renderTemplate(nearbyUrlsData) {
+  if (nearbyUrlsData['metadata'].length != 0) {
+    $('empty-list-container').hidden = true;
+  }
+  // This is a workaround with jstemplate. Jstemplate render only works on empty
+  // node. When we need to rerender things, we have to remove previous nodes.
+  let renderContainer = document.getElementById('render-container');
+  // Remove existing childNode.
+  while (renderContainer.hasChildNodes()) {
+    renderContainer.removeChild(renderContainer.lastChild);
+  }
+  let templateDiv = document.getElementById('render-template').cloneNode(true);
+  renderContainer.appendChild(templateDiv);
+
   // This is the javascript code that processes the template:
-  jstProcess(new JsEvalContext(nearbyUrlsData), $('physicalWebTemplate'));
+  jstProcess(new JsEvalContext(nearbyUrlsData), templateDiv);
 }
 
-function requestNearbyURLs() {
-  chrome.send('requestNearbyURLs');
+function physicalWebPageLoaded() {
+  chrome.send('physicalWebPageLoaded');
 }
 
 function physicalWebItemClicked(index) {
@@ -22,14 +35,9 @@ function physicalWebItemClicked(index) {
 }
 
 function returnNearbyURLs(nearbyUrlsData) {

[mmocny, 2017/03/24 17:19:11]

Nit: Also rename this function if you rename the C++ function to
"pushNearbyResults"

[Ran, 2017/03/24 18:45:20]

Done.

-  var bodyContainer = $('body-container');
+  let bodyContainer = $('body-container');
   renderTemplate(nearbyUrlsData);
   bodyContainer.hidden = false;

[mmocny, 2017/03/24 17:19:11]

Nit: I think you should also move this .hidden=false out of this function now.

[Ran, 2017/03/24 18:45:20]

Done.

-
-  if (nearbyUrlsData['metadata'].length == 0) {
-    var emptyMessage = $('empty-list-container');
-    emptyMessage.hidden = false;
-  }
 }
 
-document.addEventListener('DOMContentLoaded', requestNearbyURLs);
+document.addEventListener('DOMContentLoaded', physicalWebPageLoaded);