Cleanup mdn scripts

utils/apidoc/mdn/extract.dart

 #import ("dart:html");
 #import ("dart:htmlimpl");
 #import ("dart:dom", prefix:"dom");
 #import ("dart:json");
 
 // Workaround for HTML lib missing feature.
 Range newRange() {
   return LevelDom.wrapRange(dom.document.createRange());
 }
 
 // Temporary range object to optimize performance computing client rects
 // from text nodes.
 Range _tempRange;
 // Hacks because ASYNC measurement is annoying when just writing a script.
 ClientRect getClientRect(Node n) {
   if (n is Element) {
-    Element e = n;
-    dom.Element raw = unwrapDomObject(e.dynamic);
+    dom.Element raw = unwrapDomObject(n.dynamic);
     return LevelDom.wrapClientRect(raw.getBoundingClientRect());
   } else {
     // Crazy hacks that works for nodes.... create a range and measure it.
     if (_tempRange == null) {
       _tempRange = newRange();
     }
     _tempRange.setStartBefore(n);
     _tempRange.setEndAfter(n);
      return _tempRange.getBoundingClientRect();
   }
 }
 
-final DART_REMOVED = "dart_removed";
+/**
+ * CSS class that is added to elements in the DOM to indicate that they should
+ * be removed when extracting blocks of documentation.  This is helpful when
+ * running this script in a web browser as it is easy to visually see what
+ * blocks of information were extracted when using CSS such as DEBUG_CSS
+ * which highlights elements that should be removed. 
+ */
+final DART_REMOVED = "dart-removed";
 
 final DEBUG_CSS = """
 <style type="text/css">
-  .dart_removed {
+  .dart-removed {
     background-color: rgba(255, 0, 0, 0.5);
    }
 </style>""";
 
 final MIN_PIXELS_DIFFERENT_LINES = 10;
 
 final IDL_SELECTOR = "pre.eval, pre.idl";
 
 Map data;
 
