commit 9a89a491f801afb2a4fb0d90be397a4b2e553a95
author Mirko Bonadei <mbonadei@webrtc.org> Tue May 29 14:22:32 2018
committer Commit Bot <commit-bot@chromium.org> Tue May 29 15:17:04 2018
tree 2935f7be35b58c73ee49d4e5b9ed8bad04e2c660
parent 79445eadcc7a004333beac0148407d0607805ff6

Reland "Reland "Adding absl includes and defines to rtc_* templates.""

This reverts commit 759eb4f2ad7e23b65fe0b834f8ba7f580bf1a933.

Reason for revert:
The problem has been fixed in https://chromium-review.googlesource.com/1075889.

Original change's description:
> Revert "Reland "Adding absl includes and defines to rtc_* templates.""
> 
> This reverts commit fae51e4c95780f59f4bedf6e01403ff86152e168.
> 
> Reason for revert:
> When use_xcode_clang = true we get:
> error: unknown warning option '-Wno-unused-template'; did you mean '-Wno-unused-result'? [-Werror,-Wunknown-warning-option]
> error: unknown warning option '-Wno-zero-as-null-pointer-constant'; did you mean '-Wno-int-to-void-pointer-cast'? [-Werror,-Wunknown-warning-option]
> 
> 
> Original change's description:
> > Reland "Adding absl includes and defines to rtc_* templates."
> > 
> > This reverts commit 8436a699a998e4fa30d97786142baad08f110d2a.
> > 
> > Reason for revert: 
> > New absl roll -> https://chromium-review.googlesource.com/1071468
> > 
> > Original change's description:
> > > Revert "Reland "Adding absl includes and defines to rtc_* templates.""
> > > 
> > > This reverts commit bdb0fe42bc46d190ca45fc5a6658eddbfa5eead5.
> > > 
> > > Reason for revert: https://ci.chromium.org/buildbot/chromium.fyi/Jumbo%20Win%20x64/11502
> > > 
> > > Original change's description:
> > > > Reland "Adding absl includes and defines to rtc_* templates."
> > > > 
> > > > This reverts commit 85cb19fec7caf558dee7a09aafabe01c5ac78f3f.
> > > > 
> > > > Reason for revert: The new version of Abseil should fix the previous
> > > > issue.
> > > > 
> > > > Original change's description:
> > > > > Revert "Reland "Adding absl includes and defines to rtc_* templates.""
> > > > > 
> > > > > This reverts commit 9632112a16d70a146e917db4de761e6253dfc364.
> > > > > 
> > > > > Reason for revert: It breaks the WebRTC roll into Chromium.
> > > > > https://chromium-review.googlesource.com/c/chromium/src/+/1061476
> > > > > 
> > > > > Original change's description:
> > > > > > Reland "Adding absl includes and defines to rtc_* templates."
> > > > > > 
> > > > > > This reverts commit d161eda477491b2b97fb3f26d229c625a2a0e9b8.
> > > > > > 
> > > > > > Reason for revert: The problem with iOS trybots should be fixed.
> > > > > > 
> > > > > > Original change's description:
> > > > > > > Revert "Adding absl includes and defines to rtc_* templates."
> > > > > > >
> > > > > > > This reverts commit 9d8f3850f4c4faad5dc5ab32ab6f2c9c43df7b6c.
> > > > > > >
> > > > > > > Reason for revert: Breaks some trybots: https://build.chromium.org/p/client.webrtc/builders/iOS64%20Release/builds/12793.
> > > > > > >
> > > > > > > Original change's description:
> > > > > > > > Adding absl includes and defines to rtc_* templates.
> > > > > > > >
> > > > > > > > This CL implicitly adds the -I compiler flag and absl macros to WebRTC
> > > > > > > > templates. In order to include absl headers using relative paths, WebRTC
> > > > > > > > needs to ensure that all its build targets are able to see absl headers.
> > > > > > > >
> > > > > > > > This can also be done with public_deps, but WebRTC is trying to avoid
> > > > > > > > it because it creates problems with other build systems. Given this
> > > > > > > > constraint, using rtc_* templates is the most reliable solution.
> > > > > > > >
> > > > > > > > Please note that rtc_* templates are adding absl includes and defines
> > > > > > > > as public_configs, this means that build targets with WebRTC targets
> > > > > > > > in their public_deps will propagate these configs following the GN
> > > > > > > > guideline.
> > > > > > > >
> > > > > > > > Bug: webrtc:8821
> > > > > > > > Change-Id: I4aa594a524f4bd045bcb3e80d76cc27f06fe01d7
> > > > > > > > Reviewed-on: https://webrtc-review.googlesource.com/70367
> > > > > > > > Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> > > > > > > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > > > > Cr-Commit-Position: refs/heads/master@{#22927}
> > > > > > >
> > > > > > > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > > > > > >
> > > > > > > Change-Id: Id8e1f881c57553386566eb1970f6b9f8632cab37
> > > > > > > No-Presubmit: true
> > > > > > > No-Tree-Checks: true
> > > > > > > No-Try: true
> > > > > > > Bug: webrtc:8821
> > > > > > > Reviewed-on: https://webrtc-review.googlesource.com/71000
> > > > > > > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > > > Cr-Commit-Position: refs/heads/master@{#22928}
> > > > > > 
> > > > > > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > > > > > 
> > > > > > Bug: webrtc:8821
> > > > > > Change-Id: I6ee2eda97bbcd4c9be25c9c4073272192b0373f8
> > > > > > Reviewed-on: https://webrtc-review.googlesource.com/71700
> > > > > > Reviewed-by: Patrik Höglund <phoglund@webrtc.org>
> > > > > > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > > Cr-Commit-Position: refs/heads/master@{#23251}
> > > > > 
> > > > > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > > > > 
> > > > > Change-Id: I61fb749797314ca514691b341c66f7f39ef45491
> > > > > No-Presubmit: true
> > > > > No-Tree-Checks: true
> > > > > No-Try: true
> > > > > Bug: webrtc:8821
> > > > > Reviewed-on: https://webrtc-review.googlesource.com/77220
> > > > > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > > > Cr-Commit-Position: refs/heads/master@{#23264}
> > > > 
> > > > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > > > 
> > > > # Not skipping CQ checks because original CL landed > 1 day ago.
> > > > 
> > > > Bug: webrtc:8821
> > > > Change-Id: I71dea953a002a0d526949c627653bcad0c6518fc
> > > > Reviewed-on: https://webrtc-review.googlesource.com/77781
> > > > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > > Cr-Commit-Position: refs/heads/master@{#23317}
> > > 
> > > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > > 
> > > Change-Id: I6010f9264dba7bcc4e82c4f4bbfb2eca561e500e
> > > No-Presubmit: true
> > > No-Tree-Checks: true
> > > No-Try: true
> > > Bug: webrtc:8821, chromium:845158
> > > Reviewed-on: https://webrtc-review.googlesource.com/78061
> > > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > > Cr-Commit-Position: refs/heads/master@{#23328}
> > 
> > TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> > 
> > # Not skipping CQ checks because original CL landed > 1 day ago.
> > 
> > Bug: webrtc:8821, chromium:845158
> > Change-Id: Iebe0958012c39e1321487e5425f43904eaf5fe91
> > Reviewed-on: https://webrtc-review.googlesource.com/78705
> > Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> > Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> > Cr-Commit-Position: refs/heads/master@{#23403}
> 
> TBR=phoglund@webrtc.org,mbonadei@webrtc.org
> 
> Change-Id: I8285cf59402aa6862ea7e3ec21f885360b7050fb
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: webrtc:8821, chromium:845158
> Reviewed-on: https://webrtc-review.googlesource.com/79420
> Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
> Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
> Cr-Commit-Position: refs/heads/master@{#23416}

TBR=phoglund@webrtc.org,mbonadei@webrtc.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: webrtc:8821, chromium:845158
Change-Id: I18ffdb0d7be61daf0b6464c68d219aa352f42dde
Reviewed-on: https://webrtc-review.googlesource.com/79582
Reviewed-by: Mirko Bonadei <mbonadei@webrtc.org>
Commit-Queue: Mirko Bonadei <mbonadei@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#23435}