Update Physical Web WebUI to show new results automatically

components/physical_web/webui/resources/physical_web.js

 // Copyright (c) 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /**
  * Takes the |nearbyUrlsData| input argument which holds metadata for web pages
  * broadcast by nearby devices.
  * @param {Object} nearbyUrlsData Information about web pages broadcast by
  *      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) {
   chrome.send('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);