Remove configuration policy from chrome build files.

Remove configuration policy from chrome build files.

The define was removed from src/chrome source files in https://codereview.chromium.org/1786263002/, this patch only touches the build files in src/chrome. Build updates outside of this directory will be updated in subsequent passes.

BUG=594261
Committed: https://crrev.com/db234c06ff6c308520594591fa54f80b08411888
Cr-Commit-Position: refs/heads/master@{#382614}

[brettw, 2016-03-14 23:09:10]

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

[commit-bot: I haz the power, 2016-03-14 23:09:41]

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

[commit-bot: I haz the power, 2016-03-14 23:26:45]

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

[commit-bot: I haz the power, 2016-03-14 23:26:46]

Dry run: Try jobs failed on following builders:
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[brettw, 2016-03-15 18:07:52]

brettw@chromium.org changed reviewers:
+ atwilson@chromium.org

[brettw, 2016-03-15 18:07:53]

Sorry in advance, some of the GYP changes will be difficult to make sense of
since the variables and file lists went all over when I removed the conditions.

[brettw, 2016-03-15 18:07:56]

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

[commit-bot: I haz the power, 2016-03-15 18:08:30]

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

[commit-bot: I haz the power, 2016-03-15 19:49:02]

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

[commit-bot: I haz the power, 2016-03-15 19:49:03]

Dry run: This issue passed the CQ dry run.

[brettw, 2016-03-21 22:22:09]

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

[commit-bot: I haz the power, 2016-03-21 22:22:26]

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

[commit-bot: I haz the power, 2016-03-21 23:08:53]

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

[commit-bot: I haz the power, 2016-03-21 23:08:54]

Dry run: This issue passed the CQ dry run.

[Andrew T Wilson (Slow), 2016-03-22 00:01:27]

LGTM

https://codereview.chromium.org/1798273002/diff/40001/chrome/chrome_dll.gypi
File chrome/chrome_dll.gypi (left):

https://codereview.chromium.org/1798273002/diff/40001/chrome/chrome_dll.gypi#...
chrome/chrome_dll.gypi:409: 'policy_path_parser',
Did we lose the dependency on policy_path_parser?

https://codereview.chromium.org/1798273002/diff/40001/chrome/test/BUILD.gn
File chrome/test/BUILD.gn (right):

https://codereview.chromium.org/1798273002/diff/40001/chrome/test/BUILD.gn#ne...
chrome/test/BUILD.gn:1675: "//chrome")
nit: remove blank line?

[brettw, 2016-03-22 17:17:44]

https://codereview.chromium.org/1798273002/diff/40001/chrome/chrome_dll.gypi
File chrome/chrome_dll.gypi (left):

https://codereview.chromium.org/1798273002/diff/40001/chrome/chrome_dll.gypi#...
chrome/chrome_dll.gypi:409: 'policy_path_parser',
Oh yeah, I got confused because it's listed in the file elsewhere, but it's
actually on another target.

[brettw, 2016-03-22 17:17:48]

The CQ bit was checked by brettw@chromium.org

[brettw, 2016-03-22 17:17:48]

The patchset sent to the CQ was uploaded after l-g-t-m from
atwilson@chromium.org
Link to the patchset: https://codereview.chromium.org/1798273002/#ps60001
(title: "review comments + merge")

[commit-bot: I haz the power, 2016-03-22 17:18:04]

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

[commit-bot: I haz the power, 2016-03-22 18:19:50]

Committed patchset #4 (id:60001)

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

Description was changed from

==========
Remove configuration policy from chrome build files.

The define was removed from src/chrome source files in
https://codereview.chromium.org/1786263002/, this patch only touches the build
files in src/chrome. Build updates outside of this directory will be updated in
subsequent passes.

BUG=594261
==========

to

==========
Remove configuration policy from chrome build files.

The define was removed from src/chrome source files in
https://codereview.chromium.org/1786263002/, this patch only touches the build
files in src/chrome. Build updates outside of this directory will be updated in
subsequent passes.

BUG=594261

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

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

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

[dgozman, 2016-03-22 21:16:07]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1818423003/ by dgozman@chromium.org.

The reason for reverting is: Speculative revert: could have caused the Win8 GN
build failures. See crbug.com/597032.