<?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>18512</bug_id>
          
          <creation_ts>2008-04-15 08:33:35 -0700</creation_ts>
          <short_desc>getPresentationAttribute returns a shared object</short_desc>
          <delta_ts>2008-06-08 12:23:16 -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>Windows XP</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>Major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Jonathan Haas">myrdred</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>mitz</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>77690</commentid>
    <comment_count>0</comment_count>
    <who name="Jonathan Haas">myrdred</who>
    <bug_when>2008-04-15 08:33:35 -0700</bug_when>
    <thetext>Suppose you&apos;ve got the following SVG:

  &lt;rect id=&quot;firstrect&quot;  x=&quot;100&quot; y=&quot;100&quot; width=&quot;100&quot; height=&quot;100&quot; fill=&quot;red&quot; /&gt;
  &lt;rect id=&quot;secondrect&quot; x=&quot;100&quot; y=&quot;250&quot; width=&quot;100&quot; height=&quot;100&quot; fill=&quot;red&quot; /&gt;

The &quot;fill=&apos;red&apos;&quot; in both objects will be mapped to the same attribute. Calling getPresentationAttribute(&apos;fill&apos;) on either rectangle will return this shared attribute reference. Modifying the returned presentation attribute will modify both rectangles. Even more fun, the change is permanent until WebKit is restarted. So executing this code:

    function onLoad()
    {
        var rect = document.getElementById(&quot;firstrect&quot;);
        var fill = rect.getPresentationAttribute(&quot;fill&quot;);
        fill.setRGBColor(&quot;green&quot;);
    }

...means that every SVG object that&apos;s loaded from then on with the attribute fill=&quot;red&quot; will be drawn as green.

Verified on Windows XP with the latest service packs installed, but I&apos;ve no reason to think the bug is platform-specific.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77691</commentid>
    <comment_count>1</comment_count>
      <attachid>20556</attachid>
    <who name="Jonathan Haas">myrdred</who>
    <bug_when>2008-04-15 08:34:18 -0700</bug_when>
    <thetext>Created attachment 20556
Repro case</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77725</commentid>
    <comment_count>2</comment_count>
    <who name="Jonathan Haas">myrdred</who>
    <bug_when>2008-04-15 14:03:30 -0700</bug_when>
    <thetext>Confirmed that this happens in the Mac OS X build. Results in other browsers:

o Firefox 3.0b5 doesn&apos;t appear to support getPresentationAttribute(), or if it does, doesn&apos;t support changing the returned value. Both squares appear red.
o IE 7 doesn&apos;t support SVG, period.
o Opera 9.27 appears to work. The top square turns green, the bottom square remains red</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77825</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-04-16 11:22:24 -0700</bug_when>
    <thetext>Yeah, I guess we can&apos;t just wrap the style object.  Need to make modifications to the inline style decl:
spec: http://www.w3.org/TR/SVG/types.html#InterfaceSVGStylable
original bug: https://bugs.webkit.org/show_bug.cgi?id=13976
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77826</commentid>
    <comment_count>4</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2008-04-16 11:24:01 -0700</bug_when>
    <thetext>I don&apos;t think that &quot;every attribute which is loaded from then on&quot; will have red.  Rather any sibling element which happens to be sharing the RenderStyle with this element will suddenly turn red.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>77827</commentid>
    <comment_count>5</comment_count>
    <who name="Jonathan Haas">myrdred</who>
    <bug_when>2008-04-16 11:32:38 -0700</bug_when>
    <thetext>Try it. Load the repro case in Safari and then without restarting, visit an SVG document with fill=&quot;red&quot;. LayoutTests/svg/custom/viewport-clip.svg will do.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78247</commentid>
    <comment_count>6</comment_count>
    <who name="Dave Hyatt">hyatt</who>
    <bug_when>2008-04-19 02:25:41 -0700</bug_when>
    <thetext>Style sharing for HTML actually looks at the mapped attributes, and if they don&apos;t match it avoids sharing.  I don&apos;t know if this is helpful or not.  It sounds similar.

