Remove the rtc_relative_path GYP variable and similar defines

Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjingle/libjingle.gyp

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True

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

[kjellander_webrtc, 2016-04-19 21:28:48]

Description was changed from

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

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

to

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
==========

[kjellander_webrtc, 2016-04-19 21:28:48]

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

[kjellander_webrtc, 2016-04-19 21:29:07]

A small cleanup step towards removing talk/.

[kjellander_webrtc, 2016-04-21 12:35:51]

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

[kjellander_webrtc, 2016-04-21 12:35:51]

+perkj to offload pthatcher.

[perkj_webrtc, 2016-04-22 09:57:04]

lgtm

[kjellander_webrtc, 2016-04-22 10:30:40]

Description was changed from

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
==========

to

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
==========

[kjellander_webrtc, 2016-04-22 11:20:19]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-04-22 11:20:28]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1903553003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1903553003/1

[commit-bot: I haz the power, 2016-04-22 11:24:58]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-22 11:24:59]

Try jobs failed on following builders:
  presubmit on tryserver.webrtc (JOB_FAILED,
http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/5002)

[kjellander_webrtc, 2016-04-22 11:41:10]

Description was changed from

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
==========

to

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True
==========

[kjellander_webrtc, 2016-04-22 11:41:31]

On 2016/04/22 11:24:59, commit-bot: I haz the power wrote:
> Try jobs failed on following builders:
>   presubmit on tryserver.webrtc (JOB_FAILED,
> http://build.chromium.org/p/tryserver.webrtc/builders/presubmit/builds/5002)

Setting NOPRESUBMIT=True to bypass failing check on rtc_base dependency *sigh*

[kjellander_webrtc, 2016-04-22 11:41:36]

The CQ bit was checked by kjellander@webrtc.org

[commit-bot: I haz the power, 2016-04-22 11:41:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1903553003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1903553003/1

[commit-bot: I haz the power, 2016-04-22 11:41:54]

Description was changed from

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True
==========

to

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True
==========

[commit-bot: I haz the power, 2016-04-22 11:41:55]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-04-22 11:42:02]

Description was changed from

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True
==========

to

==========
Remove the rtc_relative_path GYP variable and similar defines

The defines that can be used to alter the include paths for Expat, SRTP
and gtest are no longer needed in WebRTC or Chromium. Let's remove them
to simplify the GYP a little.

Removed defines:
EXPAT_RELATIVE_PATH
GTEST_RELATIVE_PATH
SRTP_RELATIVE_PATH

They're all set in the Chromium build so this shouldn't affect Chromium:
https://code.google.com/p/chromium/codesearch#chromium/src/third_party/libjin...

BUG=webrtc:4256
NOTRY=True
NOPRESUBMIT=True

Committed: https://crrev.com/e19cf59eb6ee44fd4d7e7fbcfdd1a6ea75063605
Cr-Commit-Position: refs/heads/master@{#12467}
==========

[commit-bot: I haz the power, 2016-04-22 11:42:03]

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

[kjellander_webrtc, 2016-04-22 11:57:34]

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

The reason for reverting is: Breaks downstream for SRTP include paths. Will
rework this and reland without that one..