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

Side by Side Diff: chrome/browser/content_settings/content_settings_browsertest.cc

Issue 60263003: Run click to play tests on Win Aura since we have NPAPI plugins there. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: Created 7 years, 1 month 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "base/command_line.h" 5 #include "base/command_line.h"
6 #include "base/path_service.h" 6 #include "base/path_service.h"
7 #include "base/strings/stringprintf.h" 7 #include "base/strings/stringprintf.h"
8 #include "base/strings/utf_string_conversions.h" 8 #include "base/strings/utf_string_conversions.h"
9 #include "chrome/browser/chrome_notification_types.h" 9 #include "chrome/browser/chrome_notification_types.h"
10 #include "chrome/browser/content_settings/cookie_settings.h" 10 #include "chrome/browser/content_settings/cookie_settings.h"
(...skipping 270 matching lines...) Expand 10 before | Expand all | Expand 10 after
281 281
282 ui_test_utils::NavigateToURL(browser(), test_url); 282 ui_test_utils::NavigateToURL(browser(), test_url);
283 283
284 content::WebContents* web_contents = 284 content::WebContents* web_contents =
285 browser()->tab_strip_model()->GetActiveWebContents(); 285 browser()->tab_strip_model()->GetActiveWebContents();
286 286
287 EXPECT_TRUE(TabSpecificContentSettings::FromWebContents(web_contents)-> 287 EXPECT_TRUE(TabSpecificContentSettings::FromWebContents(web_contents)->
288 IsContentBlocked(CONTENT_SETTINGS_TYPE_COOKIES)); 288 IsContentBlocked(CONTENT_SETTINGS_TYPE_COOKIES));
289 } 289 }
290 290
291 #if !defined(USE_AURA) // No NPAPI plugins with Aura. 291 // NPAPI plugins only on Windows Aura.
Bernhard Bauer 2013/11/05 18:06:27 This comment seems to imply that Windows and Aura
jam 2013/11/06 00:38:21 Done.
292 #if !defined(USE_AURA) || defined(OS_WIN)
292 293
293 class ClickToPlayPluginTest : public ContentSettingsTest { 294 class ClickToPlayPluginTest : public ContentSettingsTest {
294 public: 295 public:
295 ClickToPlayPluginTest() {} 296 ClickToPlayPluginTest() {}
296 297
297 #if defined(OS_MACOSX) 298 #if defined(OS_MACOSX)
298 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE { 299 virtual void SetUpCommandLine(CommandLine* command_line) OVERRIDE {
299 base::FilePath plugin_dir; 300 base::FilePath plugin_dir;
300 PathService::Get(base::DIR_MODULE, &plugin_dir); 301 PathService::Get(base::DIR_MODULE, &plugin_dir);
301 plugin_dir = plugin_dir.AppendASCII("plugins"); 302 plugin_dir = plugin_dir.AppendASCII("plugins");
(...skipping 156 matching lines...) Expand 10 before | Expand all | Expand 10 after
458 content::RenderViewHost* host = 459 content::RenderViewHost* host =
459 browser()->tab_strip_model()->GetActiveWebContents()->GetRenderViewHost(); 460 browser()->tab_strip_model()->GetActiveWebContents()->GetRenderViewHost();
460 ChromePluginServiceFilter::GetInstance()->AuthorizeAllPlugins( 461 ChromePluginServiceFilter::GetInstance()->AuthorizeAllPlugins(
461 host->GetProcess()->GetID()); 462 host->GetProcess()->GetID());
462 host->Send(new ChromeViewMsg_LoadBlockedPlugins( 463 host->Send(new ChromeViewMsg_LoadBlockedPlugins(
463 host->GetRoutingID(), std::string())); 464 host->GetRoutingID(), std::string()));
464 465
465 EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle()); 466 EXPECT_EQ(expected_title, title_watcher.WaitAndGetTitle());
466 } 467 }
467 468
468 #endif // !defined(USE_AURA) 469 #endif // !defined(USE_AURA) || defined(OS_WIN)
469 470
470 #if defined(ENABLE_PLUGINS) 471 #if defined(ENABLE_PLUGINS)
471 472
472 class PepperContentSettingsTest : public ContentSettingsTest { 473 class PepperContentSettingsTest : public ContentSettingsTest {
473 public: 474 public:
474 PepperContentSettingsTest() {} 475 PepperContentSettingsTest() {}
475 476
476 protected: 477 protected:
477 static const char* const kExternalClearKeyMimeType; 478 static const char* const kExternalClearKeyMimeType;
478 479
(...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after
611 RunJavaScriptBlockedTest("load_widevine_no_js.html", true); 612 RunJavaScriptBlockedTest("load_widevine_no_js.html", true);
612 #endif // defined(WIDEVINE_CDM_AVAILABLE) 613 #endif // defined(WIDEVINE_CDM_AVAILABLE)
613 #endif // defined(ENABLE_PEPPER_CDMS) 614 #endif // defined(ENABLE_PEPPER_CDMS)
614 615
615 #if !defined(DISABLE_NACL) 616 #if !defined(DISABLE_NACL)
616 RunJavaScriptBlockedTest("load_nacl_no_js.html", true); 617 RunJavaScriptBlockedTest("load_nacl_no_js.html", true);
617 #endif // !defined(DISABLE_NACL) 618 #endif // !defined(DISABLE_NACL)
618 } 619 }
619 620
620 #endif // defined(ENABLE_PLUGINS) 621 #endif // defined(ENABLE_PLUGINS)
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698