Move PerfTest TraceEvent to async

Move PerfTest TraceEvent to async

Move the perf test events to use START/FINISH TraceEvent paths, which will work properly
if they are not triggered in the right nested order.

BUG=http://crbug.com/150584


Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=161942

[Yaron, 2012-10-11 21:51:38]

I don't know any of this code. Can you find someone else that's familiar with 
it?

[David Trainor- moved to gerrit, 2012-10-11 23:38:09]

Was LGTM'ed downstream by Grace and

[David Trainor- moved to gerrit, 2012-10-11 23:38:30]

On 2012/10/11 23:38:09, dtrainor wrote:
> Was LGTM'ed downstream by Grace and

Xianzhu!

[Ted C, 2012-10-11 23:46:30]

nits!

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
File content/common/android/trace_event_binding.cc (right):

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
content/common/android/trace_event_binding.cc:26: jname_(jname),
should these be aligned with env above?

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
content/common/android/trace_event_binding.cc:60: : TraceEventDataConverter(env,
jname, jarg),
should be indented 4

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
File content/public/android/java/src/org/chromium/content/common/TraceEvent.java
(right):

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
content/public/android/java/src/org/chromium/content/common/TraceEvent.java:80:
if (sEnabled) {
I would make these all one liners

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
content/public/android/java/src/org/chromium/content/common/TraceEvent.java:85:
public static void startAsync(String name, long id) {
no javadocs?

[David Trainor- moved to gerrit, 2012-10-12 00:10:49]

Addressed nits.

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
File content/common/android/trace_event_binding.cc (right):

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
content/common/android/trace_event_binding.cc:26: jname_(jname),
On 2012/10/11 23:46:30, Ted C wrote:
> should these be aligned with env above?

Done.

https://chromiumcodereview.appspot.com/11090082/diff/1/content/common/android...
content/common/android/trace_event_binding.cc:60: : TraceEventDataConverter(env,
jname, jarg),
On 2012/10/11 23:46:30, Ted C wrote:
> should be indented 4

Done.

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
File content/public/android/java/src/org/chromium/content/common/TraceEvent.java
(right):

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
content/public/android/java/src/org/chromium/content/common/TraceEvent.java:80:
if (sEnabled) {
On 2012/10/11 23:46:30, Ted C wrote:
> I would make these all one liners

Done.

https://chromiumcodereview.appspot.com/11090082/diff/1/content/public/android...
content/public/android/java/src/org/chromium/content/common/TraceEvent.java:85:
public static void startAsync(String name, long id) {
On 2012/10/11 23:46:30, Ted C wrote:
> no javadocs?

Done.

[Ted C, 2012-10-12 00:11:15]

lgtm

[David Trainor- moved to gerrit, 2012-10-12 16:24:59]

Yaron: Can I get the lgtm for content/public/android?

[Yaron, 2012-10-12 16:36:24]

lgtm

[commit-bot: I haz the power, 2012-10-12 22:37:04]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dtrainor@chromium.org/11090082/3004

[commit-bot: I haz the power, 2012-10-12 22:57:44]

Sorry for I got bad news for ya.
Compile failed with a clobber build.
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[commit-bot: I haz the power, 2012-10-15 17:48:28]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/dtrainor@chromium.org/11090082/14003

[commit-bot: I haz the power, 2012-10-15 20:33:04]

Change committed as 161942