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

Issue 788625 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Jan 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 787717



Sign in to add a comment

<marquee> is using <content> elements.

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

Issue description

fast/dom/shadow/make-marquee-bold-by-exec-command-crash.html
is crashing with IncrementalShadowDOM flag
because <marquee> is using <content> elements.

See the meta  bug 787717  for details.

 

Comment 1 by hayato@chromium.org, Nov 27 2017

Blocking: 787717

Comment 2 by kochi@chromium.org, Dec 27 2017

Status: Started (was: Assigned)
Project Member

Comment 3 by bugdroid1@chromium.org, Jan 14 2018

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

commit 424117308e9bccac62578de511eeaebfeded3199
Author: Takayoshi Kochi <kochi@chromium.org>
Date: Sun Jan 14 01:22:10 2018

Convert V0 UA shadow in <marquee> to use V1

<marquee> was implemented in V0 UA shadowroot with <content>
element, obviously for wrapping the moving parts inside its
element, and used style/animation to move.

This CL replaces <content> with <slot> to use V1 UA shadowroot.

Bug:  787717 ,  788625 
Change-Id: Ic2bbd8e1536a4736fe0f1e761eecbf7841e8a596
Reviewed-on: https://chromium-review.googlesource.com/798871
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#529178}
[modify] https://crrev.com/424117308e9bccac62578de511eeaebfeded3199/third_party/WebKit/LayoutTests/TestExpectations
[modify] https://crrev.com/424117308e9bccac62578de511eeaebfeded3199/third_party/WebKit/Source/core/html/HTMLMarqueeElement.cpp

Comment 4 by kochi@chromium.org, Jan 15 2018

Status: Fixed (was: Started)

Sign in to add a comment