Bug 148848

Summary: Add a layout test for https://bugs.webkit.org/show_bug.cgi?id=148845
Product: WebKit Reporter: Said Abou-Hallawa <sabouhallawa>
Component: SVGAssignee: Said Abou-Hallawa <sabouhallawa>
Status: RESOLVED INVALID    
Severity: Normal CC: dino, webkit-bug-importer, zimmermann
Priority: P2    
Version: WebKit Nightly Build   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch darin: review+

Description Said Abou-Hallawa 2015-09-04 16:34:59 PDT 
Add the test case attached to https://bugs.webkit.org/show_bug.cgi?id=148845 as LayoutTests/svg/as-image/svg-cached-no-intrinsic-size-draw-canvas.html. Add this test to LayoutTests/TestExpectations and mark it [ Failure ].
Comment 1 Said Abou-Hallawa 2015-09-04 17:41:46 PDT 
Created attachment 260662 [details]
Patch
Comment 2 Darin Adler 2015-09-11 10:27:03 PDT 
Comment on attachment 260662 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=260662&action=review

> LayoutTests/ChangeLog:8
> +        Add the layout test and mark it as failed in TestExpectations.

Why? That’s not usually how we do these things.
Comment 3 Said Abou-Hallawa 2015-09-27 16:56:21 PDT 
(In reply to comment #2)
> Comment on attachment 260662 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=260662&action=review
> 
> > LayoutTests/ChangeLog:8
> > +        Add the layout test and mark it as failed in TestExpectations.
> 
> Why? That’s not usually how we do these things.

My purpose of this patch was to submit the test case even if I do not have the fix ready. For me usually writing the test usually takes more time than fixing the code. 
But I agree I should not do thing differently. So I am closing this bug as WONTFIX. Fixing the other bug  https://bugs.webkit.org/show_bug.cgi?id=148845 should include both the fix and the test case.