<?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>48046</bug_id>
          
          <creation_ts>2010-10-21 00:34:02 -0700</creation_ts>
          <short_desc>WebKit shouldn&apos;t load a plug-in based on file extension if a MIME type is specified.</short_desc>
          <delta_ts>2010-10-24 23:31:58 -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>New Bugs</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>InRadar</keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Andy Estes">aestes</reporter>
          <assigned_to name="Andy Estes">aestes</assigned_to>
          <cc>abarth</cc>
    
    <cc>eric</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>297262</commentid>
    <comment_count>0</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 00:34:02 -0700</bug_when>
    <thetext>WebKit shouldn&apos;t load a plug-in when a page author specifies a MIME type that no plug-in supports.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297265</commentid>
    <comment_count>1</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 00:41:59 -0700</bug_when>
    <thetext>In the following example:

&lt;embed src=&quot;movie.m4v&quot; type=&quot;application/x-shockwave-flash&quot;&gt;

when Flash isn&apos;t installed but QuickTime is, WebKit will unexpectedly load the QuickTime plug-in. This is because WebKit recognizes that QuickTime supports files ending in &apos;.m4v&apos; and loads the file in QuickTime despite the markup specifying a MIME type QuickTime doesn&apos;t support.

In this case, WebKit should display the missing plug-in indicator if an explicit MIME type is specified that no installed plug-in claims to support.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297266</commentid>
    <comment_count>2</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 00:42:41 -0700</bug_when>
    <thetext>&lt;rdar://problem/8573762&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297275</commentid>
    <comment_count>3</comment_count>
      <attachid>71401</attachid>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 00:57:51 -0700</bug_when>
    <thetext>Created attachment 71401
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297277</commentid>
    <comment_count>4</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 01:03:16 -0700</bug_when>
    <thetext>Giving myself an r-. I need to see what other platforms need this fix in their FrameLoaderClient implementation before posting this for review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297286</commentid>
    <comment_count>5</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 01:34:27 -0700</bug_when>
    <thetext>(In reply to comment #4)
&gt; Giving myself an r-. I need to see what other platforms need this fix in their FrameLoaderClient implementation before posting this for review.

Looks okay.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297304</commentid>
    <comment_count>6</comment_count>
      <attachid>71401</attachid>
    <who name="Brian Weinstein">bweinstein</who>
    <bug_when>2010-10-21 02:12:11 -0700</bug_when>
    <thetext>Comment on attachment 71401
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=71401&amp;action=review

&gt; WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm:1632
&gt; +    if (!pluginPackage &amp;&amp; [extension length] != 0 &amp;&amp; [MIMEType length] == 0) {

Should we be using [extension length] &amp;&amp; ![MIMEType length] and get rid of the comparisons with 0?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297430</commentid>
    <comment_count>7</comment_count>
      <attachid>71401</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2010-10-21 08:21:06 -0700</bug_when>
    <thetext>Comment on attachment 71401
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=71401&amp;action=review

r=me with the suggested changes

&gt;&gt; WebKit/mac/WebCoreSupport/WebFrameLoaderClient.mm:1632
&gt;&gt; +    if (!pluginPackage &amp;&amp; [extension length] != 0 &amp;&amp; [MIMEType length] == 0) {
&gt; 
&gt; Should we be using [extension length] &amp;&amp; ![MIMEType length] and get rid of the comparisons with 0?

Indeed, please follow current WebKit style even though the old code is incorrect.

&gt; LayoutTests/plugins/invalid-mime-with-valid-extension-shows-missing-plugin.html:19
&gt;  

I would guess that the data: url will be delivered to the plug-in much faster than the test file, so I think you have a potential race condition here even with the setTimeout(0). I think you may want to use the test file for both plug-in instances and use a non-zero timeout instead.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297751</commentid>
    <comment_count>8</comment_count>
      <attachid>71514</attachid>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 17:42:08 -0700</bug_when>
    <thetext>Created attachment 71514
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>297752</commentid>
    <comment_count>9</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-21 17:43:23 -0700</bug_when>
    <thetext>(In reply to comment #8)
&gt; Created an attachment (id=71514) [details]
&gt; Patch

I rewrote the test. It changed enough that it probably warrants a new review.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298045</commentid>
    <comment_count>10</comment_count>
      <attachid>71514</attachid>
    <who name="Eric Carlson">eric.carlson</who>
    <bug_when>2010-10-22 10:29:18 -0700</bug_when>
    <thetext>Comment on attachment 71514
Patch

Much simpler test, nice! 

One very minor nit that you might want to change: you don&apos;t need to call finishTest from logFailure, just get rid of the &quot;else&quot; after calling testCallback.

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298159</commentid>
    <comment_count>11</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-22 13:58:02 -0700</bug_when>
    <thetext>Thanks Eric. Landed in http://trac.webkit.org/changeset/70332.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298162</commentid>
    <comment_count>12</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-22 14:02:26 -0700</bug_when>
    <thetext>Oops, forgot to svn add the tests. Layout tests landed in http://trac.webkit.org/changeset/70334.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298247</commentid>
    <comment_count>13</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-22 16:30:06 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/70332 might have broken GTK Linux 32-bit Release</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298249</commentid>
    <comment_count>14</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-22 16:30:36 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/70334 might have broken GTK Linux 32-bit Release</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298278</commentid>
    <comment_count>15</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-22 17:21:21 -0700</bug_when>
    <thetext>(In reply to comment #14)
&gt; http://trac.webkit.org/changeset/70334 might have broken GTK Linux 32-bit Release

Fixing.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298303</commentid>
    <comment_count>16</comment_count>
    <who name="Andy Estes">aestes</who>
    <bug_when>2010-10-22 19:27:16 -0700</bug_when>
    <thetext>I didn&apos;t handle this for WebKit2 or Win/Gtk/Qt (oops!). Fixed in http://trac.webkit.org/changeset/70379.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>298307</commentid>
    <comment_count>17</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2010-10-22 21:09:49 -0700</bug_when>
    <thetext>http://trac.webkit.org/changeset/70379 might have broken GTK Linux 32-bit Debug</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>71401</attachid>
            <date>2010-10-21 00:57:51 -0700</date>
            <delta_ts>2010-10-21 17:42:05 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-48046-20101021005750.patch</filename>
            <type>text/plain</type>
            <size>3951</size>
            <attacher name="Andy Estes">aestes</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9tYWMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9tYWMvQ2hh
bmdlTG9nCShyZXZpc2lvbiA3MDIwOSkKKysrIFdlYktpdC9tYWMvQ2hhbmdlTG9nCSh3b3JraW5n
IGNvcHkpCkBAIC0xLDMgKzEsMjEgQEAKKzIwMTAtMTAtMjEgIEFuZHkgRXN0ZXMgIDxhZXN0ZXNA
YXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIFdlYktpdCBzaG91bGRuJ3QgbG9hZCBhIHBsdWctaW4gYmFzZWQgb24gZmlsZSBleHRlbnNp
b24gaWYgYSBNSU1FIHR5cGUKKyAgICAgICAgaXMgc3BlY2lmaWVkLgorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDgwNDYKKworICAgICAgICBJZiBhIE1J
TUUgdHlwZSBpcyBzcGVjaWZpZWQgaW4gYW4gb2JqZWN0IG9yIGVtYmVkIGVsZW1lbnQsIGFuZCB0
aGF0IE1JTUUKKyAgICAgICAgdHlwZSBpc24ndCBzdXBwb3J0ZWQgYnkgYW4gaW5zdGFsbGVkIHBs
dWctaW4sIFdlYktpdCBzaG91bGRuJ3QgYXR0ZW1wdAorICAgICAgICB0byBmaW5kIGEgcGx1Zy1p
biBiYXNlZCBvbiB0aGUgZmlsZSBleHRlbnNpb24gb2YgdGhlIHVybCBhdHRyaWJ1dGUuCisgICAg
ICAgIERvaW5nIHNvIGNhbiBsZWFkIHRvIGNhc2VzIHdoZXJlIGEgcGx1Zy1pbiBpcyBsb2FkZWQg
dGhhdCBjYW4ndCBoYW5kbGUKKyAgICAgICAgcmVzb3VyY2VzIG9mIHRoZSBNSU1FIHR5cGUgc3Bl
Y2lmaWVkIGJ5IHRoZSBhdXRob3IuCisKKyAgICAgICAgKiBXZWJDb3JlU3VwcG9ydC9XZWJGcmFt
ZUxvYWRlckNsaWVudC5tbToKKyAgICAgICAgKFdlYkZyYW1lTG9hZGVyQ2xpZW50OjpjcmVhdGVQ
bHVnaW4pOiBPbmx5IGNoZWNrIGZvciBhIHBsdWdpblBhY2thZ2UgYnkKKyAgICAgICAgZXh0ZW5z
aW9uIGlmIE1JTUUgdHlwZSBpcyB0aGUgZW1wdHkgc3RyaW5nLgorCiAyMDEwLTEwLTIwICBTaW1v
biBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5
IERhbiBCZXJuc3RlaW4uCkluZGV4OiBXZWJLaXQvbWFjL1dlYkNvcmVTdXBwb3J0L1dlYkZyYW1l
TG9hZGVyQ2xpZW50Lm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9tYWMvV2ViQ29yZVN1cHBvcnQv
V2ViRnJhbWVMb2FkZXJDbGllbnQubW0JKHJldmlzaW9uIDY5ODA3KQorKysgV2ViS2l0L21hYy9X
ZWJDb3JlU3VwcG9ydC9XZWJGcmFtZUxvYWRlckNsaWVudC5tbQkod29ya2luZyBjb3B5KQpAQCAt
MTYyOSw3ICsxNjI5LDcgQEAgUGFzc1JlZlB0cjxXaWRnZXQ+IFdlYkZyYW1lTG9hZGVyQ2xpZW50
OgogICAgIH0KICAgICAKICAgICBOU1N0cmluZyAqZXh0ZW5zaW9uID0gW1twbHVnaW5VUkwgcGF0
aF0gcGF0aEV4dGVuc2lvbl07Ci0gICAgaWYgKCFwbHVnaW5QYWNrYWdlICYmIFtleHRlbnNpb24g
bGVuZ3RoXSAhPSAwKSB7CisgICAgaWYgKCFwbHVnaW5QYWNrYWdlICYmIFtleHRlbnNpb24gbGVu
Z3RoXSAhPSAwICYmIFtNSU1FVHlwZSBsZW5ndGhdID09IDApIHsKICAgICAgICAgcGx1Z2luUGFj
a2FnZSA9IFt3ZWJWaWV3IF9wbHVnaW5Gb3JFeHRlbnNpb246ZXh0ZW5zaW9uXTsKICAgICAgICAg
aWYgKHBsdWdpblBhY2thZ2UpIHsKICAgICAgICAgICAgIE5TU3RyaW5nICpuZXdNSU1FVHlwZSA9
IFtwbHVnaW5QYWNrYWdlIE1JTUVUeXBlRm9yRXh0ZW5zaW9uOmV4dGVuc2lvbl07CkluZGV4OiBM
YXlvdXRUZXN0cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9n
CShyZXZpc2lvbiA3MDIwOSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5
KQpAQCAtMSwzICsxLDE0IEBACisyMDEwLTEwLTIxICBBbmR5IEVzdGVzICA8YWVzdGVzQGFwcGxl
LmNvbT4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBX
ZWJLaXQgc2hvdWxkbid0IGxvYWQgYSBwbHVnLWluIGJhc2VkIG9uIGZpbGUgZXh0ZW5zaW9uIGlm
IGEgTUlNRSB0eXBlCisgICAgICAgIGlzIHNwZWNpZmllZC4KKyAgICAgICAgaHR0cHM6Ly9idWdz
LndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTQ4MDQ2CisKKyAgICAgICAgKiBwbHVnaW5zL2lu
dmFsaWQtbWltZS13aXRoLXZhbGlkLWV4dGVuc2lvbi1zaG93cy1taXNzaW5nLXBsdWdpbi1leHBl
Y3RlZC50eHQ6IEFkZGVkLgorICAgICAgICAqIHBsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgtdmFs
aWQtZXh0ZW5zaW9uLXNob3dzLW1pc3NpbmctcGx1Z2luLmh0bWw6IEFkZGVkLgorCiAyMDEwLTEw
LTIwICBTaGVyaWZmIEJvdCAgPHdlYmtpdC5yZXZpZXcuYm90QGdtYWlsLmNvbT4KIAogICAgICAg
ICBVbnJldmlld2VkLCByb2xsaW5nIG91dCByNzAwNDAuCkluZGV4OiBMYXlvdXRUZXN0cy9wbHVn
aW5zL2ludmFsaWQtbWltZS13aXRoLXZhbGlkLWV4dGVuc2lvbi1zaG93cy1taXNzaW5nLXBsdWdp
bi1leHBlY3RlZC50eHQKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvcGx1Z2lucy9pbnZhbGlk
LW1pbWUtd2l0aC12YWxpZC1leHRlbnNpb24tc2hvd3MtbWlzc2luZy1wbHVnaW4tZXhwZWN0ZWQu
dHh0CShyZXZpc2lvbiAwKQorKysgTGF5b3V0VGVzdHMvcGx1Z2lucy9pbnZhbGlkLW1pbWUtd2l0
aC12YWxpZC1leHRlbnNpb24tc2hvd3MtbWlzc2luZy1wbHVnaW4tZXhwZWN0ZWQudHh0CShyZXZp
c2lvbiAwKQpAQCAtMCwwICsxLDIgQEAKK1BBU1MKKyAKSW5kZXg6IExheW91dFRlc3RzL3BsdWdp
bnMvaW52YWxpZC1taW1lLXdpdGgtdmFsaWQtZXh0ZW5zaW9uLXNob3dzLW1pc3NpbmctcGx1Z2lu
Lmh0bWwKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvcGx1Z2lucy9pbnZhbGlkLW1pbWUtd2l0
aC12YWxpZC1leHRlbnNpb24tc2hvd3MtbWlzc2luZy1wbHVnaW4uaHRtbAkocmV2aXNpb24gMCkK
KysrIExheW91dFRlc3RzL3BsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgtdmFsaWQtZXh0ZW5zaW9u
LXNob3dzLW1pc3NpbmctcGx1Z2luLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMTggQEAK
KzxzY3JpcHQ+CisgICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikgeworICAgICAg
ICBsYXlvdXRUZXN0Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisgICAgICAgIGxheW91dFRlc3RD
b250cm9sbGVyLndhaXRVbnRpbERvbmUoKTsKKyAgICB9CisKKyAgICBmdW5jdGlvbiBzdHJlYW1M
b2FkZWQoKSB7CisgICAgICAgIGRvY3VtZW50LmdldEVsZW1lbnRCeUlkKCJjb25zb2xlIikuaW5u
ZXJIVE1MID0gIkZBSUwiOworICAgIH0KKworICAgIGZ1bmN0aW9uIGZpbmlzaFRlc3QoKSB7Cisg
ICAgICAgIGlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAgICAgICAgICBsYXlv
dXRUZXN0Q29udHJvbGxlci5ub3RpZnlEb25lKCk7CisgICAgfQorPC9zY3JpcHQ+Cis8ZGl2IGlk
PSJjb25zb2xlIj5QQVNTPC9kaXY+Cis8ZW1iZWQgc3JjPSJyZXNvdXJjZXMvdGVzdC50ZXN0bmV0
c2NhcGUiIHR5cGU9ImFwcGxpY2F0aW9uL3gtbm9uLWV4aXN0ZW50IiBvbnN0cmVhbWxvYWQ9InN0
cmVhbUxvYWRlZCgpOyI+Cis8ZW1iZWQgc3JjPSJkYXRhOnRleHQvcGxhaW4sIiB0eXBlPSJhcHBs
aWNhdGlvbi94LXdlYmtpdC10ZXN0LW5ldHNjYXBlIiBvbnN0cmVhbWxvYWQ9InNldFRpbWVvdXQo
ZmluaXNoVGVzdCwgMCk7Ij4K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>71514</attachid>
            <date>2010-10-21 17:42:08 -0700</date>
            <delta_ts>2010-10-22 10:29:18 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-48046-20101021174207.patch</filename>
            <type>text/plain</type>
            <size>4135</size>
            <attacher name="Andy Estes">aestes</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYktpdC9tYWMvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9tYWMvQ2hh
bmdlTG9nCShyZXZpc2lvbiA3MDIwOSkKKysrIFdlYktpdC9tYWMvQ2hhbmdlTG9nCSh3b3JraW5n
IGNvcHkpCkBAIC0xLDMgKzEsMjEgQEAKKzIwMTAtMTAtMjEgIEFuZHkgRXN0ZXMgIDxhZXN0ZXNA
YXBwbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAg
ICAgIFdlYktpdCBzaG91bGRuJ3QgbG9hZCBhIHBsdWctaW4gYmFzZWQgb24gZmlsZSBleHRlbnNp
b24gaWYgYSBNSU1FIHR5cGUKKyAgICAgICAgaXMgc3BlY2lmaWVkLgorICAgICAgICBodHRwczov
L2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDgwNDYKKworICAgICAgICBJZiBhIE1J
TUUgdHlwZSBpcyBzcGVjaWZpZWQgaW4gYW4gb2JqZWN0IG9yIGVtYmVkIGVsZW1lbnQsIGFuZCB0
aGF0IE1JTUUKKyAgICAgICAgdHlwZSBpc24ndCBzdXBwb3J0ZWQgYnkgYW4gaW5zdGFsbGVkIHBs
dWctaW4sIFdlYktpdCBzaG91bGRuJ3QgYXR0ZW1wdAorICAgICAgICB0byBmaW5kIGEgcGx1Zy1p
biBiYXNlZCBvbiB0aGUgZmlsZSBleHRlbnNpb24gb2YgdGhlIHVybCBhdHRyaWJ1dGUuCisgICAg
ICAgIERvaW5nIHNvIGNhbiBsZWFkIHRvIGNhc2VzIHdoZXJlIGEgcGx1Zy1pbiBpcyBsb2FkZWQg
dGhhdCBjYW4ndCBoYW5kbGUKKyAgICAgICAgcmVzb3VyY2VzIG9mIHRoZSBNSU1FIHR5cGUgc3Bl
Y2lmaWVkIGJ5IHRoZSBhdXRob3IuCisKKyAgICAgICAgKiBXZWJDb3JlU3VwcG9ydC9XZWJGcmFt
ZUxvYWRlckNsaWVudC5tbToKKyAgICAgICAgKFdlYkZyYW1lTG9hZGVyQ2xpZW50OjpjcmVhdGVQ
bHVnaW4pOiBPbmx5IGNoZWNrIGZvciBhIHBsdWdpblBhY2thZ2UgYnkKKyAgICAgICAgZXh0ZW5z
aW9uIGlmIE1JTUUgdHlwZSBpcyB0aGUgZW1wdHkgc3RyaW5nLgorCiAyMDEwLTEwLTIwICBTaW1v
biBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5
IERhbiBCZXJuc3RlaW4uCkluZGV4OiBXZWJLaXQvbWFjL1dlYkNvcmVTdXBwb3J0L1dlYkZyYW1l
TG9hZGVyQ2xpZW50Lm1tCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYktpdC9tYWMvV2ViQ29yZVN1cHBvcnQv
V2ViRnJhbWVMb2FkZXJDbGllbnQubW0JKHJldmlzaW9uIDY5ODA3KQorKysgV2ViS2l0L21hYy9X
ZWJDb3JlU3VwcG9ydC9XZWJGcmFtZUxvYWRlckNsaWVudC5tbQkod29ya2luZyBjb3B5KQpAQCAt
MTYyOSw3ICsxNjI5LDcgQEAgUGFzc1JlZlB0cjxXaWRnZXQ+IFdlYkZyYW1lTG9hZGVyQ2xpZW50
OgogICAgIH0KICAgICAKICAgICBOU1N0cmluZyAqZXh0ZW5zaW9uID0gW1twbHVnaW5VUkwgcGF0
aF0gcGF0aEV4dGVuc2lvbl07Ci0gICAgaWYgKCFwbHVnaW5QYWNrYWdlICYmIFtleHRlbnNpb24g
bGVuZ3RoXSAhPSAwKSB7CisgICAgaWYgKCFwbHVnaW5QYWNrYWdlICYmIFtleHRlbnNpb24gbGVu
Z3RoXSAmJiAhW01JTUVUeXBlIGxlbmd0aF0pIHsKICAgICAgICAgcGx1Z2luUGFja2FnZSA9IFt3
ZWJWaWV3IF9wbHVnaW5Gb3JFeHRlbnNpb246ZXh0ZW5zaW9uXTsKICAgICAgICAgaWYgKHBsdWdp
blBhY2thZ2UpIHsKICAgICAgICAgICAgIE5TU3RyaW5nICpuZXdNSU1FVHlwZSA9IFtwbHVnaW5Q
YWNrYWdlIE1JTUVUeXBlRm9yRXh0ZW5zaW9uOmV4dGVuc2lvbl07CkluZGV4OiBMYXlvdXRUZXN0
cy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PQotLS0gTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCShyZXZpc2lv
biA3MDIwOSkKKysrIExheW91dFRlc3RzL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwz
ICsxLDE1IEBACisyMDEwLTEwLTIxICBBbmR5IEVzdGVzICA8YWVzdGVzQGFwcGxlLmNvbT4KKwor
ICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBXZWJLaXQgc2hv
dWxkbid0IGxvYWQgYSBwbHVnLWluIGJhc2VkIG9uIGZpbGUgZXh0ZW5zaW9uIGlmIGEgTUlNRSB0
eXBlCisgICAgICAgIGlzIHNwZWNpZmllZC4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5v
cmcvc2hvd19idWcuY2dpP2lkPTQ4MDQ2CisKKyAgICAgICAgKiBwbHVnaW5zL2ludmFsaWQtbWlt
ZS13aXRoLXZhbGlkLWV4dGVuc2lvbi1zaG93cy1taXNzaW5nLXBsdWdpbi1leHBlY3RlZC50eHQ6
IEFkZGVkLgorICAgICAgICAqIHBsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgtdmFsaWQtZXh0ZW5z
aW9uLXNob3dzLW1pc3NpbmctcGx1Z2luLmh0bWw6IEFkZGVkLgorICAgICAgICAqIHBsdWdpbnMv
cmVzb3VyY2VzL3Rlc3QudGVzdG5ldHNjYXBlOiBBZGRlZC4KKwogMjAxMC0xMC0yMCAgU2hlcmlm
ZiBCb3QgIDx3ZWJraXQucmV2aWV3LmJvdEBnbWFpbC5jb20+CiAKICAgICAgICAgVW5yZXZpZXdl
ZCwgcm9sbGluZyBvdXQgcjcwMDQwLgpJbmRleDogTGF5b3V0VGVzdHMvcGx1Z2lucy9pbnZhbGlk
LW1pbWUtd2l0aC12YWxpZC1leHRlbnNpb24tc2hvd3MtbWlzc2luZy1wbHVnaW4tZXhwZWN0ZWQu
dHh0Cj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT0KLS0tIExheW91dFRlc3RzL3BsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgt
dmFsaWQtZXh0ZW5zaW9uLXNob3dzLW1pc3NpbmctcGx1Z2luLWV4cGVjdGVkLnR4dAkocmV2aXNp
b24gMCkKKysrIExheW91dFRlc3RzL3BsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgtdmFsaWQtZXh0
ZW5zaW9uLXNob3dzLW1pc3NpbmctcGx1Z2luLWV4cGVjdGVkLnR4dAkocmV2aXNpb24gMCkKQEAg
LTAsMCArMSwyIEBACitQQVNTCisKSW5kZXg6IExheW91dFRlc3RzL3BsdWdpbnMvaW52YWxpZC1t
aW1lLXdpdGgtdmFsaWQtZXh0ZW5zaW9uLXNob3dzLW1pc3NpbmctcGx1Z2luLmh0bWwKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gTGF5b3V0VGVzdHMvcGx1Z2lucy9pbnZhbGlkLW1pbWUtd2l0aC12YWxpZC1leHRl
bnNpb24tc2hvd3MtbWlzc2luZy1wbHVnaW4uaHRtbAkocmV2aXNpb24gMCkKKysrIExheW91dFRl
c3RzL3BsdWdpbnMvaW52YWxpZC1taW1lLXdpdGgtdmFsaWQtZXh0ZW5zaW9uLXNob3dzLW1pc3Np
bmctcGx1Z2luLmh0bWwJKHJldmlzaW9uIDApCkBAIC0wLDAgKzEsMjQgQEAKKzxzY3JpcHQ+Cisg
ICAgaWYgKHdpbmRvdy5sYXlvdXRUZXN0Q29udHJvbGxlcikgeworICAgICAgICBsYXlvdXRUZXN0
Q29udHJvbGxlci5kdW1wQXNUZXh0KCk7CisgICAgICAgIGxheW91dFRlc3RDb250cm9sbGVyLndh
aXRVbnRpbERvbmUoKTsKKyAgICB9CisKKyAgICBmdW5jdGlvbiBsb2dGYWlsdXJlKCkgeworICAg
ICAgICBkb2N1bWVudC5nZXRFbGVtZW50QnlJZCgiY29uc29sZSIpLmlubmVySFRNTCA9ICJGQUlM
IjsKKyAgICAgICAgZmluaXNoVGVzdCgpOworICAgIH0KKworICAgIGZ1bmN0aW9uIGZpbmlzaFRl
c3QoKSB7CisgICAgICAgIGlmICh3aW5kb3cubGF5b3V0VGVzdENvbnRyb2xsZXIpCisgICAgICAg
ICAgICBsYXlvdXRUZXN0Q29udHJvbGxlci5ub3RpZnlEb25lKCk7CisgICAgfQorPC9zY3JpcHQ+
Cis8ZGl2IGlkPSJjb25zb2xlIj5QQVNTPC9kaXY+Cis8ZW1iZWQgbmFtZT0icGx1Z2luIiBzcmM9
InJlc291cmNlcy90ZXN0LnRlc3RuZXRzY2FwZSIgdHlwZT0iYXBwbGljYXRpb24veC1ub24tZXhp
c3RlbnQiPgorPHNjcmlwdD4KKyAgICBpZiAocGx1Z2luLnRlc3RDYWxsYmFjaykKKyAgICAgICAg
cGx1Z2luLnRlc3RDYWxsYmFjaygibG9nRmFpbHVyZSIpOworICAgIGVsc2UKKyAgICAgICAgZmlu
aXNoVGVzdCgpOworPC9zY3JpcHQ+CkluZGV4OiBMYXlvdXRUZXN0cy9wbHVnaW5zL3Jlc291cmNl
cy90ZXN0LnRlc3RuZXRzY2FwZQo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Cg==
</data>
<flag name="review"
          id="61579"
          type_id="1"
          status="+"
          setter="eric.carlson"
    />
          </attachment>
      

    </bug>

</bugzilla>