clang tot: Disable -Wuser-defined-warnings

clang tot: Disable -Wuser-defined-warnings

BUG=683349
TBR=hans

Review-Url: https://codereview.chromium.org/2649163002
Cr-Commit-Position: refs/heads/master@{#445401}
Committed: https://chromium.googlesource.com/chromium/src/+/59c02c3aa8f983c2adecb654bd71e2ada5fb4d8e

[Nico, 2017-01-23 16:39:21]

Description was changed from

==========
clang tot: Disable -Wuser-defined-warnings

BUG=
==========

to

==========
clang tot: Disable -Wuser-defined-warnings

BUG=683349
==========

[Nico, 2017-01-23 16:39:29]

thakis@chromium.org changed reviewers:
+ hans@chromium.org

[Nico, 2017-01-23 16:39:35]

Description was changed from

==========
clang tot: Disable -Wuser-defined-warnings

BUG=683349
==========

to

==========
clang tot: Disable -Wuser-defined-warnings

BUG=683349
TBR=hans
==========

[Nico, 2017-01-23 16:39:40]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2017-01-23 16:39:53]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[hans, 2017-01-23 16:58:39]

lgtm

https://codereview.chromium.org/2649163002/diff/1/build/config/compiler/BUILD.gn
File build/config/compiler/BUILD.gn (right):

https://codereview.chromium.org/2649163002/diff/1/build/config/compiler/BUILD...
build/config/compiler/BUILD.gn:1086: # TODO(thakis ): https://crbug.com/683349
nit: there's an extra space here

[Nico, 2017-01-23 17:10:59]

https://codereview.chromium.org/2649163002/diff/1/build/config/compiler/BUILD.gn
File build/config/compiler/BUILD.gn (right):

https://codereview.chromium.org/2649163002/diff/1/build/config/compiler/BUILD...
build/config/compiler/BUILD.gn:1086: # TODO(thakis ): https://crbug.com/683349
On 2017/01/23 16:58:39, hans wrote:
> nit: there's an extra space here

I actually changed my username from thakis to thakis . I'm now thakis
@chromium.org.

(thanks, will address in follow-up, seems sad to delay landing of this by 2h
just to fix a comment)

[commit-bot: I haz the power, 2017-01-23 17:45:51]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485189580853780, "parent_rev":
"0e16ad41609efac7f602f45a807bce915d2181f6", "commit_rev":
"59c02c3aa8f983c2adecb654bd71e2ada5fb4d8e"}

[commit-bot: I haz the power, 2017-01-23 17:46:34]

Description was changed from

==========
clang tot: Disable -Wuser-defined-warnings

BUG=683349
TBR=hans
==========

to

==========
clang tot: Disable -Wuser-defined-warnings

BUG=683349
TBR=hans

Review-Url: https://codereview.chromium.org/2649163002
Cr-Commit-Position: refs/heads/master@{#445401}
Committed:
https://chromium.googlesource.com/chromium/src/+/59c02c3aa8f983c2adecb654bd71...
==========

[commit-bot: I haz the power, 2017-01-23 17:46:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/59c02c3aa8f983c2adecb654bd71...