Commit 0ea7d517 authored by Charlie Reis's avatar Charlie Reis Committed by Commit Bot

Revert "Origin isolation: enable by default"

This reverts commit 824b062d.

Reason for revert: Causing crashes with hosted apps in https://crbug.com/1141721.

Original change's description:
> Origin isolation: enable by default
>
> Intent to Ship: https://groups.google.com/a/chromium.org/g/blink-dev/c/WW4fGjvroWI
>
> Further work on removing origin trial code and the flags is tracked in
> https://crbug.com/1148056 and https://crbug.com/1148057.
>
> Bug: 1042415
> Change-Id: Icbeecd0b592c76d36e2111b8e4d8fe30dbb63343
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2533145
> Commit-Queue: Domenic Denicola <domenic@chromium.org>
> Reviewed-by: Yoav Weiss <yoavweiss@chromium.org>
> Reviewed-by: Jochen Eisinger <jochen@chromium.org>
> Reviewed-by: Alex Moshchuk <alexmos@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#826910}

TBR=creis@chromium.org,alexmos@chromium.org,jochen@chromium.org,domenic@chromium.org,yoavweiss@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 1042415, 1141721
Change-Id: I428b190fc4c6bdf39de5ffb43226ba91851e7544
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2553209Reviewed-by: default avatarDomenic Denicola <domenic@chromium.org>
Reviewed-by: default avatarAlex Moshchuk <alexmos@chromium.org>
Reviewed-by: default avatarCharlie Reis <creis@chromium.org>
Commit-Queue: Charlie Reis <creis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#829937}
parent b071c2e0
......@@ -432,7 +432,7 @@ const base::Feature kNotificationTriggers{"NotificationTriggers",
// Note that the origin policy-based variant of origin isolation is controlled
// by kOriginPolicy, instead.
const base::Feature kOriginIsolationHeader{"OriginIsolationHeader",
base::FEATURE_ENABLED_BY_DEFAULT};
base::FEATURE_DISABLED_BY_DEFAULT};
// Origin Policy. See https://crbug.com/751996
const base::Feature kOriginPolicy{"OriginPolicy",
......
......@@ -1370,7 +1370,7 @@
{
name: "OriginIsolationHeader",
origin_trial_feature_name: "OriginIsolationHeader",
status: "stable",
status: "experimental",
},
{
name: "OriginPolicy",
......
......@@ -171,7 +171,6 @@ PASS oldChildWindow.onwebkittransitionend is newChildWindow.onwebkittransitionen
PASS oldChildWindow.onwheel is newChildWindow.onwheel
PASS oldChildWindow.opener is newChildWindow.opener
PASS oldChildWindow.origin is newChildWindow.origin
PASS oldChildWindow.originIsolated is newChildWindow.originIsolated
PASS oldChildWindow.outerHeight is newChildWindow.outerHeight
PASS oldChildWindow.outerWidth is newChildWindow.outerWidth
PASS oldChildWindow.pageXOffset is newChildWindow.pageXOffset
......
......@@ -146,7 +146,6 @@ PASS childWindow.onwebkittransitionend is null
PASS childWindow.onwheel is null
PASS childWindow.opener is null
PASS childWindow.origin is 'file://'
PASS childWindow.originIsolated is false
PASS childWindow.outerHeight is 0
PASS childWindow.outerWidth is 0
PASS childWindow.pageXOffset is 0
......
......@@ -146,7 +146,6 @@ PASS childWindow.onwebkittransitionend is null
PASS childWindow.onwheel is null
PASS childWindow.opener is null
PASS childWindow.origin is 'file://'
PASS childWindow.originIsolated is false
PASS childWindow.outerHeight is 0
PASS childWindow.outerWidth is 0
PASS childWindow.pageXOffset is 0
......
......@@ -9708,7 +9708,6 @@ namespace CSS
getter onwheel
getter opener
getter origin
getter originIsolated
getter outerHeight
getter outerWidth
getter pageXOffset
......
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