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

Unified Diff: chrome/browser/extensions/crx_installer_browsertest.cc

Issue 1862513003: Remove NPAPI from browser and utility (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 8 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/extensions/crx_installer_browsertest.cc
diff --git a/chrome/browser/extensions/crx_installer_browsertest.cc b/chrome/browser/extensions/crx_installer_browsertest.cc
index 093311d66b7cad1644be9300c68a34980596fa3d..a485443f28c78eb3222c48b1c8ba7fc578c36885 100644
--- a/chrome/browser/extensions/crx_installer_browsertest.cc
+++ b/chrome/browser/extensions/crx_installer_browsertest.cc
@@ -305,23 +305,6 @@ class ExtensionCrxInstallerTestWithExperimentalApis
}
};
-// This test is skipped on ChromeOS because it requires the NPAPI,
-// which is not available on that platform.
-#if !defined(OS_CHROMEOS)
-IN_PROC_BROWSER_TEST_F(ExtensionCrxInstallerTest, Whitelisting) {
- std::string id = "hdgllgikmikobbofgnabhfimcfoopgnd";
- ExtensionRegistry* registry = ExtensionRegistry::Get(
- browser()->profile());
-
- // Even whitelisted extensions with NPAPI should not prompt.
- scoped_ptr<MockPromptProxy> mock_prompt =
- CreateMockPromptProxyForBrowser(browser());
- InstallWithPrompt("uitest/plugins", id, mock_prompt.get());
- EXPECT_FALSE(mock_prompt->confirmation_requested());
- EXPECT_TRUE(registry->enabled_extensions().GetByID(id));
-}
-#endif
-
IN_PROC_BROWSER_TEST_F(ExtensionCrxInstallerTest,
ExperimentalExtensionFromGallery) {
// Gallery-installed extensions should have their experimental permission

Powered by Google App Engine
This is Rietveld 408576698