Remove expectation for http/tests/fileapi/blob-url-in-subframe.html.

Remove exceptions for tests fixed by teaching GetSiteForURL() about blob URLs.

These tests got recently fixed by https://crrev.com/1911573002

BUG=602818
Committed: https://crrev.com/c95b1fe8a47162538c389ec2361332473437beda
Cr-Commit-Position: refs/heads/master@{#390732}

[Łukasz Anforowicz, 2016-04-29 17:08:57]

lukasza@chromium.org changed reviewers:
+ nick@chromium.org

[Łukasz Anforowicz, 2016-04-29 17:08:57]

Nick, can you take a look please?

[ncarter (slow), 2016-04-29 18:53:39]

lgtm

https://codereview.chromium.org/1932273002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/FlagExpectations/site-per-process (right):

https://codereview.chromium.org/1932273002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:12:
http/tests/security/opened-document-security-origin-resets-on-navigation.html [
Crash ]
I believe that CL should have fixed this one too.

(I must have accidentally reverted my FlagExpectations changes during the
review:
https://codereview.chromium.org/1911573002/diff/40001/third_party/WebKit/Layo...
)

[Łukasz Anforowicz, 2016-04-29 19:05:06]

Description was changed from

==========
Remove expectation for http/tests/fileapi/blob-url-in-subframe.html.

This test got recently fixed by https://crrev.com/1911573002

BUG=602818
==========

to

==========
Remove exceptions for tests fixed by teaching GetSiteForURL() about blob URLs.

These tests got recently fixed by https://crrev.com/1911573002

BUG=602818
==========

[Łukasz Anforowicz, 2016-04-29 19:05:45]

Thanks for reviewing.

https://codereview.chromium.org/1932273002/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/FlagExpectations/site-per-process (right):

https://codereview.chromium.org/1932273002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/FlagExpectations/site-per-process:12:
http/tests/security/opened-document-security-origin-resets-on-navigation.html [
Crash ]
On 2016/04/29 18:53:39, ncarter wrote:
> I believe that CL should have fixed this one too.

Done - thanks for pointing this out.
> 
> (I must have accidentally reverted my FlagExpectations changes during the
> review:
>
https://codereview.chromium.org/1911573002/diff/40001/third_party/WebKit/Layo...
> )

Ack.

[Łukasz Anforowicz, 2016-04-29 19:06:07]

The CQ bit was checked by lukasza@chromium.org

[Łukasz Anforowicz, 2016-04-29 19:06:07]

The patchset sent to the CQ was uploaded after l-g-t-m from nick@chromium.org
Link to the patchset: https://codereview.chromium.org/1932273002/#ps20001
(title: "One more test got fixed - removing that expectation as well.")

[commit-bot: I haz the power, 2016-04-29 19:06:39]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1932273002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1932273002/20001

[ncarter (slow), 2016-04-29 19:06:39]

lgtm

[commit-bot: I haz the power, 2016-04-29 20:14:18]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-04-30 17:28:12]

Patchset 2 (id:??) landed as
https://crrev.com/c95b1fe8a47162538c389ec2361332473437beda
Cr-Commit-Position: refs/heads/master@{#390732}

[commit-bot: I haz the power, 2016-04-30 18:02:03]

Description was changed from

==========
Remove exceptions for tests fixed by teaching GetSiteForURL() about blob URLs.

These tests got recently fixed by https://crrev.com/1911573002

BUG=602818
==========

to

==========
Remove exceptions for tests fixed by teaching GetSiteForURL() about blob URLs.

These tests got recently fixed by https://crrev.com/1911573002

BUG=602818

Committed: https://crrev.com/c95b1fe8a47162538c389ec2361332473437beda
Cr-Commit-Position: refs/heads/master@{#390732}
==========