@@ skipping 228 matching lines @@
   if (path.startsWith('/')) {
     return "$pageDomain$path";
   } else if (path.startsWith("#")) {
     return "$pageUrl$path";
   } else {
     return "$pageDir$path";
   }
 }
 
 bool inTable(Node n) {
-  while(n != null) {
+  while (n != null) {
     if (n is TableElement) return true;
     n = n.parent;
   }
   return false;
 }
 
 String escapeHTML(str) {
   Element e = new Element.tag("div");
   e.text = str;
   return e.innerHTML;
 }
 
 List<Text> getAllTextNodes(Element elem) {
-  List<Text> nodes = <Text>[];
+  final nodes = <Text>[];

[Bob Nystrom, 2012/02/01 18:41:13]

:D

[Jacob, 2012/02/01 21:12:02]

Done.

   helper(Node n) {
     if (n is Text) {
       nodes.add(n);
     } else {
       for (Node child in n.nodes) {
         helper(child);
       }
     }
   };
 
   helper(elem);
   return nodes;
 }
 
 /**
  * Whether a node and its children are all types that are safe to skip if the
  * nodes have no text content.
  */
 bool isSkippableType(Node n) {
   // TODO(jacobr): are there any types we don't want to skip even if they
   // have no text content?
   if (n is ImageElement || n is CanvasElement || n is InputElement
       || n is ObjectElement) {
     return false;
   }
   if (n is Text) return true;
 
-  for (Node child in n.nodes) {
-    if (isSkippableType(child) == false) {
+  for (final child in n.nodes) {
+    if (!isSkippableType(child)) {
       return false;
     }
   }
   return true;
 }
 
 bool isSkippable(Node n) {
   if (!isSkippableType(n)) return false;
   return n.text.trim().length == 0;
 }
 
 void onEnd() {
   // Hideous hack to send JSON back to JS.
   String dbJson = JSON.stringify(dbEntry);
   // workaround bug in JSON parser.
   dbJson = dbJson.replaceAll("ZDARTIUMDOESNTESCAPESLASHNJXXXX", "\\n");
 
   window.postMessage("START_DART_MESSAGE_UNIQUE_IDENTIFIER$dbJson", "*");
 }
 
 class SectionParseResult {
   final String html;
   final String url;
   final String idl;
   SectionParseResult(this.html, this.url, this.idl);
 }
 
 String genCleanHtml(Element root) {
-  for (Element e in root.queryAll(".$DART_REMOVED")) {
+  for (final e in root.queryAll(".$DART_REMOVED")) {
     e.classes.remove(DART_REMOVED);
   }
 
   // Ditch inline styles.
-  for (Element e in root.queryAll('[style]')) {
+  for (final e in root.queryAll('[style]')) {
     e.attributes.remove('style');
   }
 
   // These elements are just tags that we should suppress.
-  for (Element e in root.queryAll(".lang.lang-en")) {
+  for (final e in root.queryAll(".lang.lang-en")) {
     e.remove();
   }
 
+  Element parametersList;
+  Element returnValue;
+  for (final e in root.queryAll("h6")) {
+    if (e.text == 'Parameters') {
+      parametersList = e;
+    } else if (e.text == 'Return value') {
+      returnValue = e;
+    }
+  }
+  
+  if (parametersList != null) {
+    int numEmptyParameters = 0;
+    final parameterDescriptions = root.queryAll("dd");
+    for (Element parameterDescription in parameterDescriptions) {
+      if (parameterDescription.text.trim().length == 0) {
+        numEmptyParameters++;
+      }
+    }
+    if (numEmptyParameters > 0 &&
+        numEmptyParameters == parameterDescriptions.length) {
+      // Remove the parameter list as it adds zero value as all descriptions
+      // are empty.
+      parametersList.remove();
+      for (final e in root.queryAll("dl")) {
+        e.remove();
+      }   
+    } else if (parameterDescriptions.length == 0 &&
+        parametersList.nextElementSibling != null &&
+        parametersList.nextElementSibling.text.trim() == 'None.') {
+      // No need to display that the function takes 0 parameters.
+      parametersList.nextElementSibling.remove();
+      parametersList.remove();
+    }
+  }
+
+  // Heuristic: if the return value is a single word it is a type name not a
+  // useful text description so suppress it.

[Bob Nystrom, 2012/02/01 18:41:13]

Comments like this are very helpful.

[Jacob, 2012/02/01 21:12:02]

Done.

+  if (returnValue != null &&
+      returnValue.nextElementSibling != null &&
+      returnValue.nextElementSibling.text.trim().split(' ').length <= 1) {
+    returnValue.nextElementSibling.remove();
+    returnValue.remove();
+  }
+
   bool changed = true;
   while (changed) {
     changed = false;
-    while (root.nodes.length == 1) {
-      Node child = root.nodes.first;
-      if (child is Element) {
-        root = child;
-        changed = true;
-      } else {
-        // Just calling innerHTML on the parent will be sufficient...
-        // and insures the output is properly escaped.
-        break;
-      }
+    while (root.nodes.length == 1 && root.nodes.first is Element) {
+      root = root.nodes.first;
+      changed = true;
     }
 
     // Trim useless nodes from the front.
-    while(root.nodes.length > 0 &&
+    while (root.nodes.length > 0 &&
         isSkippable(root.nodes.first)) {
       root.nodes.first.remove();
       changed = true;
     }
 
     // Trim useless nodes from the back.
-    while(root.nodes.length > 0 &&
+    while (root.nodes.length > 0 &&
         isSkippable(root.nodes.last())) {
       root.nodes.last().remove();
       changed = true;
     }
   }
   return JSONFIXUPHACK(root.innerHTML);
 }
 
-String genPrettyHtml(DocumentFragment fragment) {
-  return genCleanHtml(fragment);
-}
-
 String genPrettyHtmlFromElement(Element e) {
   e = e.clone(true);
   return genCleanHtml(e);
 }
 
 class PostOrderTraversalIterator implements Iterator<Node> {
 
   Node _next;
 
   PostOrderTraversalIterator(Node start) {
     _next = _leftMostDescendent(start);
   }
 
   bool hasNext() => _next != null;
 
   Node next() {
     if (_next == null) return null;
-    Node ret = _next;
+    final ret = _next;
     if (_next.nextNode != null) {
       _next = _leftMostDescendent(_next.nextNode);
     } else {
       _next = _next.parent;
     }
     return ret;
   }
 
   static Node _leftMostDescendent(Node n) {
     while (n.nodes.length > 0) {
       n = n.nodes.first;
     }
     return n;
   }
 }
 
 class PostOrderTraversal implements Iterable<Node> {
   final Node _node;
   PostOrderTraversal(this._node);
 
   Iterator<Node> iterator() => new PostOrderTraversalIterator(_node);
 }
 
 Range findFirstLine(Range section, String prop) {
-  Range firstLine = newRange();
+  final firstLine = newRange();
   firstLine.setStart(section.startContainer, section.startOffset);
 
   num maxBottom = null;
-  for (Node n in new PostOrderTraversal(section.startContainer)) {
+  for (final n in new PostOrderTraversal(section.startContainer)) {
     int compareResult = section.comparePoint(n, 0);
     if (compareResult == -1) {
       // before range so skip.
       continue;
     } else if (compareResult > 0) {
       // After range so exit.
       break;
     }
 
     final rect = getClientRect(n);
     num bottom = rect.bottom;
     if (rect.height > 0 && rect.width > 0) {
-      if (maxBottom != null && (
-        maxBottom + MIN_PIXELS_DIFFERENT_LINES < bottom
-        )) {
+      if (maxBottom != null &&
+          maxBottom + MIN_PIXELS_DIFFERENT_LINES < bottom) {
         break;
       } else if (maxBottom == null || maxBottom > bottom) {
         maxBottom = bottom;
       }
     }
 
     firstLine.setEndAfter(n);
   }
 
-  if (firstLine.toString().indexOf(stripWebkit(prop)) == -1) {
+  // If the first line of text in the section does not contain the property
+  // name then we're not confident we are able to extract a high accuracy match
+  // so we should not return anything.
+  if (!firstLine.toString().contains(stripWebkit(prop))) {
     return null;
   }
   return firstLine;
 }
 
 AnchorElement findAnchorElement(Element root, String prop) {
   for (AnchorElement a in root.queryAll("a")) {
-    if (a.text.indexOf(prop) != -1) {
+    if (a.text.contains(prop)) {
       return a;
     }
   }
   return null;
 }
 
 // First surrounding element with an ID is safe enough.
-Element findTigherRoot(Element elem, Element root) {
+Element findTighterRoot(Element elem, Element root) {
   Element candidate = elem;
-  while(root != candidate) {
+  while (root != candidate) {
     candidate = candidate.parent;
     if (candidate.id.length > 0 && candidate.id.indexOf("section_") != 0) {
       break;
     }
   }
   return candidate;
 }
 
-// this is very slow and ugly.. consider rewriting.
+// TODO(jacobr): this is very slow and ugly.. consider rewriting or at least
+// commenting carefully.
 SectionParseResult filteredHtml(Element elem, Element root, String prop,
     Function fragmentGeneratedCallback) {
   // Using a tighter root avoids false positives at the risk of trimming
   // text we shouldn't.
-  root = findTigherRoot(elem, root);
-  Range range = newRange();
+  root = findTighterRoot(elem, root);
+  final range = newRange();
   range.setStartBefore(elem);
 
   Element current = elem;
   while (current != null) {
     range.setEndBefore(current);
-    if (current.classes.contains(DART_REMOVED)) {
-      if (range.toString().trim().length > 0) {
-        break;
-      }
+    if (current.classes.contains(DART_REMOVED) &&
+        range.toString().trim().length > 0) {
+      break;
     }
     if (current.firstElementChild != null) {
       current = current.firstElementChild;
     } else {
       while (current != null) {
         range.setEndAfter(current);
         if (current == root) {
           current = null;
           break;
         }
@@ skipping 10 matching lines @@
     Range firstLine = findFirstLine(range, prop);
     if (firstLine != null) {
       range.setStart(firstLine.endContainer, firstLine.endOffset);
       DocumentFragment firstLineClone = firstLine.cloneContents();
       AnchorElement anchor = findAnchorElement(firstLineClone, prop);
       if (anchor != null) {
         url = getAbsoluteUrl(anchor);
       }
     }
   }
-  DocumentFragment fragment = range.cloneContents();
+  final fragment = range.cloneContents();
   if (fragmentGeneratedCallback != null) {
     fragmentGeneratedCallback(fragment);
   }
   // Strip tags we don't want
   for (Element e in fragment.queryAll("script, object, style")) {
     e.remove();
   }
 
   // Extract idl
-  StringBuffer idl = new StringBuffer();
+  final idl = new StringBuffer();
   if (prop != null && prop.length > 0) {
     // Only expect properties to have HTML.
     for(Element e in fragment.queryAll(IDL_SELECTOR)) {
       idl.add(e.outerHTML);
       e.remove();
     }
     // TODO(jacobr) this is a very basic regex to see if text looks like IDL
     RegExp likelyIdl = new RegExp(" $prop\\w*\\(");
 
     for (Element e in fragment.queryAll("pre")) {
       // Check if it looks like idl...
       String txt = e.text.trim();
-      if (likelyIdl.hasMatch(txt) && txt.indexOf("\n") != -1
-          && txt.indexOf(")") != -1) {
+      if (likelyIdl.hasMatch(txt) && txt.contains("\n")
+          && txt.contains(")")) {

[Bob Nystrom, 2012/02/01 18:41:13]

This will probably fit on one line now.

[Jacob, 2012/02/01 21:12:02]

Done.

         idl.add(e.outerHTML);
         e.remove();
       }
     }
   }
-  return new SectionParseResult(genPrettyHtml(fragment), url, idl.toString());
+  return new SectionParseResult(genCleanHtml(fragment), url, idl.toString());
 }
 
-Element findBest(Element root, List<Text> allText, String prop, String propType) {
+Element findBest(Element root, List<Text> allText, String prop,
+    String propType) {
   // Best bet: match an id
   Element cand;
-  cand = root.query("#" + prop);
+  cand = root.query("#$prop");
 
   if (cand == null && propType == "methods") {
-    cand = root.query("[id=" + prop + "\\(\\)]");
+    cand = root.query("[id=$prop\\(\\)]");
+  }
+  while (cand != null && cand.text.trim().length == 0) {
+    // We found the bookmark for the element but sadly it is just an empty
+    // placeholder. Find the first real element.
+    cand = cand.nextElementSibling;
   }
   if (cand != null) {
-    while (cand != null && cand.text.trim().length == 0) {
-      // We found the bookmark for the element but sadly it is just an empty
-      // placeholder. Find the first real element.
-      cand = cand.nextElementSibling;
-    }
-    if (cand != null) {
-      return cand;
-    }
+    return cand;
   }
 
-  // If you are at least 70 pixels from the left, something is definitely fishy and we shouldn't even consider this candidate.
+  // If you are at least 70 pixels from the left, something is definitely
+  //  fishy and we shouldn't even consider this candidate.
   num candLeft = 70;
 
   for (Text text in allText) {
     Element proposed = null;
 
-//    var t = safeNameCleanup(text.text);
-// TODO(jacobr): does it hurt precision to use the full cleanup?
+    // TODO(jacobr): does it hurt precision to use the full cleanup?
     String t = fullNameCleanup(text.text);
     if (t == prop) {
       proposed = text.parent;
       ClientRect candRect = getClientRect(proposed);
 
       // TODO(jacobr): this is a good heuristic
       // if (selObj.selector.indexOf(" > DD ") == -1
       if (candRect.left < candLeft) {
         cand = proposed;
         candLeft = candRect.left;
       }
     }
   }
   return cand;
 }
 
 bool isObsolete(Element e) {
   RegExp obsoleteRegExp = new RegExp(@"(^|\s)obsolete(?=\s|$)");
   RegExp deprecatedRegExp = new RegExp(@"(^|\s)deprecated(?=\s|$)");
   for (Element child in e.queryAll("span")) {
     String t = child.text.toLowerCase();
     if (t.startsWith("obsolete") || t.startsWith("deprecated")) return true;
   }
 
   String text = e.text.toLowerCase();
   return obsoleteRegExp.hasMatch(text) || deprecatedRegExp.hasMatch(text);
 }
 
 bool isFirstCharLowerCase(String str) {
-  RegExp firstLower = new RegExp("^[a-z]");
-  return firstLower.hasMatch(str);
+  return const RegExp("^[a-z]").hasMatch(str);
 }
 
-void scrapeSection(Element root, String sectionSelector,
-                   String currentType,
-                   List members,
-                   String propType) {
+// TODO(jacobr): document this method.
+void scrapeSection(Element root, String sectionSelector, String currentType,
+    List members, String propType) {
   Map expectedProps = dartIdl[propType];
 
   Set<String> alreadyMatchedProperties = new Set<String>();
   bool onlyConsiderTables = false;
   ElementList allMatches = root.queryAll(sectionSelector);
   if (allMatches.length == 0) {
     allMatches = root.queryAll(".fullwidth-table");
     onlyConsiderTables = true;
   }
   for (Element matchElement in allMatches) {
     DivElement match = matchElement.parent;
-    if (!match.id.startsWith("section") && !(match.id == "pageText")) {
-      throw "Enexpected element $match";
+    if (!match.id.startsWith("section") && match.id != "pageText") {
+      throw "Unexpected element $match";
     }
     match.classes.add(DART_REMOVED);
 
     bool foundProps = false;
 
     // TODO(jacobr): we should really look for the table tag instead
     // add an assert if we are missing something that is a table...
     // TODO(jacobr) ignore tables in tables....
     for (Element t in match.queryAll('.standard-table, .fullwidth-table')) {
       int helpIndex = -1;
       num i = 0;
       for (Element r in t.queryAll("th, td.header")) {
-        var txt = r.text.trim().split(" ")[0].toLowerCase();
+        final txt = r.text.trim().split(" ")[0].toLowerCase();
         if (txt == "description") {
           helpIndex = i;
           break;
         }
         i++;
       }
 
       List<int> numMatches = new List<int>(i);
       for (int j = 0; j < i; j++) {
         numMatches[j] = 0;
       }
 
       // Find the row that seems to have the most names that look like
       // expected properties.
       for (Element r in t.queryAll("tbody tr")) {
-        ElementList $row = r.elements;
-        if ($row.length == 0 || $row.first.classes.contains(".header")) {
+        ElementList row = r.elements;
+        if (row.length == 0 || row.first.classes.contains(".header")) {
           continue;
         }
 
-        for (int k = 0; k < numMatches.length && k < $row.length; k++) {
-          Element e = $row[k];
-          if (expectedProps.containsKey(fullNameCleanup(e.text))) {
+        for (int k = 0; k < numMatches.length && k < row.length; k++) {
+          if (expectedProps.containsKey(fullNameCleanup(row[k].text))) {
             numMatches[k]++;
             break;
           }
         }
       }
 
       int propNameIndex = 0;
       {
         int bestCount = numMatches[0];
         for (int k = 1; k < numMatches.length; k++) {
           if (numMatches[k] > bestCount) {
             bestCount = numMatches[k];
             propNameIndex = k;
           }
         }
       }
 
       for (Element r in t.queryAll("tbody tr")) {
-        ElementList $row = r.elements;
-        if ($row.length > propNameIndex && $row.length > helpIndex ) {
-          if ($row.first.classes.contains(".header")) {
+        ElementList row = r.elements;
+        if (row.length > propNameIndex && row.length > helpIndex) {
+          if (row.first.classes.contains(".header")) {
             continue;
           }
           // TODO(jacobr): this code for determining the namestr is needlessly
           // messy.
-          Element nameRow = $row[propNameIndex];
+          Element nameRow = row[propNameIndex];
           AnchorElement a = nameRow.query("a");
           String goodName = '';
           if (a != null) {
             goodName = a.text.trim();
           }
           String nameStr = nameRow.text;
 
           Map entry = new Map<String, String>();
 
-  //        "currentType": $($row[1]).text().trim(), // find("code") ?
-          entry["name"] = fullNameCleanup(nameStr.length > 0 ? nameStr : goodName);
+          entry["name"] = fullNameCleanup(nameStr.length > 0 ?
+              nameStr : goodName);
 
           final parse = filteredHtml(nameRow, nameRow, entry["name"], null);
           String altHelp = parse.html;
 
-         // "jsSignature": nameStr,
-          entry["help"] = (helpIndex == -1 || $row[helpIndex] == null) ? altHelp : genPrettyHtmlFromElement($row[helpIndex]);
-        //  "altHelp" : altHelp,
+          entry["help"] = (helpIndex == -1 || row[helpIndex] == null) ?
+              altHelp : genPrettyHtmlFromElement(row[helpIndex]);
           if (parse.url != null) {
             entry["url"] = parse.url;
           }
 
           if (parse.idl.length > 0) {
             entry["idl"] = parse.idl;
           }
 
           entry["obsolete"] = isObsolete(r);
 
@@ skipping 20 matching lines @@
       if (alreadyMatchedProperties.contains(prop)) {
         continue;
       }
       Element e = findBest(match, allText, prop, propType);
       if (e != null && !inTable(e)) {
         pmap[prop] = e;
       }
     }
 
     for (String prop in pmap.getKeys()) {
-      Element e = pmap[prop];
-      e.classes.add(DART_REMOVED);
+      pmap[prop].classes.add(DART_REMOVED);
     }
 
     for (String prop in pmap.getKeys()) {
       Element e = pmap[prop];
       ClientRect r = getClientRect(e);
       // TODO(jacobr): a lot of these queries are identical.
       for (Element cand in match.queryAll(e.tagName)) {
-        if (!cand.classes.contains(DART_REMOVED) && !inTable(cand) ) { // XXX use a neg selector.
+        // TODO(jacobr): use a negative selector instead.
+        if (!cand.classes.contains(DART_REMOVED) && !inTable(cand)) {
           ClientRect candRect = getClientRect(cand);
           // TODO(jacobr): this is somewhat loose.
           if (candRect.left == r.left &&
             (candRect.height - r.height).abs() < 5) {
             String propName = fullNameCleanup(cand.text);
-            if (isFirstCharLowerCase(propName) && pmap.containsKey(propName) == false && alreadyMatchedProperties.contains(propName) == false) {
-              // Don't set here to avoid layouts... cand.classes.add(DART_REMOVED);
+            if (isFirstCharLowerCase(propName) && !pmap.containsKey(propName)
+                && !alreadyMatchedProperties.contains(propName)) {
               pmap[propName] = cand;
             }
           }
         }
       }
     }
 
     for (String prop in pmap.getKeys()) {
       Element e = pmap[prop];
       e.classes.add(DART_REMOVED);
     }
 
     // Find likely "subsections" of the main section and mark them with
     // DART_REMOVED so we don't include them in member descriptions... which
     // would suck.
     for (Element e in match.queryAll("[id]")) {
-      if (e.id.indexOf(matchElement.id) != -1) {
+      if (e.id.contains(matchElement.id)) {
         e.classes.add(DART_REMOVED);
       }
     }
 
     for (String prop in pmap.getKeys()) {
       Element elem = pmap[prop];
       bool obsolete = false;
       final parse = filteredHtml(
         elem, match, prop,
         (Element e) {
           obsolete = isObsolete(e);
         });
       Map entry = {
         "url" : parse.url,
         "name" : prop,
         "help" : parse.html,
         "obsolete" : obsolete
-        //"jsSignature" : nameStr
       };
       if (parse.idl.length > 0) {
         entry["idl"] = parse.idl;
       }
       cleanupEntry(members, entry);
     }
   }
 }
 
 String trimHtml(String html) {
   // TODO(jacobr): impl.
   return html;
 }
 
 bool maybeName(String name) {

[Bob Nystrom, 2012/02/01 18:41:13]

Doesn't seem to be used?

[Jacob, 2012/02/01 21:12:02]

It is used.  You must be just searching this diff instead of the entire file.

-  RegExp nameRegExp = new RegExp("^[a-z][a-z0-9A-Z]+\$");
-  if (nameRegExp.hasMatch(name)) return true;
-  RegExp constRegExp = new RegExp("^[A-Z][A-Z_]*\$");
-  if (constRegExp.hasMatch(name)) return true;
+  return const RegExp("^[a-z][a-z0-9A-Z]+\$").hasMatch(name) ||
+      const RegExp("^[A-Z][A-Z_]*\$").hasMatch(name);
 }
 
 void markRemoved(var e) {
   if (e != null) {
     // TODO( remove)
     if (e is Element) {
       e.classes.add(DART_REMOVED);
     } else {
       for (Element el in e) {
         el.classes.add(DART_REMOVED);
       }
     }
   }
 }
 
 String JSONFIXUPHACK(String value) {
   return value.replaceAll("\n", "ZDARTIUMDOESNTESCAPESLASHNJXXXX");
 }
 
 String mozToWebkit(String name) {
-  RegExp regExp = new RegExp("^moz");
-  name = name.replaceFirst(regExp, "webkit");
-  return name;
+  return name.replaceFirst(const RegExp("^moz"), "webkit");
 }
 
 String stripWebkit(String name) {
   return trimPrefix(name, "webkit");
 }
 
 String fullNameCleanup(String name) {
   int parenIndex = name.indexOf('(');
   if (parenIndex != -1) {
     // TODO(jacobr): workaround bug in:
@@ skipping 59 matching lines @@
 
 // TODO(jacobr) dup with trim start....
 String trimPrefix(String str, String prefix) {
   if (str.indexOf(prefix) == 0) {
     return str.substring(prefix.length);
   } else {
     return str;
   }
 }
 
-void resourceLoaded() {
-  if (data != null) run();
-}
-
 String trimStart(String str, String start) {
   if (str.startsWith(start) && str.length > start.length) {
     return str.substring(start.length);
   }
   return str;
 }
 
 String trimEnd(String str, String end) {
   if (str.endsWith(end) && str.length > end.length) {
     return str.substring(0, str.length - end.length);
@@ skipping 13 matching lines @@
         dbEntry[key] += html;
       } else {
         dbEntry[key] = html;
       }
     }
     e.classes.add(DART_REMOVED);
   }
 }
 
 void run() {
-  // Inject CSS to insure lines don't wrap unless it was intentional.
+  // Inject CSS to ensure lines don't wrap unless it was intentional.
   document.head.nodes.add(new Element.html("""
 <style type="text/css">
   body {
     width: 10000px;
   }
 </style>"""));
 
   String title = trimEnd(window.document.title.trim(), " - MDN");
   dbEntry['title'] = title;
 
   // TODO(rnystrom): Clean up the page a bunch. Not sure if this is the best
   // place to do this...
 
   // Remove the "Introduced in HTML <version>" boxes.
   for (Element e in document.queryAll('.htmlVersionHeaderTemplate')) {
     e.remove();
   }
 
-  // Flatten the list of known DOM types into a faster and case-insensitive map.
+  // Flatten the list of known DOM types into a faster and case-insensitive
+  // map.
   domTypes = {};
   for (final domType in domTypesRaw) {
     domTypes[domType.toLowerCase()] = domType;
   }
 
   // Fix up links.
   final SHORT_LINK = const RegExp(@'^[\w/]+$');
   final INNER_LINK = const RegExp(@'[Ee]n/(?:[\w/]+/|)([\w#.]+)(?:\(\))?$');
   final MEMBER_LINK = const RegExp(@'(\w+)[.#](\w+)');
   final RELATIVE_LINK = const RegExp(@'^(?:../)*/?[Ee][Nn]/(.+)');
 
   // - Make relative links absolute.
   // - If we can, take links that point to other MDN pages and retarget them
   //   to appropriate pages in our docs.
   // TODO(rnystrom): Add rel external to links we didn't fix.
   for (AnchorElement a in document.queryAll('a')) {
     // Get the raw attribute because we *don't* want the browser to fully-
-    // qualify the name for us since it has the wrong base address for the page.
+    // qualify the name for us since it has the wrong base address for the
+    // page.
     var href = a.attributes['href'];
 
     // Ignore busted links.
     if (href == null) continue;
 
     // If we can recognize what it's pointing to, point it to our page instead.
     tryToLinkToRealType(maybeType) {
       // See if we know a type with that name.
       final realType = domTypes[maybeType.toLowerCase()];
       if (realType != null) {
@@ skipping 25 matching lines @@
         tryToLinkToRealType(member[1]);
       } else {
         tryToLinkToRealType(match[1]);
       }
     }
 
     // Put it back into the element.
     a.attributes['href'] = href;
   }
 
-  if (title.toLowerCase().indexOf(currentTypeTiny.toLowerCase()) == -1) {
+  if (!title.toLowerCase().contains(currentTypeTiny.toLowerCase())) {
     bool foundMatch = false;
     // Test out if the title is really an HTML tag that matches the
     // current class name.
     for (String tag in [title.split(" ")[0], title.split(".").last()]) {
       try {
         dom.Element element = dom.document.createElement(tag);
         if (element.typeName == currentType) {
           foundMatch = true;
           break;
         }
       } catch(e) {}
     }
-    if (foundMatch == false) {
+    if (!foundMatch ){

[Bob Nystrom, 2012/02/01 18:41:13]

Remove space before ")".

[Jacob, 2012/02/01 21:12:02]

rem   oved        space    )

       dbEntry['skipped'] = true;
       dbEntry['cause'] = "Suspect title";
       onEnd();
       return;
     }
   }
 
   Element root = document.query(".pageText");
   if (root == null) {
     dbEntry['cause'] = '.pageText not found';
     onEnd();
     return;
   }
 
   markRemoved(root.query("#Notes"));
   List members = dbEntry['members'];
 
   markRemoved(document.queryAll(".pageToc, footer, header, #nav-toolbar"));
   markRemoved(document.queryAll("#article-nav"));
   markRemoved(document.queryAll(".hideforedit"));
   markRemoved(document.queryAll(".navbox"));
   markRemoved(document.query("#Method_overview"));
   markRemoved(document.queryAll("h1, h2"));
 
   scrapeSection(root, "#Methods", currentType, members, 'methods');
-  scrapeSection(root, "#Constants, #Error_codes, #State_constants", currentType, members, 'constants');
+  scrapeSection(root, "#Constants, #Error_codes, #State_constants",
+      currentType, members, 'constants');
   // TODO(jacobr): infer tables based on multiple matches rather than
   // using a hard coded list of section ids.
   scrapeSection(root,
-      "[id^=Properties], #Notes, [id^=Other_properties], #Attributes, #DOM_properties, #Event_handlers, #Event_Handlers",
+      "[id^=Properties], #Notes, [id^=Other_properties], #Attributes, " +
+      "#DOM_properties, #Event_handlers, #Event_Handlers",
       currentType, members, 'properties');
 
   // Avoid doing this till now to avoid messing up the section scrape.
   markRemoved(document.queryAll("h3"));
 
-  ElementList $examples = root.queryAll("span[id^=example], span[id^=Example]");
+  ElementList examples = root.queryAll("span[id^=example], span[id^=Example]");
 
   extractSection("#See_also", 'seeAlso');
   extractSection("#Specification, #Specifications", "specification");
-  // $("#Methods").parent().remove(); // not safe (e.g. Document)
 
   // TODO(jacobr): actually extract the constructor(s)
   extractSection("#Constructor, #Constructors", 'constructor');
   extractSection("#Browser_compatibility, #Compatibility", 'compatibility');
 
   List<String> exampleHtml = [];
-  for (Element e in $examples) {
+  for (Element e in examples) {
     e.classes.add(DART_REMOVED);
   }
-  for (Element e in $examples) {
+  for (Element e in examples) {
     String html = filteredHtml(e, root, null,
       (DocumentFragment fragment) {
         removeHeaders(fragment);
         if (fragment.text.trim().toLowerCase() == "example") {
           // Degenerate example.
           fragment.nodes.clear();
         }
       }).html;
     if (html.length > 0) {
       exampleHtml.add(html);
@@ skipping 35 matching lines @@
 }
 
 void main() {
   window.on.load.add(documentLoaded);
 }
 
 void documentLoaded(event) {
   new XMLHttpRequest.getTEMPNAME('${window.location}.json', (req) {
     data = JSON.parse(req.responseText);
     dbEntry = {'members': [], 'srcUrl': pageUrl};
-    resourceLoaded();
+    run();
   });
 }