DevTools: [Documentation] Update parser for WikiText

Source/devtools/front_end/documentation/DocumentationView.js

Index: Source/devtools/front_end/documentation/DocumentationView.js
diff --git a/Source/devtools/front_end/documentation/DocumentationView.js b/Source/devtools/front_end/documentation/DocumentationView.js
index e9ce67a045719a85d94c0e618491a9ce95dacbb6..2221ff94f3c0c355bdda042dbd8518145e794185 100644
--- a/Source/devtools/front_end/documentation/DocumentationView.js
+++ b/Source/devtools/front_end/documentation/DocumentationView.js
@@ -22,6 +22,7 @@ WebInspector.DocumentationView.showDocumentationURL = function(url, searchItem)
     if (!WebInspector.DocumentationView._view)
         WebInspector.DocumentationView._view = new WebInspector.DocumentationView();
     var view = WebInspector.DocumentationView._view;
+    view.element.removeChildren();
     WebInspector.inspectorView.showCloseableViewInDrawer("documentation", WebInspector.UIString("Documentation"), view);
     view.showDocumentation(url, searchItem);
 }
@@ -61,14 +62,21 @@ WebInspector.DocumentationView.prototype = {
             return;
         }
         var wikiMarkupText = pages[wikiKeys[0]]["revisions"]["0"]["*"];
-        var article = WebInspector.JSArticle.parse(wikiMarkupText);
+        var article;
+        try {
+            article = WebInspector.JSArticle.parse(wikiMarkupText);
+        } catch (error) {
+            this._createEmptyPage();

[apavlov, 2014/09/10 14:09:23]

This and the following line can be dropped:
if (!article) {...} below will _createEmptyPage the second time otherwise.

[iliia, 2014/09/11 09:26:30]

Done.

+            article = null;
+            console.error(error);

[apavlov, 2014/09/10 14:09:23]

Should this be error.message?

[iliia, 2014/09/11 09:26:30]

Done.

[apavlov, 2014/09/11 10:40:26]

Not done - still, the "error" object is used instead of error.message

[iliia, 2014/09/11 13:26:28]

Done.

+        }
         if (!article) {
             this._createEmptyPage();
             return;
         }
 
-        var renderer = new WebInspector.DocumentationView.Renderer(article, searchItem);
         this.element.removeChildren();
+        var renderer = new WebInspector.DocumentationView.Renderer(article, searchItem);
         this.element.appendChild(renderer.renderJSArticle());
     },
 
@@ -258,7 +266,7 @@ WebInspector.DocumentationView.Renderer.prototype = {
 
     /**
      * @param {!WebInspector.WikiParser.ArticleElement} article
-     * @return {!Element}
+     * @return {?Element}

[apavlov, 2014/09/10 14:09:23]

Any reason for this being nullable?

[iliia, 2014/09/11 09:26:30]

element could be undefined if something's going wrong.
On 2014/09/10 14:09:23, apavlov (OOO until Sept.19) wrote:

      */
     _renderBlock: function(article)
     {