Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(505)

Unified Diff: third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js

Issue 2873153002: DevTools: Draw screenshot independently of frames bar on flamechart. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js
diff --git a/third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js b/third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js
index bba71855f9af3a95385968d5e710e935e7008cfa..5ac5dd82f6e29a7ce98b1976b153dbd25516aba6 100644
--- a/third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js
+++ b/third_party/WebKit/Source/devtools/front_end/perf_ui/FlameChart.js
@@ -1120,7 +1120,8 @@ PerfUI.FlameChart = class extends PerfUI.ChartViewport {
currentOffset += style.height;
}
var isFirstOnLevel = groupIndex >= 0 && level === groups[groupIndex].startLevel;
- var thisLevelIsVisible = visible || isFirstOnLevel && groups[groupIndex].style.useFirstLineForOverview;
+ var thisLevelIsVisible =
+ parentGroupIsVisible && (visible || isFirstOnLevel && groups[groupIndex].style.useFirstLineForOverview);
if (level < levelCount) {
var height;
if (groupIndex >= 0) {

Powered by Google App Engine
This is Rietveld 408576698