New issue
Advanced search Search tips

Issue 849057 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 1
Type: Bug-Regression



Sign in to add a comment

Performance memory chart is blank.

Reported by hall...@gmail.com, Jun 3 2018

Issue description

Chrome Version       : 69.0.3448.0 (Developer Build) (64-bit)

What steps will reproduce the problem?
(1) Open the dev tools, performance tab and check the memory box.
(2) Record a session.
(3) Note the output memory profile chart is blank.

What is the expected result?
A useful chart that has the full Y axis range.


What happens instead?
It looks like the Y axis is 0 - 1 only hiding all the data.


Please provide any additional information below. Attach a screenshot if
possible.

 

Comment 1 by hall...@gmail.com, Jun 3 2018

Screen Shot 2018-06-03 at 11.53.40 AM.png
161 KB View Download
Labels: Needs-Triage-M69
Cc: phanindra.mandapaka@chromium.org
Components: UI>Browser
Labels: Needs-Feedback Triaged-ET OS-Linux OS-Mac OS-Windows
Unable to reproduce the issue reported chrome version on 69.0.3448.0, latest stable 67.0.3396.62 and latest chrome 69.0.3450.0 using Windows 10 and ubuntu 17.10. Attaching Screenshot for reference.

Steps 
--------
1. Launched Chrome.
2.Opened dev tools and navigated to performance tab
3.Recorded session 
We are able to see the memory profile chart with result.

@Reporter: As we are unable to reproduce the issue from our end. Could you please review the attached screen-cast and confirm if anything being missed here.

Thanks! 
849057.PNG
138 KB View Download

Comment 5 Deleted

Comment 6 by hall...@gmail.com, Jun 5 2018

The screenshot doesn't show the memory chart. It's the memory chart that is affected by this bug. Please follow the reproduction instructions and don't forget to *check the memory box*.

Cheers.
Project Member

Comment 7 by sheriffbot@chromium.org, Jun 5 2018

Labels: -Needs-Feedback
Thank you for providing more feedback. Adding the requester to the cc list.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 8 by hall...@gmail.com, Jun 5 2018

Screen Shot 2018-06-06 at 9.55.03 AM.png
68.7 KB View Download

Comment 9 by hall...@gmail.com, Jun 5 2018

Also this bug was found on macOS so if it's not present in the windows build then you can try and reproduce in the macOS build. Thanks.
Cc: viswa.karala@chromium.org
Labels: -Type-Bug -Pri-3 hasbisect-per-revision ReleaseBlock-Stable RegressedIn-67 FoundIn-67 M-67 Target-67 Target-69 Target-68 FoundIn-68 FoundIn-69 Pri-1 Type-Bug-Regression
Owner: alph@chromium.org
Status: Assigned (was: Unconfirmed)
Able to reproduce the issue on reported chrome 69.0.3448.0  & latest canary 69.0.3453.0 also on latest stable 67.0.3396.79 using Windows10,Mac OS10.13.3 ,Ubuntu14.04.Below is the bisect information for same.

Bisect Info:
================
Good build: 67.0.3394.0
Bad build: 67.0.3395.0

You are probably looking for a change made after 549820 (known good), but no later than 549821 (first known bad).

CHANGELOG URL:
https://chromium.googlesource.com/chromium/src/+log/1b2b5547d17eeb4f9013b00a1ea611854223dc1e..94942045f8a0d26994609c6c578170a6ebf5131b

Reviewed-on: https://chromium-review.googlesource.com/1003833

Suspecting same from change log.

Alexei Filippov@: Please confirm the issue and help in re-assigning if it is not related to your change.Adding RB-stable blocker as it is recent regression.Please feel free to remove if not the case.

Thanks!
Cc: dgozman@chromium.org pfeldman@chromium.org
Project Member

Comment 12 by bugdroid1@chromium.org, Jun 9 2018

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

commit d28dccea0d7bba1da1d54b6165e207ae4b5e802f
Author: Alexei Filippov <alph@chromium.org>
Date: Sat Jun 09 01:09:28 2018

DevTools: Fix timeline memory graph is empty right after recording.

BUG= 849057 

Change-Id: Id63654558ab18ae960dcdcc989939f521ea807c2
Reviewed-on: https://chromium-review.googlesource.com/1093423
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565809}
[modify] https://crrev.com/d28dccea0d7bba1da1d54b6165e207ae4b5e802f/third_party/blink/renderer/devtools/front_end/timeline/CountersGraph.js
[modify] https://crrev.com/d28dccea0d7bba1da1d54b6165e207ae4b5e802f/third_party/blink/renderer/devtools/front_end/timeline/TimelineDetailsView.js

Comment 13 by alph@chromium.org, Jun 9 2018

Labels: -M-67 Merge-Request-68

Comment 14 by alph@chromium.org, Jun 9 2018

Status: Fixed (was: Assigned)
Project Member

Comment 15 by sheriffbot@chromium.org, Jun 10 2018

Labels: -Merge-Request-68 Hotlist-Merge-Review Merge-Review-68
This bug requires manual review: M68 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Target-67 -Merge-Review-68 Merge-Approved-68
Approving merge for M68. Branch:3440
Project Member

Comment 17 by bugdroid1@chromium.org, Jun 11 2018

Labels: -merge-approved-68 merge-merged-3440
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/aa624a92064352adb5ccdbbcbcd337bdc56ed48b

commit aa624a92064352adb5ccdbbcbcd337bdc56ed48b
Author: Alexei Filippov <alph@chromium.org>
Date: Mon Jun 11 19:17:18 2018

DevTools: Fix timeline memory graph is empty right after recording.

BUG= 849057 
TBR=alph@chromium.org

(cherry picked from commit d28dccea0d7bba1da1d54b6165e207ae4b5e802f)

Change-Id: Id63654558ab18ae960dcdcc989939f521ea807c2
Reviewed-on: https://chromium-review.googlesource.com/1093423
Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
Commit-Queue: Alexei Filippov <alph@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#565809}
Reviewed-on: https://chromium-review.googlesource.com/1096022
Reviewed-by: Alexei Filippov <alph@chromium.org>
Cr-Commit-Position: refs/branch-heads/3440@{#280}
Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733}
[modify] https://crrev.com/aa624a92064352adb5ccdbbcbcd337bdc56ed48b/third_party/blink/renderer/devtools/front_end/timeline/CountersGraph.js
[modify] https://crrev.com/aa624a92064352adb5ccdbbcbcd337bdc56ed48b/third_party/blink/renderer/devtools/front_end/timeline/TimelineDetailsView.js

Sign in to add a comment