<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>28827</bug_id>
          
          <creation_ts>2009-08-28 23:45:42 -0700</creation_ts>
          <short_desc>SVGSVGElement.unsuspendRedraw() shouldn&apos;t throw</short_desc>
          <delta_ts>2009-08-31 04:06:44 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>SVG</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Trivial</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Cameron McCormack (:heycam)">heycam</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>eric</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>143504</commentid>
    <comment_count>0</comment_count>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2009-08-28 23:45:42 -0700</bug_when>
    <thetext>Erratum removes the exception on unsuspendRedraw():

http://www.w3.org/2003/01/REC-SVG11-20030114-errata#remove_exception_on_unsuspendredraw</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143505</commentid>
    <comment_count>1</comment_count>
      <attachid>38773</attachid>
    <who name="Cameron McCormack (:heycam)">heycam</who>
    <bug_when>2009-08-28 23:53:30 -0700</bug_when>
    <thetext>Created attachment 38773
Patch v1

Remove the exception.  (The IDL was using &quot;setter raises&quot; erroneously, too.)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143604</commentid>
    <comment_count>2</comment_count>
      <attachid>38773</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 03:18:39 -0700</bug_when>
    <thetext>Comment on attachment 38773
Patch v1

Please link to the errata in your ChangeLog in the future. :)  ChangeLogs are much quicker lookup than pulling up the bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143612</commentid>
    <comment_count>3</comment_count>
      <attachid>38773</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 03:30:03 -0700</bug_when>
    <thetext>Comment on attachment 38773
Patch v1

Rejecting patch 38773 from commit-queue.  This patch will require manual commit.

[&apos;WebKitTools/Scripts/run-webkit-tests&apos;, &apos;--no-launch-safari&apos;, &apos;--quiet&apos;, &apos;--exit-after-n-failures=1&apos;] failed with exit code 1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143626</commentid>
    <comment_count>4</comment_count>
      <attachid>38773</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 03:48:18 -0700</bug_when>
    <thetext>Comment on attachment 38773
Patch v1

Flakey Test:
Testing 11178 test cases.
media/video-source-add-src.html -&gt; crashed</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143633</commentid>
    <comment_count>5</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 04:01:26 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; (From update of attachment 38773 [details])
&gt; Flakey Test:
&gt; Testing 11178 test cases.
&gt; media/video-source-add-src.html -&gt; crashed

Logged as bug 28845.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143634</commentid>
    <comment_count>6</comment_count>
      <attachid>38773</attachid>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 04:06:41 -0700</bug_when>
    <thetext>Comment on attachment 38773
Patch v1

Clearing flags on attachment: 38773

Committed r47890: &lt;http://trac.webkit.org/changeset/47890&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>143635</commentid>
    <comment_count>7</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-08-31 04:06:44 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>38773</attachid>
            <date>2009-08-28 23:53:30 -0700</date>
            <delta_ts>2009-08-31 04:06:41 -0700</delta_ts>
            <desc>Patch v1</desc>
            <filename>28827.patch</filename>
            <type>text/plain</type>
            <size>2401</size>
            <attacher name="Cameron McCormack (:heycam)">heycam</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA0Nzg4MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTUgQEAKKzIwMDktMDgtMjggIENhbWVyb24gTWNDb3JtYWNrICA8Y2FtQG1jYy5p
