Pass CapabilityFilter via CreateInstanceForHandle

Pass CapabilityFilter via CreateInstanceForHandle

http://crbug.com/555392

Committed: https://crrev.com/6fbb41395a587c506cbf3bad18bcd89dec5c1803
Cr-Commit-Position: refs/heads/master@{#361255}

[Ben Goodger (Google), 2015-11-21 02:39:09]

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

[commit-bot: I haz the power, 2015-11-21 02:41:16]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1465793005/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1465793005/1

[commit-bot: I haz the power, 2015-11-21 02:50:57]

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

[commit-bot: I haz the power, 2015-11-21 02:50:58]

Dry run: Try jobs failed on following builders:
  cast_shell_android on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_andr...)
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_amd64-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_amd64-...)

[Ben Goodger (Google), 2015-11-21 23:38:49]

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

[commit-bot: I haz the power, 2015-11-21 23:39:17]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1465793005/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1465793005/40001

[Ben Goodger (Google), 2015-11-22 04:31:48]

ben@chromium.org changed reviewers:
+ sky@chromium.org

[commit-bot: I haz the power, 2015-11-22 04:52:05]

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

[commit-bot: I haz the power, 2015-11-22 04:52:06]

Dry run: This issue passed the CQ dry run.

[sky, 2015-11-22 15:27:31]

https://codereview.chromium.org/1465793005/diff/60001/mojo/application/public...
File mojo/application/public/interfaces/application_manager.mojom (right):

https://codereview.chromium.org/1465793005/diff/60001/mojo/application/public...
mojo/application/public/interfaces/application_manager.mojom:17:
mojo.CapabilityFilter? filter);
Document the capabilities a null filter gets.

https://codereview.chromium.org/1465793005/diff/60001/mojo/shell/application_...
File mojo/shell/application_manager.cc (right):

https://codereview.chromium.org/1465793005/diff/60001/mojo/shell/application_...
mojo/shell/application_manager.cc:129: if (!filter.is_null())
Do you want to use the permissive filter if no filter is specified?

[Ben Goodger (Google), 2015-11-24 00:29:36]

Removed the ability to pass a null filter. Will land after the other. PTAL

https://codereview.chromium.org/1465793005/diff/60001/mojo/shell/application_...
File mojo/shell/application_manager.cc (right):

https://codereview.chromium.org/1465793005/diff/60001/mojo/shell/application_...
mojo/shell/application_manager.cc:129: if (!filter.is_null())
On 2015/11/22 15:27:31, sky wrote:
> Do you want to use the permissive filter if no filter is specified?

Nope we specifically decided a null filter should be default (principle of least
privilege).

[sky, 2015-11-24 00:30:20]

LGTM

[Ben Goodger (Google), 2015-11-24 03:12:21]

The CQ bit was checked by ben@chromium.org

[commit-bot: I haz the power, 2015-11-24 03:13:34]

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

[commit-bot: I haz the power, 2015-11-24 04:44:28]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2015-11-24 04:45:28]

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