Commit 3bb3bb8f authored by Christian Dullweber's avatar Christian Dullweber Committed by Commit Bot

Add field trial for PageInfo redesign

Bug: 1077766
Change-Id: Ia91cfc1d8f72bf56c18236e7ac926bca90aa3936
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2532218Reviewed-by: default avatarLiquan (Max) Gu <maxlg@chromium.org>
Reviewed-by: default avatarWeilun Shi <sweilun@chromium.org>
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#827736}
parent ecd22f2f
......@@ -205,6 +205,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowOnInsecureHttpWebsite() throws IOException {
mTestServerRule.setServerUsesHttps(false);
loadUrlAndOpenPageInfo(mTestServerRule.getServer().getURL(sSimpleHtml));
......@@ -217,6 +218,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowOnSecureWebsite() throws IOException {
loadUrlAndOpenPageInfo(mTestServerRule.getServer().getURL(sSimpleHtml));
mRenderTestRule.render(getPageInfoView(), "PageInfo_SecureWebsite");
......@@ -229,6 +231,7 @@ public class PageInfoViewTest {
@MediumTest
@Feature({"RenderTest"})
@DisabledTest(message = "https://crbug.com/1133770")
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowOnExpiredCertificateWebsite() throws IOException {
mTestServerRule.setCertificateType(ServerCertificate.CERT_EXPIRED);
loadUrlAndOpenPageInfo(mTestServerRule.getServer().getURL(sSimpleHtml));
......@@ -241,6 +244,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testChromePage() throws IOException {
loadUrlAndOpenPageInfo("chrome://version/");
mRenderTestRule.render(getPageInfoView(), "PageInfo_InternalSite");
......@@ -253,6 +257,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowWithPermissions() throws IOException {
mIsSystemLocationSettingEnabled = false;
addSomePermissions(mTestServerRule.getServer().getURL("/"));
......@@ -266,6 +271,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowWithCookieBlocking() throws IOException {
setThirdPartyCookieBlocking(CookieControlsMode.BLOCK_THIRD_PARTY);
loadUrlAndOpenPageInfo(mTestServerRule.getServer().getURL(sSimpleHtml));
......@@ -278,6 +284,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowWithPermissionsAndCookieBlocking() throws IOException {
addSomePermissions(mTestServerRule.getServer().getURL("/"));
setThirdPartyCookieBlocking(CookieControlsMode.BLOCK_THIRD_PARTY);
......@@ -291,6 +298,7 @@ public class PageInfoViewTest {
@Test
@MediumTest
@Feature({"RenderTest"})
@Features.DisableFeatures(PageInfoFeatureList.PAGE_INFO_V2)
public void testShowWithDefaultSettingPermissions() throws IOException {
addDefaultSettingPermissions(mTestServerRule.getServer().getURL("/"));
loadUrlAndOpenPageInfo(mTestServerRule.getServer().getURL(sSimpleHtml));
......
......@@ -41,6 +41,7 @@ import org.chromium.chrome.browser.flags.ChromeSwitches;
import org.chromium.chrome.browser.payments.handler.PaymentHandlerCoordinator;
import org.chromium.chrome.browser.payments.handler.PaymentHandlerCoordinator.PaymentHandlerUiObserver;
import org.chromium.chrome.test.ChromeJUnit4RunnerDelegate;
import org.chromium.components.page_info.PageInfoFeatureList;
import org.chromium.content_public.browser.LoadUrlParams;
import org.chromium.content_public.browser.WebContents;
import org.chromium.net.test.EmbeddedTestServer;
......@@ -334,6 +335,11 @@ public class ExpandablePaymentHandlerTest {
String paymentAppUrl = mServer.getURL(
"/components/test/data/payments/maxpay.com/payment_handler_window.html");
if (PageInfoFeatureList.isEnabled(PageInfoFeatureList.PAGE_INFO_V2)) {
// The new UI only shows a hostname by default. Expand to full URL.
onView(withId(R.id.page_info_url_wrapper)).perform(click());
}
onView(withId(R.id.page_info_url))
.check(matches(isDisplayed()))
.check(matches(withText(paymentAppUrl)));
......
......@@ -4968,6 +4968,21 @@
]
}
],
"PageInfoV2": [
{
"platforms": [
"android"
],
"experiments": [
{
"name": "Enabled",
"enable_features": [
"PageInfoV2"
]
}
]
}
],
"PaintHoldingCrossOrigin": [
{
"platforms": [
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment