[DevTools] Remove SidebarTreeElement from ProfilesPanel.

[DevTools] Remove SidebarTreeElement from ProfilesPanel.

This is a step towards removing SidebarTreeElement, and moving
all tree outlines into shadow dom.

BUG=none

Committed: https://crrev.com/40a78688dffd36ab1ace3c9cdc06242425067b19
Cr-Commit-Position: refs/heads/master@{#418708}

[dgozman, 2016-09-13 00:37:18]

dgozman@chromium.org changed reviewers:
+ alph@chromium.org

[dgozman, 2016-09-13 00:37:18]

Could you please take a look?

[alph, 2016-09-13 00:59:07]

lgtm

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
(right):

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js:1165:
this._titlesElement.classList.remove("no-subtitle");
nit:
this._subtitleElement.textContent = statusUpdate.subtitle || "";
this._titlesElement.classList.toggle("no-subtitle", !statusUpdate.subtitle);

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js:1296: *
@return {boolean}
@param

[dgozman, 2016-09-14 18:34:11]

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
File third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js
(right):

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js:1165:
this._titlesElement.classList.remove("no-subtitle");
On 2016/09/13 00:59:07, alph wrote:
> nit:
> this._subtitleElement.textContent = statusUpdate.subtitle || "";
> this._titlesElement.classList.toggle("no-subtitle", !statusUpdate.subtitle);

Done.

https://codereview.chromium.org/2335983002/diff/1/third_party/WebKit/Source/d...
third_party/WebKit/Source/devtools/front_end/profiler/ProfilesPanel.js:1296: *
@return {boolean}
On 2016/09/13 00:59:07, alph wrote:
> @param

Done.

[dgozman, 2016-09-14 19:11:01]

The CQ bit was checked by dgozman@chromium.org

[dgozman, 2016-09-14 19:11:02]

The patchset sent to the CQ was uploaded after l-g-t-m from alph@chromium.org
Link to the patchset: https://codereview.chromium.org/2335983002/#ps40001
(title: "rebased")

[commit-bot: I haz the power, 2016-09-14 19:11:55]

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-09-14 21:14:01]

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

[commit-bot: I haz the power, 2016-09-14 21:14:02]

Try jobs failed on following builders:
  win_chromium_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[dgozman, 2016-09-14 21:26:37]

The CQ bit was checked by dgozman@chromium.org

[commit-bot: I haz the power, 2016-09-14 21:27:31]

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-09-14 22:59:16]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-09-14 23:00:55]

Description was changed from

==========
[DevTools] Remove SidebarTreeElement from ProfilesPanel.

This is a step towards removing SidebarTreeElement, and moving
all tree outlines into shadow dom.

BUG=none
==========

to

==========
[DevTools] Remove SidebarTreeElement from ProfilesPanel.

This is a step towards removing SidebarTreeElement, and moving
all tree outlines into shadow dom.

BUG=none

Committed: https://crrev.com/40a78688dffd36ab1ace3c9cdc06242425067b19
Cr-Commit-Position: refs/heads/master@{#418708}
==========

[commit-bot: I haz the power, 2016-09-14 23:00:56]

Patchset 3 (id:??) landed as
https://crrev.com/40a78688dffd36ab1ace3c9cdc06242425067b19
Cr-Commit-Position: refs/heads/master@{#418708}