ZC5hdT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mjg4MjcKKyAgICAgICAgU1ZH
U1ZHRWxlbWVudC51bnN1c3BlbmRSZWRyYXcoKSBzaG91bGRuJ3QgdGhyb3cKKworICAgICAgICAq
IHN2Zy9TVkdTVkdFbGVtZW50LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OlNWR1NWR0VsZW1lbnQ6
OnVuc3VzcGVuZFJlZHJhdyk6CisgICAgICAgICogc3ZnL1NWR1NWR0VsZW1lbnQuaDoKKyAgICAg
ICAgKiBzdmcvU1ZHU1ZHRWxlbWVudC5pZGw6CisKIDIwMDktMDgtMjggIFNpbW9uIEZyYXNlciAg
PHNpbW9uLmZyYXNlckBhcHBsZS5jb20+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgTWFyayBSb3dl
CkluZGV4OiBXZWJDb3JlL3N2Zy9TVkdTVkdFbGVtZW50LmNwcAo9PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJD
b3JlL3N2Zy9TVkdTVkdFbGVtZW50LmNwcAkocmV2aXNpb24gNDc3NzQpCisrKyBXZWJDb3JlL3N2
Zy9TVkdTVkdFbGVtZW50LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMzEzLDkgKzMxMyw4IEBAIHVu
c2lnbmVkIGxvbmcgU1ZHU1ZHRWxlbWVudDo6c3VzcGVuZFJlZHIKICAgICByZXR1cm4gMDsKIH0K
IAotdm9pZCBTVkdTVkdFbGVtZW50Ojp1bnN1c3BlbmRSZWRyYXcodW5zaWduZWQgbG9uZyAvKiBz
dXNwZW5kX2hhbmRsZV9pZCAqLywgRXhjZXB0aW9uQ29kZSYpCit2b2lkIFNWR1NWR0VsZW1lbnQ6
OnVuc3VzcGVuZFJlZHJhdyh1bnNpZ25lZCBsb25nIC8qIHN1c3BlbmRfaGFuZGxlX2lkICovKQog
ewotICAgIC8vIGlmIHN1c3BlbmRfaGFuZGxlX2lkIGlzIG5vdCBmb3VuZCwgdGhyb3cgZXhjZXB0
aW9uCiAgICAgLy8gRklYTUU6IEltcGxlbWVudCBtZSAoc2VlIGJ1ZyAxMTI3NSkKIH0KIApJbmRl
eDogV2ViQ29yZS9zdmcvU1ZHU1ZHRWxlbWVudC5oCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvc3Zn
L1NWR1NWR0VsZW1lbnQuaAkocmV2aXNpb24gNDc3NzQpCisrKyBXZWJDb3JlL3N2Zy9TVkdTVkdF
bGVtZW50LmgJKHdvcmtpbmcgY29weSkKQEAgLTk1LDcgKzk1LDcgQEAgbmFtZXNwYWNlIFdlYkNv
cmUgewogICAgICAgICB2b2lkIHNldEN1cnJlbnRUaW1lKGZsb2F0IHNlY29uZHMpOwogCiAgICAg
ICAgIHVuc2lnbmVkIGxvbmcgc3VzcGVuZFJlZHJhdyh1bnNpZ25lZCBsb25nIG1heF93YWl0X21p
bGxpc2Vjb25kcyk7Ci0gICAgICAgIHZvaWQgdW5zdXNwZW5kUmVkcmF3KHVuc2lnbmVkIGxvbmcg
c3VzcGVuZF9oYW5kbGVfaWQsIEV4Y2VwdGlvbkNvZGUmKTsKKyAgICAgICAgdm9pZCB1bnN1c3Bl
bmRSZWRyYXcodW5zaWduZWQgbG9uZyBzdXNwZW5kX2hhbmRsZV9pZCk7CiAgICAgICAgIHZvaWQg
dW5zdXNwZW5kUmVkcmF3QWxsKCk7CiAgICAgICAgIHZvaWQgZm9yY2VSZWRyYXcoKTsKIApJbmRl
eDogV2ViQ29yZS9zdmcvU1ZHU1ZHRWxlbWVudC5pZGwKPT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9z
dmcvU1ZHU1ZHRWxlbWVudC5pZGwJKHJldmlzaW9uIDQ3Nzc0KQorKysgV2ViQ29yZS9zdmcvU1ZH
U1ZHRWxlbWVudC5pZGwJKHdvcmtpbmcgY29weSkKQEAgLTU2LDggKzU2LDcgQEAgbW9kdWxlIHN2
ZyB7CiAgICAgICAgIHJlYWRvbmx5IGF0dHJpYnV0ZSBTVkdQb2ludCBjdXJyZW50VHJhbnNsYXRl
OwogCiAgICAgICAgIHVuc2lnbmVkIGxvbmcgc3VzcGVuZFJlZHJhdyhpbiB1bnNpZ25lZCBsb25n
IG1heFdhaXRNaWxsaXNlY29uZHMpOwotICAgICAgICB2b2lkIHVuc3VzcGVuZFJlZHJhdyhpbiB1
bnNpZ25lZCBsb25nIHN1c3BlbmRIYW5kbGVJZCkKLSAgICAgICAgICAgIHNldHRlciByYWlzZXMo
RE9NRXhjZXB0aW9uKTsKKyAgICAgICAgdm9pZCB1bnN1c3BlbmRSZWRyYXcoaW4gdW5zaWduZWQg
bG9uZyBzdXNwZW5kSGFuZGxlSWQpOwogICAgICAgICB2b2lkIHVuc3VzcGVuZFJlZHJhd0FsbCgp
OwogICAgICAgICB2b2lkIGZvcmNlUmVkcmF3KCk7CiAgICAgICAgIHZvaWQgcGF1c2VBbmltYXRp
b25zKCk7Cg==
</data>

          </attachment>
      

    </bug>

</bugzilla>