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

Issue 606834 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit 29 days ago
Closed: Aug 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

-webkit-padding-start value not listed in Computed Style panel

Reported by teo8...@gmail.com, Apr 26 2016

Issue description

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

Steps to reproduce the problem:
See the two screenshot.
The first is from the "Styles" tab, the second is the "Computed" tab, both from a same <ul> node.

What is the expected behavior?
The -webkit-padding-start property should be displayed in the Computed tab. It should have a triangle next to it which, when clicked, would unfold and display the source of the value for the property which is "user agent stylesheet".

What went wrong?
the -webkit-padding-start property is not displayed in the Computed tab at all, making it impossible to figure out where the left padding comes from (it is already painfully difficult to figure out when padddings and margins are dictated by -webkit-margin-before and the like, because no indication is given that this is what determines the computed value of margin or padding, but that's another issue)

Did this work before? N/A 

Chrome version: 50.0.2661.86  Channel: stable
OS Version: 
Flash Version: Shockwave Flash 21.0 r0
 

Comment 1 by teo8...@gmail.com, Apr 26 2016

Forgot the screenshots
styles.png
11.4 KB View Download
computed.png
52.1 KB View Download

Comment 2 by allada@chromium.org, Apr 26 2016

Owner: lushnikov@chromium.org
Status: Available (was: Unconfirmed)

Comment 3 by teo8...@gmail.com, May 29 2016

And the same goes for the -webkit-margin-{before|after|start|end} properties
Status: Assigned (was: Available)
Project Member

Comment 5 by bugdroid1@chromium.org, Aug 3 2016

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

commit 4e200d9789699ba9d52d8650babb12664b2d1b5d
Author: lushnikov <lushnikov@chromium.org>
Date: Wed Aug 03 01:23:55 2016

DevTools: fix CSS.getComputedStyle protocol method

Currently, the CSS.getComputedStyle upcasts CSSComputedStyleDeclaration
to the CSSStyleDeclaration and, unwillingly, was using the length() and
item() methods to iterate the CSSComputedStyleDeclaration.

However, these are private methods of CSSComputedStyleDeclaration which
are not supposed to be used by the clients.

BUG= 606834 
R=dgozman

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

[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/LayoutTests/inspector/elements/styles-2/inject-stylesheet-expected.txt
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/LayoutTests/inspector/elements/styles-2/lazy-computed-style-expected.txt
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/LayoutTests/inspector/elements/styles-2/region-style-crash-expected.txt
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/LayoutTests/inspector/elements/styles-3/styles-computed-trace-expected.txt
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/Source/core/inspector/InspectorCSSAgent.cpp
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/Source/core/inspector/InspectorStyleSheet.cpp
[modify] https://crrev.com/4e200d9789699ba9d52d8650babb12664b2d1b5d/third_party/WebKit/Source/core/inspector/InspectorStyleSheet.h

Status: WontFix (was: Assigned)

Comment 7 by teo8...@gmail.com, Aug 3 2016

Why WontFix??
Status: Fixed (was: WontFix)
Indeed, it's actually fixed

Sign in to add a comment