search

catapult-project / catapult

Deprecated Catapult GitHub. Please instead use http://crbug.com "Speed>Benchmarks" component for bugs and https://chromium.googlesource.com/catapult for downloading and editing source code..
https://chromium.googlesource.com/catapult
BSD 3-Clause "New" or "Revised" License
1.93k stars 563 forks source link

Clicking on scroll bar in timing mode gets handled by the tool not the scrollbar #554

Closed catapult-bot closed 9 years ago

catapult-bot commented 9 years ago

Issue by natduca Monday Sep 22, 2014 at 20:25 GMT Originally opened as https://github.com/google/trace-viewer/issues/554

From vku...@etouch.net on April 18, 2014 05:54:43

Chrome Version: 36.0.1946.0 (Official Build 264462) m OS: All, Win 7(Aero enabled) What steps will reproduce the problem? 1.Launch chrome, navigate to "chrome://tracing". 2.Click on 'Record' button, again click on 'Record' then click on'Stop' recording button of the overlay. 3.Now, click on the button of 'Thread Times' from right side of window and select 'Wall time' option from thread times section. 4.Click on 'Timing mode' button from 'drag handler', try to click on the vertical scroll bar & observe.

Actual: On clicking scroll bar of 'timing mode' ,window shift to right side and user is unable to scroll down.

Expected: On clicking scroll bar of 'timing mode' ,window should not shift to right side and user should be able to scroll down.

This is a regression issue broken in 'M36' and below is the narrow bisect info. http://build.chromium.org/f/chromium/perf/dashboard/ui/changelog.html?url=%2Ftrunk%2Fsrc&range=262544%3A262565&mode=html Suspecting: 262548 ?

Could you please reassign to right developer if your change is not cause for this issue.

Attachment: ActualResult.mp4

Original issue: http://code.google.com/p/trace-viewer/issues/detail?id=549

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From smok...@chromium.org on April 18, 2014 07:24:16

Labels: ReleaseBlock-Stable

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From nd...@chromium.org on April 18, 2014 11:44:10

Thanks for the bug report. But, please note this comment with regard to about:tracing reports: https://code.google.com/p/chromium/issues/detail?id=362458#c3 I dont know that I have bandwidth to fix this right now but I'll leave it around in case someone gets to it.

Summary: Clicking on scroll bar in timing mode gets handled by the tool not the scrollbar (was: Regression: On clicking scroll bar of 'timing mode' ,window shift to right side and user is unable to scroll down.)
Status: Available
Owner: ---
Labels: -Type-Bug-Regression -Pri-1 -M-36 -ReleaseBlock-Stable Type-Bug Pri-2

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From nd...@chromium.org on April 18, 2014 11:44:48

Labels: -Pri-2 -OS-All -Cr-Internals Pri-3 Area-Views

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From kpha...@chromium.org on September 02, 2014 04:44:31

I'll handle this one.

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From kpha...@chromium.org on September 02, 2014 04:45:02

Cc: kpha...@chromium.org

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From kpha...@chromium.org on September 11, 2014 00:46:49

Added a patch for review @ https://codereview.appspot.com/138330043/

catapult-bot commented 9 years ago

Comment by natduca Monday Sep 22, 2014 at 20:25 GMT

From r.kasibh...@samsung.com on September 20, 2014 04:11:39

The patch fixing the issue has been merged into the github repo. The SHA id of the merge fixing the issue 5b0446a2f335aba1629fb629f02fb6b5253ef516

Status: Fixed