Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398
Committed: https://crrev.com/ac9f876bc06b8152ecc3a328147cdab19136361b
Cr-Commit-Position: refs/heads/master@{#14464}

[kwiberg-webrtc, 2016-09-29 12:41:23]

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

[kwiberg-webrtc, 2016-09-29 12:41:25]

You asked for it, so you get to review it. :-)

I had to refrain from sorting the includes in four files that brought in both
gtest stuff and X11 stuff. Apparently X11 defines macros that clash with
internal gtest type names...

[kjellander_webrtc, 2016-09-29 19:21:52]

Woho, lgtm
If you used/wrote a tool for this, would you mind sharing it or a link to it in
the CL description?

[kjellander_webrtc, 2016-09-29 19:22:38]

On 2016/09/29 19:21:52, kjellander_webrtc wrote:
> Woho, lgtm
> If you used/wrote a tool for this, would you mind sharing it or a link to it
in
> the CL description?

I think you can use NOTRY=True as long as this passes compilation.

[kwiberg-webrtc, 2016-09-30 08:52:42]

On 2016/09/29 19:21:52, kjellander_webrtc wrote:
> Woho, lgtm
> If you used/wrote a tool for this, would you mind sharing it or a link to it
in
> the CL description?

No, not really. :-( I visited each line where we #included gmock.h or gtest.h,
and executed an emacs macro that sorted the lines of the current paragraph
(where "paragraph" is defined as a block of text surrounded by blank lines). I
took a second to inspect each place manually to see if there were blank lines
that I should remove first, or anything else strange.

[kwiberg-webrtc, 2016-09-30 23:06:04]

Description was changed from

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

BUG=webrtc:6398
==========

to

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398
==========

[kwiberg-webrtc, 2016-10-01 01:42:41]

The CQ bit was checked by kwiberg@webrtc.org

[kwiberg-webrtc, 2016-10-01 01:42:42]

The patchset sent to the CQ was uploaded after l-g-t-m from
kjellander@webrtc.org
Link to the patchset: https://codereview.webrtc.org/2381013002/#ps40001 (title:
"rebase")

[commit-bot: I haz the power, 2016-10-01 01:42:52]

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-10-01 03:43:23]

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

[commit-bot: I haz the power, 2016-10-01 03:43:26]

Try jobs failed on following builders:
  android_arm64_rel on master.tryserver.webrtc (JOB_TIMED_OUT, no build URL)

[kwiberg-webrtc, 2016-10-01 05:28:28]

The CQ bit was checked by kwiberg@webrtc.org

[commit-bot: I haz the power, 2016-10-01 05:28:35]

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-10-01 05:29:47]

Description was changed from

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398
==========

to

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398
==========

[commit-bot: I haz the power, 2016-10-01 05:29:49]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-10-01 05:29:57]

Description was changed from

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398
==========

to

==========
Sort #includes that got unsorted when gmock.h and gtest.h moved to webrtc/test/

gmock.h and gtest.h were moved (or rather, got wrappers so that we
could put some icky compatibility hacks in one place instead of 500)
in this CL: https://codereview.webrtc.org/2358993004/

NOPRESUBMIT=true
BUG=webrtc:6398

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

[commit-bot: I haz the power, 2016-10-01 05:29:59]

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