RESOLVED FIXED 53179
[chromium] Implement reflections for composited layers
https://bugs.webkit.org/show_bug.cgi?id=53179
Summary [chromium] Implement reflections for composited layers
Vangelis Kokkevis
Reported 2011-01-26 10:39:48 PST
Add support for the -webkit-box-reflect property
Attachments
Patch (16.51 KB, patch)
2011-02-01 14:19 PST, Vangelis Kokkevis
kbr: review+
vangelis: commit-queue-
Vangelis Kokkevis
Comment 1 2011-02-01 14:19:18 PST
Kenneth Russell
Comment 2 2011-02-01 14:48:04 PST
Comment on attachment 80825 [details] Patch Looks good to me, in particular the drawableContentRect() refactoring. Do we need test expectations updates for the chromium-gpu port?
Vangelis Kokkevis
Comment 3 2011-02-02 15:08:52 PST
Vangelis Kokkevis
Comment 4 2011-02-02 15:10:06 PST
(In reply to comment #2) > (From update of attachment 80825 [details]) > Looks good to me, in particular the drawableContentRect() refactoring. Do we need test expectations updates for the chromium-gpu port? New test expectations will be harvested from build bots and updated once the patch lands. At the moment all compositing tests are marked as PASS FAIL so they won't turn the chromium-gpu bot red.
Jonathan Backer
Comment 5 2011-02-03 06:38:08 PST
Regression on http://webkit.org/blog/386/3d-transforms/ ASSERTION FAILED: m_ptr (third_party/WebKit/Source/JavaScriptCore/wtf/OwnPtr.h:66 typename WTF::RemovePointer<T>::Type* WTF::OwnPtr<T>::operator->() const [with T = WebCore::RenderSurfaceChromium]) This is on the Linux build. The regression first occurred on a WebKit roll (chromium r73597). From this, I narrowed it down to this particular WebKit commit.
Jonathan Backer
Comment 6 2011-02-03 06:55:58 PST
Based on advice from #webkit, I've filed a separate bug for the regression (https://bugs.webkit.org/show_bug.cgi?id=53679).
Note You need to log in before you can comment on or make changes to this bug.