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

Issue 788904 link

Starred by 6 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Mac
Pri: 2
Type: Bug

Blocking:
issue 785132



Sign in to add a comment

Computed value for 'content' property is incorrect.

Project Member Reported by shend@chromium.org, Nov 27 2017

Issue description

Chrome Version: 64.0.3269.3 (Official Build) dev (64-bit)

What steps will reproduce the problem?
Open https://jsfiddle.net/w0t1q980/1

What is the expected result?
Should say: "content: 'normal'; pseudo-content: 'none'"

What happens instead?
Says: "content: ''; pseudo-content: ''"

According to https://www.w3.org/TR/CSS2/generate.html#content:
   On elements, always computes to 'normal'. On :before and :after, if 'normal' is specified, computes to 'none'.

Since the default value is 'normal', we should get 'normal' for the element itself and 'none' for the pseudo element.

Firefox says "content: 'none'; pseudo-content: 'none'".
 
Labels: Needs-Bisect
Cc: divya.pa...@techmahindra.com
Labels: -Pri-3 -Needs-Bisect Triaged-ET M-64 Needs-Triage-M64 OS-Linux OS-Mac OS-Windows Pri-2
Status: Untriaged (was: Unconfirmed)
Able to reproduce this issue on reported version 64.0.3269.3 and latest canary 64.0.3278.0 using Mac 10.12.6, Windows 10, Ubuntu 14.04 with steps mentioned in comment#0.
Please find the attached screeshots of 64.0.3269.3, 64.0.3278.0 and 50.0.2661.0 chrome versions.

This issue is seen from M50 series[50.0.2661.0]. Hence considering this issue as Non-regression and marking as Untriaged.

Thanks!
788904_64.0.3278.0.png
191 KB View Download
788904_64.0.3269.3.png
185 KB View Download
788904_50.0.2661.26.png
196 KB View Download

Comment 3 by nainar@chromium.org, Nov 29 2017

Labels: Hotlist-GoodFirstBug
Status: Available (was: Untriaged)
Labels: -Update-Quarterly

Comment 5 by shend@chromium.org, Dec 27 2017

Blocking: 785132
Owner: ramy...@samsung.com
Status: Assigned (was: Available)
Would like to work on this bug.
@Darren Shen/ NainaR 
Added a patch at 
https://chromium-review.googlesource.com/c/chromium/src/+/942126

Can you please help me to know why StylePropertyMap returning null for correct initial value of content?
Project Member

Comment 8 by bugdroid1@chromium.org, Mar 7 2018

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

commit 11fb2ae26b6e84211adf8270eb4f0cf42b945fe8
Author: Ramya Vadlamudi <ramya.v@samsung.com>
Date: Wed Mar 07 10:22:30 2018

Computed value for 'content' property is incorrect.

Computed value on elements is by default 'normal'
and on :before and :after computes to 'none'
Spec: https://www.w3.org/TR/CSS2/generate.html#content

BUG= 788904 

Change-Id: Iefd07961445b057d9f893e057f35b082c214454d
Reviewed-on: https://chromium-review.googlesource.com/942126
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Commit-Queue: Ramya Vadlamudi <ramya.v@samsung.com>
Cr-Commit-Position: refs/heads/master@{#541400}
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/fast/css/content-property-quote-types.html
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/fast/css/getComputedStyle/computed-style-listing-expected.txt
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/fast/css/getComputedStyle/computed-style-without-renderer-listing-expected.txt
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/fast/dom/hover-after-dom-delete-expected.txt
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/fast/dom/hover-after-dom-delete.html
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/LayoutTests/svg/css/getComputedStyle-listing-expected.txt
[modify] https://crrev.com/11fb2ae26b6e84211adf8270eb4f0cf42b945fe8/third_party/WebKit/Source/core/css/properties/ComputedStyleUtils.cpp

Comment 9 by ramy...@samsung.com, Mar 12 2018

Status: Fixed (was: Assigned)

Comment 10 Deleted

Comment 11 by phi...@heede.dk, Jun 4 2018

This issue reintroduced a breaking change first tracked in  BUG 609848 

Sign in to add a comment