Add UI for RuntimeErrors in the ErrorConsole

chrome/browser/resources/extensions/extension_list.js

Index: chrome/browser/resources/extensions/extension_list.js
diff --git a/chrome/browser/resources/extensions/extension_list.js b/chrome/browser/resources/extensions/extension_list.js
index ec0b53e578e8b12f2296b3883925af97f15e0bf1..1806516a97a4969a36d7cbadfce577e5895127c2 100644
--- a/chrome/browser/resources/extensions/extension_list.js
+++ b/chrome/browser/resources/extensions/extension_list.js
@@ -305,13 +305,21 @@ cr.define('options', function() {
         });
       }
 
-      // The manifest errors and warnings, in one of two formats (depending on
-      // if the ErrorConsole is enabled).
-      if (extension.manifestErrors) {
-        var manifestErrors = node.querySelector('.manifest-errors');
-        manifestErrors.hidden = false;
-        manifestErrors.appendChild(
-            new extensions.ExtensionErrorList(extension.manifestErrors));
+      // If the ErrorConsole is enabled, handle the manifest and runtime errors.
+      // Otherwise, handle any install warnings.
+      if (extension.manifestErrors || extension.runtimeErrors) {
+        if (extension.manifestErrors) {
+          var panel = node.querySelector('.manifest-errors-wrapper');
+          panel.hidden = false;
+          panel.appendChild(new extensions.ExtensionErrorList(
+              extension.manifestErrors, 'manifest'));
+        }
+        if (extension.runtimeErrors) {
+          var panel = node.querySelector('.runtime-errors-wrapper');
+          panel.hidden = false;
+          panel.appendChild(new extensions.ExtensionErrorList(
+              extension.runtimeErrors, 'runtime'));
+        }
       } else if (extension.installWarnings) {

[not at google - send to devlin, 2013/09/05 00:37:16]

I'm puzzled by the else here. Why can't this be:

if (extension.manifestErrors) {
  ... manifest error stuff
}

if (extension.runtimeErrors) {
  ... runtime error stuff
}

if (extension.installWarnings) {
  ... install warning stuff
}

[Devlin, 2013/09/05 17:53:55]

It can be. I had it like that for two reasons:
1. It makes the logic for possible entities more clear: if we have
runtime/manifest errors, we do not have install warnings.
2. It adds an extra layer to make sure we don't show something silly if our
assumptions are ever wrong (even though they shouldn't be).

But, of course, if the block is more puzzling than the alternative, clearly #1
failed. So changed. :)

[not at google - send to devlin, 2013/09/06 17:05:19]

Yeah - it would be simpler to just render what we're told to. Let the C++ code
(which is actually testable) enforce consistency.

         var panel = node.querySelector('.install-warnings');
         panel.hidden = false;