</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78297</commentid>
    <comment_count>7</comment_count>
      <attachid>20694</attachid>
    <who name="Rob Buis">rwlbuis</who>
    <bug_when>2008-04-20 00:54:19 -0700</bug_when>
    <thetext>Created attachment 20694
First attempt

This small patch fixes the problem. However I wonder if there is a nicer way to do it. I guess there is nothing wrong with sharing the style declaration for this attribute though, since in the testcase the values are equal and we do not know up front that we are going to use getPresentationAttribute, which is used very rarely anyway. So I really can&apos;t see another way, though maybe the unsharing can be done with less statements or some helper method in StyledElement. I&apos;ll do the testcase later, I think the original testcase in the bug is a good base for an automated test.
Cheers,

Rob.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>78300</commentid>
    <comment_count>8</comment_count>
      <attachid>20696</attachid>
    <who name="Rob Buis">rwlbuis</who>
    <bug_when>2008-04-20 07:04:02 -0700</bug_when>
    <thetext>Created attachment 20696
Now with testcase

Just added a testcase, code is the same.
Cheers,

Rob.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>79184</commentid>
    <comment_count>9</comment_count>
      <attachid>20696</attachid>
    <who name="">mitz</who>
    <bug_when>2008-04-28 18:00:40 -0700</bug_when>
    <thetext>Comment on attachment 20696
Now with testcase

Is it possible that the style will not be shared at the time you call getPresentationAttribute() but a later addition to the DOM will make it shared?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>81414</commentid>
    <comment_count>10</comment_count>
      <attachid>20696</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2008-05-24 23:22:03 -0700</bug_when>
    <thetext>Comment on attachment 20696
Now with testcase

