Reland of Android: Add support for TraceEvent before the native library is loaded.

Reland of Android: Add support for TraceEvent before the native library is loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
> TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam

Committed: https://crrev.com/0267e3c29d74e8a4b0cab629206e14fa02ae2009
Cr-Commit-Position: refs/heads/master@{#405120}

[Benoit L, 2016-07-12 08:36:29]

Created Reland of Android: Add support for TraceEvent before the native library
is loaded.

[pasko, 2016-07-12 09:29:47]

what are the changes on top of the trivial revert of the revert?

[Benoit L, 2016-07-12 11:18:09]

On 2016/07/12 09:29:47, pasko wrote:
> what are the changes on top of the trivial revert of the revert?

Patch set 1 is the reland, 2 is the fix.

[Benoit L, 2016-07-12 11:18:16]

The CQ bit was checked by lizeb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 11:18:54]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-12 11:19:05]

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

[commit-bot: I haz the power, 2016-07-12 11:19:06]

Dry run: This issue passed the CQ dry run.

[Benoit L, 2016-07-12 11:24:07]

Description was changed from

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
> 
> Reason for revert:
> https://crbug.com/627142
> 
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
> 
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
> 
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,wychen@chromiu...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=564041
==========

to

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
==========

[Benoit L, 2016-07-12 11:24:18]

The CQ bit was checked by lizeb@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-07-12 11:24:45]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[pasko, 2016-07-12 11:37:13]

lgtm, thank you

[Yaron, 2016-07-12 13:23:10]

lgtm

[commit-bot: I haz the power, 2016-07-12 13:26:31]

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

[commit-bot: I haz the power, 2016-07-12 13:26:32]

Dry run: Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Benoit L, 2016-07-12 14:23:46]

Description was changed from

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
==========

to

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam
==========

[Benoit L, 2016-07-12 14:24:32]

The CQ bit was checked by lizeb@chromium.org

[commit-bot: I haz the power, 2016-07-12 14:24:56]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[jam, 2016-07-12 16:20:15]

lgtm

[commit-bot: I haz the power, 2016-07-12 17:06:26]

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

[commit-bot: I haz the power, 2016-07-12 17:06:27]

Try jobs failed on following builders:
  linux_chromium_chromeos_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)

[Benoit L, 2016-07-13 09:49:35]

The CQ bit was checked by lizeb@chromium.org

[commit-bot: I haz the power, 2016-07-13 09:49:52]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-07-13 11:03:59]

Description was changed from

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam
==========

to

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam
==========

[commit-bot: I haz the power, 2016-07-13 11:04:00]

Committed patchset #2 (id:230001)

[commit-bot: I haz the power, 2016-07-13 11:04:07]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-13 11:05:44]

Description was changed from

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam
==========

to

