Roll chromium_revision 0868029..33f8768

Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

There was no other way to get the Android test to build
than renaming the APK to yuv_unittests. That made it possible
to use build/apk_test.gypi (there's some bug somewhere that assumes
names doesn't start with "lib", see http://crbug.com/543820).

TBR=fbarchard@chromium.org
BUG=libyuv:626
Committed: https://chromium.googlesource.com/libyuv/libyuv/+/d19f1c319a49a2e9f3b2bf07b0ebccfcc1e49380

[kjellander_chromium, 2016-08-04 06:56:51]

Description was changed from

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

TBR=fbarchard@chromium.org
BUG=
==========

to

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

[kjellander_chromium, 2016-08-04 07:24:23]

Description was changed from

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

to

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

Had to copy a bunch of stuff from
https://chromium.googlesource.com/chromium/src/+/master/build/apk_test.gypi
since we haven't figured out how to use that directly instead
for libyuv.

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

[kjellander_chromium, 2016-08-04 08:01:18]

Description was changed from

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

Had to copy a bunch of stuff from
https://chromium.googlesource.com/chromium/src/+/master/build/apk_test.gypi
since we haven't figured out how to use that directly instead
for libyuv.

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

to

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

There was no other way to get the Android test to build
than renaming the APK to yuv_unittests. That made it possible
to use build/apk_test.gypi (there's some bug somewhere that assumes
names doesn't start with "lib", see http://crbug.com/543820).

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

[kjellander_chromium, 2016-08-04 08:03:58]

Patchset #1 (id:1) has been deleted

[kjellander_chromium, 2016-08-04 08:05:35]

I successfully executed the test on Android:

$ out/Release/bin/run_yuv_unittest 
W    0.042s Main  No isolate file provided. No data deps will be pushed.
C   36.061s Main 
********************************************************************************
C   36.061s Main  Summary
C   36.061s Main 
********************************************************************************
C   36.063s Main  [==========] 981 tests ran.
C   36.063s Main  [  PASSED  ] 981 tests.
C   36.063s Main 
********************************************************************************

I hope you're fine with the renaming of it (it's unlikely somebody wants to
track down what needs to change to support the name starting with lib).

[kjellander_chromium, 2016-08-04 11:42:59]

landing this assuming you're fine with the annoying (but I don't see any other
way) rename of the Android test.

[kjellander_chromium, 2016-08-04 11:43:23]

Description was changed from

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

There was no other way to get the Android test to build
than renaming the APK to yuv_unittests. That made it possible
to use build/apk_test.gypi (there's some bug somewhere that assumes
names doesn't start with "lib", see http://crbug.com/543820).

TBR=fbarchard@chromium.org
BUG=libyuv:626
==========

to

==========
Roll chromium_revision 0868029..33f8768

Add a lot of new dependencies for the Android toolchain
to setup_links.py (added for WebRTC in https://codereview.webrtc.org/2190893002
and https://codereview.webrtc.org/2215633002).

There was no other way to get the Android test to build
than renaming the APK to yuv_unittests. That made it possible
to use build/apk_test.gypi (there's some bug somewhere that assumes
names doesn't start with "lib", see http://crbug.com/543820).

TBR=fbarchard@chromium.org
BUG=libyuv:626

Committed:
https://chromium.googlesource.com/libyuv/libyuv/+/d19f1c319a49a2e9f3b2bf07b0e...
==========

[kjellander_chromium, 2016-08-04 11:43:25]

Committed patchset #2 (id:40001) manually as
d19f1c319a49a2e9f3b2bf07b0ebccfcc1e49380 (presubmit successful).

[fbarchard1, 2016-08-04 22:30:13]

fbarchard@google.com changed reviewers:
+ fbarchard@google.com

[fbarchard1, 2016-08-04 22:30:13]

lgtm!

[fbarchard1, 2016-08-04 22:30:47]

On 2016/08/04 22:30:13, fbarchard1 wrote:
> lgtm!

lgtm for the code review.  But awesome!

[fbarchard1, 2016-08-04 22:30:49]

On 2016/08/04 22:30:13, fbarchard1 wrote:
> lgtm!

lgtm for the code review.  But awesome!