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

Issue 921333006: DevTools: Merge main thread strips on timeline overview into a single one. (Closed)

Created:
5 years, 10 months ago by alph
Modified:
5 years, 10 months ago
Reviewers:
CC:
blink-reviews, caseq+blink_chromium.org, loislo+blink_chromium.org, eustas+blink_chromium.org, malch+blink_chromium.org, yurys+blink_chromium.org, lushnikov+blink_chromium.org, pfeldman+blink_chromium.org, apavlov+blink_chromium.org, devtools-reviews_chromium.org, sergeyv+blink_chromium.org, aandrey+blink_chromium.org, kozyatinskiy+blink_chromium.org
Target Ref:
refs/heads/master
Project:
blink
Visibility:
Public.

Description

DevTools: Merge main thread strips on timeline overview into a single one. Merge all the strips corresponding to main renderer thread into one and show only the top level activity on it. All the background threads activity, e.g. workers, compositor threads, etc are shown on the second strip. BUG=460072

Patch Set 1 #

Patch Set 2 : Put background activities onto their own strips. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+75 lines, -61 lines) Patch
M Source/devtools/front_end/timeline/TimelineEventOverview.js View 1 3 chunks +64 lines, -50 lines 0 comments Download
M Source/devtools/front_end/timeline/TimelinePanel.js View 1 1 chunk +1 line, -1 line 0 comments Download
M Source/devtools/front_end/timeline/TimelineUIUtils.js View 2 chunks +10 lines, -10 lines 0 comments Download

Powered by Google App Engine
This is Rietveld 408576698