BlobUrlBrowserTest: Add a test exercising blob: URLs from file:// documents.

BlobUrlBrowserTest: Add a test exercising blob: URLs from file:// documents.

Small tweaks and renaming to other tests.

BUG=644966
TEST=content_browsertests
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

[ncarter (slow), 2016-09-28 20:26:59]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-09-28 20:27:25]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[ncarter (slow), 2016-09-28 20:27:47]

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

[ncarter (slow), 2016-09-28 20:27:48]

Charlie, please review

[commit-bot: I haz the power, 2016-09-28 21:04:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-09-28 21:04:19]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Charlie Reis, 2016-09-28 22:48:10]

Looks like we learned some things from this!  Maybe disable
kAllowFileAccessFromFiles and document the surprising origin inheritance
behavior?

https://codereview.chromium.org/2376083002/diff/1/content/browser/blob_storag...
File content/browser/blob_storage/blob_url_browsertest.cc (right):

https://codereview.chromium.org/2376083002/diff/1/content/browser/blob_storag...
content/browser/blob_storage/blob_url_browsertest.cc:132: "blob:file:///*"));
I was surprised by this.  In practice, we get a "null" origin on the blob URL,
since file URLs aren't allowed to script each other.

Looks like this is happening due to the following line in
BrowserTestBase::SetUp:
command_line->AppendSwitch(switches::kAllowFileAccessFromFiles);

https://codereview.chromium.org/2376083002/diff/1/content/browser/blob_storag...
content/browser/blob_storage/blob_url_browsertest.cc:146:
"domAutomationController.send(window.opener.document.body.innerText);",
Amazingly, this still works even with my null origin blob!  That's pretty
surprising.  According to dcheng@, that's because the blob inherits the
SecurityOrigin of the creator (in PublicURLManager::registerURL).

[Charlie Reis, 2016-09-28 22:48:40]

[CC dcheng, based on discussion]

[ncarter (slow), 2016-10-07 21:06:49]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[ncarter (slow), 2016-10-07 21:07:06]

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

[ncarter (slow), 2016-10-07 21:07:07]

charlie: PTAL
alexmos: FYI

[commit-bot: I haz the power, 2016-10-07 21:07:34]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[Charlie Reis, 2016-10-07 21:48:22]

LGTM, though the new LinkToSameOriginFileBlob  test failed in
content_site_isolation_browsertests on linux_chromium_rel_ng.

https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium...

[commit-bot: I haz the power, 2016-10-07 22:16:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-07 22:16:35]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ncarter (slow), 2016-10-18 19:34:01]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-18 19:34:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-18 21:03:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-18 21:03:19]

Dry run: Try jobs failed on following builders:
  linux_chromium_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[ncarter (slow), 2016-10-24 22:33:41]

Description was changed from

==========
BlobUrlBrowserTest: Add a test exercising blob: URLs from file:// documents.

Small tweaks and renaming to other tests.

BUG=644966
TEST=content_browsertests
==========

to

==========
BlobUrlBrowserTest: Add a test exercising blob: URLs from file:// documents.

Small tweaks and renaming to other tests.

BUG=644966
TEST=content_browsertests
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[ncarter (slow), 2016-10-24 22:34:01]

The CQ bit was checked by nick@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-10-24 22:34:36]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-10-24 22:53:01]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-10-24 22:53:02]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)