Adding error handling to ONC validation.

chrome/browser/resources/net_internals/chromeos_view.js

 // Copyright (c) 2012 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.
 
 /**
  * This view displays information on ChromeOS specific features.
  */
 var CrosView = (function() {
   'use strict';
 
@@ skipping 15 matching lines @@
   /**
    *  Send file contents and passcode to C++ cros network library.
    *
    *  @private
    */
   function importONCFile_() {
     clearParseStatus_();
     if (fileContent)
       g_browser.importONCFile(fileContent, passcode);
     else
-      setParseStatus_('ONC file parse failed: cannot read file');
+      setParseStatus_('ONC file parse failed: cannot read file', false);
     clearFileInput_();
   }
 
   /**
    *  Set the passcode var, and trigger onc import.
    *
    *  @param {string} value The passcode value.
    *  @private
    */
   function setPasscode_(value) {
@@ skipping 36 matching lines @@
     } else {
       importONCFile_();
     }
   }
 
   /**
    *  Clear ONC file parse status.  Clears and hides the parse status div.
    *
    *  @private
    */
-  function clearParseStatus_(error) {
+  function clearParseStatus_() {
     var parseStatus = $(CrosView.PARSE_STATUS_ID);
     parseStatus.hidden = true;
     parseStatus.textContent = '';
   }
 
   /**
    *  Set ONC file parse status.
    *
    *  @private
    */
-  function setParseStatus_(error) {
+  function setParseStatus_(message, success) {
     var parseStatus = $(CrosView.PARSE_STATUS_ID);
     parseStatus.hidden = false;
-    parseStatus.textContent = error ?
-        'ONC file parse failed: ' + error : 'ONC file successfully parsed';
+
+    if (success) {
+      if (message) {
+        message = 'ONC file parsed and imported, however with warnings:\n' +
+            message;
+      } else {
+        message = 'ONC file parsed and imported.';
+      }
+    } else {
+      message = 'ONC file parse failed:\n' + message;
+    }
+
+    parseStatus.innerHTML = message.replace(/\n/g, '<br>');
     reset_();
   }
 
   /**
    *  Set storing debug logs status.
    *
    *  @private
    */
   function setStoreDebugLogsStatus_(status) {
     $(CrosView.STORE_DEBUG_LOGS_STATUS_ID).innerText = status;
@@ skipping 118 matching lines @@
     // Inherit from DivView.
     __proto__: DivView.prototype,
 
     onONCFileParse: setParseStatus_,
     onStoreDebugLogs: setStoreDebugLogsStatus_,
     onSetNetworkDebugMode: setNetworkDebugModeStatus_,
   };
 
   return CrosView;
 })();