MD Settings: fix auto-focus on sync page.

chrome/test/data/webui/settings/cr_settings_interactive_ui_tests.js

 // Copyright 2015 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.
 
 /** @fileoverview Runs the Polymer Settings interactive UI tests. */
 
 /** @const {string} Path to source root. */
 var ROOT_PATH = '../../../../../';
 
 // Polymer BrowserTest fixture.
@@ skipping 49 matching lines @@
     'focus_row_behavior_test.js',
   ]),
 };
 
 TEST_F('CrSettingsFocusRowBehavior', 'FocusTest', function() {
   mocha.run();
 });
 
 
 /**
+ * Test fixture for Sync Page.
+ * @constructor
+ * @extends {CrSettingsInteractiveUITest}
+ */
+function CrSettingsSyncPageTest() {}
+
+CrSettingsSyncPageTest.prototype = {
+  __proto__: CrSettingsInteractiveUITest.prototype,

[dpapad, 2017/06/14 01:02:00]

Does the new test have to be an interactive UI test? Asking because we already
have tests for sync_page.html, at
https://cs.chromium.org/chromium/src/chrome/test/data/webui/settings/cr_setti...

It is currently a bit confusing that we have two files that test the same
element,

people_page_sync_page_test.js (existing)
settings_sync_page_test.js (new in this CL)

So I suggest to either
1) Run your new test as a normal test and move the test in the existing file.
2) If 1 not possible, maybe still move the test in the existing file, and use
mocha.grep() at line 80 to only run your new test case as an interactive ui
test.
3) If not 1 or 2, rename the files to be more consistent/descriptive?

+
+  /** @override */
+  browsePreload: 'chrome://md-settings/people_page/sync_page.html',
+
+  /** @override */
+  extraLibraries: CrSettingsInteractiveUITest.prototype.extraLibraries.concat([
+    'settings_sync_page_test.js',
+  ]),
+};
+
+TEST_F('CrSettingsSyncPageTest', 'All', function() {
+  mocha.run();
+});
+
+
+/**
  * @constructor
  * @extends {CrSettingsInteractiveUITest}
  */
 function CrSettingsAnimatedPagesTest() {}
 
 CrSettingsAnimatedPagesTest.prototype = {
   __proto__: CrSettingsInteractiveUITest.prototype,
 
   /** @override */
   browsePreload:
       'chrome://md-settings/settings_page/settings_animated_pages.html',
 
   extraLibraries: CrSettingsInteractiveUITest.prototype.extraLibraries.concat([
     'settings_animated_pages_test.js',
   ]),
 };
 
 TEST_F('CrSettingsAnimatedPagesTest', 'All', function() {
   mocha.run()
 });