<?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>36702</bug_id>
          
          <creation_ts>2010-03-27 07:18:54 -0700</creation_ts>
          <short_desc>[Qt] NPP_SetWindow seems to not be called when TestNetscapePlugin is moved</short_desc>
          <delta_ts>2011-01-31 05:55:23 -0800</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>Plug-ins</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Other</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>Qt, QtTriaged</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          <dependson>40730</dependson>
    
    <dependson>53420</dependson>
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Robert Hogan">robert</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>kbalazs</cc>
    
    <cc>ossy</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>204984</commentid>
    <comment_count>0</comment_count>
      <attachid>51828</attachid>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-03-27 07:18:54 -0700</bug_when>
    <thetext>Created attachment 51828
Test Case

In the attached test, replacing

            window.addEventListener(&apos;load&apos;, doTest, false);

with

            window.addEventListener(&apos;load&apos;, function() {
                window.setTimeout(doTest, 0);

will cause the test to fail.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>205014</commentid>
    <comment_count>1</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-03-27 11:51:25 -0700</bug_when>
    <thetext>The test expectation is &quot;should not crash&quot;. Are you saying that it crashes if doTest is deferred?

Note that WebKit Qt component is for bugs in the WebKit Qt API layer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>205018</commentid>
    <comment_count>2</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-03-27 12:04:35 -0700</bug_when>
    <thetext>(In reply to comment #1)
&gt; The test expectation is &quot;should not crash&quot;. Are you saying that it crashes if
&gt; doTest is deferred?
&gt; 
&gt; Note that WebKit Qt component is for bugs in the WebKit Qt API layer.

No, it times out in Qt. Note that you need to have the patch at https://bugs.webkit.org/show_bug.cgi?id=36675 applied to encounter the bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>205049</commentid>
    <comment_count>3</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2010-03-27 14:10:59 -0700</bug_when>
    <thetext>This seems to imply that NPP_SetWindow is not called on plug-in if it&apos;s moved after initial layout. I think that&apos;s correct behavior for windowed plug-ins (but I&apos;m not sure whether TestNetscapePlugin is windowed or windowless, particularly on Qt).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>233720</commentid>
    <comment_count>4</comment_count>
    <who name="Balazs Kelemen">kbalazs</who>
    <bug_when>2010-06-03 05:49:39 -0700</bug_when>
    <thetext>Please clarify the situation:
  * is it crashing or just not working correctly?
  * is this the correct behavior as ap suggested or not?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>233865</commentid>
    <comment_count>5</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-03 12:11:57 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Please clarify the situation:
&gt;   * is it crashing or just not working correctly?
&gt;   * is this the correct behavior as ap suggested or not?

The two tests: LayoutTests/plugins/update-widgets-crash.html and LayoutTests/plugins/reentrant-update-widget-positions.html are the same except for that the former has:
           window.addEventListener(&apos;load&apos;, doTest, false);

instead of

            window.addEventListener(&apos;load&apos;, function() {
                window.setTimeout(doTest, 0);

LayoutTests/plugins/update-widgets-crash.html passes while LayoutTests/plugins/reentrant-update-widget-positions.html fails.

It fails by timing out.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>238050</commentid>
    <comment_count>6</comment_count>
      <attachid>58702</attachid>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-14 14:40:53 -0700</bug_when>
    <thetext>Created attachment 58702
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239267</commentid>
    <comment_count>7</comment_count>
      <attachid>58702</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-17 00:33:21 -0700</bug_when>
    <thetext>Comment on attachment 58702
Patch

Clearing flags on attachment: 58702

Committed r61311: &lt;http://trac.webkit.org/changeset/61311&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239268</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2010-06-17 00:33:26 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239303</commentid>
    <comment_count>9</comment_count>
    <who name="Csaba Osztrogonác">ossy</who>
    <bug_when>2010-06-17 02:02:31 -0700</bug_when>
    <thetext>It was rolled out by http://trac.webkit.org/changeset/61315,
because it make 2 tests fail:
http://build.webkit.org/results/Qt%20Linux%20Release/r61314%20%2813453%29/results.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239456</commentid>
    <comment_count>10</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-17 10:35:38 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; It was rolled out by http://trac.webkit.org/changeset/61315,
&gt; because it make 2 tests fail:
&gt; http://build.webkit.org/results/Qt%20Linux%20Release/r61314%20%2813453%29/results.html

That was sloppy, sorry. I&apos;ll look into this now.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>239468</commentid>
    <comment_count>11</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-17 10:50:07 -0700</bug_when>
    <thetext>(In reply to comment #9)
&gt; It was rolled out by http://trac.webkit.org/changeset/61315,
&gt; because it make 2 tests fail:
&gt; http://build.webkit.org/results/Qt%20Linux%20Release/r61314%20%2813453%29/results.html

Qt expected results for resize-from-plugin.html were wrong anyway by the looks of it, the new results are slightly less wrong but still problematic. 

The mac expected result is:

x: 18, y: 52, width: 200, height: 200, clipRect: (18, 52, 200, 142)

Qt result was (without patch):

x: 18, y: 54, width: 100, height: 100, clipRect: (0, 0, 100, 100)

And with patch:

x: 18, y: 54, width: 100, height: 100, clipRect: (0, 0, 200, 142)

So closer, but still no cigar.

The other test looks like a minor layout difference due to the extra resizing now taking place in DRT. So results just need to be updated when I land the corrected patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>240379</commentid>
    <comment_count>12</comment_count>
      <attachid>59192</attachid>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-19 13:02:47 -0700</bug_when>
    <thetext>Created attachment 59192
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>240440</commentid>
    <comment_count>13</comment_count>
    <who name="Robert Hogan">robert</who>
    <bug_when>2010-06-20 05:54:47 -0700</bug_when>
    <thetext>Committed r61505: &lt;http://trac.webkit.org/changeset/61505&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="0"
              isprivate="0"
          >
            <attachid>51828</attachid>
            <date>2010-03-27 07:18:54 -0700</date>
            <delta_ts>2010-06-03 12:05:23 -0700</delta_ts>
            <desc>Test Case</desc>
            <filename>reentrant-update-widget-positions-2.html</filename>
            <type>text/plain</type>
            <size>1480</size>
            <attacher name="Robert Hogan">robert</attacher>
            
              <data encoding="base64">PCFET0NUWVBFIGh0bWw+Cgo8aHRtbD4KPGhlYWQ+CiAgICA8ZGl2IGlkPSJjb25zb2xlIj48L2Rp
dj4KICAgIDxsaW5rIHJlbD0ic3R5bGVzaGVldCIgaHJlZj0iLi4vZmFzdC9qcy9yZXNvdXJjZXMv
anMtdGVzdC1zdHlsZS5jc3MiPgogICAgPHNjcmlwdCBzcmM9Ii4uL2Zhc3QvanMvcmVzb3VyY2Vz
L2pzLXRlc3QtcHJlLmpzIj48L3NjcmlwdD4KICAgIDxzdHlsZSB0eXBlPSJ0ZXh0L2NzcyIgbWVk
aWE9InNjcmVlbiI+CiAgICAgICAgI3BsdWdpbiB7CiAgICAgICAgICAgIHBvc2l0aW9uOiBhYnNv
bHV0ZTsKICAgICAgICAgICAgd2lkdGg6IDEwMHB4OwogICAgICAgICAgICBoZWlnaHQ6IDEwMHB4
OwogICAgICAgIH0KICAgIDwvc3R5bGU+CiAgICA8c2NyaXB0IHR5cGU9InRleHQvamF2YXNjcmlw
dCIgY2hhcnNldD0idXRmLTgiPgogICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikg
ewogICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLmR1bXBBc1RleHQoKTsKICAgICAgICBsYXlv
dXRUZXN0Q29udHJvbGxlci53YWl0VW50aWxEb25lKCk7CiAgICB9CiAgICAvLyBUaGlzIGdldHMg
Y2FsbGVkIGF1dG9tYXRpY2FsbHkgZnJvbSB0aGUgdGVzdCBwbHVnaW4uCiAgICB2YXIgZG9pbmdU
ZXN0ID0gZmFsc2U7CiAgICBmdW5jdGlvbiBzZXRXaW5kb3dDYWxsZWQoKQogICAgewogICAgICAg
IGlmICghZG9pbmdUZXN0KQogICAgICAgICAgICByZXR1cm47CgogICAgICAgIHZhciB2aWN0aW0g
PSBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgndmljdGltJyk7CiAgICAgICAgdmljdGltLnBhcmVu
dE5vZGUucmVtb3ZlQ2hpbGQodmljdGltKTsKICAgICAgICBkb2N1bWVudC5ib2R5Lm9mZnNldFRv
cDsKCiAgICAgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikKICAgICAgICAgICAg
bGF5b3V0VGVzdENvbnRyb2xsZXIubm90aWZ5RG9uZSgpOwogICAgfQogICAgCiAgICBmdW5jdGlv
biBkb1Rlc3QoKQogICAgewogICAgICAgIGRvaW5nVGVzdCA9IHRydWU7CiAgICAgICAgZG9jdW1l
bnQuZ2V0RWxlbWVudEJ5SWQoJ3BsdWdpbicpLnN0eWxlLnRvcCA9ICIxMjBweCI7CiAgICAgICAg
ZG9jdW1lbnQuYm9keS5vZmZzZXRUb3A7CiAgICAgICAgZG9pbmdUZXN0ID0gZmFsc2U7CiAgICB9
CiAgICAKICAgIHdpbmRvdy5hZGRFdmVudExpc3RlbmVyKCdsb2FkJywgZG9UZXN0LCBmYWxzZSk7
CiAgICA8L3NjcmlwdD4KPC9oZWFkPgo8Ym9keT4KCiAgICA8cD5UaGlzIHRlc3Qgc2hvdWxkIG5v
dCBjcmFzaC48L3A+CiAgICA8ZW1iZWQgdHlwZT0iYXBwbGljYXRpb24veC13ZWJraXQtdGVzdC1u
ZXRzY2FwZSIKICAgICAgICBvblNldFdpbmRvdz0ic2V0V2luZG93Q2FsbGVkKCkiCiAgICAgICAg
aWQ9InBsdWdpbiI+CiAgICA8ZW1iZWQgbmFtZT0idmljdGltIiB0eXBlPSJhcHBsaWNhdGlvbi94
LXdlYmtpdC10ZXN0LW5ldHNjYXBlIiBpZD0idmljdGltIj4KICAKPC9ib2R5Pgo8L2h0bWw+Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>58702</attachid>
            <date>2010-06-14 14:40:53 -0700</date>
            <delta_ts>2010-06-19 13:02:41 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36702-20100614224051.patch</filename>
            <type>text/plain</type>
            <size>3978</size>
            <attacher name="Robert Hogan">robert</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA5ZGQ3MTMwNjJhMjgwNDhkZTMzODBiMGEyNjlhM2NhZTgwZjJhMWJhLi5iNTQyZjhm
OTJkNWI5ZWFkYzQzM2QyYTUxYzU0NzljMDljZjk1NGI3IDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkgQEAK
KzIwMTAtMDYtMTQgIFJvYmVydCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgorCisgICAgICAg
IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gTlBQX1NldFdpbmRv
dyBzZWVtcyB0byBub3QgYmUgY2FsbGVkIHdoZW4gVGVzdE5ldHNjYXBlUGx1Z2luIGlzIG1vdmVk
CisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM2NzAy
CisKKyAgICAgICAgc2V0TlBXaW5kb3dJZk5lZWRlZCgpIGlzIGNhbGxlZCBvbiBwYWludCgpIGlu
IFBsdWdpblZpZXdRdCwgd2hpY2ggZG9lc24ndAorICAgICAgICB3b3JrIGZvciBEUlQuIFNvIGNh
bGwgaXQgaWYgd2UgYXJlIGluIERSVCBtb2RlIGFuZCB0aGUgd2luZG93IGdlb21ldHJ5CisgICAg
ICAgIGhhcyBjaGFuZ2VkLgorCisgICAgICAgIFVuc2tpcHMgcGx1Z2lucy9yZWVudHJhbnQtdXBk
YXRlLXdpZGdldC1wb3NpdGlvbnMuaHRtbAorCisgICAgICAgICogcGxhdGZvcm0vcXQvU2tpcHBl
ZDoKKwogMjAxMC0wNi0xNCAgRG1pdHJ5IFRpdG92ICA8ZGltaWNoQGNocm9taXVtLm9yZz4KIAog
ICAgICAgICBVbnJldmlld2VkLCB0ZW1wb3JhcnkgdXBkYXRlIENocm9taXVtIGV4cGVjdGF0aW9u
cyBhZnRlciBodHRwOi8vdHJhYy53ZWJraXQub3JnL2NoYW5nZXNldC82MTEyOApkaWZmIC0tZ2l0
IGEvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQvU2tpcHBlZCBiL0xheW91dFRlc3RzL3BsYXRmb3Jt
L3F0L1NraXBwZWQKaW5kZXggMjlhMGNmNjIwZGYzMmJjMDRkNDI3NDYxNDU1NDUwZTNhZWMwMzQw
Ni4uNjYyNjk2YWY1MDYwMGQwYjgwMjFmY2ExZTkwNjQyNWY4ZjZmZGE1NiAxMDA2NDQKLS0tIGEv
TGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQvU2tpcHBlZAorKysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9y
bS9xdC9Ta2lwcGVkCkBAIC00MjQsOCArNDI0LDYgQEAgcGx1Z2lucy9nZXQtdXJsLXdpdGgtYmxh
bmstdGFyZ2V0Lmh0bWwKIHBsdWdpbnMvb3Blbi1hbmQtY2xvc2Utd2luZG93LXdpdGgtcGx1Z2lu
Lmh0bWwKIHBsdWdpbnMvZ2V0dXJsbm90aWZ5LWR1cmluZy1kb2N1bWVudC10ZWFyZG93bi5odG1s
CiAKLSMgLS0tLSBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY3MDIK
LXBsdWdpbnMvcmVlbnRyYW50LXVwZGF0ZS13aWRnZXQtcG9zaXRpb25zLmh0bWwKICMgLS0tLSBw
bHVnaW5zIHJlbmRlcmVkIGluIGluY29ycmVjdCBwb3NpdGlvbgogcGx1Z2lucy9lbWJlZC1hdHRy
aWJ1dGVzLXN0eWxlLmh0bWwKICMgLS0tLSBkZXBlbmRzIG9uIFF1aWNrVGltZSBzdXBwb3J0CmRp
ZmYgLS1naXQgYS9XZWJDb3JlL0NoYW5nZUxvZyBiL1dlYkNvcmUvQ2hhbmdlTG9nCmluZGV4IDAy
YzYyNDJmZjEwNTBhZDU4M2RiOWZiODU2MmYxYzkyMGEwN2YyMDMuLmRmMzhjODQ0ZmJkZTQ3MTk0
MTNmNWE3MDA4NzQ2NjA4Yjg4ZDE4ZGYgMTAwNjQ0Ci0tLSBhL1dlYkNvcmUvQ2hhbmdlTG9nCisr
KyBiL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMjAgQEAKKzIwMTAtMDYtMTQgIFJvYmVy
dCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gTlBQX1NldFdpbmRvdyBzZWVtcyB0byBub3QgYmUg
Y2FsbGVkIHdoZW4gVGVzdE5ldHNjYXBlUGx1Z2luIGlzIG1vdmVkCisKKyAgICAgICAgaHR0cHM6
Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM2NzAyCisKKyAgICAgICAgc2V0TlBX
aW5kb3dJZk5lZWRlZCgpIGlzIGNhbGxlZCBvbiBwYWludCgpIGluIFBsdWdpblZpZXdRdCwgd2hp
Y2ggZG9lc24ndAorICAgICAgICB3b3JrIGZvciBEUlQuIFNvIGNhbGwgaXQgaWYgd2UgYXJlIGlu
IERSVCBtb2RlIGFuZCB0aGUgd2luZG93IGdlb21ldHJ5CisgICAgICAgIGhhcyBjaGFuZ2VkLgor
CisgICAgICAgIFVuc2tpcHMgcGx1Z2lucy9yZWVudHJhbnQtdXBkYXRlLXdpZGdldC1wb3NpdGlv
bnMuaHRtbAorCisgICAgICAgICogcGx1Z2lucy9xdC9QbHVnaW5WaWV3UXQuY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6UGx1Z2luVmlldzo6dXBkYXRlUGx1Z2luV2lkZ2V0KToKKwogMjAxMC0wNi0x
NCAgQW5kZXJzIENhcmxzc29uICA8YW5kZXJzY2FAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmll
d2VkIGJ5IERhcmluIEFkbGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wbHVnaW5zL3F0L1BsdWdp
blZpZXdRdC5jcHAgYi9XZWJDb3JlL3BsdWdpbnMvcXQvUGx1Z2luVmlld1F0LmNwcAppbmRleCBm
ZjUyMDcwZmIzZjZmM2Q2ZjI5MTg5ZTNkZmU5MjViMjI5MTk3MDkyLi42YTBmYTM5ZWUxYTM5YjE5
MzI5MWQ5ZTQ1NzI3MmU1OTQzOTAwOGMzIDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsdWdpbnMvcXQv
UGx1Z2luVmlld1F0LmNwcAorKysgYi9XZWJDb3JlL3BsdWdpbnMvcXQvUGx1Z2luVmlld1F0LmNw
cApAQCAtNjAsNiArNjAsNyBAQAogI2luY2x1ZGUgIlNjcmlwdENvbnRyb2xsZXIuaCIKICNpbmNs
dWRlICJTZXR0aW5ncy5oIgogI2luY2x1ZGUgIm5wcnVudGltZV9pbXBsLmgiCisjaW5jbHVkZSAi
cXdlYnBhZ2VfcC5oIgogI2luY2x1ZGUgInJ1bnRpbWVfcm9vdC5oIgogCiAjaW5jbHVkZSA8UUFw
cGxpY2F0aW9uPgpAQCAtMTIxLDEyICsxMjIsMTUgQEAgdm9pZCBQbHVnaW5WaWV3Ojp1cGRhdGVQ
bHVnaW5XaWRnZXQoKQogICAgIC8vIGRvIG5vdCBjYWxsIHNldE5QV2luZG93SWZOZWVkZWQgaW1t
ZWRpYXRlbHksIHdpbGwgYmUgY2FsbGVkIG9uIHBhaW50KCkKICAgICBtX2hhc1BlbmRpbmdHZW9t
ZXRyeUNoYW5nZSA9IHRydWU7CiAKLSAgICAvLyBpbiBvcmRlciB0byBtb3ZlL3Jlc2l6ZSB0aGUg
cGx1Z2luIHdpbmRvdyBhdCB0aGUgc2FtZSB0aW1lIGFzIHRoZQorICAgIC8vIChpKSBpbiBvcmRl
ciB0byBtb3ZlL3Jlc2l6ZSB0aGUgcGx1Z2luIHdpbmRvdyBhdCB0aGUgc2FtZSB0aW1lIGFzIHRo
ZQogICAgIC8vIHJlc3Qgb2YgZnJhbWUgZHVyaW5nIGUuZy4gc2Nyb2xsaW5nLCB3ZSBzZXQgdGhl
IHdpbmRvdyBnZW9tZXRyeQogICAgIC8vIGluIHRoZSBwYWludCgpIGZ1bmN0aW9uLCBidXQgYXMg
cGFpbnQoKSBpc24ndCBjYWxsZWQgd2hlbiB0aGUKICAgICAvLyBwbHVnaW4gd2luZG93IGlzIG91
dHNpZGUgdGhlIGZyYW1lIHdoaWNoIGNhbiBiZSBjYXVzZWQgYnkgYQogICAgIC8vIHNjcm9sbCwg
d2UgbmVlZCB0byBtb3ZlL3Jlc2l6ZSBpbW1lZGlhdGVseS4KLSAgICBpZiAoIW1fd2luZG93UmVj
dC5pbnRlcnNlY3RzKGZyYW1lVmlldy0+ZnJhbWVSZWN0KCkpKQorICAgIC8vIChpaSkgaWYgd2Ug
YXJlIHJ1bm5pbmcgbGF5b3V0IHRlc3RzIGZyb20gRFJULCBwYWludCgpIHdvbid0IGV2ZXIgZ2V0
IGNhbGxlZAorICAgIC8vIHNvIHdlIG5lZWQgdG8gY2FsbCBzZXROUFdpbmRvd0lmTmVlZGVkKCkg
aWYgd2luZG93IGdlb21ldHJ5IGhhcyBjaGFuZ2VkCisgICAgaWYgKCFtX3dpbmRvd1JlY3QuaW50
ZXJzZWN0cyhmcmFtZVZpZXctPmZyYW1lUmVjdCgpKQorICAgICAgICB8fCAoUVdlYlBhZ2VQcml2
YXRlOjpkcnRSdW4gJiYgcGxhdGZvcm1QbHVnaW5XaWRnZXQoKSAmJiAobV93aW5kb3dSZWN0ICE9
IG9sZFdpbmRvd1JlY3QgfHwgbV9jbGlwUmVjdCAhPSBvbGRDbGlwUmVjdCkpKQogICAgICAgICBz
ZXROUFdpbmRvd0lmTmVlZGVkKCk7CiAKICAgICAvLyBNYWtlIHN1cmUgd2UgZ2V0IHJlcGFpbnRl
ZCBhZnRlcndhcmRzLiBUaGlzIGlzIG5lY2Vzc2FyeSBmb3IgZG93bndhcmQK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>59192</attachid>
            <date>2010-06-19 13:02:47 -0700</date>
            <delta_ts>2010-06-20 04:24:02 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-36702-20100619210245.patch</filename>
            <type>text/plain</type>
            <size>5349</size>
            <attacher name="Robert Hogan">robert</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL0NoYW5nZUxvZyBiL0xheW91dFRlc3RzL0NoYW5nZUxv
ZwppbmRleCA0OGFjNTM4NWUwM2E3MTA1OGM4OWRhODc0Mjc5YTRhNWE4YjNhZGJlLi5mYTU5ZWJk
OGRlMGY3YzMwZmMzNWMyODg0NWFjYjNhM2U0OGVhOTFlIDEwMDY0NAotLS0gYS9MYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKKysrIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTkgQEAK
KzIwMTAtMDYtMTQgIFJvYmVydCBIb2dhbiAgPHJvYmVydEB3ZWJraXQub3JnPgorCisgICAgICAg
IFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIFtRdF0gTlBQX1NldFdpbmRv
dyBzZWVtcyB0byBub3QgYmUgY2FsbGVkIHdoZW4gVGVzdE5ldHNjYXBlUGx1Z2luIGlzIG1vdmVk
CisKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTM2NzAy
CisKKyAgICAgICAgc2V0TlBXaW5kb3dJZk5lZWRlZCgpIGlzIGNhbGxlZCBvbiBwYWludCgpIGlu
IFBsdWdpblZpZXdRdCwgd2hpY2ggZG9lc24ndAorICAgICAgICB3b3JrIGZvciBEUlQuIFNvIGNh
bGwgaXQgaWYgd2UgYXJlIGluIERSVCBtb2RlIGFuZCB0aGUgd2luZG93IGdlb21ldHJ5CisgICAg
ICAgIGhhcyBjaGFuZ2VkLgorCisgICAgICAgIFVuc2tpcHMgcGx1Z2lucy9yZWVudHJhbnQtdXBk
YXRlLXdpZGdldC1wb3NpdGlvbnMuaHRtbAorCisgICAgICAgICogcGxhdGZvcm0vcXQvU2tpcHBl
ZDoKKwogMjAxMC0wNi0xOSAgS2VudCBUYW11cmEgIDx0a2VudEBjaHJvbWl1bS5vcmc+CiAKICAg
ICAgICAgVW5yZXZpZXdlZCwgdGVzdCBleHBlY3RhdGlvbiB1cGRhdGUuCmRpZmYgLS1naXQgYS9M
YXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkIGIvTGF5b3V0VGVzdHMvcGxhdGZvcm0vcXQv
U2tpcHBlZAppbmRleCBmMDFhNmJlNmU2YzdjMjlkNDdmOWQyZmRkNWI5NzRkYTgwOTc0M2YwLi42
NmE0MTI5ZjgxMDlmNGNhZjI1YjAzNmZkOWY2MmExMDBiOGEzNjUyIDEwMDY0NAotLS0gYS9MYXlv
dXRUZXN0cy9wbGF0Zm9ybS9xdC9Ta2lwcGVkCisrKyBiL0xheW91dFRlc3RzL3BsYXRmb3JtL3F0
L1NraXBwZWQKQEAgLTQyNSwxMiArNDI1LDE4IEBAIHBsdWdpbnMvZ2V0LXVybC13aXRoLWJsYW5r
LXRhcmdldC5odG1sCiAKICMgLS0tLSBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5j
Z2k/aWQ9MzY3MDIKIHBsdWdpbnMvcmVlbnRyYW50LXVwZGF0ZS13aWRnZXQtcG9zaXRpb25zLmh0
bWwKKworIyAtLS0tIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00MDg3
OQorcGx1Z2lucy9yZXNpemUtZnJvbS1wbHVnaW4uaHRtbAorCiAjIC0tLS0gcGx1Z2lucyByZW5k
ZXJlZCBpbiBpbmNvcnJlY3QgcG9zaXRpb24KIHBsdWdpbnMvZW1iZWQtYXR0cmlidXRlcy1zdHls
ZS5odG1sCiAjIC0tLS0gZGVwZW5kcyBvbiBRdWlja1RpbWUgc3VwcG9ydAogcGx1Z2lucy9lbWJl
ZC1hdHRyaWJ1dGVzLXNldHRpbmcuaHRtbAogZmFzdC9kb20vb2JqZWN0LWVtYmVkLXBsdWdpbi1z
Y3JpcHRpbmcuaHRtbAorCiAjIC0tLS0gZXZlbnRzIG5vdCBnZXR0aW5nIHRvIHBsdWdpbgorIyAt
LS0tIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD00MDczMAogcGx1Z2lu
cy9tb3VzZS1ldmVudHMuaHRtbAogcGx1Z2lucy9rZXlib2FyZC1ldmVudHMuaHRtbAogcGx1Z2lu
cy9tb3VzZS1ldmVudHMtZml4ZWRwb3MuaHRtbApkaWZmIC0tZ2l0IGEvTGF5b3V0VGVzdHMvcGxh
dGZvcm0vcXQvcGx1Z2lucy9uZXRzY2FwZS1kb20tYWNjZXNzLWV4cGVjdGVkLnR4dCBiL0xheW91
dFRlc3RzL3BsYXRmb3JtL3F0L3BsdWdpbnMvbmV0c2NhcGUtZG9tLWFjY2Vzcy1leHBlY3RlZC50
eHQKaW5kZXggN2E2MDkzMDljZWI1YjlmODQwOWQxYzljN2Q5YzhlYWFhNTgxMGY1Ny4uZmRmMDVm
MzM1NmYzNDY4ZDQxZDBiY2M3MzEwZTg5M2FjMTYzMmYxMyAxMDA2NDQKLS0tIGEvTGF5b3V0VGVz
dHMvcGxhdGZvcm0vcXQvcGx1Z2lucy9uZXRzY2FwZS1kb20tYWNjZXNzLWV4cGVjdGVkLnR4dAor
KysgYi9MYXlvdXRUZXN0cy9wbGF0Zm9ybS9xdC9wbHVnaW5zL25ldHNjYXBlLWRvbS1hY2Nlc3Mt
ZXhwZWN0ZWQudHh0CkBAIC05LDcgKzksNyBAQCBsYXllciBhdCAoMCwwKSBzaXplIDgwMHg2MDAK
ICAgICAgICAgICB0ZXh0IHJ1biBhdCAoNTczLDApIHdpZHRoIDE4NTogIlRoZSBwbHVnLWluIHNo
b3VsZCBoYXZlIGEiCiAgICAgICAgICAgdGV4dCBydW4gYXQgKDAsMjApIHdpZHRoIDEwNzogIjNw
eCByZWQgYm9yZGVyLiIKICAgICAgIFJlbmRlckJsb2NrIChhbm9ueW1vdXMpIGF0ICgwLDU2KSBz
aXplIDc4NHgyMDYKLSAgICAgICAgUmVuZGVyRW1iZWRkZWRPYmplY3Qge0VNQkVEfSBhdCAoMCww
KSBzaXplIDIwNngyMDYgW2JvcmRlcjogKDNweCBzb2xpZCAjRkYwMDAwKV0gW1FUOiBnZW9tZXRy
eToge2F0ICg4LDY0KSBzaXplIDIwMHgyMDB9IGlzSGlkZGVuOiAwIGlzU2VsZlZpc2libGU6IDEg
aXNQYXJlbnRWaXNpYmxlOiAxIG1hc2s6IHthdCAoMCwwKSBzaXplIDIwMHgyMDB9IF0gCisgICAg
ICAgIFJlbmRlckVtYmVkZGVkT2JqZWN0IHtFTUJFRH0gYXQgKDAsMCkgc2l6ZSAyMDZ4MjA2IFti
b3JkZXI6ICgzcHggc29saWQgI0ZGMDAwMCldIFtRVDogZ2VvbWV0cnk6IHthdCAoMTEsNjcpIHNp
emUgMjAweDIwMH0gaXNIaWRkZW46IDAgaXNTZWxmVmlzaWJsZTogMSBpc1BhcmVudFZpc2libGU6
IDEgbWFzazoge2F0ICgwLDApIHNpemUgMjAweDIwMH0gXSAKICAgICAgICAgUmVuZGVyVGV4dCB7
I3RleHR9IGF0ICgwLDApIHNpemUgMHgwCiAgICAgICAgIFJlbmRlclRleHQgeyN0ZXh0fSBhdCAo
MCwwKSBzaXplIDB4MAogICAgICAgICBSZW5kZXJUZXh0IHsjdGV4dH0gYXQgKDAsMCkgc2l6ZSAw
eDAKZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5k
ZXggOTk0ZjUyYzZlOGYwMDUxNmExZDA2NTA0MzFmMjMwZmQxYmEzODM1OS4uMzM4Mjg3ZjY3Mjll
NmQ0NzhlMjkxNDQ2MjAwOGU1MjBmNGE3NmRiYiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VM
b2cKKysrIGIvV2ViQ29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwyMCBAQAorMjAxMC0wNi0xNCAg
Um9iZXJ0IEhvZ2FuICA8cm9iZXJ0QHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkg
Tk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgW1F0XSBOUFBfU2V0V2luZG93IHNlZW1zIHRvIG5v
dCBiZSBjYWxsZWQgd2hlbiBUZXN0TmV0c2NhcGVQbHVnaW4gaXMgbW92ZWQKKworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MzY3MDIKKworICAgICAgICBz
ZXROUFdpbmRvd0lmTmVlZGVkKCkgaXMgY2FsbGVkIG9uIHBhaW50KCkgaW4gUGx1Z2luVmlld1F0
LCB3aGljaCBkb2Vzbid0CisgICAgICAgIHdvcmsgZm9yIERSVC4gU28gY2FsbCBpdCBpZiB3ZSBh
cmUgaW4gRFJUIG1vZGUgYW5kIHRoZSB3aW5kb3cgZ2VvbWV0cnkKKyAgICAgICAgaGFzIGNoYW5n
ZWQuCisKKyAgICAgICAgVW5za2lwcyBwbHVnaW5zL3JlZW50cmFudC11cGRhdGUtd2lkZ2V0LXBv
c2l0aW9ucy5odG1sCisKKyAgICAgICAgKiBwbHVnaW5zL3F0L1BsdWdpblZpZXdRdC5jcHA6Cisg
ICAgICAgIChXZWJDb3JlOjpQbHVnaW5WaWV3Ojp1cGRhdGVQbHVnaW5XaWRnZXQpOgorCiAyMDEw
LTA2LTE5ICBHZW9yZ2UgV3JpZ2h0ICA8Z3dyaWdodEByaW0uY29tPgogCiAgICAgICAgIFJldmll
d2VkIGJ5IEdlb3JnZSBTdGFpa29zLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9wbHVnaW5zL3F0L1Bs
dWdpblZpZXdRdC5jcHAgYi9XZWJDb3JlL3BsdWdpbnMvcXQvUGx1Z2luVmlld1F0LmNwcAppbmRl
eCBmZjUyMDcwZmIzZjZmM2Q2ZjI5MTg5ZTNkZmU5MjViMjI5MTk3MDkyLi42YTBmYTM5ZWUxYTM5
YjE5MzI5MWQ5ZTQ1NzI3MmU1OTQzOTAwOGMzIDEwMDY0NAotLS0gYS9XZWJDb3JlL3BsdWdpbnMv
cXQvUGx1Z2luVmlld1F0LmNwcAorKysgYi9XZWJDb3JlL3BsdWdpbnMvcXQvUGx1Z2luVmlld1F0
LmNwcApAQCAtNjAsNiArNjAsNyBAQAogI2luY2x1ZGUgIlNjcmlwdENvbnRyb2xsZXIuaCIKICNp
bmNsdWRlICJTZXR0aW5ncy5oIgogI2luY2x1ZGUgIm5wcnVudGltZV9pbXBsLmgiCisjaW5jbHVk
ZSAicXdlYnBhZ2VfcC5oIgogI2luY2x1ZGUgInJ1bnRpbWVfcm9vdC5oIgogCiAjaW5jbHVkZSA8
UUFwcGxpY2F0aW9uPgpAQCAtMTIxLDEyICsxMjIsMTUgQEAgdm9pZCBQbHVnaW5WaWV3Ojp1cGRh
dGVQbHVnaW5XaWRnZXQoKQogICAgIC8vIGRvIG5vdCBjYWxsIHNldE5QV2luZG93SWZOZWVkZWQg
aW1tZWRpYXRlbHksIHdpbGwgYmUgY2FsbGVkIG9uIHBhaW50KCkKICAgICBtX2hhc1BlbmRpbmdH
ZW9tZXRyeUNoYW5nZSA9IHRydWU7CiAKLSAgICAvLyBpbiBvcmRlciB0byBtb3ZlL3Jlc2l6ZSB0
aGUgcGx1Z2luIHdpbmRvdyBhdCB0aGUgc2FtZSB0aW1lIGFzIHRoZQorICAgIC8vIChpKSBpbiBv
cmRlciB0byBtb3ZlL3Jlc2l6ZSB0aGUgcGx1Z2luIHdpbmRvdyBhdCB0aGUgc2FtZSB0aW1lIGFz
IHRoZQogICAgIC8vIHJlc3Qgb2YgZnJhbWUgZHVyaW5nIGUuZy4gc2Nyb2xsaW5nLCB3ZSBzZXQg
dGhlIHdpbmRvdyBnZW9tZXRyeQogICAgIC8vIGluIHRoZSBwYWludCgpIGZ1bmN0aW9uLCBidXQg
YXMgcGFpbnQoKSBpc24ndCBjYWxsZWQgd2hlbiB0aGUKICAgICAvLyBwbHVnaW4gd2luZG93IGlz
IG91dHNpZGUgdGhlIGZyYW1lIHdoaWNoIGNhbiBiZSBjYXVzZWQgYnkgYQogICAgIC8vIHNjcm9s
bCwgd2UgbmVlZCB0byBtb3ZlL3Jlc2l6ZSBpbW1lZGlhdGVseS4KLSAgICBpZiAoIW1fd2luZG93
UmVjdC5pbnRlcnNlY3RzKGZyYW1lVmlldy0+ZnJhbWVSZWN0KCkpKQorICAgIC8vIChpaSkgaWYg
d2UgYXJlIHJ1bm5pbmcgbGF5b3V0IHRlc3RzIGZyb20gRFJULCBwYWludCgpIHdvbid0IGV2ZXIg
Z2V0IGNhbGxlZAorICAgIC8vIHNvIHdlIG5lZWQgdG8gY2FsbCBzZXROUFdpbmRvd0lmTmVlZGVk
KCkgaWYgd2luZG93IGdlb21ldHJ5IGhhcyBjaGFuZ2VkCisgICAgaWYgKCFtX3dpbmRvd1JlY3Qu
aW50ZXJzZWN0cyhmcmFtZVZpZXctPmZyYW1lUmVjdCgpKQorICAgICAgICB8fCAoUVdlYlBhZ2VQ
cml2YXRlOjpkcnRSdW4gJiYgcGxhdGZvcm1QbHVnaW5XaWRnZXQoKSAmJiAobV93aW5kb3dSZWN0
ICE9IG9sZFdpbmRvd1JlY3QgfHwgbV9jbGlwUmVjdCAhPSBvbGRDbGlwUmVjdCkpKQogICAgICAg
ICBzZXROUFdpbmRvd0lmTmVlZGVkKCk7CiAKICAgICAvLyBNYWtlIHN1cmUgd2UgZ2V0IHJlcGFp
bnRlZCBhZnRlcndhcmRzLiBUaGlzIGlzIG5lY2Vzc2FyeSBmb3IgZG93bndhcmQK
</data>
<flag name="review"
          id="45891"
          type_id="1"
          status="+"
          setter="hausmann"
    />
          </attachment>
      

    </bug>

</bugzilla>