Add some tracing to NewTabPage.

Add some tracing to NewTabPage.

Review-Url: https://codereview.chromium.org/2645393002
Cr-Commit-Position: refs/heads/master@{#448002}
Committed: https://chromium.googlesource.com/chromium/src/+/4cca33f07610e310cc5504110a102f747eff4319

[Michael van Ouwerkerk, 2017-01-23 15:44:21]

mvanouwerkerk@chromium.org changed reviewers:
+ bauerb@chromium.org

[Michael van Ouwerkerk, 2017-01-23 15:44:21]

Bernhard, could you take a look please?

[Bernhard Bauer, 2017-01-23 16:15:24]

lgtm

https://codereview.chromium.org/2645393002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
(right):

https://codereview.chromium.org/2645393002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java:325:
TraceEvent.begin("NewTabPageView.initialize()");
Would it make sense to use `TAG + ".initialize()"` or something? Would save some
string space and make sure we stay consistent.

[Michael van Ouwerkerk, 2017-02-03 15:07:18]

https://codereview.chromium.org/2645393002/diff/1/chrome/android/java/src/org...
File chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
(right):

https://codereview.chromium.org/2645393002/diff/1/chrome/android/java/src/org...
chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java:325:
TraceEvent.begin("NewTabPageView.initialize()");
On 2017/01/23 16:15:24, Bernhard Bauer wrote:
> Would it make sense to use `TAG + ".initialize()"` or something? Would save
some
> string space and make sure we stay consistent.

Done.

[Michael van Ouwerkerk, 2017-02-03 15:07:31]

Description was changed from

==========
Add some tracing to NewTabPage.

BUG=??
==========

to

==========
Add some tracing to NewTabPage.
==========

[Michael van Ouwerkerk, 2017-02-03 15:08:04]

The CQ bit was checked by mvanouwerkerk@chromium.org

[Michael van Ouwerkerk, 2017-02-03 15:08:04]

The patchset sent to the CQ was uploaded after l-g-t-m from bauerb@chromium.org
Link to the patchset: https://codereview.chromium.org/2645393002/#ps20001
(title: "Address review comments from bauerb.")

[commit-bot: I haz the power, 2017-02-03 15:08:18]

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, 2017-02-03 16:38:23]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486134484455620,
"parent_rev": "385db906ffa3937dab92438ee6fc43b6485514ae", "commit_rev":
"4cca33f07610e310cc5504110a102f747eff4319"}

[commit-bot: I haz the power, 2017-02-03 16:39:03]

Description was changed from

==========
Add some tracing to NewTabPage.
==========

to

==========
Add some tracing to NewTabPage.

Review-Url: https://codereview.chromium.org/2645393002
Cr-Commit-Position: refs/heads/master@{#448002}
Committed:
https://chromium.googlesource.com/chromium/src/+/4cca33f07610e310cc5504110a10...
==========

[commit-bot: I haz the power, 2017-02-03 16:39:08]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/4cca33f07610e310cc5504110a10...