Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

A traced value's argument name is changed from "RenderFrame" to
"renderFrame", to be better consistent with the naming style in Trace
Viewer.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/7cee1e976a770ec9a99022824ca99d0114774e7f
Cr-Commit-Position: refs/heads/master@{#397293}

[Xiaocheng, 2016-06-01 05:05:21]

Description was changed from

==========
Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

A traced value's argument name is changed from "RenderFrame" to
"renderFrame", to be better consistent with the naming style in Trace
Viewer.
==========

to

==========
Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

A traced value's argument name is changed from "RenderFrame" to
"renderFrame", to be better consistent with the naming style in Trace
Viewer.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

[Xiaocheng, 2016-06-01 05:06:11]

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

[commit-bot: I haz the power, 2016-06-01 05:06:25]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2028913003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2028913003/1

[Xiaocheng, 2016-06-01 05:07:54]

xiaochengh@chromium.org changed reviewers:
+ clamy@chromium.org, skyostil@chromium.org

[Xiaocheng, 2016-06-01 05:07:55]

Just a minor change in the naming style. PTAL.

[commit-bot: I haz the power, 2016-06-01 06:39:08]

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

[commit-bot: I haz the power, 2016-06-01 06:39:09]

Dry run: This issue passed the CQ dry run.

[Sami, 2016-06-01 09:11:48]

lgtm with a nit.

https://codereview.chromium.org/2028913003/diff/1/content/browser/frame_host/...
File content/browser/frame_host/frame_tree_node_blame_context.cc (right):

https://codereview.chromium.org/2028913003/diff/1/content/browser/frame_host/...
content/browser/frame_host/frame_tree_node_blame_context.cc:59:
value->BeginDictionary("renderFrame");
nit: render_frame to match the other properties here.

[Xiaocheng, 2016-06-01 09:19:20]

On 2016/06/01 at 09:11:48, skyostil wrote:
> lgtm with a nit.

Thanks.

> 
>
https://codereview.chromium.org/2028913003/diff/1/content/browser/frame_host/...
> File content/browser/frame_host/frame_tree_node_blame_context.cc (right):
> 
>
https://codereview.chromium.org/2028913003/diff/1/content/browser/frame_host/...
> content/browser/frame_host/frame_tree_node_blame_context.cc:59:
value->BeginDictionary("renderFrame");
> nit: render_frame to match the other properties here.

... but didn't we choose "renderFrame" as the name in
https://codereview.chromium.org/2016213002?

[clamy, 2016-06-01 13:08:43]

rs lgtm

[Xiaocheng, 2016-06-02 01:46:04]

The CQ bit was checked by xiaochengh@chromium.org

[commit-bot: I haz the power, 2016-06-02 01:48:01]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/2028913003/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/2028913003/1

[commit-bot: I haz the power, 2016-06-02 02:09:57]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-06-02 02:11:34]

Description was changed from

==========
Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

A traced value's argument name is changed from "RenderFrame" to
"renderFrame", to be better consistent with the naming style in Trace
Viewer.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Renaming in traced FrameTreeNodeBlameContext snapshot's argument name

A traced value's argument name is changed from "RenderFrame" to
"renderFrame", to be better consistent with the naming style in Trace
Viewer.
CQ_INCLUDE_TRYBOTS=tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/7cee1e976a770ec9a99022824ca99d0114774e7f
Cr-Commit-Position: refs/heads/master@{#397293}
==========

[commit-bot: I haz the power, 2016-06-02 02:11:35]

Patchset 1 (id:??) landed as
https://crrev.com/7cee1e976a770ec9a99022824ca99d0114774e7f
Cr-Commit-Position: refs/heads/master@{#397293}