Add GN files for google_update target.

Add GN files for google_update target.

Committed: https://crrev.com/ae7a0aef128043e8d51810e5dabc43c1b167bada
Cr-Commit-Position: refs/heads/master@{#311547}

[Slava Chigrin, 2015-01-14 18:34:17]

vchigrin@yandex-team.ru changed reviewers:
+ grt@chromium.org, thakis@chromium.org

[Slava Chigrin, 2015-01-14 18:43:09]

vchigrin@yandex-team.ru changed reviewers:
- grt@chromium.org, thakis@chromium.org

[Slava Chigrin, 2015-01-14 18:54:43]

vchigrin@yandex-team.ru changed reviewers:
+ grt@chromium.org, robertshield@chromium.org

[Slava Chigrin, 2015-01-14 19:06:11]

Patchset #1 (id:1) has been deleted

[Slava Chigrin, 2015-01-14 19:08:17]

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn
File google_update/BUILD.gn (right):

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn#n...
google_update/BUILD.gn:8: include_dirs = [ "$root_gen_dir" ]
Used root_gen_dir instead of target_gen_dir because of chrome includes file like
that
#include "google_update/google_update_idl.h"
See
https://code.google.com/p/chromium/codesearch#search/&q=google_update_idl.h&s...

[robertshield, 2015-01-14 19:30:06]

lgtm

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn
File google_update/BUILD.gn (right):

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn#n...
google_update/BUILD.gn:8: include_dirs = [ "$root_gen_dir" ]
On 2015/01/14 19:08:17, Slava Chigrin wrote:
> Used root_gen_dir instead of target_gen_dir because of chrome includes file
like
> that
> #include "google_update/google_update_idl.h"
> See
>
https://code.google.com/p/chromium/codesearch#search/&q=google_update_idl.h&s...

I don't really know how GN works, but if this is worth mentioning here, is it
also worth placing in a comment?

[grt (UTC plus 2), 2015-01-14 19:30:16]

i don't yet have GN readability, so rubberstamp lgtm on the assumption that it
works.

[Slava Chigrin, 2015-01-14 20:14:29]

Thank everybody much for review, I've added comment.

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn
File google_update/BUILD.gn (right):

https://codereview.chromium.org/853643004/diff/20001/google_update/BUILD.gn#n...
google_update/BUILD.gn:8: include_dirs = [ "$root_gen_dir" ]
On 2015/01/14 19:30:06, robertshield wrote:
> On 2015/01/14 19:08:17, Slava Chigrin wrote:
> > Used root_gen_dir instead of target_gen_dir because of chrome includes file
> like
> > that
> > #include "google_update/google_update_idl.h"
> > See
> >
>
https://code.google.com/p/chromium/codesearch#search/&q=google_update_idl.h&s...
> 
> I don't really know how GN works, but if this is worth mentioning here, is it
> also worth placing in a comment?

Done.

[Slava Chigrin, 2015-01-14 20:14:38]

The CQ bit was checked by vchigrin@yandex-team.ru

[commit-bot: I haz the power, 2015-01-14 20:16:08]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/853643004/40001

[commit-bot: I haz the power, 2015-01-14 21:40:21]

Committed patchset #2 (id:40001)

[commit-bot: I haz the power, 2015-01-14 21:41:10]

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