| Index: chrome/browser/permissions/permissions_browsertest.cc
|
| diff --git a/chrome/browser/permissions/permissions_browsertest.cc b/chrome/browser/permissions/permissions_browsertest.cc
|
| index c4a77d6ff31182b16319aa2e83ce849dcaea57ef..ba33f1f0f78392bed8ca827addece7347af59de9 100644
|
| --- a/chrome/browser/permissions/permissions_browsertest.cc
|
| +++ b/chrome/browser/permissions/permissions_browsertest.cc
|
| @@ -30,8 +30,7 @@ void PermissionsBrowserTest::SetUpOnMainThread() {
|
|
|
| ASSERT_TRUE(embedded_test_server()->Start());
|
|
|
| - ui_test_utils::NavigateToURL(browser(),
|
| - embedded_test_server()->GetURL(test_url()));
|
| + ui_test_utils::NavigateToURL(browser(), GetTestUrl());
|
| }
|
|
|
| void PermissionsBrowserTest::TearDownOnMainThread() {
|
| @@ -51,6 +50,10 @@ content::WebContents* PermissionsBrowserTest::GetWebContents() {
|
| return browser()->tab_strip_model()->GetActiveWebContents();
|
| }
|
|
|
| +GURL PermissionsBrowserTest::GetTestUrl() {
|
| + return embedded_test_server()->GetURL(test_url());
|
| +}
|
| +
|
| void PermissionsBrowserTest::CommonFailsBeforeRequesting() {
|
| EXPECT_EQ(0, prompt_factory()->total_request_count());
|
| // Dismiss any prompts if they are shown when using the feature.
|
| @@ -63,8 +66,8 @@ void PermissionsBrowserTest::CommonFailsIfDismissed() {
|
| prompt_factory()->set_response_type(PermissionRequestManager::DISMISS);
|
| TriggerPrompt();
|
|
|
| - EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| EXPECT_FALSE(FeatureUsageSucceeds());
|
| + EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| }
|
|
|
| void PermissionsBrowserTest::CommonFailsIfBlocked() {
|
| @@ -72,8 +75,8 @@ void PermissionsBrowserTest::CommonFailsIfBlocked() {
|
| prompt_factory()->set_response_type(PermissionRequestManager::DENY_ALL);
|
| TriggerPrompt();
|
|
|
| - EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| EXPECT_FALSE(FeatureUsageSucceeds());
|
| + EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| }
|
|
|
| void PermissionsBrowserTest::CommonSucceedsIfAllowed() {
|
| @@ -81,6 +84,6 @@ void PermissionsBrowserTest::CommonSucceedsIfAllowed() {
|
| prompt_factory()->set_response_type(PermissionRequestManager::ACCEPT_ALL);
|
| TriggerPrompt();
|
|
|
| - EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| EXPECT_TRUE(FeatureUsageSucceeds());
|
| + EXPECT_EQ(1, prompt_factory()->total_request_count());
|
| }
|
|
|