==========
Reland of Android: Add support for TraceEvent before the native library is
loaded. (patchset #1 id:1 of https://codereview.chromium.org/2145443002/ )

Reason for revert:
Reland.

Original issue's description:
> Revert of Android: Add support for TraceEvent before the native library is
loaded. (patchset #12 id:220001 of https://codereview.chromium.org/1486053003/ )
>
> Reason for revert:
> https://crbug.com/627142
>
> Original issue's description:
> > Android: Add support for TraceEvent before the native library is loaded.
> >
> > Currently TraceEvents recorded before the native library load are
> > silently discarded. This CL adds support for such "early" events.
> >
> > The events are buffered in Java, and then sent to the native side once
> > it is available. This support is intentionally simple now. The aim is to
> > ease performance investigation of early startup, and to ease tracking of
> > early startup performance regression, since timings are then accessible
> > to TBM tests.
> >
> > Also use the newly-introduced events in a couple places, and fix a bug
> > in the TraceEvent we use to record the launcher activity lifetime (when
> > an Activity calls finish from onCreate(), onPause() is not called).
> >
> > BUG=564041
> >
> > Committed: https://crrev.com/fb0784294541c53f12e283124672750ab2e291f1
> > Cr-Commit-Position: refs/heads/master@{#404635}
>
>
TBR=pasko@chromium.org,yfriedman@chromium.org,jam@chromium.org,lizeb@chromium...
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=564041
>
> Committed: https://crrev.com/ffda73103b9a84e4739088175b9250a0c28e6f8c
> Cr-Commit-Position: refs/heads/master@{#404752}

BUG=564041
TBR=jam

Committed: https://crrev.com/0267e3c29d74e8a4b0cab629206e14fa02ae2009
Cr-Commit-Position: refs/heads/master@{#405120}
==========

[commit-bot: I haz the power, 2016-07-13 11:05:45]

Patchset 2 (id:??) landed as
https://crrev.com/0267e3c29d74e8a4b0cab629206e14fa02ae2009
Cr-Commit-Position: refs/heads/master@{#405120}

[phoglund_chromium, 2016-07-14 11:50:49]

phoglund@chromium.org changed reviewers:
+ phoglund@chromium.org

[phoglund_chromium, 2016-07-14 11:50:49]

This is breaking the WebRTC bots when we try to roll Chromium into WebRTC:
https://codereview.webrtc.org/2147203002/. Example:
https://build.chromium.org/p/tryserver.webrtc/builders/android_clang_dbg/buil...:

FAILED: lib/libwebrtc_nonparallel_tests.so
lib/libwebrtc_nonparallel_tests.so.TOC 
if [ ! -e lib/libwebrtc_nonparallel_tests.so -o ! -e
lib/libwebrtc_nonparallel_tests.so.TOC ]; then
/b/build/slave/cache/cipd/goma/gomacc
../../third_party/llvm-build/Release+Asserts/bin/clang++ -shared -Wl,-z,now
-Wl,-z,relro -Wl,--fatal-warnings -Wl,-z,defs -Wl,-z,noexecstack -fPIC
-B/b/build/slave/android_clang/build/src/third_party/binutils/Linux_x64/Release/bin
-Wl,-wrap,calloc -Wl,-wrap,free -Wl,-wrap,malloc -Wl,-wrap,memalign
-Wl,-wrap,posix_memalign -Wl,-wrap,pvalloc -Wl,-wrap,realloc -Wl,-wrap,valloc
-Wl,-z,relro -Wl,-z,now -fuse-ld=gold -Wl,--build-id=sha1 -Wl,--no-undefined
--sysroot=../../third_party/android_tools/ndk//platforms/android-16/arch-arm
-nostdlib
-L../../third_party/android_tools/ndk//sources/cxx-stl/llvm-libc++/libs/armeabi-v7a
-Wl,--exclude-libs=libgcc.a -Wl,--exclude-libs=libc++_static.a
-Wl,--exclude-libs=libcommon_audio.a -Wl,--exclude-libs=libcommon_audio_neon.a
-Wl,--exclude-libs=libcommon_audio_sse2.a -Wl,--exclude-libs=libiSACFix.a
-Wl,--exclude-libs=libisac_neon.a -Wl,--exclude-libs=libopus.a
-Wl,--exclude-libs=libvpx.a -target arm-linux-androideabi -Wl,--icf=all
-Wl,-shared,-Bsymbolic
../../third_party/android_tools/ndk//platforms/android-16/arch-arm/usr/lib/crtbegin_so.o
-Wl,--version-script=/b/build/slave/android_clang/build/src/build/android/android_no_jni_exports.lst
-Wl,--warn-shared-textrel -Wl,-O1 -Wl,--as-needed -o
lib/libwebrtc_nonparallel_tests.so -Wl,-soname=libwebrtc_nonparallel_tests.so
@lib/libwebrtc_nonparallel_tests.so.rsp && { readelf -d
lib/libwebrtc_nonparallel_tests.so | grep SONAME ; nm -gD -f p
lib/libwebrtc_nonparallel_tests.so | cut -f1-2 -d' '; } >
lib/libwebrtc_nonparallel_tests.so.TOC; else
/b/build/slave/cache/cipd/goma/gomacc
../../third_party/llvm-build/Release+Asserts/bin/clang++ -shared -Wl,-z,now
-Wl,-z,relro -Wl,--fatal-warnings -Wl,-z,defs -Wl,-z,noexecstack -fPIC
-B/b/build/slave/android_clang/build/src/third_party/binutils/Linux_x64/Release/bin
-Wl,-wrap,calloc -Wl,-wrap,free -Wl,-wrap,malloc -Wl,-wrap,memalign
-Wl,-wrap,posix_memalign -Wl,-wrap,pvalloc -Wl,-wrap,realloc -Wl,-wrap,valloc
-Wl,-z,relro -Wl,-z,now -fuse-ld=gold -Wl,--build-id=sha1 -Wl,--no-undefined
--sysroot=../../third_party/android_tools/ndk//platforms/android-16/arch-arm
-nostdlib
-L../../third_party/android_tools/ndk//sources/cxx-stl/llvm-libc++/libs/armeabi-v7a
-Wl,--exclude-libs=libgcc.a -Wl,--exclude-libs=libc++_static.a
-Wl,--exclude-libs=libcommon_audio.a -Wl,--exclude-libs=libcommon_audio_neon.a
-Wl,--exclude-libs=libcommon_audio_sse2.a -Wl,--exclude-libs=libiSACFix.a
-Wl,--exclude-libs=libisac_neon.a -Wl,--exclude-libs=libopus.a
-Wl,--exclude-libs=libvpx.a -target arm-linux-androideabi -Wl,--icf=all
-Wl,-shared,-Bsymbolic
../../third_party/android_tools/ndk//platforms/android-16/arch-arm/usr/lib/crtbegin_so.o
-Wl,--version-script=/b/build/slave/android_clang/build/src/build/android/android_no_jni_exports.lst
-Wl,--warn-shared-textrel -Wl,-O1 -Wl,--as-needed -o
lib/libwebrtc_nonparallel_tests.so -Wl,-soname=libwebrtc_nonparallel_tests.so
@lib/libwebrtc_nonparallel_tests.so.rsp && { readelf -d
lib/libwebrtc_nonparallel_tests.so | grep SONAME ; nm -gD -f p
lib/libwebrtc_nonparallel_tests.so | cut -f1-2 -d' '; } >
lib/libwebrtc_nonparallel_tests.so.tmp && if ! cmp -s
lib/libwebrtc_nonparallel_tests.so.tmp lib/libwebrtc_nonparallel_tests.so.TOC;
then mv lib/libwebrtc_nonparallel_tests.so.tmp
lib/libwebrtc_nonparallel_tests.so.TOC ; fi; fi
obj/chromium/src/base/libbase.a(obj/chromium/src/base/android/base.base_jni_registrar.o):../../chromium/src/base/android/base_jni_registrar.cc:base::android::kBaseRegisteredMethods:
error: undefined reference to 'base::android::RegisterEarlyTraceEvent(_JNIEnv*)'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Do we need to add something to our test binaries here? Please revert if there is
no quick way to fix this.

[pasko, 2016-07-14 17:37:44]

Feel free to revert, the next working day in PAR is next Monday (2016-07-17).

[pasko, 2016-07-14 17:40:27]

On 2016/07/14 17:37:44, pasko wrote:
> Feel free to revert, the next working day in PAR is next Monday (2016-07-17).

Pardon, next Monday is going to be 18th.

[phoglund_chromium, 2016-07-15 10:52:09]

A revert of this CL (patchset #2 id:230001) has been created in
https://codereview.chromium.org/2149993004/ by phoglund@chromium.org.

The reason for reverting is: This is breaking the WebRTC bots when we try to
roll Chromium into WebRTC:
https://codereview.webrtc.org/2147203002/

https://build.chromium.org/p/tryserver.webrtc/builders/android_clang_dbg/buil...

FAILED: lib/libwebrtc_nonparallel_tests.so
...
error: undefined reference to 'base::android::RegisterEarlyTraceEvent(_JNIEnv*)'
.