Bug 140826 - Move height/width implementation for use element from RenderSVGViewportContainer to SVGUseElement
Summary: Move height/width implementation for use element from RenderSVGViewportContai...
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks: 140602
  Show dependency treegraph
 
Reported: 2015-01-23 09:33 PST by Darin Adler
Modified: 2015-01-24 11:56 PST (History)
3 users (show)

See Also:


Attachments
Patch (61.68 KB, patch)
2015-01-23 09:42 PST, Darin Adler
andersca: review+
buildbot: commit-queue-
Details | Formatted Diff | Diff
Archive of layout-test-results from ews105 for mac-mavericks-wk2 (704.06 KB, application/zip)
2015-01-23 10:02 PST, Build Bot
no flags Details
Archive of layout-test-results from ews102 for mac-mavericks (526.07 KB, application/zip)
2015-01-23 10:30 PST, Build Bot
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2015-01-23 09:33:38 PST
Move height/width implementation for use element from RenderSVGViewportContainer to SVGUseElement
Comment 1 Darin Adler 2015-01-23 09:42:00 PST
Created attachment 245232 [details]
Patch
Comment 2 Build Bot 2015-01-23 10:02:31 PST
Comment on attachment 245232 [details]
Patch

Attachment 245232 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5273941429977088

New failing tests:
imported/mozilla/svg/dynamic-use-remove-width.svg
Comment 3 Build Bot 2015-01-23 10:02:35 PST
Created attachment 245235 [details]
Archive of layout-test-results from ews105 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews105  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 4 Build Bot 2015-01-23 10:30:11 PST
Comment on attachment 245232 [details]
Patch

Attachment 245232 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/6384392339456000

New failing tests:
imported/mozilla/svg/dynamic-use-remove-width.svg
Comment 5 Build Bot 2015-01-23 10:30:13 PST
Created attachment 245236 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 6 Anders Carlsson 2015-01-23 10:41:27 PST
Comment on attachment 245232 [details]
Patch

Test failures seem unrelated, but I'm only going to r+, not cq+.
Comment 7 Darin Adler 2015-01-23 17:57:30 PST
I think the test failure is related. I will fix before landing.
Comment 8 Darin Adler 2015-01-24 11:56:10 PST
Committed r179069: <http://trac.webkit.org/changeset/179069>