[Webkit-unassigned] [Bug 158247] New: Video elements are painted twice, in PaintPhaseForeground and PaintPhaseSelfOutline

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=158247

            Bug ID: 158247
           Summary: Video elements are painted twice, in
                    PaintPhaseForeground and PaintPhaseSelfOutline
    Classification: Unclassified
           Product: WebKit
           Version: WebKit Nightly Build
          Hardware: Unspecified
                OS: Unspecified
            Status: NEW
          Severity: Normal
          Priority: P2
         Component: Layout and Rendering
          Assignee: webkit-unassigned at lists.webkit.org
          Reporter: Hironori.Fujii at sony.com
                CC: simon.fraser at apple.com

Video elements are painted twice, in PaintPhaseForeground and PaintPhaseSelfOutline

In r105247 <http://trac.webkit.org/changeset/105247>, a following change has been made.
http://trac.webkit.org/changeset/105247/trunk/Source/WebCore/rendering/RenderReplaced.cpp

> Index: trunk/Source/WebCore/rendering/RenderReplaced.cpp
> ===================================================================
> --- a/trunk/Source/WebCore/rendering/RenderReplaced.cpp
> +++ b/trunk/Source/WebCore/rendering/RenderReplaced.cpp
> @@ -123,5 +123,5 @@
>          paintOutline(paintInfo.context, paintRect);
>      
> -    if (paintInfo.phase != PaintPhaseForeground && paintInfo.phase != PaintPhaseSelection)
> +    if (paintInfo.phase != PaintPhaseForeground && paintInfo.phase != PaintPhaseSelection && !canHaveChildren())
>          return;

Before this change, it was ensured paintReplaced is called only in PaintPhaseForeground and PaintPhaseSelection.
After this change, paintReplaced is called even in PaintPhaseSelfOutline if canHaveChildren().

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20160601/0965d7df/attachment-0001.html>