+    if (cssSVGAttr-&gt;style()-&gt;refCount() &gt; 1) {

The hasOneRef() function was made for uses like this one. I think it&apos;d be better to use that.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>82632</commentid>
    <comment_count>11</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2008-06-08 12:23:16 -0700</bug_when>
    <thetext>Committed revision 34448.
</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>20556</attachid>
            <date>2008-04-15 08:34:18 -0700</date>
            <delta_ts>2008-04-15 08:34:18 -0700</delta_ts>
            <desc>Repro case</desc>
            <filename>getPresentationAttribute-bug-repro-case.svg</filename>
            <type>image/svg+xml</type>
            <size>618</size>
            <attacher name="Jonathan Haas">myrdred</attacher>
            
              <data encoding="base64">PD94bWwgdmVyc2lvbj0iMS4wIiBlbmNvZGluZz0iSVNPLTg4NTktMSIgc3RhbmRhbG9uZT0ibm8i
Pz4KPCFET0NUWVBFIHN2ZyBQVUJMSUMgIi0vL1czQy8vRFREIFNWRyAyMDAxMDkwNC8vRU4iICJo
dHRwOi8vd3d3LnczLm9yZy9UUi8yMDAxL1JFQy1TVkctMjAwMTA5MDQvRFREL3N2ZzEwLmR0ZCI+
CjxzdmcgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiBvbmxvYWQ9Im9uTG9hZCgp
Ij4KCjxzY3JpcHQ+CjwhW0NEQVRBWwogICAgZnVuY3Rpb24gb25Mb2FkKCkKICAgIHsKICAgICAg
ICB2YXIgcmVjdCA9IGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJmaXJzdHJlY3QiKTsKICAgICAg
ICB2YXIgZmlsbCA9IHJlY3QuZ2V0UHJlc2VudGF0aW9uQXR0cmlidXRlKCJmaWxsIik7CiAgICAg
ICAgZmlsbC5zZXRSR0JDb2xvcigiZ3JlZW4iKTsKICAgIH0KICAgIF1dPgo8L3NjcmlwdD4KICA8
cmVjdCBpZD0iZmlyc3RyZWN0IiAgeD0iMTAwIiB5PSIxMDAiIHdpZHRoPSIxMDAiIGhlaWdodD0i
MTAwIiBmaWxsPSJyZWQiIC8+CiAgPHJlY3QgaWQ9InNlY29uZHJlY3QiIHg9IjEwMCIgeT0iMjUw
IiB3aWR0aD0iMTAwIiBoZWlnaHQ9IjEwMCIgZmlsbD0icmVkIiAvPgo8L3N2Zz4K
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>20694</attachid>
            <date>2008-04-20 00:54:19 -0700</date>
            <delta_ts>2008-04-20 07:04:02 -0700</delta_ts>
            <desc>First attempt</desc>
            <filename>18512.diff</filename>
            <type>text/plain</type>
            <size>1838</size>
            <attacher name="Rob Buis">rwlbuis</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAzMjI4MCkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTggQEAKKzIwMDgtMDQtMjAgIFJvYiBCdWlzICA8YnVpc0BrZGUub3JnPgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFdBUk5JTkc6IE5P
IFRFU1QgQ0FTRVMgQURERUQgT1IgQ0hBTkdFRAorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODUxMgorICAgICAgICBnZXRQcmVzZW50YXRpb25BdHRy
aWJ1dGUgcmV0dXJucyBhIHNoYXJlZCBvYmplY3QKKworICAgICAgICBVbnNoYXJlIHRoZSBtYXBw
ZWQgc3R5bGUgZGVjbGFyYXRpb24gaWYgaXQgaXMgc2hhcmVkCisgICAgICAgIGJlZm9yZSByZXR1
cm5pbmcgdGhlIHByZXNlbnRhdGlvbiBhdHRyaWJ1dGUuCisKKyAgICAgICAgKiBzdmcvU1ZHU3R5
bGVkRWxlbWVudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTVkdTdHlsZWRFbGVtZW50OjpnZXRQ
cmVzZW50YXRpb25BdHRyaWJ1dGUpOgorCiAyMDA4LTA0LTIwICBTaW1vbiBIYXVzbWFubiAgPGhh
dXNtYW5uQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQWxwIFRva2VyLgpJbmRl
eDogV2ViQ29yZS9zdmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29y
ZS9zdmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAJKHJldmlzaW9uIDMyMjgwKQorKysgV2ViQ29yZS9z
dmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEsNiArMSw2IEBACiAv
KgogICAgIENvcHlyaWdodCAoQykgMjAwNCwgMjAwNSwgMjAwNiwgMjAwNywgMjAwOCBOaWtvbGFz
IFppbW1lcm1hbm4gPHppbW1lcm1hbm5Aa2RlLm9yZz4KLSAgICAgICAgICAgICAgICAgIDIwMDQs
IDIwMDUsIDIwMDcgUm9iIEJ1aXMgPGJ1aXNAa2RlLm9yZz4KKyAgICAgICAgICAgICAgICAgIDIw
MDQsIDIwMDUsIDIwMDcsIDIwMDggUm9iIEJ1aXMgPGJ1aXNAa2RlLm9yZz4KIAogICAgIFRoaXMg
bGlicmFyeSBpcyBmcmVlIHNvZnR3YXJlOyB5b3UgY2FuIHJlZGlzdHJpYnV0ZSBpdCBhbmQvb3IK
ICAgICBtb2RpZnkgaXQgdW5kZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgTGlicmFyeSBHZW5lcmFs
IFB1YmxpYwpAQCAtMjgxLDYgKzI4MSwxMSBAQCBQYXNzUmVmUHRyPENTU1ZhbHVlPiBTVkdTdHls
ZWRFbGVtZW50OjpnCiAgICAgTWFwcGVkQXR0cmlidXRlKiBjc3NTVkdBdHRyID0gbWFwcGVkQXR0
cmlidXRlcygpLT5nZXRBdHRyaWJ1dGVJdGVtKG5hbWUpOwogICAgIGlmICghY3NzU1ZHQXR0ciB8
fCAhY3NzU1ZHQXR0ci0+c3R5bGUoKSkKICAgICAgICAgcmV0dXJuIDA7CisgICAgaWYgKGNzc1NW
R0F0dHItPnN0eWxlKCktPnJlZkNvdW50KCkgPiAxKSB7CisgICAgICAgIGNzc1NWR0F0dHItPnNl
dERlY2woMCk7CisgICAgICAgIGludCBwcm9wSWQgPSBTVkdTdHlsZWRFbGVtZW50Ojpjc3NQcm9w
ZXJ0eUlkRm9yU1ZHQXR0cmlidXRlTmFtZShjc3NTVkdBdHRyLT5uYW1lKCkpOworICAgICAgICBh
ZGRDU1NQcm9wZXJ0eShjc3NTVkdBdHRyLCBwcm9wSWQsIGNzc1NWR0F0dHItPnZhbHVlKCkpOwor
ICAgIH0KICAgICByZXR1cm4gY3NzU1ZHQXR0ci0+c3R5bGUoKS0+Z2V0UHJvcGVydHlDU1NWYWx1
ZShuYW1lKTsKIH0KIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>20696</attachid>
            <date>2008-04-20 07:04:02 -0700</date>
            <delta_ts>2008-05-24 23:22:03 -0700</delta_ts>
            <desc>Now with testcase</desc>
            <filename>18512-2.diff</filename>
            <type>text/plain</type>
            <size>5460</size>
            <attacher name="Rob Buis">rwlbuis</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAzMjI4MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTYgQEAKKzIwMDgtMDQtMjAgIFJvYiBCdWlzICA8YnVpc0BrZGUub3JnPgorCisg
ICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIGh0dHBzOi8vYnVn
cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xODUxMgorICAgICAgICBnZXRQcmVzZW50YXRp
b25BdHRyaWJ1dGUgcmV0dXJucyBhIHNoYXJlZCBvYmplY3QKKworICAgICAgICBVbnNoYXJlIHRo
ZSBtYXBwZWQgc3R5bGUgZGVjbGFyYXRpb24gaWYgaXQgaXMgc2hhcmVkCisgICAgICAgIGJlZm9y
ZSByZXR1cm5pbmcgdGhlIHByZXNlbnRhdGlvbiBhdHRyaWJ1dGUuCisKKyAgICAgICAgKiBzdmcv
U1ZHU3R5bGVkRWxlbWVudC5jcHA6CisgICAgICAgIChXZWJDb3JlOjpTVkdTdHlsZWRFbGVtZW50
OjpnZXRQcmVzZW50YXRpb25BdHRyaWJ1dGUpOgorCiAyMDA4LTA0LTIwICBTaW1vbiBIYXVzbWFu
biAgPGhhdXNtYW5uQHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgQWxwIFRva2Vy
LgpJbmRleDogV2ViQ29yZS9zdmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
V2ViQ29yZS9zdmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAJKHJldmlzaW9uIDMyMjgxKQorKysgV2Vi
Q29yZS9zdmcvU1ZHU3R5bGVkRWxlbWVudC5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEsNiArMSw2
IEBACiAvKgogICAgIENvcHlyaWdodCAoQykgMjAwNCwgMjAwNSwgMjAwNiwgMjAwNywgMjAwOCBO
aWtvbGFzIFppbW1lcm1hbm4gPHppbW1lcm1hbm5Aa2RlLm9yZz4KLSAgICAgICAgICAgICAgICAg
IDIwMDQsIDIwMDUsIDIwMDcgUm9iIEJ1aXMgPGJ1aXNAa2RlLm9yZz4KKyAgICAgICAgICAgICAg
ICAgIDIwMDQsIDIwMDUsIDIwMDcsIDIwMDggUm9iIEJ1aXMgPGJ1aXNAa2RlLm9yZz4KIAogICAg
IFRoaXMgbGlicmFyeSBpcyBmcmVlIHNvZnR3YXJlOyB5b3UgY2FuIHJlZGlzdHJpYnV0ZSBpdCBh
bmQvb3IKICAgICBtb2RpZnkgaXQgdW5kZXIgdGhlIHRlcm1zIG9mIHRoZSBHTlUgTGlicmFyeSBH
ZW5lcmFsIFB1YmxpYwpAQCAtMjgxLDYgKzI4MSwxMSBAQCBQYXNzUmVmUHRyPENTU1ZhbHVlPiBT
VkdTdHlsZWRFbGVtZW50OjpnCiAgICAgTWFwcGVkQXR0cmlidXRlKiBjc3NTVkdBdHRyID0gbWFw
cGVkQXR0cmlidXRlcygpLT5nZXRBdHRyaWJ1dGVJdGVtKG5hbWUpOwogICAgIGlmICghY3NzU1ZH
QXR0ciB8fCAhY3NzU1ZHQXR0ci0+c3R5bGUoKSkKICAgICAgICAgcmV0dXJuIDA7CisgICAgaWYg
KGNzc1NWR0F0dHItPnN0eWxlKCktPnJlZkNvdW50KCkgPiAxKSB7CisgICAgICAgIGNzc1NWR0F0
dHItPnNldERlY2woMCk7CisgICAgICAgIGludCBwcm9wSWQgPSBTVkdTdHlsZWRFbGVtZW50Ojpj
c3NQcm9wZXJ0eUlkRm9yU1ZHQXR0cmlidXRlTmFtZShjc3NTVkdBdHRyLT5uYW1lKCkpOworICAg
ICAgICBhZGRDU1NQcm9wZXJ0eShjc3NTVkdBdHRyLCBwcm9wSWQsIGNzc1NWR0F0dHItPnZhbHVl
KCkpOworICAgIH0KICAgICByZXR1cm4gY3NzU1ZHQXR0ci0+c3R5bGUoKS0+Z2V0UHJvcGVydHlD
U1NWYWx1ZShuYW1lKTsKIH0KIApJbmRleDogTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIExheW91dFRlc3RzL0NoYW5nZUxvZwkocmV2aXNpb24gMzIyODEpCisrKyBMYXlvdXRU
ZXN0cy9DaGFuZ2VMb2cJKHdvcmtpbmcgY29weSkKQEAgLTEsMyArMSwxNCBAQAorMjAwOC0wNC0y
MCAgUm9iIEJ1aXMgIDxidWlzQGtkZS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ
IChPT1BTISkuCisKKyAgICAgICAgVGVzdGNhc2UgZm9yOgorICAgICAgICBodHRwczovL2J1Z3Mu
d2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTg1MTIKKyAgICAgICAgZ2V0UHJlc2VudGF0aW9u
QXR0cmlidXRlIHJldHVybnMgYSBzaGFyZWQgb2JqZWN0CisKKyAgICAgICAgKiBzdmcvY3VzdG9t
L2dldFByZXNlbnRhdGlvbkF0dHJpYnV0ZS1tb2RpZnktZXhwZWN0ZWQudHh0OiBBZGRlZC4KKyAg
ICAgICAgKiBzdmcvY3VzdG9tL2dldFByZXNlbnRhdGlvbkF0dHJpYnV0ZS1tb2RpZnkuc3ZnOiBB
ZGRlZC4KKwogMjAwOC0wNC0xOSAgRGFuIEJlcm5zdGVpbiAgPG1pdHpAYXBwbGUuY29tPgogCiAg
ICAgICAgIFJldmlld2VkIGJ5IERhdmUgSHlhdHQuCkluZGV4OiBMYXlvdXRUZXN0cy9zdmcvY3Vz
dG9tL2dldFByZXNlbnRhdGlvbkF0dHJpYnV0ZS1tb2RpZnktZXhwZWN0ZWQudHh0Cj09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT0KLS0tIExheW91dFRlc3RzL3N2Zy9jdXN0b20vZ2V0UHJlc2VudGF0aW9uQXR0cmlidXRlLW1v
ZGlmeS1leHBlY3RlZC50eHQJKHJldmlzaW9uIDApCisrKyBMYXlvdXRUZXN0cy9zdmcvY3VzdG9t
L2dldFByZXNlbnRhdGlvbkF0dHJpYnV0ZS1tb2RpZnktZXhwZWN0ZWQudHh0CShyZXZpc2lvbiAw
KQpAQCAtMCwwICsxLDMgQEAKK1RoaXMgdGVzdCB0ZXN0cyB0aGF0IGdldHRpbmcgYSBDU1NWYWx1
ZSB1c2luZyBnZXRQcmVzZW50YXRpb25BdHRyaWJ1dGUgYW5kIHRoZW4gY2hhbmdpbmcgdGhhdCB2
YWx1ZSBkb2VzIG5vdCBpbmZsdWVuY2Ugb3RoZXIgcHJlc2VudGF0aW9uIGF0dHJpYnV0ZXMgdGhh
dCBzaGFyZSB0aGF0IHZhbHVlLiBTZWUgQnVnIDE4NTEyLgorUGFzc2VkCisKSW5kZXg6IExheW91
dFRlc3RzL3N2Zy9jdXN0b20vZ2V0UHJlc2VudGF0aW9uQXR0cmlidXRlLW1vZGlmeS5zdmcKPT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PQotLS0gTGF5b3V0VGVzdHMvc3ZnL2N1c3RvbS9nZXRQcmVzZW50YXRpb25BdHRyaWJ1
dGUtbW9kaWZ5LnN2ZwkocmV2aXNpb24gMCkKKysrIExheW91dFRlc3RzL3N2Zy9jdXN0b20vZ2V0
UHJlc2VudGF0aW9uQXR0cmlidXRlLW1vZGlmeS5zdmcJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEs
NTYgQEAKKzw/eG1sIHZlcnNpb249IjEuMCIgZW5jb2Rpbmc9IklTTy04ODU5LTEiIHN0YW5kYWxv
bmU9Im5vIj8+Cis8IURPQ1RZUEUgc3ZnIFBVQkxJQyAiLS8vVzNDLy9EVEQgU1ZHIDIwMDEwOTA0
Ly9FTiIgImh0dHA6Ly93d3cudzMub3JnL1RSLzIwMDEvUkVDLVNWRy0yMDAxMDkwNC9EVEQvc3Zn
MTAuZHRkIj4KKzxzdmcgeG1sbnM9Imh0dHA6Ly93d3cudzMub3JnLzIwMDAvc3ZnIiBvbmxvYWQ9
InJ1blRlc3QoKSI+CisKKzxzY3JpcHQ+Cis8IVtDREFUQVsKKyAgICBmdW5jdGlvbiBsb2coc3Ry
aW5nKSB7CisgICAgICB2YXIgbmV3RGl2ID0gZG9jdW1lbnQuY3JlYXRlRWxlbWVudE5TKCJodHRw
Oi8vd3d3LnczLm9yZy8xOTk5L3hodG1sIiwgImRpdiIpOworICAgICAgbmV3RGl2LnRleHRDb250
ZW50ID0gc3RyaW5nOworICAgICAgZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoJ2xvZycpLmFwcGVu
ZENoaWxkKG5ld0Rpdik7CisgICAgfQorICAgIGZ1bmN0aW9uIHJ1blRlc3QoKQorICAgIHsKKyAg
ICAgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKKyAgICAgICAgICAgIGxheW91
dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKKyAgICAgICAgdmFyIHJlY3QgPSBkb2N1bWVu
dC5nZXRFbGVtZW50QnlJZCgic2Vjb25kcmVjdCIpOworICAgICAgICB2YXIgZmlsbCA9IHJlY3Qu
Z2V0UHJlc2VudGF0aW9uQXR0cmlidXRlKCJmaWxsIik7CisgICAgICAgIGlmIChmaWxsID09IG51
bGwpIHsKKyAgICAgICAgICAgIGxvZygiRmFpbGVkIGdldHRpbmcgdGhlIGZpbGwgUEEiKTsKKyAg
ICAgICAgICAgIHJldHVybjsKKyAgICAgICAgfQorICAgICAgICBmaWxsLnNldFJHQkNvbG9yKCJn
cmVlbiIpOworICAgICAgICByZWN0ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoImZpcnN0cmVj
dCIpOworICAgICAgICBmaWxsID0gcmVjdC5nZXRQcmVzZW50YXRpb25BdHRyaWJ1dGUoImZpbGwi
KTsKKyAgICAgICAgaWYgKGZpbGwuY3NzVGV4dCAhPSAnI0ZGMDAwMCcpIHsKKyAgICAgICAgICAg
IGxvZygiQ2hhbmdpbmcgZmlsbCBQQSBzaG91bGQgbm90IGNoYW5nZSBmaXJzdHJlY3QgZmlsbCBQ
QSIpOworICAgICAgICAgICAgcmV0dXJuOworICAgICAgICB9CisgICAgICAgIHJlY3QgPSBkb2N1
bWVudC5nZXRFbGVtZW50QnlJZCgic2Vjb25kcmVjdCIpOworICAgICAgICBmaWxsID0gcmVjdC5n
ZXRQcmVzZW50YXRpb25BdHRyaWJ1dGUoImZpbGwiKTsKKyAgICAgICAgZmlsbC5zZXRSR0JDb2xv
cigiYmx1ZSIpOworICAgICAgICByZWN0ID0gZG9jdW1lbnQuZ2V0RWxlbWVudEJ5SWQoInRoaXJk
cmVjdCIpOworICAgICAgICBmaWxsID0gcmVjdC5nZXRQcmVzZW50YXRpb25BdHRyaWJ1dGUoImZp
bGwiKTsKKyAgICAgICAgaWYgKGZpbGwuY3NzVGV4dCAhPSAnIzAwODAwMCcpIHsKKyAgICAgICAg
ICAgIGxvZygiQ2hhbmdpbmcgZmlsbCBQQSBzaG91bGQgbm90IGNoYW5nZSB0aGlyZHJlY3QgZmls
bCBQQSIpOworICAgICAgICAgICAgcmV0dXJuOworICAgICAgICB9CisgICAgICAgIGxvZygiUGFz
c2VkIik7CisgICAgfQorICAgIF1dPgorPC9zY3JpcHQ+Cis8Zm9yZWlnbk9iamVjdD4KKyAgICA8
aHRtbCB4bWxucz0iaHR0cDovL3d3dy53My5vcmcvMTk5OS94aHRtbCI+CisgICAgICAgIDxkaXYg
aWQ9ImxvZyI+CitUaGlzIHRlc3QgdGVzdHMgdGhhdCBnZXR0aW5nIGEgQ1NTVmFsdWUgdXNpbmcg
Z2V0UHJlc2VudGF0aW9uQXR0cmlidXRlCithbmQgdGhlbiBjaGFuZ2luZyB0aGF0IHZhbHVlIGRv
ZXMgbm90IGluZmx1ZW5jZSBvdGhlciBwcmVzZW50YXRpb24KK2F0dHJpYnV0ZXMgdGhhdCBzaGFy
ZSB0aGF0IHZhbHVlLgorU2VlIDxhIGhyZWY9Imh0dHA6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTE4NTEyIj5CdWcgMTg1MTI8L2E+LgorICAgICAgICA8L2Rpdj4KKyAgICA8L2h0
bWw+Cis8L2ZvcmVpZ25PYmplY3Q+CisKKyAgPHJlY3QgaWQ9ImZpcnN0cmVjdCIgIHg9IjEwMCIg
eT0iMTAwIiB3aWR0aD0iMTAwIiBoZWlnaHQ9IjEwMCIgZmlsbD0icmVkIiAvPgorICA8cmVjdCBp
ZD0ic2Vjb25kcmVjdCIgeD0iMTAwIiB5PSIyNTAiIHdpZHRoPSIxMDAiIGhlaWdodD0iMTAwIiBm
aWxsPSJyZWQiIC8+CisgIDxyZWN0IGlkPSJ0aGlyZHJlY3QiIHg9IjEwMCIgeT0iNDAwIiB3aWR0
aD0iMTAwIiBoZWlnaHQ9IjEwMCIgZmlsbD0iZ3JlZW4iIC8+Cis8L3N2Zz4K
</data>
<flag name="review"
          id="9039"
          type_id="1"
          status="+"
          setter="darin"
    />
          </attachment>
      

    </bug>

</bugzilla>