Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10634)

Unified Diff: chrome/test/data/webui/options.js

Issue 8333013: Allow generator javascript test files to go anywhere in the source tree (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix escaping from windows file paths. Created 9 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/test/data/webui/options.js
diff --git a/chrome/test/data/webui/options.js b/chrome/test/data/webui/options.js
deleted file mode 100644
index e3ec6abe6d90ff4014465de2501f5b9140f3cd7c..0000000000000000000000000000000000000000
--- a/chrome/test/data/webui/options.js
+++ /dev/null
@@ -1,121 +0,0 @@
-// Copyright (c) 2011 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.
-
-/**
- * TestFixture for OptionsPage WebUI testing.
- * @extends {testing.Test}
- * @constructor
- */
-function OptionsWebUITest() {}
-
-OptionsWebUITest.prototype = {
- __proto__: testing.Test.prototype,
-
- /**
- * Browse to the options page & call our preLoad().
- */
- browsePreload: 'chrome://settings',
-
- /**
- * Register a mock handler to ensure expectations are met and options pages
- * behave correctly.
- */
- preLoad: function() {
-
- /**
- * Create handler class with empty methods to allow mocking to register
- * expectations and for registration of handlers with chrome.send.
- */
- function MockOptionsHandler() {}
-
- MockOptionsHandler.prototype = {
- coreOptionsInitialize: function() {},
- fetchPrefs: function() {},
- observePrefs: function() {},
- setBooleanPref: function() {},
- setIntegerPref: function() {},
- setDoublePref: function() {},
- setStringPref: function() {},
- setObjectPref: function() {},
- clearPref: function() {},
- coreOptionsUserMetricsAction: function() {},
- // TODO(scr): Handle this new message:
- // getInstantFieldTrialStatus: function() {},
- };
-
- // Create the actual mock and register stubs for methods expected to be
- // called before our tests run. Specific expectations can be made in the
- // tests themselves.
- var mockHandler = this.mockHandler = mock(MockOptionsHandler);
- mockHandler.stubs().fetchPrefs(ANYTHING);
- mockHandler.stubs().observePrefs(ANYTHING);
- mockHandler.stubs().coreOptionsInitialize();
-
- // Register our mock as a handler of the chrome.send messages.
- registerMockMessageCallbacks(mockHandler, MockOptionsHandler);
- },
-};
-
-// Crashes on Mac only. See http://crbug.com/79181
-GEN('#if defined(OS_MACOSX)');
-GEN('#define MAYBE_testSetBooleanPrefTriggers ' +
- 'DISABLED_testSetBooleanPrefTriggers');
-GEN('#else');
-GEN('#define MAYBE_testSetBooleanPrefTriggers testSetBooleanPrefTriggers');
-GEN('#endif // defined(OS_MACOSX)');
-
-TEST_F('OptionsWebUITest', 'MAYBE_testSetBooleanPrefTriggers', function() {
- // TODO(dtseng): make generic to click all buttons.
- var showHomeButton = $('toolbarShowHomeButton');
- var trueListValue = [
- 'browser.show_home_button',
- true,
- 'Options_Homepage_HomeButton',
- ];
- // Note: this expectation is checked in testing::Test::tearDown.
- this.mockHandler.expects(once()).setBooleanPref(trueListValue);
-
- // Cause the handler to be called.
- showHomeButton.click();
- showHomeButton.blur();
-});
-
-// Not meant to run on ChromeOS at this time.
-// Not finishing in windows. http://crbug.com/81723
-GEN('#if defined(OS_CHROMEOS) || defined(OS_MACOSX) || defined(OS_WIN) \\');
-GEN(' || defined(TOUCH_UI)');
-GEN('#define MAYBE_testRefreshStaysOnCurrentPage \\');
-GEN(' DISABLED_testRefreshStaysOnCurrentPage');
-GEN('#else');
-GEN('#define MAYBE_testRefreshStaysOnCurrentPage ' +
- 'testRefreshStaysOnCurrentPage');
-GEN('#endif');
-
-TEST_F('OptionsWebUITest', 'MAYBE_testRefreshStaysOnCurrentPage', function() {
- var item = $('advancedPageNav');
- item.onclick();
- window.location.reload();
- var pageInstance = AdvancedOptions.getInstance();
- var topPage = OptionsPage.getTopmostVisiblePage();
- var expectedTitle = pageInstance.title;
- var actualTitle = document.title;
- assertEquals("chrome://settings/advanced", document.location.href);
- assertEquals(expectedTitle, actualTitle);
- assertEquals(pageInstance, topPage);
-});
-
-// Test that there are no console errors after opening all registered pages.
-// Crashes on chromium os, flaky on other platforms. See crbug.com/90420.
-// Disabling on Mac OS X since the failure rate is now >50%.
-GEN('#if defined(TOOLKIT_VIEWS) || defined(OS_MACOSX)');
-GEN('#define MAYBE_testOpenAllOptionsPages DISABLED_testOpenAllOptionsPages');
-GEN('#else');
-GEN('#define MAYBE_testOpenAllOptionsPages FLAKY_testOpenAllOptionsPages');
-GEN('#endif // defined(TOOLKIT_VIEWS)');
-TEST_F('OptionsWebUITest', 'MAYBE_testOpenAllOptionsPages', function() {
- expectTrue(!!OptionsPage.registeredPages);
- for (var name in OptionsPage.registeredPages) {
- OptionsPage.showPageByName(name, false);
- }
-});

Powered by Google App Engine
This is Rietveld 408576698