Extracted shell_interfaces and shell_public into a separate gyp file.

Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/builds/2023/steps/gclient%20runhooks%20%28with%20patch%29/logs/stdio

BUG=610554
Committed: https://crrev.com/9b143c65c2e8b243e54b39359d4509d85f5fa72a
Cr-Commit-Position: refs/heads/master@{#393196}

[Eugene But (OOO till 7-30), 2016-05-10 02:23:49]

Description was changed from

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=None
==========

to

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=610554
==========

[Eugene But (OOO till 7-30), 2016-05-10 02:24:43]

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

[commit-bot: I haz the power, 2016-05-10 02:25:12]

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

[commit-bot: I haz the power, 2016-05-10 02:29:27]

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

[commit-bot: I haz the power, 2016-05-10 02:29:28]

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

[Eugene But (OOO till 7-30), 2016-05-10 02:52:04]

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

[commit-bot: I haz the power, 2016-05-10 02:52:18]

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

[commit-bot: I haz the power, 2016-05-10 02:55:51]

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

[commit-bot: I haz the power, 2016-05-10 02:55:51]

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

[Eugene But (OOO till 7-30), 2016-05-10 15:25:55]

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

[commit-bot: I haz the power, 2016-05-10 15:26:09]

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

[commit-bot: I haz the power, 2016-05-10 15:29:58]

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

[commit-bot: I haz the power, 2016-05-10 15:29:59]

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

[Eugene But (OOO till 7-30), 2016-05-10 16:08:38]

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

[commit-bot: I haz the power, 2016-05-10 16:09:01]

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

[commit-bot: I haz the power, 2016-05-10 17:24:17]

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

[commit-bot: I haz the power, 2016-05-10 17:24:18]

Dry run: This issue passed the CQ dry run.

[Eugene But (OOO till 7-30), 2016-05-10 17:29:15]

eugenebut@chromium.org changed reviewers:
+ rockot@chromium.org

[Eugene But (OOO till 7-30), 2016-05-10 17:29:16]

Please add ben@ if necessary.

[Ken Rockot(use gerrit already), 2016-05-10 23:17:08]

LGTM, I think you can safely TBR ben for the mechanical changes

[Eugene But (OOO till 7-30), 2016-05-11 00:29:37]

avi@ for content/
chcunningham@ for media/
erg@ for components, services/user and services/tracing

[Eugene But (OOO till 7-30), 2016-05-11 22:36:22]

eugenebut@chromium.org changed reviewers:
+ avi@chromium.org, chcunningham@chromium.org, erg@chromium.org

[Eugene But (OOO till 7-30), 2016-05-11 22:36:23]

Actually adding reviewers :)

avi@ for content/
chcunningham@ for media/
erg@ for components, services/user and services/tracing

[Elliot Glaysher, 2016-05-11 22:43:37]

mechanical change lgtm

[chcunningham, 2016-05-12 00:54:20]

lgtm

[Avi (use Gerrit), 2016-05-12 01:47:20]

lgtm

[Eugene But (OOO till 7-30), 2016-05-12 05:00:53]

The CQ bit was checked by eugenebut@chromium.org

[commit-bot: I haz the power, 2016-05-12 05:01:04]

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

[commit-bot: I haz the power, 2016-05-12 06:26:22]

Description was changed from

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=610554
==========

to

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=610554
==========

[commit-bot: I haz the power, 2016-05-12 06:26:24]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-05-12 06:27:48]

Description was changed from

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=610554
==========

to

==========
Extracted shell_interfaces and shell_public into a separate gyp file.

This change breaks components/filesystem/filesystem.gyp ->
services/shell/shell.gyp -> components/filesystem/filesystem.gyp
circular dependency which has been introduced here:
https://codereview.chromium.org/1942473002/diff/320001/services/shell/shell.gyp

and breaks runhooks on iOS as follows:
https://build.chromium.org/p/tryserver.chromium.mac/builders/ios-device/build...

BUG=610554

Committed: https://crrev.com/9b143c65c2e8b243e54b39359d4509d85f5fa72a
Cr-Commit-Position: refs/heads/master@{#393196}
==========

[commit-bot: I haz the power, 2016-05-12 06:27:49]

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