<?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>41864</bug_id>
          
          <creation_ts>2010-07-08 09:19:53 -0700</creation_ts>
          <short_desc>[V8] Navigation policy doesn&apos;t play nicely with pepper plugins</short_desc>
          <delta_ts>2010-07-08 16:59:52 -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></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Adam Barth">abarth</reporter>
          <assigned_to name="Adam Barth">abarth</assigned_to>
          <cc>fishd</cc>
    
    <cc>jam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>248258</commentid>
    <comment_count>0</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 09:19:53 -0700</bug_when>
    <thetext>[V8] Navigation policy doesn&apos;t play nicely with pepper plugins</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248260</commentid>
    <comment_count>1</comment_count>
      <attachid>60898</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 09:21:54 -0700</bug_when>
    <thetext>Created attachment 60898
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248346</commentid>
    <comment_count>2</comment_count>
      <attachid>60913</attachid>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 11:05:08 -0700</bug_when>
    <thetext>Created attachment 60913
Updated patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248417</commentid>
    <comment_count>3</comment_count>
      <attachid>60913</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 12:14:40 -0700</bug_when>
    <thetext>Comment on attachment 60913
Updated patch

This patch does not have a ChangeLog.

V8Utilities:103
 +  Frame* callingFrameForNavigation() {
This isn&apos;t the right name for this function.  The calling frame isn&apos;t changing because we&apos;re interested in navigation.  Maybe the name &quot;activeFrameForNavigation&quot; is better?  It&apos;s really the same thing we do for the canAccess check, so it might be worth sharing there too.  Maybe principalFrame (to steal some terminology from Mozilla)?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248445</commentid>
    <comment_count>4</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-07-08 12:48:22 -0700</bug_when>
    <thetext>I was struggling to suggest a name too.  I like principalFrame since it helps specify how we intend to use the frame.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248447</commentid>
    <comment_count>5</comment_count>
    <who name="Darin Fisher (:fishd, Google)">fishd</who>
    <bug_when>2010-07-08 12:49:15 -0700</bug_when>
    <thetext>Another choice is callingOrEnteredFrame.  It seems common in WebKit to use
function names like that.  It doesn&apos;t add any new terms to understand and
just states what the function does.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248467</commentid>
    <comment_count>6</comment_count>
      <attachid>60937</attachid>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 13:03:20 -0700</bug_when>
    <thetext>Created attachment 60937
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248470</commentid>
    <comment_count>7</comment_count>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 13:03:59 -0700</bug_when>
    <thetext>callingOrEnteredFrame sounds good to me.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248584</commentid>
    <comment_count>8</comment_count>
      <attachid>60937</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 15:19:51 -0700</bug_when>
    <thetext>Comment on attachment 60937
Patch

-    if (!callingFrame)
-        return;

You removed this null check.  Retrieving the frame can fail if the entered context is associated with an inactive document.  In that case, this patch introduces a crash.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248585</commentid>
    <comment_count>9</comment_count>
      <attachid>60937</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 15:20:20 -0700</bug_when>
    <thetext>Comment on attachment 60937
Patch

WebCore/bindings/v8/V8Utilities.cpp:119
 +      Frame* callingFrame = callingOrEnteredFrame();
Also, &quot;callingFrame&quot; is not an accurate name anymore.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248600</commentid>
    <comment_count>10</comment_count>
      <attachid>60975</attachid>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 15:53:41 -0700</bug_when>
    <thetext>Created attachment 60975
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248622</commentid>
    <comment_count>11</comment_count>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 16:28:32 -0700</bug_when>
    <thetext>Thanks John.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248632</commentid>
    <comment_count>12</comment_count>
      <attachid>60975</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-07-08 16:42:46 -0700</bug_when>
    <thetext>Comment on attachment 60975
Patch

This will actually fail to land in the commit-queue because the ChangeLog says OOPS.  I can land it manually for John.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248635</commentid>
    <comment_count>13</comment_count>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 16:43:55 -0700</bug_when>
    <thetext>Thanks for the initial pointer and review.  I can take care of landing it.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>248640</commentid>
    <comment_count>14</comment_count>
    <who name="John Abd-El-Malek">jam</who>
    <bug_when>2010-07-08 16:59:52 -0700</bug_when>
    <thetext>Committed r62868: &lt;http://trac.webkit.org/changeset/62868&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>60898</attachid>
            <date>2010-07-08 09:21:54 -0700</date>
            <delta_ts>2010-07-08 11:05:08 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-41864-20100708092152.patch</filename>
            <type>text/plain</type>
            <size>1711</size>
            <attacher name="Adam Barth">abarth</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA2Mjc5MSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMTAtMDctMDggIEFkYW0gQmFydGggIDxhYmFydGhAd2Via2l0Lm9y
Zz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBbVjhd
IE5hdmlnYXRpb24gcG9saWN5IGRvZXNuJ3QgcGxheSBuaWNlbHkgd2l0aCBwZXBwZXIgcGx1Z2lu
cworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDE4NjQK
KworICAgICAgICBTYW1wbGUgcGF0Y2guICBTdGlsbCBuZWVkcyBhIHRlc3QuCisKKyAgICAgICAg
Tm8gbmV3IHRlc3RzLiAoT09QUyEpCisKKyAgICAgICAgKiBiaW5kaW5ncy92OC9WOFV0aWxpdGll
cy5jcHA6CisgICAgICAgIChXZWJDb3JlOjpzaG91bGRBbGxvd05hdmlnYXRpb24pOgorCiAyMDEw
LTA3LTA4ICBWaXRhbHkgUmVwZXNoa28gIDx2aXRhbHlyQGNocm9taXVtLm9yZz4KIAogICAgICAg
ICBSZXZpZXdlZCBieSBQYXZlbCBGZWxkbWFuLgpJbmRleDogV2ViQ29yZS9iaW5kaW5ncy92OC9W
OFV0aWxpdGllcy5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9iaW5kaW5ncy92OC9WOFV0aWxp
dGllcy5jcHAJKHJldmlzaW9uIDYyNzkxKQorKysgV2ViQ29yZS9iaW5kaW5ncy92OC9WOFV0aWxp
dGllcy5jcHAJKHdvcmtpbmcgY29weSkKQEAgLTEwMiw4ICsxMDIsMTYgQEAgYm9vbCBwcm9jZXNz
aW5nVXNlckdlc3R1cmUoKQogCiBib29sIHNob3VsZEFsbG93TmF2aWdhdGlvbihGcmFtZSogZnJh
bWUpCiB7Ci0gICAgRnJhbWUqIGNhbGxpbmdGcmFtZSA9IFY4UHJveHk6OnJldHJpZXZlRnJhbWVG
b3JDYWxsaW5nQ29udGV4dCgpOwotICAgIHJldHVybiBjYWxsaW5nRnJhbWUgJiYgY2FsbGluZ0Zy
YW1lLT5sb2FkZXIoKS0+c2hvdWxkQWxsb3dOYXZpZ2F0aW9uKGZyYW1lKTsKKyAgICBGcmFtZSog
ZnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9yQ2FsbGluZ0NvbnRleHQoKTsKKyAgICBp
ZiAoIWZyYW1lKSB7CisgICAgICAgIC8vIFVuZm9ydHVuYXRlbHksIHdoZW4gcHJvY2Vzc2luZyBz
Y3JpcHQgZnJvbSBhIHBsdWctaW4sIHdlIG1pZ2h0IG5vdAorICAgICAgICAvLyBoYXZlIGEgY2Fs
bGluZyBjb250ZXh0LiAgSW4gdGhvc2UgY2FzZXMsIHdlIGZhbGwgYmFjayB0byB0aGUKKyAgICAg
ICAgLy8gZW50ZXJlZCBjb250ZXh0IGZvciBzZWN1cml0eSBjaGVja3MuCisgICAgICAgIC8vIEZJ
WE1FOiBXZSBuZWVkIGEgYmV0dGVyIEFQSSBmb3IgcmV0cmlldmluZyBmcmFtZXMgdGhhdCBhYnN0
cmFjdHMKKyAgICAgICAgLy8gICAgICAgIGF3YXkgdGhpcyBjb25jZXJuLgorICAgICAgICBmcmFt
ZSA9IFY4UHJveHk6OnJldHJpZXZlRnJhbWVGb3JFbnRlcmVkQ29udGV4dCgpOworICAgIH0KKyAg
ICByZXR1cm4gZnJhbWUgJiYgZnJhbWUtPmxvYWRlcigpLT5zaG91bGRBbGxvd05hdmlnYXRpb24o
ZnJhbWUpOwogfQogCiBLVVJMIGNvbXBsZXRlVVJMKGNvbnN0IFN0cmluZyYgcmVsYXRpdmVVUkwp
Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>60913</attachid>
            <date>2010-07-08 11:05:08 -0700</date>
            <delta_ts>2010-07-08 13:03:16 -0700</delta_ts>
            <desc>Updated patch</desc>
            <filename>navpatch.diff</filename>
            <type>text/plain</type>
            <size>1618</size>
            <attacher name="John Abd-El-Malek">jam</attacher>
            
              <data encoding="base64">SW5kZXg6IFY4VXRpbGl0aWVzLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBWOFV0aWxpdGllcy5jcHAJKHJl
dmlzaW9uIDYyODAyKQorKysgVjhVdGlsaXRpZXMuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC0xMDAs
OSArMTAwLDIyIEBACiAgICAgcmV0dXJuIGZyYW1lICYmIGZyYW1lLT5zY3JpcHQoKS0+cHJvY2Vz
c2luZ1VzZXJHZXN0dXJlKCk7CiB9CiAKK0ZyYW1lKiBjYWxsaW5nRnJhbWVGb3JOYXZpZ2F0aW9u
KCkgeworICAgIEZyYW1lKiBjYWxsaW5nRnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9y
Q2FsbGluZ0NvbnRleHQoKTsKKyAgICBpZiAoIWNhbGxpbmdGcmFtZSkgeworICAgICAgICAvLyBV
bmZvcnR1bmF0ZWx5LCB3aGVuIHByb2Nlc3Npbmcgc2NyaXB0IGZyb20gYSBwbHVnLWluLCB3ZSBt
aWdodCBub3QKKyAgICAgICAgLy8gaGF2ZSBhIGNhbGxpbmcgY29udGV4dC4gIEluIHRob3NlIGNh
c2VzLCB3ZSBmYWxsIGJhY2sgdG8gdGhlCisgICAgICAgIC8vIGVudGVyZWQgY29udGV4dCBmb3Ig
c2VjdXJpdHkgY2hlY2tzLgorICAgICAgICAvLyBGSVhNRTogV2UgbmVlZCBhIGJldHRlciBBUEkg
Zm9yIHJldHJpZXZpbmcgZnJhbWVzIHRoYXQgYWJzdHJhY3RzCisgICAgICAgIC8vICAgICAgICBh
d2F5IHRoaXMgY29uY2Vybi4KKyAgICAgICAgY2FsbGluZ0ZyYW1lID0gVjhQcm94eTo6cmV0cmll
dmVGcmFtZUZvckVudGVyZWRDb250ZXh0KCk7CisgICAgfQorICAgIHJldHVybiBjYWxsaW5nRnJh
bWU7Cit9CisKIGJvb2wgc2hvdWxkQWxsb3dOYXZpZ2F0aW9uKEZyYW1lKiBmcmFtZSkKIHsKLSAg
ICBGcmFtZSogY2FsbGluZ0ZyYW1lID0gVjhQcm94eTo6cmV0cmlldmVGcmFtZUZvckNhbGxpbmdD
b250ZXh0KCk7CisgICAgRnJhbWUqIGNhbGxpbmdGcmFtZSA9IGNhbGxpbmdGcmFtZUZvck5hdmln
YXRpb24oKTsKICAgICByZXR1cm4gY2FsbGluZ0ZyYW1lICYmIGNhbGxpbmdGcmFtZS0+bG9hZGVy
KCktPnNob3VsZEFsbG93TmF2aWdhdGlvbihmcmFtZSk7CiB9CiAKQEAgLTExNywxMCArMTMwLDcg
QEAKIAogdm9pZCBuYXZpZ2F0ZUlmQWxsb3dlZChGcmFtZSogZnJhbWUsIGNvbnN0IEtVUkwmIHVy
bCwgYm9vbCBsb2NrSGlzdG9yeSwgYm9vbCBsb2NrQmFja0ZvcndhcmRMaXN0KQogewotICAgIEZy
YW1lKiBjYWxsaW5nRnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9yQ2FsbGluZ0NvbnRl
eHQoKTsKLSAgICBpZiAoIWNhbGxpbmdGcmFtZSkKLSAgICAgICAgcmV0dXJuOwotCisgICAgRnJh
bWUqIGNhbGxpbmdGcmFtZSA9IGNhbGxpbmdGcmFtZUZvck5hdmlnYXRpb24oKTsKICAgICBpZiAo
IXByb3RvY29sSXNKYXZhU2NyaXB0KHVybCkgfHwgU2NyaXB0Q29udHJvbGxlcjo6aXNTYWZlU2Ny
aXB0KGZyYW1lKSkKICAgICAgICAgZnJhbWUtPnJlZGlyZWN0U2NoZWR1bGVyKCktPnNjaGVkdWxl
TG9jYXRpb25DaGFuZ2UodXJsLnN0cmluZygpLCBjYWxsaW5nRnJhbWUtPmxvYWRlcigpLT5vdXRn
b2luZ1JlZmVycmVyKCksIGxvY2tIaXN0b3J5LCBsb2NrQmFja0ZvcndhcmRMaXN0LCBwcm9jZXNz
aW5nVXNlckdlc3R1cmUoKSk7CiB9Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>60937</attachid>
            <date>2010-07-08 13:03:20 -0700</date>
            <delta_ts>2010-07-08 15:53:38 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-41864-20100708130247.patch</filename>
            <type>text/plain</type>
            <size>2406</size>
            <attacher name="John Abd-El-Malek">jam</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA2MjgyMSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMTAtMDctMDggIEpvaG4gQWJkLUVsLU1hbGVrICA8amFtQGNocm9t
aXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICBbVjhdIE5hdmlnYXRpb24gcG9saWN5IGRvZXNuJ3QgcGxheSBuaWNlbHkgd2l0aCBwZXBwZXIg
cGx1Z2lucworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
NDE4NjQKKworICAgICAgICBObyBuZXcgdGVzdHMuIChPT1BTISkKKworICAgICAgICAqIGJpbmRp
bmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OmNhbGxpbmdPckVudGVy
ZWRGcmFtZSk6CisgICAgICAgIChXZWJDb3JlOjpzaG91bGRBbGxvd05hdmlnYXRpb24pOgorICAg
ICAgICAoV2ViQ29yZTo6bmF2aWdhdGVJZkFsbG93ZWQpOgorCiAyMDEwLTA3LTA4ICBFcmljIENh
cmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh
biBCZXJuc3RlaW4uCkluZGV4OiBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAo9
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09Ci0tLSBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAkocmV2aXNp
b24gNjI4MDIpCisrKyBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAkod29ya2lu
ZyBjb3B5KQpAQCAtMTAwLDkgKzEwMCwyMyBAQCBib29sIHByb2Nlc3NpbmdVc2VyR2VzdHVyZSgp
CiAgICAgcmV0dXJuIGZyYW1lICYmIGZyYW1lLT5zY3JpcHQoKS0+cHJvY2Vzc2luZ1VzZXJHZXN0
dXJlKCk7CiB9CiAKLWJvb2wgc2hvdWxkQWxsb3dOYXZpZ2F0aW9uKEZyYW1lKiBmcmFtZSkKK0Zy
YW1lKiBjYWxsaW5nT3JFbnRlcmVkRnJhbWUoKQogewogICAgIEZyYW1lKiBjYWxsaW5nRnJhbWUg
PSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9yQ2FsbGluZ0NvbnRleHQoKTsKKyAgICBpZiAoIWNh
bGxpbmdGcmFtZSkgeworICAgICAgICAvLyBVbmZvcnR1bmF0ZWx5LCB3aGVuIHByb2Nlc3Npbmcg
c2NyaXB0IGZyb20gYSBwbHVnLWluLCB3ZSBtaWdodCBub3QKKyAgICAgICAgLy8gaGF2ZSBhIGNh
bGxpbmcgY29udGV4dC4gIEluIHRob3NlIGNhc2VzLCB3ZSBmYWxsIGJhY2sgdG8gdGhlCisgICAg
ICAgIC8vIGVudGVyZWQgY29udGV4dCBmb3Igc2VjdXJpdHkgY2hlY2tzLgorICAgICAgICAvLyBG
SVhNRTogV2UgbmVlZCBhIGJldHRlciBBUEkgZm9yIHJldHJpZXZpbmcgZnJhbWVzIHRoYXQgYWJz
dHJhY3RzCisgICAgICAgIC8vICAgICAgICBhd2F5IHRoaXMgY29uY2Vybi4KKyAgICAgICAgY2Fs
bGluZ0ZyYW1lID0gVjhQcm94eTo6cmV0cmlldmVGcmFtZUZvckVudGVyZWRDb250ZXh0KCk7Cisg
ICAgfQorICAgIHJldHVybiBjYWxsaW5nRnJhbWU7Cit9CisKK2Jvb2wgc2hvdWxkQWxsb3dOYXZp
Z2F0aW9uKEZyYW1lKiBmcmFtZSkKK3sKKyAgICBGcmFtZSogY2FsbGluZ0ZyYW1lID0gY2FsbGlu
Z09yRW50ZXJlZEZyYW1lKCk7CiAgICAgcmV0dXJuIGNhbGxpbmdGcmFtZSAmJiBjYWxsaW5nRnJh
bWUtPmxvYWRlcigpLT5zaG91bGRBbGxvd05hdmlnYXRpb24oZnJhbWUpOwogfQogCkBAIC0xMTcs
MTAgKzEzMSw3IEBAIEtVUkwgY29tcGxldGVVUkwoY29uc3QgU3RyaW5nJiByZWxhdGl2ZVUKIAog
dm9pZCBuYXZpZ2F0ZUlmQWxsb3dlZChGcmFtZSogZnJhbWUsIGNvbnN0IEtVUkwmIHVybCwgYm9v
bCBsb2NrSGlzdG9yeSwgYm9vbCBsb2NrQmFja0ZvcndhcmRMaXN0KQogewotICAgIEZyYW1lKiBj
YWxsaW5nRnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9yQ2FsbGluZ0NvbnRleHQoKTsK
LSAgICBpZiAoIWNhbGxpbmdGcmFtZSkKLSAgICAgICAgcmV0dXJuOwotCisgICAgRnJhbWUqIGNh
bGxpbmdGcmFtZSA9IGNhbGxpbmdPckVudGVyZWRGcmFtZSgpOwogICAgIGlmICghcHJvdG9jb2xJ
c0phdmFTY3JpcHQodXJsKSB8fCBTY3JpcHRDb250cm9sbGVyOjppc1NhZmVTY3JpcHQoZnJhbWUp
KQogICAgICAgICBmcmFtZS0+cmVkaXJlY3RTY2hlZHVsZXIoKS0+c2NoZWR1bGVMb2NhdGlvbkNo
YW5nZSh1cmwuc3RyaW5nKCksIGNhbGxpbmdGcmFtZS0+bG9hZGVyKCktPm91dGdvaW5nUmVmZXJy
ZXIoKSwgbG9ja0hpc3RvcnksIGxvY2tCYWNrRm9yd2FyZExpc3QsIHByb2Nlc3NpbmdVc2VyR2Vz
dHVyZSgpKTsKIH0K
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>60975</attachid>
            <date>2010-07-08 15:53:41 -0700</date>
            <delta_ts>2010-07-08 16:42:46 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-41864-20100708155309.patch</filename>
            <type>text/plain</type>
            <size>2816</size>
            <attacher name="John Abd-El-Malek">jam</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiA2MjgyMSkKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTcgQEAKKzIwMTAtMDctMDggIEpvaG4gQWJkLUVsLU1hbGVrICA8amFtQGNocm9t
aXVtLm9yZz4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICBbVjhdIE5hdmlnYXRpb24gcG9saWN5IGRvZXNuJ3QgcGxheSBuaWNlbHkgd2l0aCBwZXBwZXIg
cGx1Z2lucworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9
NDE4NjQKKworICAgICAgICBObyBuZXcgdGVzdHMuIChPT1BTISkKKworICAgICAgICAqIGJpbmRp
bmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OmNhbGxpbmdPckVudGVy
ZWRGcmFtZSk6CisgICAgICAgIChXZWJDb3JlOjpzaG91bGRBbGxvd05hdmlnYXRpb24pOgorICAg
ICAgICAoV2ViQ29yZTo6bmF2aWdhdGVJZkFsbG93ZWQpOgorCiAyMDEwLTA3LTA4ICBFcmljIENh
cmxzb24gIDxlcmljLmNhcmxzb25AYXBwbGUuY29tPgogCiAgICAgICAgIFJldmlld2VkIGJ5IERh
biBCZXJuc3RlaW4uCkluZGV4OiBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAo9
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09Ci0tLSBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAkocmV2aXNp
b24gNjI4MDIpCisrKyBXZWJDb3JlL2JpbmRpbmdzL3Y4L1Y4VXRpbGl0aWVzLmNwcAkod29ya2lu
ZyBjb3B5KQpAQCAtMTAwLDEwICsxMDAsMjQgQEAgYm9vbCBwcm9jZXNzaW5nVXNlckdlc3R1cmUo
KQogICAgIHJldHVybiBmcmFtZSAmJiBmcmFtZS0+c2NyaXB0KCktPnByb2Nlc3NpbmdVc2VyR2Vz
dHVyZSgpOwogfQogCitGcmFtZSogY2FsbGluZ09yRW50ZXJlZEZyYW1lKCkKK3sKKyAgICBGcmFt
ZSogZnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2ZUZyYW1lRm9yQ2FsbGluZ0NvbnRleHQoKTsKKyAg
ICBpZiAoIWZyYW1lKSB7CisgICAgICAgIC8vIFVuZm9ydHVuYXRlbHksIHdoZW4gcHJvY2Vzc2lu
ZyBzY3JpcHQgZnJvbSBhIHBsdWctaW4sIHdlIG1pZ2h0IG5vdAorICAgICAgICAvLyBoYXZlIGEg
Y2FsbGluZyBjb250ZXh0LiAgSW4gdGhvc2UgY2FzZXMsIHdlIGZhbGwgYmFjayB0byB0aGUKKyAg
ICAgICAgLy8gZW50ZXJlZCBjb250ZXh0IGZvciBzZWN1cml0eSBjaGVja3MuCisgICAgICAgIC8v
IEZJWE1FOiBXZSBuZWVkIGEgYmV0dGVyIEFQSSBmb3IgcmV0cmlldmluZyBmcmFtZXMgdGhhdCBh
YnN0cmFjdHMKKyAgICAgICAgLy8gICAgICAgIGF3YXkgdGhpcyBjb25jZXJuLgorICAgICAgICBm
cmFtZSA9IFY4UHJveHk6OnJldHJpZXZlRnJhbWVGb3JFbnRlcmVkQ29udGV4dCgpOworICAgIH0K
KyAgICByZXR1cm4gZnJhbWU7Cit9CisKIGJvb2wgc2hvdWxkQWxsb3dOYXZpZ2F0aW9uKEZyYW1l
KiBmcmFtZSkKIHsKLSAgICBGcmFtZSogY2FsbGluZ0ZyYW1lID0gVjhQcm94eTo6cmV0cmlldmVG
cmFtZUZvckNhbGxpbmdDb250ZXh0KCk7Ci0gICAgcmV0dXJuIGNhbGxpbmdGcmFtZSAmJiBjYWxs
aW5nRnJhbWUtPmxvYWRlcigpLT5zaG91bGRBbGxvd05hdmlnYXRpb24oZnJhbWUpOworICAgIEZy
YW1lKiBjYWxsaW5nT3JFbnRlcmVkID0gY2FsbGluZ09yRW50ZXJlZEZyYW1lKCk7CisgICAgcmV0
dXJuIGNhbGxpbmdPckVudGVyZWQgJiYgY2FsbGluZ09yRW50ZXJlZC0+bG9hZGVyKCktPnNob3Vs
ZEFsbG93TmF2aWdhdGlvbihmcmFtZSk7CiB9CiAKIEtVUkwgY29tcGxldGVVUkwoY29uc3QgU3Ry
aW5nJiByZWxhdGl2ZVVSTCkKQEAgLTExNywxMiArMTMxLDExIEBAIEtVUkwgY29tcGxldGVVUkwo
Y29uc3QgU3RyaW5nJiByZWxhdGl2ZVUKIAogdm9pZCBuYXZpZ2F0ZUlmQWxsb3dlZChGcmFtZSog
ZnJhbWUsIGNvbnN0IEtVUkwmIHVybCwgYm9vbCBsb2NrSGlzdG9yeSwgYm9vbCBsb2NrQmFja0Zv
cndhcmRMaXN0KQogewotICAgIEZyYW1lKiBjYWxsaW5nRnJhbWUgPSBWOFByb3h5OjpyZXRyaWV2
ZUZyYW1lRm9yQ2FsbGluZ0NvbnRleHQoKTsKLSAgICBpZiAoIWNhbGxpbmdGcmFtZSkKKyAgICBG
cmFtZSogY2FsbGluZ09yRW50ZXJlZCA9IGNhbGxpbmdPckVudGVyZWRGcmFtZSgpOworICAgIGlm
ICghY2FsbGluZ09yRW50ZXJlZCkKICAgICAgICAgcmV0dXJuOwotCiAgICAgaWYgKCFwcm90b2Nv
bElzSmF2YVNjcmlwdCh1cmwpIHx8IFNjcmlwdENvbnRyb2xsZXI6OmlzU2FmZVNjcmlwdChmcmFt
ZSkpCi0gICAgICAgIGZyYW1lLT5yZWRpcmVjdFNjaGVkdWxlcigpLT5zY2hlZHVsZUxvY2F0aW9u
Q2hhbmdlKHVybC5zdHJpbmcoKSwgY2FsbGluZ0ZyYW1lLT5sb2FkZXIoKS0+b3V0Z29pbmdSZWZl
cnJlcigpLCBsb2NrSGlzdG9yeSwgbG9ja0JhY2tGb3J3YXJkTGlzdCwgcHJvY2Vzc2luZ1VzZXJH
ZXN0dXJlKCkpOworICAgICAgICBmcmFtZS0+cmVkaXJlY3RTY2hlZHVsZXIoKS0+c2NoZWR1bGVM
b2NhdGlvbkNoYW5nZSh1cmwuc3RyaW5nKCksIGNhbGxpbmdPckVudGVyZWQtPmxvYWRlcigpLT5v
dXRnb2luZ1JlZmVycmVyKCksIGxvY2tIaXN0b3J5LCBsb2NrQmFja0ZvcndhcmRMaXN0LCBwcm9j
ZXNzaW5nVXNlckdlc3R1cmUoKSk7CiB9CiAKIFNjcmlwdEV4ZWN1dGlvbkNvbnRleHQqIGdldFNj
cmlwdEV4ZWN1dGlvbkNvbnRleHQoKQo=
</data>
<flag name="review"
          id="48857"
          type_id="1"
          status="+"
          setter="fishd"
    />
    <flag name="commit-queue"
          id="48868"
          type_id="3"
          status="-"
          setter="abarth"
    />
          </attachment>
      

    </bug>

</bugzilla>