Commit a31ee25a authored by caseq@chromium.org's avatar caseq@chromium.org

Timeline: fix expansion arrow appearing on rows with no visible children

We used to check TimelinePresentationRecord.expandable(), but it was accidrently replaced
with hasPresentationChildren(), which is not the same now, since we keep hidden records
in the presentation model.

R=pfeldman@chromium.org

Review URL: https://codereview.chromium.org/201823003

git-svn-id: svn://svn.chromium.org/blink/trunk@169620 bbb929c8-8fbe-4397-9dbb-9b2b20218538
parent a434069c
......@@ -343,14 +343,6 @@ WebInspector.TimelinePresentationModel.Record.prototype = {
return this._presentationChildren;
},
/**
* @return {boolean}
*/
hasPresentationChildren: function()
{
return !!this._presentationChildren.length;
},
/**
* @return {boolean}
*/
......
......@@ -1031,7 +1031,7 @@ WebInspector.TimelineRecordListRow.prototype = {
}
}
this._expandArrowElement.classList.toggle("parent", presentationRecord.hasPresentationChildren());
this._expandArrowElement.classList.toggle("parent", presentationRecord.expandable());
this._expandArrowElement.classList.toggle("expanded", !!presentationRecord.visibleChildrenCount());
this._record.setListRow(this);
},
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment