Re-design the "data" tab on about:net-internals.

chrome/browser/resources/net_internals/import_view.js

Index: chrome/browser/resources/net_internals/import_view.js
===================================================================
--- chrome/browser/resources/net_internals/import_view.js	(revision 94563)
+++ chrome/browser/resources/net_internals/import_view.js	(working copy)
@@ -3,61 +3,20 @@
 // found in the LICENSE file.
 
 /**
- * This view displays options for importing/exporting the captured data.  This
- * view is responsible for the interface and file I/O.  The BrowserBridge
- * handles creation and loading of the data dumps.
- *
- *   - Has a button to generate a JSON dump.
- *
- *   - Has a button to import a JSON dump.
- *
- *   - Shows how many events have been captured.
+ * This view displays options for importing data from a log file.
  *  @constructor
  */
-function DataView() {
-  const mainBoxId = 'dataTabContent';
-  const downloadIframeId = 'dataViewDownloadIframe';
-  const saveFileButtonId = 'dataViewSaveLogFile';
-  const dataViewSaveStatusTextId = 'dataViewSaveStatusText';
-  const securityStrippingCheckboxId = 'securityStrippingCheckbox';
-  const byteLoggingCheckboxId = 'byteLoggingCheckbox';
-  const passivelyCapturedCountId = 'passivelyCapturedCount';
-  const activelyCapturedCountId = 'activelyCapturedCount';
-  const deleteAllId = 'dataViewDeleteAll';
-  const dumpDataDivId = 'dataViewDumpDataDiv';
+function ImportView() {
+  const mainBoxId = 'importTabContent';
   const loadedDivId = 'dataViewLoadedDiv';
   const loadedClientInfoTextId = 'dataViewLoadedClientInfoText';
   const loadLogFileDropTargetId = 'dataViewDropTarget';
   const loadLogFileId = 'dataViewLoadLogFile';
   const dataViewLoadStatusTextId = 'dataViewLoadStatusText';
-  const capturingTextSpanId = 'dataViewCapturingTextSpan';
-  const loggingTextSpanId = 'dataViewLoggingTextSpan';
 
   DivView.call(this, mainBoxId);
 
-  var securityStrippingCheckbox = $(securityStrippingCheckboxId);
-  securityStrippingCheckbox.onclick =
-      this.onSetSecurityStripping_.bind(this, securityStrippingCheckbox);
 
-  var byteLoggingCheckbox = $(byteLoggingCheckboxId);
-  byteLoggingCheckbox.onclick =
-      this.onSetByteLogging_.bind(this, byteLoggingCheckbox);
-
-  this.downloadIframe_ = $(downloadIframeId);
-
-  this.saveFileButton_ = $(saveFileButtonId);
-  this.saveFileButton_.onclick = this.onSaveFile_.bind(this);
-  this.saveStatusText_ = $(dataViewSaveStatusTextId);
-
-  this.activelyCapturedCountBox_ = $(activelyCapturedCountId);
-  this.passivelyCapturedCountBox_ = $(passivelyCapturedCountId);
-  $(deleteAllId).onclick = g_browser.sourceTracker.deleteAllSourceEntries.bind(
-      g_browser.sourceTracker);
-
-  this.dumpDataDiv_ = $(dumpDataDivId);
-  this.capturingTextSpan_ = $(capturingTextSpanId);
-  this.loggingTextSpan_ = $(loggingTextSpanId);
-
   this.loadedDiv_ = $(loadedDivId);
   this.loadedClientInfoText_ = $(loadedClientInfoTextId);
 
@@ -69,98 +28,29 @@
   dropTarget.ondragenter = this.onDrag.bind(this);
   dropTarget.ondragover = this.onDrag.bind(this);
   dropTarget.ondrop = this.onDrop.bind(this);
-
-  this.updateEventCounts_();
-
-  // Track blob for previous log dump so it can be revoked when a new dump is
-  // saved.
-  this.lastBlobURL_ = null;
-
-  g_browser.sourceTracker.addObserver(this);
 }
 
-inherits(DataView, DivView);
+inherits(ImportView, DivView);
 
 /**
- * Called whenever a new event is received.
- */
-DataView.prototype.onSourceEntryUpdated = function(sourceEntry) {
-  this.updateEventCounts_();
-};
-
-/**
- * Called whenever some log events are deleted.  |sourceIds| lists
- * the source IDs of all deleted log entries.
- */
-DataView.prototype.onSourceEntriesDeleted = function(sourceIds) {
-  this.updateEventCounts_();
-};
-
-/**
- * Called whenever all log events are deleted.
- */
-DataView.prototype.onAllSourceEntriesDeleted = function() {
-  this.updateEventCounts_();
-};
-
-/**
  * Called when a log file is loaded, after clearing the old log entries and
- * loading the new ones.  Hides parts of the page related to saving data, and
- * shows those related to loading.  Returns true to indicate the view should
+ * loading the new ones.  Returns true to indicate the view should
  * still be visible.
  */
-DataView.prototype.onLoadLogFinish = function(data) {
-  setNodeDisplay(this.dumpDataDiv_, false);
-  setNodeDisplay(this.capturingTextSpan_, false);
-  setNodeDisplay(this.loggingTextSpan_, true);
+ImportView.prototype.onLoadLogFinish = function(data) {
   setNodeDisplay(this.loadedDiv_, true);
   this.updateLoadedClientInfo();
   return true;
 };
 
 /**
- * Updates the counters showing how many events have been captured.
- */
-DataView.prototype.updateEventCounts_ = function() {
-  this.activelyCapturedCountBox_.textContent =
-      g_browser.sourceTracker.getNumActivelyCapturedEvents();
-  this.passivelyCapturedCountBox_.textContent =
-      g_browser.sourceTracker.getNumPassivelyCapturedEvents();
-};
-
-/**
- * Depending on the value of the checkbox, enables or disables logging of
- * actual bytes transferred.
- */
-DataView.prototype.onSetByteLogging_ = function(byteLoggingCheckbox) {
-  if (byteLoggingCheckbox.checked) {
-    g_browser.setLogLevel(LogLevelType.LOG_ALL);
-  } else {
-    g_browser.setLogLevel(LogLevelType.LOG_ALL_BUT_BYTES);
-  }
-};
-
-/**
- * Depending on the value of the checkbox, enables or disables stripping
- * cookies and passwords from log dumps and displayed events.
- */
-DataView.prototype.onSetSecurityStripping_ =
-    function(securityStrippingCheckbox) {
-  g_browser.sourceTracker.setSecurityStripping(
-      securityStrippingCheckbox.checked);
-};
-
-DataView.prototype.onSecurityStrippingChanged = function() {
-};
-
-/**
  * Called when something is dragged over the drop target.
  *
  * Returns false to cancel default browser behavior when a single file is being
  * dragged.  When this happens, we may not receive a list of files for security
  * reasons, which is why we allow the |files| array to be empty.
  */
-DataView.prototype.onDrag = function(event) {
+ImportView.prototype.onDrag = function(event) {
   return event.dataTransfer.types.indexOf('Files') == -1 ||
          event.dataTransfer.files.length > 1;
 };
@@ -169,16 +59,16 @@
  * Called when something is dropped onto the drop target.  If it's a single
  * file, tries to load it as a log file.
  */
-DataView.prototype.onDrop = function(event) {
+ImportView.prototype.onDrop = function(event) {
   if (event.dataTransfer.types.indexOf('Files') == -1 ||
       event.dataTransfer.files.length != 1) {
     return;
   }
   event.preventDefault();
 
-  // Loading a log file may hide the currently active tab.  Switch to the data
+  // Loading a log file may hide the currently active tab.  Switch to the import
   // tab to prevent this.
-  document.location.hash = 'data';
+  document.location.hash = 'import';
 
   this.loadLogFile(event.dataTransfer.files[0]);
 };
@@ -188,14 +78,14 @@
  *
  * Gets the log file from the input element and tries to read from it.
  */
-DataView.prototype.logFileChanged = function() {
+ImportView.prototype.logFileChanged = function() {
   this.loadLogFile(this.loadFileElement_.files[0]);
 };
 
 /**
  * Attempts to read from the File |logFile|.
  */
-DataView.prototype.loadLogFile = function(logFile) {
+ImportView.prototype.loadLogFile = function(logFile) {
   if (logFile) {
     this.setLoadFileStatus('Loading log...', true);
     var fileReader = new FileReader();
@@ -211,7 +101,7 @@
  * Displays an error message when unable to read the selected log file.
  * Also clears the file input control, so the same file can be reloaded.
  */
-DataView.prototype.onLoadLogFileError = function(event) {
+ImportView.prototype.onLoadLogFileError = function(event) {
   this.loadFileElement_.value = null;
   this.setLoadFileStatus(
       'Error ' + getKeyWithValue(FileError, event.target.error.code) +
@@ -219,24 +109,22 @@
       false);
 };
 
-DataView.prototype.onLoadLogFile = function(logFile, event) {
+ImportView.prototype.onLoadLogFile = function(logFile, event) {
   var result = loadLogFile(event.target.result, logFile.fileName);
   this.setLoadFileStatus(result, false);
 };
 
 /**
  * Sets the load from file status text, displayed below the load file button,
- * to |text|.  Also enables or disables the save/load buttons based on the value
+ * to |text|.  Also enables or disables the load buttons based on the value
  * of |isLoading|, which must be true if the load process is still ongoing, and
  * false when the operation has stopped, regardless of success of failure.
  * Also, when loading is done, replaces the load button so the same file can be
  * loaded again.
  */
-DataView.prototype.setLoadFileStatus = function(text, isLoading) {
+ImportView.prototype.setLoadFileStatus = function(text, isLoading) {
   this.enableLoadFileElement_(!isLoading);
-  this.enableSaveFileButton_(!isLoading);
   this.loadStatusText_.textContent = text;
-  this.saveStatusText_.textContent = '';
 
   if (!isLoading) {
     // Clear the button, so the same file can be reloaded.  Recreating the
@@ -249,60 +137,14 @@
   }
 };
 
-/**
- * Sets the save to file status text, displayed below the save to file button,
- * to |text|.  Also enables or disables the save/load buttons based on the value
- * of |isSaving|, which must be true if the save process is still ongoing, and
- * false when the operation has stopped, regardless of success of failure.
- */
-DataView.prototype.setSaveFileStatus = function(text, isSaving) {
-  this.enableSaveFileButton_(!isSaving);
-  this.enableLoadFileElement_(!isSaving);
-  this.loadStatusText_.textContent = '';
-  this.saveStatusText_.textContent = text;
-};
-
-DataView.prototype.enableSaveFileButton_ = function(enabled) {
-  this.saveFileButton_.disabled = !enabled;
-};
-
-DataView.prototype.enableLoadFileElement_ = function(enabled) {
+ImportView.prototype.enableLoadFileElement_ = function(enabled) {
   this.loadFileElement_.disabled = !enabled;
 };
 
 /**
- * If not already busy loading or saving a log dump, triggers asynchronous
- * generation of log dump and starts waiting for it to complete.
- */
-DataView.prototype.onSaveFile_ = function() {
-  if (this.saveFileButton_.disabled)
-    return;
-  // Clean up previous blob, if any, to reduce resource usage.
-  if (this.lastBlobURL_) {
-    window.webkitURL.revokeObjectURL(this.lastBlobURL_);
-    this.lastBlobURL_ = null;
-  }
-  this.setSaveFileStatus('Preparing data...', true);
-
-  createLogDumpAsync(this.onLogDumpCreated_.bind(this));
-};
-
-/**
- * Creates a blob url and starts downloading it.
- */
-DataView.prototype.onLogDumpCreated_ = function(dumpText) {
-  var blobBuilder = new WebKitBlobBuilder();
-  blobBuilder.append(dumpText, 'native');
-  var textBlob = blobBuilder.getBlob('octet/stream');
-  this.lastBlobURL_ = window.webkitURL.createObjectURL(textBlob);
-  this.downloadIframe_.src = this.lastBlobURL_;
-  this.setSaveFileStatus('Dump successful', false);
-};
-
-/**
  * Prints some basic information about the environment when the log was made.
  */
-DataView.prototype.updateLoadedClientInfo = function() {
+ImportView.prototype.updateLoadedClientInfo = function() {
   this.loadedClientInfoText_.textContent = '';
   if (typeof(ClientInfo) != 'object')
     return;