Do not kill renderer when enumerateDevices is called from about:blank.

Do not kill renderer when enumerateDevices is called from unique origins (e.g., about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the Mojo migration.

BUG=660204
Committed: https://crrev.com/7ab0eddda935e0f3bf84d73d7806c9f1d5fccd99
Cr-Commit-Position: refs/heads/master@{#431090}

[Guido Urdaneta, 2016-11-01 09:55:12]

Description was changed from

==========
Do not kill renderer when enumerateDevices is called from about:blank.

BUG=660204
==========

to

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

BUG=660204
==========

[Guido Urdaneta, 2016-11-01 09:55:56]

Description was changed from

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

BUG=660204
==========

to

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the
Mojo migration.

BUG=660204
==========

[Guido Urdaneta, 2016-11-01 09:56:18]

guidou@chromium.org changed reviewers:
+ mkwst@chromium.org

[Guido Urdaneta, 2016-11-01 10:22:46]

mkwst@: please review this change to IPC parameter validation. See
http://crbug.com/660204 for more context.

[Mike West, 2016-11-03 13:05:33]

Can you add a test that doesn't crash? :)

[Guido Urdaneta, 2016-11-03 16:49:32]

Patchset #3 (id:40001) has been deleted

[Guido Urdaneta, 2016-11-03 16:50:56]

Patchset #3 (id:60001) has been deleted

[Guido Urdaneta, 2016-11-03 16:51:31]

On 2016/11/03 13:05:33, Mike West wrote:
> Can you add a test that doesn't crash? :)

There you go!

[Mike West, 2016-11-07 09:54:34]

LGTM, thanks!

[Guido Urdaneta, 2016-11-07 09:56:05]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-11-07 09:56:21]

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-11-07 10:02:57]

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

[commit-bot: I haz the power, 2016-11-07 10:02:58]

Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Guido Urdaneta, 2016-11-07 10:26:42]

guidou@chromium.org changed reviewers:
+ tommi@chromium.org

[Guido Urdaneta, 2016-11-07 10:27:24]

tommi@: PTAL

[tommi (sloooow) - chröme, 2016-11-09 13:13:27]

lgtm

[Guido Urdaneta, 2016-11-09 20:27:48]

The CQ bit was checked by guidou@chromium.org

[commit-bot: I haz the power, 2016-11-09 20:28:24]

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-11-09 20:32:38]

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

[commit-bot: I haz the power, 2016-11-09 20:32:39]

Try jobs failed on following builders:
  ios-device on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds...)
  ios-simulator on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios-simulator/bui...)

[Guido Urdaneta, 2016-11-09 22:22:44]

The CQ bit was checked by guidou@chromium.org

[Guido Urdaneta, 2016-11-09 22:22:44]

The patchset sent to the CQ was uploaded after l-g-t-m from mkwst@chromium.org,
tommi@chromium.org
Link to the patchset: https://codereview.chromium.org/2468853002/#ps100001
(title: "rebase")

[commit-bot: I haz the power, 2016-11-09 22:23:41]

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-11-10 00:09:30]

Description was changed from

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the
Mojo migration.

BUG=660204
==========

to

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the
Mojo migration.

BUG=660204
==========

[commit-bot: I haz the power, 2016-11-10 00:09:32]

Committed patchset #4 (id:100001)

[commit-bot: I haz the power, 2016-11-10 00:20:18]

Description was changed from

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the
Mojo migration.

BUG=660204
==========

to

==========
Do not kill renderer when enumerateDevices is called from unique origins (e.g.,
about:blank).

This CL restores the behavior of enumerateDevices with unique origins before the
Mojo migration.

BUG=660204

Committed: https://crrev.com/7ab0eddda935e0f3bf84d73d7806c9f1d5fccd99
Cr-Commit-Position: refs/heads/master@{#431090}
==========

[commit-bot: I haz the power, 2016-11-10 00:20:19]

Patchset 4 (id:??) landed as
https://crrev.com/7ab0eddda935e0f3bf84d73d7806c9f1d5fccd99
Cr-Commit-Position: refs/heads/master@{#431090}