diff options
| author | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-09 02:33:27 +0000 |
|---|---|---|
| committer | dmazzoni@chromium.org <dmazzoni@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-01-09 02:33:27 +0000 |
| commit | 30fdb36651388788b318cde3e9e86c29fd2bd79b (patch) | |
| tree | c6f8d5411b084f8832fadc3192e52bd7744abf13 /chrome | |
| parent | 2380c6bdf6dc0f4f6929f1b9702ec3ffce747e23 (diff) | |
| download | chromium_src-30fdb36651388788b318cde3e9e86c29fd2bd79b.zip chromium_src-30fdb36651388788b318cde3e9e86c29fd2bd79b.tar.gz chromium_src-30fdb36651388788b318cde3e9e86c29fd2bd79b.tar.bz2 | |
Add test for accessibility histograms.
The goal is just to make sure this code has test coverage, not to test
correctness. There was an issue in the past where the code that computed
the histograms crashed in debug mode, this will help prevent something like
that from landing in the future.
BUG=99504
Review URL: https://chromiumcodereview.appspot.com/11778045
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@175677 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome')
| -rw-r--r-- | chrome/browser/accessibility/browser_accessibility_state_browsertest.cc | 30 | ||||
| -rw-r--r-- | chrome/chrome_tests.gypi | 1 |
2 files changed, 31 insertions, 0 deletions
diff --git a/chrome/browser/accessibility/browser_accessibility_state_browsertest.cc b/chrome/browser/accessibility/browser_accessibility_state_browsertest.cc new file mode 100644 index 0000000..f23f218 --- /dev/null +++ b/chrome/browser/accessibility/browser_accessibility_state_browsertest.cc @@ -0,0 +1,30 @@ +// Copyright (c) 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. + +#include "content/public/browser/browser_accessibility_state.h" +#include "chrome/test/base/in_process_browser_test.h" +#include "chrome/test/base/uma_histogram_helper.h" + +namespace content { + +// Note: even though BrowserAccessibilityStateImpl is in content, this +// test should be in Chrome because otherwise the Chrome-OS-specific +// histograms won't get updated. +class BrowserAccessibilityStateImplTest : public InProcessBrowserTest { +}; + +IN_PROC_BROWSER_TEST_F(BrowserAccessibilityStateImplTest, TestHistograms) { + UMAHistogramHelper histograms; + + BrowserAccessibilityState::GetInstance()->UpdateHistogramsForTesting(); + histograms.Fetch(); + histograms.ExpectTotalCount("Accessibility.State", 1); +#if defined(OS_WIN) + histograms.ExpectTotalCount("Accessibility.WinScreenReader", 1); +#elif defined(OS_CHROMEOS) + histograms.ExpectTotalCount("Accessibility.CrosSpokenFeedback", 1); +#endif +} + +} // namespace content diff --git a/chrome/chrome_tests.gypi b/chrome/chrome_tests.gypi index 5073a8f..3624732 100644 --- a/chrome/chrome_tests.gypi +++ b/chrome/chrome_tests.gypi @@ -878,6 +878,7 @@ 'app/chrome_dll_resource.h', 'app/chrome_version.rc.version', 'browser/accessibility/accessibility_extension_apitest.cc', + 'browser/accessibility/browser_accessibility_state_browsertest.cc', 'browser/app_controller_mac_browsertest.mm', 'browser/autocomplete/autocomplete_browsertest.cc', 'browser/autofill/autofill_browsertest.cc', |