Fix inconsistent setting of the rtc_include_unittests flag.

Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org

Committed: https://crrev.com/9d34148714773339a4e8396bd28aceb571554d36
Cr-Commit-Position: refs/heads/master@{#13484}

[phoglund, 2016-07-15 08:31:48]

Description was changed from

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/
==========

to

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.
==========

[phoglund, 2016-07-15 08:31:48]

phoglund@webrtc.org changed reviewers:
+ tommi@webrtc.org

[phoglund, 2016-07-15 08:32:13]

Description was changed from

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.
==========

to

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.
==========

[phoglund, 2016-07-15 10:54:19]

Description was changed from

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.
==========

to

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org
==========

[phoglund, 2016-07-15 10:54:25]

The CQ bit was checked by phoglund@webrtc.org

[commit-bot: I haz the power, 2016-07-15 10:54:31]

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, 2016-07-15 10:56:10]

Description was changed from

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org
==========

to

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org
==========

[commit-bot: I haz the power, 2016-07-15 10:56:11]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-07-15 10:56:20]

Description was changed from

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org
==========

to

==========
Fix inconsistent setting of the rtc_include_unittests flag.

On advice from brettw, se discussion here:
https://codereview.webrtc.org/2149543002/

The problem was we were setting the flag to both false and true,
and the the true happened to win out for WebRTC checkouts and
false for Chromium checkouts. This change should make this
mechanic more obvious.

This change _should_ have no effect downstream.

Doing tbr to see if we can get the chromium import back into a good state today.

TBR=tommi@webrtc.org

Committed: https://crrev.com/9d34148714773339a4e8396bd28aceb571554d36
Cr-Commit-Position: refs/heads/master@{#13484}
==========

[commit-bot: I haz the power, 2016-07-15 10:56:21]

Patchset 1 (id:??) landed as
https://crrev.com/9d34148714773339a4e8396bd28aceb571554d36
Cr-Commit-Position: refs/heads/master@{#13484}

[phoglund, 2016-07-15 11:13:07]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.webrtc.org/2150293002/ by phoglund@webrtc.org.

The reason for reverting is: Nope, breaks chromium.