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

Unified Diff: chrome/browser/storage/durable_storage_browsertest.cc

Issue 1294913009: Durable Storage: Add "deny" test (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 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/browser/storage/durable_storage_browsertest.cc
diff --git a/chrome/browser/storage/durable_storage_browsertest.cc b/chrome/browser/storage/durable_storage_browsertest.cc
index 50628db41aec49ffb74739ad4ca91d14c629a4c7..225bdbe36bd2a488356820b4edad45a9dd6f8e3f 100644
--- a/chrome/browser/storage/durable_storage_browsertest.cc
+++ b/chrome/browser/storage/durable_storage_browsertest.cc
@@ -21,65 +21,69 @@
class DurableStorageBrowserTest : public InProcessBrowserTest {
public:
- DurableStorageBrowserTest();
- ~DurableStorageBrowserTest() override;
+ DurableStorageBrowserTest() = default;
+ ~DurableStorageBrowserTest() override = default;
void SetUpCommandLine(base::CommandLine*) override;
+ void SetUpOnMainThread() override;
+
+ protected:
+ content::RenderFrameHost* GetRenderFrameHost() {
+ return browser()->tab_strip_model()->GetActiveWebContents()->GetMainFrame();
+ }
+ GURL url_;
private:
DISALLOW_COPY_AND_ASSIGN(DurableStorageBrowserTest);
};
-DurableStorageBrowserTest::DurableStorageBrowserTest() {}
-
-DurableStorageBrowserTest::~DurableStorageBrowserTest() {}
-
void DurableStorageBrowserTest::SetUpCommandLine(
base::CommandLine* command_line) {
command_line->AppendSwitch(
switches::kEnableExperimentalWebPlatformFeatures);
}
-// TODO(dgrogan): Reenable after https://codereview.chromium.org/1302643002
-// lands.
-IN_PROC_BROWSER_TEST_F(DurableStorageBrowserTest, FirstTabSeesResult) {
- EXPECT_TRUE(embedded_test_server()->Started() ||
- embedded_test_server()->InitializeAndWaitUntilReady());
+void DurableStorageBrowserTest::SetUpOnMainThread() {
+ if (embedded_test_server()->Started())
+ return;
+ ASSERT_TRUE(embedded_test_server()->InitializeAndWaitUntilReady());
+ url_ = embedded_test_server()->GetURL("/durable/durability-permissions.html");
+}
- Browser* current_browser = browser();
+IN_PROC_BROWSER_TEST_F(DurableStorageBrowserTest, DenyString) {
+ ui_test_utils::NavigateToURL(browser(), url_);
+ PermissionBubbleManager::FromWebContents(
+ browser()->tab_strip_model()->GetActiveWebContents())
+ ->set_auto_response_for_test(PermissionBubbleManager::DENY_ALL);
+ bool default_box_is_persistent;
+ EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
+ GetRenderFrameHost(), "requestPermission()", &default_box_is_persistent));
+ EXPECT_EQ(false, default_box_is_persistent);
+ std::string permission_string;
+ EXPECT_TRUE(content::ExecuteScriptAndExtractString(
+ GetRenderFrameHost(), "checkPermission()", &permission_string));
+ EXPECT_EQ("denied", permission_string);
+}
- const GURL& url =
- embedded_test_server()->GetURL("/durable/durability-window1.html");
- ui_test_utils::NavigateToURL(current_browser, url);
- content::RenderFrameHost* render_frame_host =
- current_browser->tab_strip_model()
- ->GetActiveWebContents()
- ->GetMainFrame();
+IN_PROC_BROWSER_TEST_F(DurableStorageBrowserTest, FirstTabSeesResult) {
+ ui_test_utils::NavigateToURL(browser(), url_);
std::string permission_string;
EXPECT_TRUE(content::ExecuteScriptAndExtractString(
- render_frame_host, "checkPermission()", &permission_string));
+ GetRenderFrameHost(), "checkPermission()", &permission_string));
EXPECT_EQ("default", permission_string);
- chrome::NewTab(current_browser);
- const GURL& url2 =
- embedded_test_server()->GetURL("/durable/durability-window2.html");
- ui_test_utils::NavigateToURL(current_browser, url2);
- render_frame_host = current_browser->tab_strip_model()
- ->GetActiveWebContents()
- ->GetMainFrame();
+ chrome::NewTab(browser());
+ ui_test_utils::NavigateToURL(browser(), url_);
PermissionBubbleManager::FromWebContents(
- current_browser->tab_strip_model()->GetActiveWebContents())
+ browser()->tab_strip_model()->GetActiveWebContents())
->set_auto_response_for_test(PermissionBubbleManager::ACCEPT_ALL);
bool default_box_is_persistent = false;
EXPECT_TRUE(content::ExecuteScriptAndExtractBool(
- render_frame_host, "requestPermission()", &default_box_is_persistent));
+ GetRenderFrameHost(), "requestPermission()", &default_box_is_persistent));
EXPECT_EQ(true, default_box_is_persistent);
- current_browser->tab_strip_model()->ActivateTabAt(0, false);
- render_frame_host = current_browser->tab_strip_model()
- ->GetActiveWebContents()
- ->GetMainFrame();
+ browser()->tab_strip_model()->ActivateTabAt(0, false);
EXPECT_TRUE(content::ExecuteScriptAndExtractString(
- render_frame_host, "checkPermission()", &permission_string));
+ GetRenderFrameHost(), "checkPermission()", &permission_string));
EXPECT_EQ("granted", permission_string);
}

Powered by Google App Engine
This is Rietveld 408576698