Service Worker: Clients.openWindow() should allow opening x-origin URLs

Service Worker: Clients.openWindow() should allow opening x-origin URLs

Clients.openWindow(url) with a cross-origin url currently rejects with
a SecurityError exception. As per spec:
https://slightlyoff.github.io/ServiceWorker/spec/service_worker/index.html#clients-openwindow
it should open a window and resolve with null.

Checked the expected behavior with @jakearchibald.

Corresponding content changes: https://codereview.chromium.org/984853003/

BUG=464680

[jungkees, 2015-03-06 10:50:30]

jungkee.song@samsung.com changed reviewers:
+ falken@chromium.org, jakearchibald@google.com, jsbell@chromium.org,
mlamouri@chromium.org

[jsbell, 2015-03-06 16:59:42]

lgtm

[falken, 2015-03-06 17:02:09]

Oops, you should coordinate with mlamouri who has patches up for this:
https://codereview.chromium.org/980383004/

See also his blink-dev post and bug.

[mlamouri (slow - plz ping), 2015-03-06 17:04:20]

not lgtm, we don't want openWindow() to be able to open urls like "file://" or
"chrome://". This CL would make this possible.

[jungkees, 2015-03-09 05:51:49]

On 2015/03/06 17:04:20, Mounir Lamouri wrote:
> not lgtm, we don't want openWindow() to be able to open urls like "file://" or
> "chrome://". This CL would make this possible.

Yes, I'm fine with mlamouri@'s patch: https://codereview.chromium.org/985043002

So, should I close this issue by myself? (Trying to follow the convention.)

[falken, 2015-03-09 06:31:05]

On 2015/03/09 05:51:49, jungkees wrote:
> On 2015/03/06 17:04:20, Mounir Lamouri wrote:
> > not lgtm, we don't want openWindow() to be able to open urls like "file://"
or
> > "chrome://". This CL would make this possible.
> 
> Yes, I'm fine with mlamouri@'s patch:
https://codereview.chromium.org/985043002
> 
> So, should I close this issue by myself? (Trying to follow the convention.)

Yep please close this... apologies and thanks for taking on the bug!

[jungkees, 2015-03-09 07:31:01]

On 2015/03/09 06:31:05, falken wrote:
> On 2015/03/09 05:51:49, jungkees wrote:
> > On 2015/03/06 17:04:20, Mounir Lamouri wrote:
>
> Yep please close this... apologies and thanks for taking on the bug!

Thanks for reviewing!