Add GN Build file for extensions_browser target.

Add GN Build file for extensions_browser target.

BUG=None
TEST=gn gen out/Debug_gn && ninja -C out/Debug_gn extensions/browser
R=brettw@chromium.org
TBR=yoz@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=285800

[tfarina, 2014-07-24 22:23:05]

Brett, Yoyo, ping??

[Yoyo Zhou, 2014-07-24 22:25:59]

I'm sorry, but I'm not familiar enough with GN to review. Brett should look at
it.

[brettw, 2014-07-24 22:26:12]

LGTM with change

https://codereview.chromium.org/418763002/diff/20001/extensions/browser/BUILD.gn
File extensions/browser/BUILD.gn (right):

https://codereview.chromium.org/418763002/diff/20001/extensions/browser/BUILD...
extensions/browser/BUILD.gn:242: sources -= [
Personally I think it would be clearer to invert this so you have

sources = [ <always used sources> ]
if (enable_extensions) {
  sources += [ <extensions enabled sources> ]
}

[tfarina, 2014-07-26 02:05:07]

The CQ bit was checked by tfarina@chromium.org

[tfarina, 2014-07-26 02:05:13]

https://codereview.chromium.org/418763002/diff/20001/extensions/browser/BUILD.gn
File extensions/browser/BUILD.gn (right):

https://codereview.chromium.org/418763002/diff/20001/extensions/browser/BUILD...
extensions/browser/BUILD.gn:242: sources -= [
On 2014/07/24 22:26:12, brettw wrote:
> Personally I think it would be clearer to invert this so you have
> 
> sources = [ <always used sources> ]
> if (enable_extensions) {
>   sources += [ <extensions enabled sources> ]
> }

Done.

[commit-bot: I haz the power, 2014-07-26 05:00:40]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/tfarina@chromium.org/418763002/60001

[commit-bot: I haz the power, 2014-07-26 20:06:39]

Change committed as 285800