Bug 177060

Summary: Make display: contents text children inherit from their parent element, not the parent renderer.
Product: WebKit Reporter: Emilio Cobos Álvarez (:emilio) <emilio>
Component: CSSAssignee: Nobody <webkit-unassigned>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: buildbot, koivisto, mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on:    
Bug Blocks: 157477    
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch
buildbot: commit-queue-
Archive of layout-test-results from ews116 for mac-elcapitan none

Description Emilio Cobos Álvarez (:emilio) 2017-09-17 15:53:00 PDT
See https://github.com/w3c/csswg-drafts/issues/1118.

This is needed to make WebKit's display: contents implementation compatible with Chromium's and Gecko's.
Comment 1 Emilio Cobos Álvarez (:emilio) 2017-09-17 18:49:01 PDT
Created attachment 321061 [details]
Patch
Comment 2 Emilio Cobos Álvarez (:emilio) 2017-09-17 18:57:55 PDT
Created attachment 321062 [details]
Patch
Comment 3 Emilio Cobos Álvarez (:emilio) 2017-09-17 22:55:20 PDT
Created attachment 321079 [details]
Patch
Comment 4 Emilio Cobos Álvarez (:emilio) 2017-09-18 00:24:15 PDT
Created attachment 321080 [details]
Patch
Comment 5 Emilio Cobos Álvarez (:emilio) 2017-09-18 01:19:34 PDT
Gah, I can't repro the build failures on Linux... I guess I'll need to rework the patch to avoid moving RenderObject::style's definition.
Comment 6 Build Bot 2017-09-18 01:57:43 PDT
Comment on attachment 321080 [details]
Patch

Attachment 321080 [details] did not pass mac-debug-ews (mac):
Output: http://webkit-queues.webkit.org/results/4581293

New failing tests:
fast/shadow-dom/activate-over-slotted-content.html
fast/shadow-dom/hover-over-nested-slotted-content.html
fast/shadow-dom/click-text-inside-linked-slot.html
fast/shadow-dom/hover-over-slotted-content.html
Comment 7 Build Bot 2017-09-18 01:57:44 PDT
Created attachment 321083 [details]
Archive of layout-test-results from ews116 for mac-elcapitan

The attached test failures were seen while running run-webkit-tests on the mac-debug-ews.
Bot: ews116  Port: mac-elcapitan  Platform: Mac OS X 10.11.6
Comment 8 Michael Catanzaro 2017-09-22 07:22:08 PDT
Note all the iOS and macOS bots are failing, so it's not likely this will get a review.
Comment 9 Emilio Cobos Álvarez (:emilio) 2017-09-22 07:58:56 PDT
(In reply to Michael Catanzaro from comment #8)
> Note all the iOS and macOS bots are failing, so it's not likely this will
> get a review.

I know, release builds are failing because I moved the definition of RenderObject::style, and now there are a few translation units that can't access it.

I was able to repro the build failure on WebKitGTK release, so I just need to shuffle includes around.
Comment 10 Antti Koivisto 2017-10-16 02:28:43 PDT
Bug 178332 has a patch that takes a different approach and wraps text renderers with display:content into anonymous inline boxes with inherited style.
Comment 11 Antti Koivisto 2017-10-17 02:57:36 PDT

*** This bug has been marked as a duplicate of bug 178332 ***
Comment 12 Radar WebKit Bug Importer 2017-11-13 15:36:11 PST
<rdar://problem/35517680>