New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 598919 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit 29 days ago
Closed: Apr 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 1
Type: Bug-Regression



Sign in to add a comment

[REGRESSION] DevTools: cannot accept property edits with mouse click

Project Member Reported by lushnikov@chromium.org, Mar 30 2016

Issue description

Chrome Version       : 50.0.2661.49

What steps will reproduce the problem?
1. Navigate to google.com
2. Open DevTools, select "body" in elements panel
3. Start editing name of the "background: #fff" property
4. Put "color" instead 
5. Mouse click outside of editable field

Expected: the new value gets committed
Actual: the old value gets restored


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

UserAgentString: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.110 Safari/537.36



 
Status: Started (was: Assigned)
started: https://codereview.chromium.org/1846433002/
Project Member

Comment 2 by bugdroid1@chromium.org, Mar 30 2016

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

commit 5d556d4423750cbbcc7d8ac7cd28c1e3b5f6c9fd
Author: lushnikov <lushnikov@chromium.org>
Date: Wed Mar 30 01:21:45 2016

DevTools: [SSP] accept editing on mouse click

This behavior was regressed back in http://crrev.com/1649533003.

BUG= 598919 
R=pfeldman, dgozman

Review URL: https://codereview.chromium.org/1846433002

Cr-Commit-Position: refs/heads/master@{#383889}

[modify] https://crrev.com/5d556d4423750cbbcc7d8ac7cd28c1e3b5f6c9fd/third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

Project Member

Comment 3 by bugdroid1@chromium.org, Mar 30 2016

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

commit 5d556d4423750cbbcc7d8ac7cd28c1e3b5f6c9fd
Author: lushnikov <lushnikov@chromium.org>
Date: Wed Mar 30 01:21:45 2016

DevTools: [SSP] accept editing on mouse click

This behavior was regressed back in http://crrev.com/1649533003.

BUG= 598919 
R=pfeldman, dgozman

Review URL: https://codereview.chromium.org/1846433002

Cr-Commit-Position: refs/heads/master@{#383889}

[modify] https://crrev.com/5d556d4423750cbbcc7d8ac7cd28c1e3b5f6c9fd/third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

Components: Platform>DevTools>Editing
Labels: Merge-Request-50
Verified working as expected on Mac Chrome Canary

Comment 5 by tin...@google.com, Apr 1 2016

Labels: -Merge-Request-50 Merge-Approved-50 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M50 (branch: 2661)
Please merge your change to M50 branch 2661 by 5:00 PM on Monday, 04/04/16 so we can take it for next week beta. Thank you.
Project Member

Comment 7 by bugdroid1@chromium.org, Apr 1 2016

Labels: -merge-approved-50 merge-merged-2661
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/883dbb3191fbcd5c1c8f29ecc37b5e6f63f21fb7

commit 883dbb3191fbcd5c1c8f29ecc37b5e6f63f21fb7
Author: Andrey Lushnikov <lushnikov@chromium.org>
Date: Fri Apr 01 22:28:19 2016

DevTools: [SSP] accept editing on mouse click

This behavior was regressed back in http://crrev.com/1649533003.

BUG= 598919 
R=pfeldman, dgozman

Review URL: https://codereview.chromium.org/1846433002

Cr-Commit-Position: refs/heads/master@{#383889}
(cherry picked from commit 5d556d4423750cbbcc7d8ac7cd28c1e3b5f6c9fd)

Review URL: https://codereview.chromium.org/1847303005 .

Cr-Commit-Position: refs/branch-heads/2661@{#463}
Cr-Branched-From: ef6f6ae5e4c96622286b563658d5cd62a6cf1197-refs/heads/master@{#378081}

[modify] https://crrev.com/883dbb3191fbcd5c1c8f29ecc37b5e6f63f21fb7/third_party/WebKit/Source/devtools/front_end/elements/StylesSidebarPane.js

Status: Fixed (was: Started)
Cc: ashej...@chromium.org
Labels: TE-Verified-50.0.2661.66 TE-Verified-M50
Retested the above on Ubuntu 14.04 with chrome version '50.0.2661.66'(378081)
and working as intended, the new values gets committed. Hence marking the same as TE-Verified-50.0.2661.66.
Components: Platform>DevTools>Authoring
Components: Platform>DevTools

Sign in to add a comment