views: Remove the canvas bounds from View traces.

views: Remove the canvas bounds from View traces.

These traces include getBaseLayerSize for the recording canvas which
would have made more sense when they were added as that represented
the area being rastered. As we are now recording instead, these bounds
do not really exist except for the purpose of tracing here, so just
remove them from the trace events.

They were added in https://codereview.chromium.org/8440008/ which is
sparse in its explanations, but I can only assume was tracking down
some bad performance. We keep the traces so that they show time
distribution still, but the bounds are no longer very interesting
and have no relation to the time spent anyhow.

R=sky@chromium.org
BUG=671433
Review-Url: https://codereview.chromium.org/2825783002
Cr-Commit-Position: refs/heads/master@{#465323}
Committed: https://chromium.googlesource.com/chromium/src/+/952bde5d7daf1e3627e116ce6f2ed770ef0a4913

[danakj, 2017-04-18 17:32:11]

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

[commit-bot: I haz the power, 2017-04-18 17:32:39]

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

[danakj, 2017-04-18 17:32:47]

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

[danakj, 2017-04-18 17:32:55]

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

[commit-bot: I haz the power, 2017-04-18 17:32:57]

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, 2017-04-18 18:21:52]

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

[commit-bot: I haz the power, 2017-04-18 18:21:53]

Dry run: This issue passed the CQ dry run.

[sky, 2017-04-18 19:10:12]

LGTM

[danakj, 2017-04-18 19:10:41]

The CQ bit was checked by danakj@chromium.org

[commit-bot: I haz the power, 2017-04-18 19:11:30]

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-04-18 19:21:15]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1492542641331200,
"parent_rev": "4ec47030b3d06cb198562b4647094191ba83b06e", "commit_rev":
"952bde5d7daf1e3627e116ce6f2ed770ef0a4913"}

[commit-bot: I haz the power, 2017-04-18 19:21:52]

Description was changed from

==========
views: Remove the canvas bounds from View traces.

These traces include getBaseLayerSize for the recording canvas which
would have made more sense when they were added as that represented
the area being rastered. As we are now recording instead, these bounds
do not really exist except for the purpose of tracing here, so just
remove them from the trace events.

They were added in https://codereview.chromium.org/8440008/ which is
sparse in its explanations, but I can only assume was tracking down
some bad performance. We keep the traces so that they show time
distribution still, but the bounds are no longer very interesting
and have no relation to the time spent anyhow.

R=sky@chromium.org
BUG=671433
==========

to

==========
views: Remove the canvas bounds from View traces.

These traces include getBaseLayerSize for the recording canvas which
would have made more sense when they were added as that represented
the area being rastered. As we are now recording instead, these bounds
do not really exist except for the purpose of tracing here, so just
remove them from the trace events.

They were added in https://codereview.chromium.org/8440008/ which is
sparse in its explanations, but I can only assume was tracking down
some bad performance. We keep the traces so that they show time
distribution still, but the bounds are no longer very interesting
and have no relation to the time spent anyhow.

R=sky@chromium.org
BUG=671433

Review-Url: https://codereview.chromium.org/2825783002
Cr-Commit-Position: refs/heads/master@{#465323}
Committed:
https://chromium.googlesource.com/chromium/src/+/952bde5d7daf1e3627e116ce6f2e...
==========

[commit-bot: I haz the power, 2017-04-18 19:21:53]

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