Clean-up serviceworker-internals page.

- Adhere to the 80-column limit
- Don't show the partition if there is no registrations nor versions.

BUG=358657

Review URL: https://codereview.chromium.org/305353002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@274187 0039d316-1c4b-4281-b951-d872f2087c98

content/browser/resources/service_worker/serviceworker_internals.html

@@ -42,16 +42,20 @@
             rows="3" cols="120" readonly jscontent="$this.log"></textarea>
       </div>
       <div class="worker-controls">
-                <button href="#" class="stop" jsdisplay="$this.running_status == 'RUNNING'"
-                   jsvalues=".cmdArgs:{partition_id:$partition_id,version_id:version_id}">Stop</button>
-                <button href="#" class="sync" jsdisplay="$this.running_status == 'RUNNING'"
-                   jsvalues=".cmdArgs:{partition_id:$partition_id,version_id:version_id}">Sync</button>
-                <button href="#" class="inspect" jsdisplay="$this.running_status == 'RUNNING'"
-                   jsvalues=".cmdArgs:{process_id:process_id,devtools_agent_route_id:devtools_agent_route_id}">Inspect</button>
+        <button href="#" class="stop"
+            jsvalues=".cmdArgs:{partition_id:$partition_id,version_id:version_id}"
+            jsdisplay="$this.running_status == 'RUNNING'">Stop</button>
+        <button href="#" class="sync"
+            jsvalues=".cmdArgs:{partition_id:$partition_id,version_id:version_id}"
+            jsdisplay="$this.running_status == 'RUNNING'">Sync</button>
+        <button href="#" class="inspect"
+            jsvalues=".cmdArgs:{process_id:process_id,devtools_agent_route_id:devtools_agent_route_id}"
+            jsdisplay="$this.running_status == 'RUNNING'">Inspect</button>
         <span class="operation-status" style="display: none">Running...</span>
       </div>
     </div>
-        <div id="serviceworker-registration-template" class="serviceworker-registration">
+    <div id="serviceworker-registration-template"
+        class="serviceworker-registration">
       <div class="serviceworker-scope">
         <span>Scope:</span>
         <span jscontent="scope"></span>
@@ -75,14 +79,20 @@
       </div>
       <div class="registration-controls" jsdisplay="!$this.unregistered">
         <button href="#" class="unregister"
-                   jsvalues=".cmdArgs:{partition_id:$partition_id,scope:scope}">Unregister</button>
-                <button href="#" class="start" jsdisplay="$this.active.running_status != 'RUNNING'"
-                   jsvalues=".cmdArgs:{partition_id:$partition_id,scope:scope}">Start</button>
+            jsvalues=".cmdArgs:{partition_id:$partition_id,scope:scope}">
+          Unregister
+        </button>
+        <button href="#" class="start"
+            jsdisplay="$this.active.running_status != 'RUNNING'"
+             jsvalues=".cmdArgs:{partition_id:$partition_id,scope:scope}">
+          Start
+        </button>
         <span class="operation-status" style="display: none">Running...</span>
       </div>
     </div>
     <div id="serviceworker-list-template"
-             jsvalues="$partition_id:$this.partition_id;$partition_path:$this.partition_path;.partition_path:$this.partition_path">
+        jsvalues="$partition_id:$this.partition_id;.partition_id:$this.partition_id"
+        jsdisplay="$this.stored_registrations.length + $this.unregistered_registrations.length + $this.unregistered_versions.length > 0">
       <div class="serviceworker-summary">
         <span>Registrations in: </span>
         <span jscontent="$this.partition_path"></span>
@@ -91,7 +101,8 @@
       <div class="serviceworker-item" jsselect="$this.stored_registrations">
         <div transclude="serviceworker-registration-template"></div>
       </div>
-            <div class="serviceworker-item" jsselect="$this.unregistered_registrations">
+      <div class="serviceworker-item"
+          jsselect="$this.unregistered_registrations">
         <div transclude="serviceworker-registration-template"></div>
       </div>
       <div class="serviceworker-item" jsselect="$this.unregistered_versions">
@@ -102,9 +113,12 @@
     <div id="serviceworker-options-template">
       <div>
         <span>
-                    <input type="checkbox" class="debug_on_start" jsvalues=".checked:$this.debug_on_start">
+          <input type="checkbox" class="debug_on_start"
+              jsvalues=".checked:$this.debug_on_start">
+        </span>
+        <span>
+          Opens the DevTools window for ServiceWorker on start for debugging.
         </span>
-                <span>Opens the DevTools window for ServiceWorker on start for debugging.</span>
       </div>
     </div>
   </div>

content/browser/resources/service_worker/serviceworker_internals.js

@@ -185,11 +185,11 @@ cr.define('serviceworker', function() {
                            unregistered_versions);
     var template;
     var container = $('serviceworker-list');
-    // Existing templates are keyed by partition_path. This allows
+    // Existing templates are keyed by partition_id. This allows
     // the UI to be updated in-place rather than refreshing the
     // whole page.
     for (var i = 0; i < container.childNodes.length; ++i) {
-      if (container.childNodes[i].partition_path == partition_path) {
+      if (container.childNodes[i].partition_id == partition_id) {
         template = container.childNodes[i];
       }
     }