Make getKeys, getValues getters (keys, values).

utils/apidoc/mdn/extract.dart

 #import ("dart:html");
 #import ("dart:json");
 
 // Workaround for HTML lib missing feature.
 Range newRange() {
   return document.createRange();
 }
 
 // Temporary range object to optimize performance computing client rects
 // from text nodes.
@@ skipping 816 matching lines @@
       continue;
     }
 
     // After this point we have higher risk tests that attempt to perform
     // rudimentary page segmentation.  This approach is much more error-prone
     // than using tables because the HTML is far less clearly structured.
 
     final allText = getAllTextNodes(match);
 
     final pmap = new Map<String, Element>();
-    for (final prop in expectedProps.getKeys()) {
+    for (final prop in expectedProps.keys) {
       if (alreadyMatchedProperties.contains(prop)) {
         continue;
       }
       final e = findBest(match, allText, prop, propType);
       if (e != null && !inTable(e)) {
         pmap[prop] = e;
       }
     }
 
-    for (final prop in pmap.getKeys()) {
+    for (final prop in pmap.keys) {
       pmap[prop].classes.add(DART_REMOVED);
     }
 
     // The problem is the MDN docs do place documentation for each method in a
     // nice self contained subtree. Instead you will see something like:
 
     // <h3>drawImage</h3>
     // <p>Draw image is an awesome method</p>
     // some more info on drawImage here
     // <h3>mozDrawWindow</h3>
@@ skipping 10 matching lines @@
     // careful the definition for the drawImage method will contain the
     // definition for the mozDrawWindow method as well which would result in
     // broken docs.  We solve this problem by finding all content with similar
     // visual structure to the already found method definitions.  It turns out
     // that using the visual position of each element on the page is much
     // more reliable than using the DOM structure
     // (e.g. section_root > div > h3) for the MDN docs because MDN authors
     // carefully check that the documentation for each method comment is
     // visually consistent but take less care to check that each
     // method comment has identical markup structure.
-    for (String prop in pmap.getKeys()) {
+    for (String prop in pmap.keys) {
       Element e = pmap[prop];
       ClientRect r = getClientRect(e);
       // TODO(jacobr): a lot of these queries are identical and this code
       // could easily be optimized.
       for (final cand in match.queryAll(e.tagName)) {
         // TODO(jacobr): use a negative selector instead.
         if (!cand.classes.contains(DART_REMOVED) && !inTable(cand)) {
           final candRect = getClientRect(cand);
           // Only consider matches that have similar heights and identical left
           // coordinates.
           if (candRect.left == r.left &&
             (candRect.height - r.height).abs() < 5) {
             String propName = fullNameCleanup(cand.text);
             if (isFirstCharLowerCase(propName) && !pmap.containsKey(propName)
                 && !alreadyMatchedProperties.contains(propName)) {
               pmap[propName] = cand;
             }
           }
         }
       }
     }
 
     // We mark these elements in batch to reduce the number of layouts
     // triggered. TODO(jacobr): use new batch based async measurement to make
     // this code flow simpler.
-    for (String prop in pmap.getKeys()) {
+    for (String prop in pmap.keys) {
       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.contains(matchElement.id)) {
         e.classes.add(DART_REMOVED);
       }
     }
 
-    for (String prop in pmap.getKeys()) {
+    for (String prop in pmap.keys) {
       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,
@@ skipping 95 matching lines @@
  * values performing some simple normalization and only adding the entry to
  * [members] if it has a valid name.
  */
 void cleanupEntry(List members, Map entry) {
   if (entry.containsKey('help')) {
     entry['help'] = trimHtml(entry['help']);
   }
   String name = fullNameCleanup(entry['name']);
   entry['name'] = name;
   if (maybeName(name)) {
-    for (String key in entry.getKeys()) {
+    for (String key in entry.keys) {
       var value = entry[key];
       if (value == null) {
         entry.remove(key);
         continue;
       }
       if (value is String) {
         entry[key] = JSONFIXUPHACK(value);
       }
     }
     members.add(entry);
@@ skipping 260 matching lines @@
 }
 
 void documentLoaded(event) {
   // Load the database of expected methods and properties with an HttpRequest.
   new HttpRequest.get('${window.location}.json', (req) {
     data = JSON.parse(req.responseText);
     dbEntry = {'members': [], 'srcUrl': pageUrl};
     run();
   });
 }