Add chrome:://device-log

chrome/browser/resources/chromeos/network_ui/network_ui.js

 // Copyright 2013 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.
 
 var NetworkUI = (function() {
   'use strict';
 
   // Properties to display in the network state table. Each entry can be either
   // a single state field or an array of state fields. If more than one is
   // specified then the first non empty value is used.
@@ skipping 17 matching lines @@
   var FAVORITE_STATE_FIELDS = [
     'GUID',
     'service_path',
     'Name',
     'Type',
     'profile_path',
     'visible',
     'onc_source'
   ];
 
-  var LOG_LEVEL_CLASSNAME = {
-    'Error': 'network-log-level-error',
-    'User': 'network-log-level-user',
-    'Event': 'network-log-level-event',
-    'Debug': 'network-log-level-debug'
-  };
-
-  var LOG_LEVEL_CHECKBOX = {
-    'Error': 'log-error',
-    'User': 'log-user',
-    'Event': 'log-event',
-    'Debug': 'log-debug'
-  };
-
-  /**
-   * Create a tag of log level.
-   *
-   * @param {string} level A string that represents log level.
-   * @return {DOMElement} The created span element.
-   */
-  var createLevelTag = function(level) {
-    var tag = document.createElement('span');
-    tag.className = 'network-level-tag';
-    tag.textContent = level;
-    tag.classList.add(LOG_LEVEL_CLASSNAME[level]);
-    return tag;
-  };
-
-  /**
-   * Creates an element that contains the time, the event, the level and
-   * the description of the given log entry.
-   *
-   * @param {Object} logEntry An object that represents a single line of log.
-   * @return {DOMElement}  The created p element that represents the log entry.
-   */
-  var createLogEntryText = function(logEntry) {
-    var level = logEntry['level'];
-    if (!$(LOG_LEVEL_CHECKBOX[level]).checked)
-      return null;
-    var res = document.createElement('p');
-    var textWrapper = document.createElement('span');
-    var fileinfo = '';
-    if ($('log-fileinfo').checked)
-      fileinfo = logEntry['file'];
-    var timestamp = '';
-    if ($('log-timedetail').checked)
-      timestamp = logEntry['timestamp'];
-    else
-      timestamp = logEntry['timestampshort'];
-    textWrapper.textContent = loadTimeData.getStringF(
-      'logEntryFormat',
-      timestamp,
-      fileinfo,
-      logEntry['event']);
-    res.appendChild(createLevelTag(level));
-    res.appendChild(textWrapper);
-    return res;
-  };
-
-  /**
-   * Create event log entries.
-   *
-   * @param {Array.<string>} logEntries A array of strings that each string
-   *     represents a log event in JSON format.
-   */
-  var createEventLog = function(logEntries) {
-    var container = $('network-log-container');
-    container.textContent = '';
-    for (var i = 0; i < logEntries.length; ++i) {
-      var entry = createLogEntryText(JSON.parse(logEntries[i]));
-      if (entry)
-        container.appendChild(entry);
-    }
-  };
-
   /**
    * Create a cell with a button for expanding a network state table row.
    *
    * @param {string} guid The GUID identifying the network.
    * @return {DOMElement} The created td element that displays the given value.
    */
   var createStateTableExpandButton = function(guid) {
     var cell = document.createElement('td');
     cell.className = 'state-table-expand-button-cell';
     var button = document.createElement('button');
@@ skipping 61 matching lines @@
     var table = $(tablename);
     var oldRows = table.querySelectorAll('.state-table-row');
     for (var i = 0; i < oldRows.length; ++i)
       table.removeChild(oldRows[i]);
     states.forEach(function(state) {
       table.appendChild(createStateTableRow(stateFields, state));
     });
   };
 
   /**
-   * This callback function is triggered when the network log is received.
-   *
-   * @param {Object} data A JSON structure of event log entries.
-   */
-  var getNetworkLogCallback = function(data) {
-    createEventLog(JSON.parse(data));
-  };
-
-  /**
    * This callback function is triggered when visible networks are received.
    *
    * @param {Array} data A list of network state information for each
    *     visible network.
    */
   var onVisibleNetworksReceived = function(states) {
     createStateTable('network-state-table', NETWORK_STATE_FIELDS, states);
   };
 
   /**
@@ skipping 42 matching lines @@
     var detailCell = document.createElement('td');
     detailCell.className = 'state-table-expanded-cell';
     detailCell.colSpan = baseRow.childNodes.length - 1;
     expandedRow.appendChild(detailCell);
     var showDetail = function(state) {
       if (networkConfig.lastError)
         detailCell.textContent = networkConfig.lastError;
       else
         detailCell.textContent = JSON.stringify(state, null, '\t');
     };
-    var selected = $('get-network-type').selectedIndex;
-    var selectedId = $('get-network-type').options[selected].id;
+    var selected = $('get-property-format').selectedIndex;
+    var selectedId = $('get-property-format').options[selected].value;
     if (selectedId == 'shill')
       networkConfig.getShillProperties(guid, showDetail);
     else if (selectedId == 'managed')
       networkConfig.getManagedProperties(guid, showDetail);
     else
       networkConfig.getProperties(guid, showDetail);
     return expandedRow;
   };
 
-  /**
-   * Requests a network log update.
-   */
-  var requestLog = function() {
-    chrome.send('NetworkUI.getNetworkLog');
+  var addPropertyFormatOption = function(format) {

[xiyuan, 2014/12/11 20:14:32]

nit: document this?

[stevenjb, 2014/12/11 20:53:14]

Done.

+    var opt = document.createElement('option');
+    opt.value = format;
+    opt.innerHTML = loadTimeData.getValue(format + 'FormatOption');

[xiyuan, 2014/12/11 20:14:32]

nit: Can we use .textContent instead of .innerHTML?

[stevenjb, 2014/12/11 20:53:14]

Done.

+    return opt;
   };
 
   /**
    * Requests an update of all network info.
    */
   var requestNetworks = function() {
     networkConfig.getNetworks(
         { 'type': 'All', 'visible': true },
         onVisibleNetworksReceived);
     networkConfig.getNetworks(
         { 'type': 'All', 'configured': true },
         onFavoriteNetworksReceived);
   };
 
   /**
    * Sets refresh rate if the interval is found in the url.
    */
   var setRefresh = function() {
     var interval = parseQueryParams(window.location)['refresh'];
     if (interval && interval != '')
       setInterval(requestNetworks, parseInt(interval) * 1000);
   };
 
   /**
    * Get network information from WebUI.
    */
   document.addEventListener('DOMContentLoaded', function() {
-    $('log-refresh').onclick = requestLog;
-    $('log-error').checked = true;
-    $('log-error').onclick = requestLog;
-    $('log-user').checked = true;
-    $('log-user').onclick = requestLog;
-    $('log-event').checked = true;
-    $('log-event').onclick = requestLog;
-    $('log-debug').checked = false;
-    $('log-debug').onclick = requestLog;
-    $('log-fileinfo').checked = false;
-    $('log-fileinfo').onclick = requestLog;
-    $('log-timedetail').checked = false;
-    $('log-timedetail').onclick = requestLog;
+    $('refresh').onclick = requestNetworks;
+    var propertySelect = $('get-property-format');
+    propertySelect.appendChild(addPropertyFormatOption('normal'));
+    propertySelect.appendChild(addPropertyFormatOption('managed'));
+    propertySelect.appendChild(addPropertyFormatOption('shill'));

[xiyuan, 2014/12/11 20:14:32]

Why moving this to js? Couldn't we do the following in html?

<select id="get-property-format">
  <option value="normal" i18n-content="normalFormatOption"></option>
  <option value="managed" i18n-content="managedFormatOption"></option>
  <option value="shill" i18n-content="shillFormatOption"></option>
</select>

[stevenjb, 2014/12/11 20:53:14]

I didn't see any examples of that so I assumed you couldn't. It also seemed a
little better to me to populate the dropdown in code (model) than in the html
(view). But, it seems to work, so I'm fine going that way.
Done.

     setRefresh();
-    requestLog();
     requestNetworks();
   });
 
-  return {
-    getNetworkLogCallback: getNetworkLogCallback
-  };
+  return {};
 })();