New issue
Advanced search Search tips

Issue 681258 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jan 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 3
Type: Bug



Sign in to add a comment

DevTools: make timeline power-user-friendly again

Project Member Reported by pfeldman@chromium.org, Jan 14 2017

Issue description

See the screenshot.

 
Screenshot from 2017-01-13 17:47:25.png
45.9 KB View Download
Project Member

Comment 2 by bugdroid1@chromium.org, Jan 17 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/064f04c82c2f6cd3e9a77703362f9373034c248a

commit 064f04c82c2f6cd3e9a77703362f9373034c248a
Author: pfeldman <pfeldman@chromium.org>
Date: Tue Jan 17 21:18:58 2017

DevTools: bring back old landing page in timeline/performance.

BUG= 681258 

Review-Url: https://codereview.chromium.org/2637443004
Cr-Commit-Position: refs/heads/master@{#444146}

[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/http/tests/inspector/tracing/timeline-receive-response-event.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/frame-model-instrumentation.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-injected-script-eval.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-js/timeline-script-tag-1.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-misc/timeline-load-event.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/layer-tree.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-paint/paint-profiler-update.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-time.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/LayoutTests/inspector/tracing/timeline-time/timeline-timer.html
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/BUILD.gn
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/extensions/ExtensionServer.js
[delete] https://crrev.com/b71b7d5967c31b07766cf0ad10be89a9e526ed4a/third_party/WebKit/Source/devtools/front_end/timeline/TimelineLandingPage.js
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/timeline/TimelinePanel.js
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/timeline/module.json
[delete] https://crrev.com/b71b7d5967c31b07766cf0ad10be89a9e526ed4a/third_party/WebKit/Source/devtools/front_end/timeline/timelineLandingPage.css
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/timeline/timelinePanel.css
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/ui/Toolbar.js
[modify] https://crrev.com/064f04c82c2f6cd3e9a77703362f9373034c248a/third_party/WebKit/Source/devtools/front_end/ui/toolbar.css

Comment 3 by alph@chromium.org, Jan 18 2017

Screenshot from 2017-01-18 14:10:31.png
37.8 KB View Download
Labels: TE-Verified-57.0.2986.0 TE-Verified-M57
Verified the issue on linux Ubuntu 14.04 using chrome version 57.0.2986.0.

Observed that the fix is working as expected.

Attaching screencast for reference.

Hence, adding the verified labels.

Thanks..
681258.png
287 KB View Download
Tested the issue on Latest Dev# 57.0.2986.0 on Windows, Mac and Linux as is as displayed in Comment# 3.

Able to see Javascript profiler under performance tab. Attaching a screen shot for reference.
Hence adding TE-Verified Labels.
Thank You.

681258.png
122 KB View Download
Status: Fixed (was: Assigned)

Comment 8 by kayce@google.com, Jan 26 2017

Labels: DevTools-Release-Notes-Change

Sign in to add a comment