Moving sanitizers from build/ to base/

Moving sanitizers from build/ to base/

We have to move the content of the 'webrtc/build/' directory up of one level (see crbug.com/611808 for further details).

To avoid a collision with the DEPSed build/ at the top level we are moving all the content of 'webrtc/build/' to different locations (see webrtc:7030 for the suggested locations).

BUG=webrtc:7030
NOTRY=True

Review-Url: https://codereview.webrtc.org/2651913002
Cr-Commit-Position: refs/heads/master@{#16259}
Committed: https://chromium.googlesource.com/external/webrtc/+/18e83ea598ae8ed0446169d35a34cc5d14dc71a0

[mbonadei, 2017-01-24 08:04:50]

Description was changed from

==========
Moving sanitizers from build/ to base/

BUG=webrtc:7030
==========

to

==========
Moving sanitizers from build/ to base/

BUG=webrtc:7030
NOTRY=True
==========

[mbonadei, 2017-01-24 08:05:50]

mbonadei@webrtc.org changed reviewers:
+ henrika@webrtc.org, kjellander@webrtc.org

[kjellander_webrtc, 2017-01-24 08:11:36]

kjellander@webrtc.org changed reviewers:
+ solenberg@webrtc.org

[kjellander_webrtc, 2017-01-24 08:11:37]

+solenberg due to ongoing work on webrtc/base.
Fredrik: do you think this is a good spot or do we want to avoid polluting
webrtc/base with a new dir (I was the one suggesting webrtc/base)?
We could move them to tools-webrtc as well.

https://codereview.webrtc.org/2651913002/diff/1/webrtc/base/sanitizers/README
File webrtc/base/sanitizers/README (right):

https://codereview.webrtc.org/2651913002/diff/1/webrtc/base/sanitizers/README...
webrtc/base/sanitizers/README:3: webrtc/supplement.gypi file.
Change to:
They're overriding the default ones for Chromium using our
build_overrides/webrtc.gni file.

[henrika_webrtc, 2017-01-24 08:26:13]

I have not touched webrtc/base and am not comfortable in making this review.
Adding Tommi as CC in case he has input before Fredrik is back.

[henrika_webrtc, 2017-01-24 08:26:50]

henrika@webrtc.org changed reviewers:
- henrika@webrtc.org

[the sun, 2017-01-24 08:44:39]

On 2017/01/24 08:26:13, henrika_webrtc wrote:
> I have not touched webrtc/base and am not comfortable in making this review.
> Adding Tommi as CC in case he has input before Fredrik is back.

To me, tools-webrtc appears a better place, in order to avoid mixing up library
code with support infrastructure.

[mbonadei, 2017-01-24 10:54:51]

Description was changed from

==========
Moving sanitizers from build/ to base/

BUG=webrtc:7030
NOTRY=True
==========

to

==========
Moving sanitizers from build/ to base/

We have to move the content of the 'webrtc/build/' directory up of one level
(see crbug.com/611808 for further details).

To avoid a collision with the DEPSed build/ at the top level we are moving all
the content of 'webrtc/build/' to different locations (see webrtc:7030 for the
suggested locations).

BUG=webrtc:7030
NOTRY=True
==========

[mbonadei, 2017-01-24 14:13:02]

On 2017/01/24 08:44:39, the sun (OOO until Jan 30th) wrote:
> On 2017/01/24 08:26:13, henrika_webrtc wrote:
> > I have not touched webrtc/base and am not comfortable in making this review.
> > Adding Tommi as CC in case he has input before Fredrik is back.
> 
> To me, tools-webrtc appears a better place, in order to avoid mixing up
library
> code with support infrastructure.

Ok, moved everything to tools-webrtc/.

[kjellander_webrtc, 2017-01-24 17:41:35]

lgtm

[mbonadei, 2017-01-25 07:02:05]

The CQ bit was checked by mbonadei@webrtc.org

[commit-bot: I haz the power, 2017-01-25 07:02:17]

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

[commit-bot: I haz the power, 2017-01-25 07:04:52]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1485327725708260,
"parent_rev": "f534659ee6896b068e562bb873e0dbae5080352e", "commit_rev":
"18e83ea598ae8ed0446169d35a34cc5d14dc71a0"}

[commit-bot: I haz the power, 2017-01-25 07:04:54]

Description was changed from

==========
Moving sanitizers from build/ to base/

We have to move the content of the 'webrtc/build/' directory up of one level
(see crbug.com/611808 for further details).

To avoid a collision with the DEPSed build/ at the top level we are moving all
the content of 'webrtc/build/' to different locations (see webrtc:7030 for the
suggested locations).

BUG=webrtc:7030
NOTRY=True
==========

to

==========
Moving sanitizers from build/ to base/

We have to move the content of the 'webrtc/build/' directory up of one level
(see crbug.com/611808 for further details).

To avoid a collision with the DEPSed build/ at the top level we are moving all
the content of 'webrtc/build/' to different locations (see webrtc:7030 for the
suggested locations).

BUG=webrtc:7030
NOTRY=True

Review-Url: https://codereview.webrtc.org/2651913002
Cr-Commit-Position: refs/heads/master@{#16259}
Committed:
https://chromium.googlesource.com/external/webrtc/+/18e83ea598ae8ed0446169d35...
==========

[commit-bot: I haz the power, 2017-01-25 07:04:55]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/external/webrtc/+/18e83ea598ae8ed0446169d35...