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

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

Issue 9269020: Handle paths and ports in verified domains in inline install. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix copyright header.x Created 8 years, 11 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/webstore_inline_install_browsertest.cc
diff --git a/chrome/browser/extensions/webstore_inline_install_browsertest.cc b/chrome/browser/extensions/webstore_inline_install_browsertest.cc
index ea82dd048f10f931a331343caf655b34f11e8b6e..81dbdab234f70622ebc6ee419498218bce2cfbc4 100644
--- a/chrome/browser/extensions/webstore_inline_install_browsertest.cc
+++ b/chrome/browser/extensions/webstore_inline_install_browsertest.cc
@@ -105,6 +105,14 @@ IN_PROC_BROWSER_TEST_F(
RunInlineInstallTest("runTest1");
RunInlineInstallTest("runTest2");
+
+ // Even though we're now on the app's domain, the verified domain has a
+ // different port number, so installation should still fail.
+ ui_test_utils::NavigateToURL(
+ browser(),
+ GenerateTestServerUrl(kAppDomain, "install_non_verified_domain.html"));
+
+ RunInlineInstallTest("runTest3");
}
IN_PROC_BROWSER_TEST_F(WebstoreInlineInstallTest, FindLink) {

Powered by Google App Engine
This is Rietveld 408576698