Bug 58726 - ASSERT failure when svg <use> element changes href
Summary: ASSERT failure when svg <use> element changes href
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: SVG (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-04-16 01:27 PDT by Leo Yang
Modified: 2011-04-16 11:38 PDT (History)
4 users (show)

See Also:

Attachments
Patch (4.90 KB, patch)
2011-04-16 02:05 PDT, Leo Yang 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Leo Yang 2011-04-16 01:27:39 PDT 
ASSERT failure in SVGUseElement.cpp:509 when load the following document.

<svg xmlns="http://www.w3.org/2000/svg" xmlns:xlink="http://www.w3.org/1999/xlink">

<g>
    <use id="use" xlink:href="#rect" />
</g>

<defs>
    <rect id="rect" x="0" y="0" width="100" height="60" />
</defs>

<script>
    if (window.layoutTestController)
        layoutTestController.waitUntilDone();

    function runTest()
    {   
        document.getElementById("use").setAttribute("xlink:href", "");
        if (window.layoutTestController)
            layoutTestController.notifyDone();
    }   

    window.setTimeout("runTest()", 0); 
</script>

</svg>
Comment 1 Leo Yang 2011-04-16 02:05:16 PDT 
Created attachment 89921 [details]
Patch
Comment 2 Nikolas Zimmermann 2011-04-16 02:07:25 PDT 
Comment on attachment 89921 [details]
Patch

Good catch!
Comment 3 WebKit Commit Bot 2011-04-16 02:11:18 PDT 
Comment on attachment 89921 [details]
Patch

Rejecting attachment 89921 [details] from commit-queue.

Failed to run "['./Tools/Scripts/webkit-patch', '--status-host=queues.webkit.org', '--bot-id=cr-jail-7', 'apply-..." exit_code: 1

Last 500 characters of output:
autoinstalled/mechanize/_urllib2_fork.py", line 332, in _call_chain
    result = func(*args)
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_urllib2_fork.py", line 1170, in https_open
    return self.do_open(conn_factory, req)
  File "/mnt/git/webkit-commit-queue/Tools/Scripts/webkitpy/thirdparty/autoinstalled/mechanize/_urllib2_fork.py", line 1118, in do_open
    raise URLError(err)
urllib2.URLError: <urlopen error [Errno 60] Operation timed out>

Full output: http://queues.webkit.org/results/8459397
Comment 4 Leo Yang 2011-04-16 03:29:00 PDT 
Comment on attachment 89921 [details]
Patch

Let the cq try again
Comment 5 Eric Seidel (no email) 2011-04-16 09:14:30 PDT 
Sorry about that. :(
Comment 6 WebKit Commit Bot 2011-04-16 11:35:24 PDT 
The commit-queue encountered the following flaky tests while processing attachment 89921 [details]:

http/tests/xmlhttprequest/basic-auth-nopassword.html bug 53170
The commit-queue is continuing to process your patch.
Comment 7 WebKit Commit Bot 2011-04-16 11:38:04 PDT 
Comment on attachment 89921 [details]
Patch

Clearing flags on attachment: 89921

Committed r84090: <http://trac.webkit.org/changeset/84090>
Comment 8 WebKit Commit Bot 2011-04-16 11:38:08 PDT 
All reviewed patches have been landed.  Closing bug.