RESOLVED FIXED 175936
Weird getComputedStyle behavior for pseudo-elements on layout-dependent properties. 
https://bugs.webkit.org/show_bug.cgi?id=175936
Summary Weird getComputedStyle behavior for pseudo-elements on layout-dependent prope...
Emilio Cobos Álvarez (:emilio)
Reported 2017-08-24 08:07:16 PDT
Created attachment 318985 [details] testcase See the test-case. The width of the actual element is reported. I'd expect "auto" to be alerted, given it's the default value for the "width" property.
Attachments
testcase (102 bytes, text/html)
2017-08-24 08:07 PDT, Emilio Cobos Álvarez (:emilio) 		no flags
Details
Patch (7.14 KB, patch)
2017-08-24 11:01 PDT, Emilio Cobos Álvarez (:emilio) 		no flags
DetailsFormatted DiffDiff
Patch (8.68 KB, patch)
2017-09-01 14:15 PDT, Emilio Cobos Álvarez (:emilio) 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Emilio Cobos Álvarez (:emilio)
Comment 1 2017-08-24 11:01:28 PDT
Created attachment 318997 [details] Patch
Emilio Cobos Álvarez (:emilio)
Comment 2 2017-08-24 11:02:36 PDT
Antti, could you review this? Also, I don't know what the deal is with wpt, do the tests get auto-exported? Or do I need to submit a PR manually? Thanks!
Antti Koivisto
Comment 3 2017-08-25 05:17:05 PDT
Comment on attachment 318997 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=318997&action=review > Source/WebCore/css/CSSComputedStyleDeclaration.cpp:2386 > +RenderObject* ComputedStyleExtractor::styledRenderObject() const styledRenderer() would be a better name. Also this can return RenderElement*
Emilio Cobos Álvarez
Comment 4 2017-08-27 12:59:48 PDT
Per https://www.mail-archive.com/webkit-dev@lists.webkit.org/msg27587.html I submitted tests to https://github.com/w3c/web-platform-tests/pull/7020. Will upload a patch with the review comments addressed when that is merged. Thanks for the review Antti!
Emilio Cobos Álvarez (:emilio)
Comment 5 2017-09-01 14:15:38 PDT
Created attachment 319644 [details] Patch
WebKit Commit Bot
Comment 6 2017-09-01 15:16:05 PDT
Comment on attachment 319644 [details] Patch Clearing flags on attachment: 319644 Committed r221501: <http://trac.webkit.org/changeset/221501>
WebKit Commit Bot
Comment 7 2017-09-01 15:16:06 PDT
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 8 2017-09-27 12:55:07 PDT
<rdar://problem/34694264>
Note You need to log in before you can comment on or make changes to this bug.