[Offline Pages] Add network status, save request textbox, and offline url link to offline internals…

chrome/browser/resources/offline_pages/offline_internals.js

 // Copyright 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.
 
 cr.define('offlineInternals', function() {
   'use strict';
 
   /** @type {!Array<OfflinePage>} */
   var offlinePages = [];
 
@@ skipping 19 matching lines @@
       var checkboxCell = document.createElement('td');
       var checkbox = document.createElement('input');
       checkbox.setAttribute('type', 'checkbox');
       checkbox.setAttribute('name', 'stored');
       checkbox.setAttribute('value', pages[i].id);
 
       checkboxCell.appendChild(checkbox);
       row.appendChild(checkboxCell);
 
       var cell = document.createElement('td');
-      cell.textContent = pages[i].onlineUrl;
+      var link = document.createElement('a');
+      link.setAttribute('href', pages[i].filePath);
+      link.textContent = pages[i].onlineUrl;
+      cell.appendChild(link);
       row.appendChild(cell);
 
       cell = document.createElement('td');
       cell.textContent = pages[i].namespace;
       row.appendChild(cell);
 
       cell = document.createElement('td');
       cell.textContent = Math.round(pages[i].size / 1024);
       row.appendChild(cell);
 
@@ skipping 48 matching lines @@
       element.appendChild(logItem);
     }
   }
 
   /**
    * Refresh all displayed information.
    */
   function refreshAll() {
     browserProxy_.getStoredPages().then(fillStoredPages);
     browserProxy_.getRequestQueue().then(fillRequestQueue);
+    browserProxy_.getNetworkStatus().then(function(networkStatus) {
+      $('current-status').textContent = networkStatus;
+    });
     refreshLog();
   }
 
   /**
    * Delete all pages in the offline store.
    */
   function deleteAllPages() {
     browserProxy_.deleteAllPages().then(pagesDeleted);
   }
 
@@ skipping 47 matching lines @@
 
   /**
    * Refreshes the logs.
    */
   function refreshLog() {
     browserProxy_.getEventLogs().then(fillEventLog);
     browserProxy_.getLoggingState().then(updateLogStatus);
   }
 
   function initialize() {
+    /**
+     * @param {!boolean} enabled Whether to enable Logging.
+     */
+    function togglePageModelLog(enabled) {
+      browserProxy_.setRecordPageModel(enabled);
+      $('model-status').textContent = enabled ? 'On' : 'Off';
+    }
+
+    /**
+     * @param {!boolean} enabled Whether to enable Logging.
+     */
+    function toggleRequestQueueLog(enabled) {
+      browserProxy_.setRecordRequestQueue(enabled);
+      $('request-status').textContent = enabled ? 'On' : 'Off';
+    }
+
     $('clear-all').onclick = deleteAllPages;
     $('clear-selected').onclick = deleteSelectedPages;
     $('refresh').onclick = refreshAll;
     $('download').onclick = download;
-    $('log-model-on').onclick =
-        browserProxy_.setRecordPageModel.bind(browserProxy_, true);
-    $('log-model-off').onclick =
-        browserProxy_.setRecordPageModel.bind(browserProxy_, false);
-    $('log-request-on').onclick =
-        browserProxy_.setRecordRequestQueue.bind(browserProxy_, true);
-    $('log-request-off').onclick =
-        browserProxy_.setRecordRequestQueue.bind(browserProxy_, false);
+    $('log-model-on').onclick = togglePageModelLog.bind(this, true);
+    $('log-model-off').onclick = togglePageModelLog.bind(this, false);
+    $('log-request-on').onclick = toggleRequestQueueLog.bind(this, true);
+    $('log-request-off').onclick = toggleRequestQueueLog.bind(this, false);
     $('refresh-logs').onclick = refreshLog;
+    $('add-to-queue').onclick = function() {
+      var saveUrl = $('url').value;
+      browserProxy_.addToRequestQueue(saveUrl)
+        .then(function(state) {
+          if (state) {
+            $('save-url-state').textContent =
+              saveUrl + ' has been added to queue.';
+            $('url').textContent = '';

[dpapad, 2016/07/21 18:57:25]

Is this equivalent to 
$('url').value = '';

If so let's use |value| instead of |textContent| for consistency with line 208.

[chili, 2016/07/21 23:57:20]

Done.

+            browserProxy_.getRequestQueue().then(fillRequestQueue);
+          } else {
+            $('save-url-state').textContent =
+              saveUrl + ' failed to be added to queue.';
+          }
+        });
+    };
+
     refreshAll();
   }
 
   // Return an object with all of the exports.
   return {
     initialize: initialize,
   };
 });
 
 document.addEventListener('DOMContentLoaded', offlineInternals.initialize);