[Offline Pages] Connect the offline page logs to the internals page.

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 64 matching lines @@
       cell = document.createElement('td');
       cell.textContent = requests[i].status;
       row.appendChild(cell);
 
       requestQueueTable.appendChild(row);
     }
     savePageRequests = requests;
   }
 
   /**
+   * Fills the event logs section.
+   * @param {!Array<string>} logs A list of log strings.
+   */
+  function fillEventLog(logs) {
+    var element = $('logs');
+    element.textContent = '';
+    for (let log of logs) {
+      var logItem = document.createElement('li');
+      logItem.textContent = log;
+      element.appendChild(logItem);
+    }
+  }
+
+  /**
    * Refresh all displayed information.
    */
   function refreshAll() {
     browserProxy_.getStoredPages().then(fillStoredPages);
     browserProxy_.getRequestQueue().then(fillRequestQueue);
+    browserProxy_.getEventLogs().then(fillEventLog);
   }
 
   /**
    * Delete all pages in the offline store.
    */
   function deleteAllPages() {
     browserProxy_.deleteAllPages().then(pagesDeleted);
   }
 
   /**
@@ skipping 28 matching lines @@
     var selectedIds = [];
 
     for (var i = 0; i < checkboxes.length; i++) {
       if (checkboxes[i].checked)
         selectedIds.push(checkboxes[i].value);
     }
 
     browserProxy_.deleteSelectedPages(selectedIds).then(pagesDeleted);
   }
 
+  /**
+   * Turns on logging for stored pages.
+   */
+  function toggleModelOn() {
+    browserProxy_.setRecordPageModel(true);
+  }
+
+  /**
+   * Turns off logging for stored pages.
+   */
+  function toggleModelOff() {
+    browserProxy_.setRecordPageModel(false);
+  }
+
+  /**
+   * Turns on logging for request queue.
+   */
+  function toggleRequestOn() {
+    browserProxy_.setRecordRequestQueue(true);
+  }
+
+  /**
+   * Turns off logging for request queue.
+   */
+  function toggleRequestOff() {
+    browserProxy_.setRecordRequestQueue(false);
+  }
+
+  /**
+   * Refreshes the logs.
+   */
+  function refreshLog() {
+    browserProxy_.getEventLogs().then(fillEventLog);
+  }
+
   function initialize() {
     $('clear-all').onclick = deleteAllPages;
     $('clear-selected').onclick = deleteSelectedPages;
     $('refresh').onclick = refreshAll;
     $('download').onclick = download;
+    $('log-model-on').onclick = toggleModelOn;

[dpapad, 2016/06/24 20:33:39]

Nit: The four different callback methods can be removed and inlined as follows,

$('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);

[chili, 2016/06/24 21:43:17]

Done.

+    $('log-model-off').onclick = toggleModelOff;
+    $('log-request-on').onclick = toggleRequestOn;
+    $('log-request-off').onclick = toggleRequestOff;
+    $('refresh-logs').onclick = refreshLog;
     browserProxy_ =
         offlineInternals.OfflineInternalsBrowserProxyImpl.getInstance();
     refreshAll();
   }
 
   // Return an object with all of the exports.
   return {
     initialize: initialize,
   };
 });
 
 document.addEventListener('DOMContentLoaded', offlineInternals.initialize);