web_dev_style: check webui browser tests as well.

chrome/browser/ui/webui/options/startup_page_list_browsertest.js

 // Copyright 2013 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.
 
 /**
  * Fixture for startup pages WebUI tests.
  * @extends {testing.Test}
  * @constructor
  */
 function StartupPageListWebUITest() {}
 
 StartupPageListWebUITest.prototype = {
   __proto__: testing.Test.prototype,
 
   /**
    * Browse to the options page & call our preLoad().
    * @override
    */
   browsePreload: 'chrome://settings-frame/startup',
 
   /** @override */
   setUp: function() {
     StartupOverlay.updateStartupPages(this.fakeStartupList);
     // 1 item for entering data, 1+ from |this.fakeStartupList|.
     assertGE(this.getList().items.length, 2);
   },
 
   /**
    * Returns the list to be tested.
-   * @return {Element}
+   * @return {Element} The start-up pages list.

[Tyler Breisacher (Chromium), 2014/04/28 23:13:26]

Is it ever null? If not, {!Element}

[Dan Beam, 2014/04/28 23:54:07]

it could be, in theory (as this is getElementById())

    * @protected
    */
   getList: function() {
     return $('startupPagesList');
   },
 
   /**
    * Register a mock handler to ensure expectations are met and options pages
    * behave correctly.
    * @override
@@ skipping 55 matching lines @@
    * @param {string} type The text of data being set.
    * @param {*} val The data to set. Will be stringified.
    */
   setData: function(type, val) {
     this.data_[type] = String(val);
   },
 
   /**
    * Gets data associated with this fake data transfer.
    * @param {string} type The type of data to get.
-   * @returns {string} The requested type of data or '' if not set.
+   * @return {string} The requested type of data or '' if not set.
    */
   getData: function(type) {
     return this.data_[type] || '';
   },
 };
 
 /**
  * Creates a fake bubbling, cancelable mouse event with a mock data transfer
  * installed.
  * @param {string} type A type of mouse event (e.g. 'drop').
@@ skipping 22 matching lines @@
   // TODO(dbeam): mock4js doesn't handle complex arguments well. Fix this.
   this.mockHandler.expects(once()).dragDropStartupPage([0, [1].join()]);
 
   this.getList().selectionModel.selectedIndex = 1;
   expectEquals(1, this.getList().selectionModel.selectedIndexes.length);
 
   this.getList().items[0].dispatchEvent(createMouseEvent('drop'));
 });
 
 }());