Possible JavaScript errors caught by the closure compiler

chrome/browser/resources/options2/cookies_list.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.
 
 cr.define('options', function() {
   const DeletableItemList = options.DeletableItemList;
   const DeletableItem = options.DeletableItem;
   const ArrayDataModel = cr.ui.ArrayDataModel;
   const ListSingleSelectionModel = cr.ui.ListSingleSelectionModel;
 
@@ skipping 516 matching lines @@
       while (element.childNodes.length > 1)
         element.removeChild(element.firstChild);
       if (table)
         element.insertBefore(table, element.firstChild);
     },
 
     /**
      * The parent of this cookie tree node.
      * @type {?CookieTreeNode|CookieListItem}
      */
-    get parent(parent) {

[Tyler Breisacher (Chromium), 2012/03/09 22:36:17]

I assume the parameter is just ignored, so I don't see how this could break
anything.

[arv (Not doing code reviews), 2012/03/10 00:18:54]

This is a V8 bug. It is illegal to have paramaters in getters.

http://code.google.com/p/v8/issues/detail?id=1998

+    get parent() {
       // See below for an explanation of this special case.
       if (typeof this.parent_ == 'number')
         return this.list_.getListItemByIndex(this.parent_);
       return this.parent_;
     },
     set parent(parent) {
       if (parent == this.parent)
         return;
       if (parent instanceof CookieListItem) {
         // If the parent is to be a CookieListItem, then we keep the reference
@@ skipping 298 matching lines @@
       parent.clear();
       this.addByParent_(parent, 0, children);
       parent.endBatchUpdates();
     },
   };
 
   return {
     CookiesList: CookiesList
   };
 });