Fix up dependency chain around net_features.h

Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970
Committed: https://crrev.com/135b2ed3347a44635ae459396b4f7ab5c49849d9
Cr-Commit-Position: refs/heads/master@{#381910}

[droger, 2016-03-17 09:05:30]

droger@chromium.org changed reviewers:
+ droger@chromium.org

[droger, 2016-03-17 09:05:31]

Is this BUG=594970?

[blundell, 2016-03-17 09:08:11]

Description was changed from

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.
==========

to

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970
==========

[droger, 2016-03-17 09:09:16]

lgtm

[blundell, 2016-03-17 09:09:48]

blundell@chromium.org changed reviewers:
+ rsleevi@chromium.org, zea@chromium.org

[blundell, 2016-03-17 09:09:48]

rsleevi: please review //net

zea: please review //sync

An iOS dev uncovered this issue using xcodebuild, which is "better" at tickling
these types of issues than ninja since it's dumber.

[Ryan Sleevi, 2016-03-17 16:11:42]

lgtm

[Nicolas Zea, 2016-03-17 23:04:39]

lgtm

[blundell, 2016-03-18 08:10:44]

The CQ bit was checked by blundell@chromium.org

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

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

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

Description was changed from

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970
==========

to

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970
==========

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

Committed patchset #1 (id:1)

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

Description was changed from

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970
==========

to

==========
Fix up dependency chain around net_features.h

//net exposes the generated net_features.h header as part of its public API
(e.g., via http_network_session.h). That means that the GYP build of //net
needs to export dependent settings on net_features.

Similarly, //sync exposes //net headers that transitively include
net_features.h via *its* public API (e.g., via http_bridge.h). This means that
//sync needs to export dependent settings on //net.

The GN build already gets this mostly right, so fewer fixes are needed there.

BUG=594970

Committed: https://crrev.com/135b2ed3347a44635ae459396b4f7ab5c49849d9
Cr-Commit-Position: refs/heads/master@{#381910}
==========

[commit-bot: I haz the power, 2016-03-18 08:17:06]

Patchset 1 (id:??) landed as
https://crrev.com/135b2ed3347a44635ae459396b4f7ab5c49849d9
Cr-Commit-Position: refs/heads/master@{#381910}