<?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>78316</bug_id>
          
          <creation_ts>2012-02-09 19:12:51 -0800</creation_ts>
          <short_desc>Disable IndexedDB permissions check from shared workers</short_desc>
          <delta_ts>2012-02-10 17:20:58 -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>New Bugs</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Unspecified</rep_platform>
          <op_sys>Unspecified</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="David Grogan">dgrogan</reporter>
          <assigned_to name="David Grogan">dgrogan</assigned_to>
          <cc>jsbell</cc>
    
    <cc>michaeln</cc>
    
    <cc>tony</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>553732</commentid>
    <comment_count>0</comment_count>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-09 19:12:51 -0800</bug_when>
    <thetext>Disable IndexedDB permissions check from workers</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553734</commentid>
    <comment_count>1</comment_count>
      <attachid>126434</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-09 19:14:20 -0800</bug_when>
    <thetext>Created attachment 126434
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553738</commentid>
    <comment_count>2</comment_count>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-09 19:17:42 -0800</bug_when>
    <thetext>I left the rest of the code for 2 reasons:
1) To prevent it from not compiling.
2) To preserve the diff when I update it.

I foresee a quite skeptical webkit reviewer.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553748</commentid>
    <comment_count>3</comment_count>
      <attachid>126434</attachid>
    <who name="Michael Nordman">michaeln</who>
    <bug_when>2012-02-09 19:41:29 -0800</bug_when>
    <thetext>Comment on attachment 126434
Patch

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

&gt; Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp:165
&gt;      WebWorkerClientImpl* webWorkerClientImpl = static_cast&lt;WebWorkerClientImpl*&gt;(&amp;workerContext-&gt;thread()-&gt;workerLoaderProxy());

david, you had mentioned an invalid cast in the shared worker case... is this it on line 165?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>553751</commentid>
    <comment_count>4</comment_count>
    <who name="Michael Nordman">michaeln</who>
    <bug_when>2012-02-09 19:44:59 -0800</bug_when>
    <thetext>maybe test if isSharedWorkerContext() and only short circuit that case to not regress for dedicated workers</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554169</commentid>
    <comment_count>5</comment_count>
      <attachid>126533</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 10:25:28 -0800</bug_when>
    <thetext>Created attachment 126533
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554172</commentid>
    <comment_count>6</comment_count>
      <attachid>126533</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 10:27:09 -0800</bug_when>
    <thetext>Comment on attachment 126533
Patch

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

Changed to only disable the test for shared workers.

&gt; Source/WebKit/chromium/src/IDBFactoryBackendProxy.cpp:166
&gt;      WebWorkerClientImpl* webWorkerClientImpl = static_cast&lt;WebWorkerClientImpl*&gt;(&amp;workerContext-&gt;thread()-&gt;workerLoaderProxy());

Yes, this is the cast that needs to be changed to a WebWorkerBase.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554354</commentid>
    <comment_count>7</comment_count>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 13:42:31 -0800</bug_when>
    <thetext>Tony, could you review this?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554374</commentid>
    <comment_count>8</comment_count>
      <attachid>126533</attachid>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-02-10 14:06:05 -0800</bug_when>
    <thetext>Comment on attachment 126533
Patch

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

Can you write a LayoutTest for this?

&gt; Source/WebKit/chromium/ChangeLog:6
&gt; +
&gt; +        Reviewed by NOBODY (OOPS!).

Please put include some of the comment here to explain why you&apos;re making this change.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554401</commentid>
    <comment_count>9</comment_count>
      <attachid>126533</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 14:40:39 -0800</bug_when>
    <thetext>Comment on attachment 126533
Patch

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

The behavior that&apos;s changing here (for shared workers) can&apos;t be tested with the current DRT because it doesn&apos;t run multiple processes.  I think a layout test for the dedicated worker case is possible, though it would involve changing layoutTestController.  Which I don&apos;t want to do right now on account of expediency.  (The only reason we&apos;re adding this bypass is to get the feature in front of developers ASAP.)

Though I&apos;ve been meaning to add a permissions test for a while.  Bug filed at http://crbug.com/113738.  Is that ok?

&gt;&gt; Source/WebKit/chromium/ChangeLog:6
&gt;&gt; +        Reviewed by NOBODY (OOPS!).
&gt; 
&gt; Please put include some of the comment here to explain why you&apos;re making this change.

Done.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554403</commentid>
    <comment_count>10</comment_count>
      <attachid>126586</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 14:42:07 -0800</bug_when>
    <thetext>Created attachment 126586
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554412</commentid>
    <comment_count>11</comment_count>
    <who name="Tony Chang">tony</who>
    <bug_when>2012-02-10 14:58:39 -0800</bug_when>
    <thetext>(In reply to comment #9)
&gt; The behavior that&apos;s changing here (for shared workers) can&apos;t be tested with the current DRT because it doesn&apos;t run multiple processes.

Please mention this in the ChangeLog (e.g., No new tests because ....).

&gt; I think a layout test for the dedicated worker case is possible, though it would involve changing layoutTestController.  Which I don&apos;t want to do right now on account of expediency.  (The only reason we&apos;re adding this bypass is to get the feature in front of developers ASAP.)
&gt; Though I&apos;ve been meaning to add a permissions test for a while.  Bug filed at http://crbug.com/113738.  Is that ok?

Having a bug for this is fine for now.  I would have filed it in webkit, but whatever works for you.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554428</commentid>
    <comment_count>12</comment_count>
      <attachid>126591</attachid>
    <who name="David Grogan">dgrogan</who>
    <bug_when>2012-02-10 15:12:20 -0800</bug_when>
    <thetext>Created attachment 126591
Patch for landing</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554512</commentid>
    <comment_count>13</comment_count>
      <attachid>126591</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-02-10 17:20:51 -0800</bug_when>
    <thetext>Comment on attachment 126591
Patch for landing

Clearing flags on attachment: 126591

Committed r107463: &lt;http://trac.webkit.org/changeset/107463&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>554513</commentid>
    <comment_count>14</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-02-10 17:20:58 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>126434</attachid>
            <date>2012-02-09 19:14:20 -0800</date>
            <delta_ts>2012-02-10 10:25:25 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-78316-20120209191417.patch</filename>
            <type>text/plain</type>
            <size>1642</size>
            <attacher name="David Grogan">dgrogan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2No
cm9taXVtL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCmluZGV4
IGMzODM3ZDFjZmZjNGY1Yjg1MDQ0MDFjY2IwNDJiZDE5NWNiMzM0OWUuLmFlYzM0MDdlOTEyNDA2
NjQwZWYyOTdjYmY2ODZkNzc5NjU1MzczNzIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvY2hy
b21pdW0vQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTMgQEAKKzIwMTItMDItMDkgIERhdmlkIEdyb2dhbiAgPGRncm9nYW5AY2hyb21p
dW0ub3JnPgorCisgICAgICAgIERpc2FibGUgSW5kZXhlZERCIHBlcm1pc3Npb25zIGNoZWNrIGZy
b20gd29ya2VycworICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/
aWQ9NzgzMTYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAg
ICAqIHNyYy9JREJGYWN0b3J5QmFja2VuZFByb3h5LmNwcDoKKyAgICAgICAgKFdlYktpdDo6SURC
RmFjdG9yeUJhY2tlbmRQcm94eTo6YWxsb3dJREJGcm9tV29ya2VyVGhyZWFkKToKKwogMjAxMi0w
Mi0wOCAgUmF5bW9uZCBMaXUgIDxyYXltb25kLmxpdUBpbnRlbC5jb20+CiAKICAgICAgICAgRml4
IGEgdHlwbyBmb3IgcjEwNzEyNQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9jaHJvbWl1bS9z
cmMvSURCRmFjdG9yeUJhY2tlbmRQcm94eS5jcHAgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3Ny
Yy9JREJGYWN0b3J5QmFja2VuZFByb3h5LmNwcAppbmRleCA5OWEwYWZhZTRhN2NjMDIxOGZmZGU0
ZTM1NTU4NjQ2NGNiZTRmODc1Li5iYzNiMzIxMDIyZWQ4YmQxMjlkYTM5OGFhY2NkYjRiYTY2Zjlk
MmM3IDEwMDc1NQotLS0gYS9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJGYWN0b3J5QmFj
a2VuZFByb3h5LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJGYWN0b3J5
QmFja2VuZFByb3h5LmNwcApAQCAtMTU3LDYgKzE1NywxMSBAQCBwcml2YXRlOgogCiBib29sIElE
QkZhY3RvcnlCYWNrZW5kUHJveHk6OmFsbG93SURCRnJvbVdvcmtlclRocmVhZChXb3JrZXJDb250
ZXh0KiB3b3JrZXJDb250ZXh0LCBjb25zdCBTdHJpbmcmIG5hbWUsIGNvbnN0IFdlYlNlY3VyaXR5
T3JpZ2luJikKIHsKKyAgICAvLyBGSVhNRTogQnlwYXNzIGNoZWNraW5nIGZvciBwZXJtaXNzaW9u
IHNvIGFzIG5vdCB0byBibG9jayBzaGFyZWQgd29ya2VyCisgICAgLy8gdGVzdGluZyB1bnRpbCBh
IHBlcm1pc3Npb25zIGNoZWNrIGlzIGltcGxlbWVudGVkLiBUaGlzIGhhcyB0byBiZSBmaXhlZAor
ICAgIC8vIGJlZm9yZSBtMTkgZ29lcyB0byBiZXRhLiBodHRwOi8vY3JidWcuY29tLzExMjg1NQor
ICAgIHJldHVybiB0cnVlOworCiAgICAgV2ViV29ya2VyQ2xpZW50SW1wbCogd2ViV29ya2VyQ2xp
ZW50SW1wbCA9IHN0YXRpY19jYXN0PFdlYldvcmtlckNsaWVudEltcGwqPigmd29ya2VyQ29udGV4
dC0+dGhyZWFkKCktPndvcmtlckxvYWRlclByb3h5KCkpOwogICAgIFdvcmtlclJ1bkxvb3AmIHJ1
bkxvb3AgPSB3b3JrZXJDb250ZXh0LT50aHJlYWQoKS0+cnVuTG9vcCgpOwogCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>126533</attachid>
            <date>2012-02-10 10:25:28 -0800</date>
            <delta_ts>2012-02-10 14:42:03 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-78316-20120210102527.patch</filename>
            <type>text/plain</type>
            <size>1702</size>
            <attacher name="David Grogan">dgrogan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2No
cm9taXVtL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCmluZGV4
IGMzODM3ZDFjZmZjNGY1Yjg1MDQ0MDFjY2IwNDJiZDE5NWNiMzM0OWUuLmFhYThkZDlkZjdhNTc4
NDFiOTM2MGUwYmVjODRiYzAzNmQ1Nzc0MDYgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvY2hy
b21pdW0vQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTMgQEAKKzIwMTItMDItMDkgIERhdmlkIEdyb2dhbiAgPGRncm9nYW5AY2hyb21p
dW0ub3JnPgorCisgICAgICAgIERpc2FibGUgSW5kZXhlZERCIHBlcm1pc3Npb25zIGNoZWNrIGZy
b20gc2hhcmVkIHdvcmtlcnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTc4MzE2CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisK
KyAgICAgICAgKiBzcmMvSURCRmFjdG9yeUJhY2tlbmRQcm94eS5jcHA6CisgICAgICAgIChXZWJL
aXQ6OklEQkZhY3RvcnlCYWNrZW5kUHJveHk6OmFsbG93SURCRnJvbVdvcmtlclRocmVhZCk6CisK
IDIwMTItMDItMDggIFJheW1vbmQgTGl1ICA8cmF5bW9uZC5saXVAaW50ZWwuY29tPgogCiAgICAg
ICAgIEZpeCBhIHR5cG8gZm9yIHIxMDcxMjUKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQvY2hy
b21pdW0vc3JjL0lEQkZhY3RvcnlCYWNrZW5kUHJveHkuY3BwIGIvU291cmNlL1dlYktpdC9jaHJv
bWl1bS9zcmMvSURCRmFjdG9yeUJhY2tlbmRQcm94eS5jcHAKaW5kZXggOTlhMGFmYWU0YTdjYzAy
MThmZmRlNGUzNTU1ODY0NjRjYmU0Zjg3NS4uNDI5NTM0MDI2NzJmYzMzMzk0ZmYyOWIxYzYyNDRh
YjA5ZGE4MmY3MiAxMDA3NTUKLS0tIGEvU291cmNlL1dlYktpdC9jaHJvbWl1bS9zcmMvSURCRmFj
dG9yeUJhY2tlbmRQcm94eS5jcHAKKysrIGIvU291cmNlL1dlYktpdC9jaHJvbWl1bS9zcmMvSURC
RmFjdG9yeUJhY2tlbmRQcm94eS5jcHAKQEAgLTE1Nyw2ICsxNTcsMTIgQEAgcHJpdmF0ZToKIAog
Ym9vbCBJREJGYWN0b3J5QmFja2VuZFByb3h5OjphbGxvd0lEQkZyb21Xb3JrZXJUaHJlYWQoV29y
a2VyQ29udGV4dCogd29ya2VyQ29udGV4dCwgY29uc3QgU3RyaW5nJiBuYW1lLCBjb25zdCBXZWJT
ZWN1cml0eU9yaWdpbiYpCiB7CisgICAgLy8gRklYTUU6IEJ5cGFzcyBjaGVja2luZyBmb3IgcGVy
bWlzc2lvbiBzbyBhcyBub3QgdG8gYmxvY2sgc2hhcmVkIHdvcmtlcgorICAgIC8vIHRlc3Rpbmcg
dW50aWwgYSBwZXJtaXNzaW9ucyBjaGVjayBpcyBpbXBsZW1lbnRlZC4gVGhpcyBoYXMgdG8gYmUg
Zml4ZWQKKyAgICAvLyBiZWZvcmUgbTE5IGdvZXMgdG8gYmV0YS4gaHR0cDovL2NyYnVnLmNvbS8x
MTI4NTUKKyAgICBpZiAod29ya2VyQ29udGV4dC0+aXNTaGFyZWRXb3JrZXJDb250ZXh0KCkpCisg
ICAgICAgIHJldHVybiB0cnVlOworCiAgICAgV2ViV29ya2VyQ2xpZW50SW1wbCogd2ViV29ya2Vy
Q2xpZW50SW1wbCA9IHN0YXRpY19jYXN0PFdlYldvcmtlckNsaWVudEltcGwqPigmd29ya2VyQ29u
dGV4dC0+dGhyZWFkKCktPndvcmtlckxvYWRlclByb3h5KCkpOwogICAgIFdvcmtlclJ1bkxvb3Am
IHJ1bkxvb3AgPSB3b3JrZXJDb250ZXh0LT50aHJlYWQoKS0+cnVuTG9vcCgpOwogCg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>126586</attachid>
            <date>2012-02-10 14:42:07 -0800</date>
            <delta_ts>2012-02-10 15:12:16 -0800</delta_ts>
            <desc>Patch</desc>
            <filename>bug-78316-20120210144206.patch</filename>
            <type>text/plain</type>
            <size>1989</size>
            <attacher name="David Grogan">dgrogan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2No
cm9taXVtL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCmluZGV4
IGMzODM3ZDFjZmZjNGY1Yjg1MDQ0MDFjY2IwNDJiZDE5NWNiMzM0OWUuLmY2YjRiZDMzN2VjY2Rl
MjQwY2NjZWM2NzUwMGFjMzU0N2YwOGIxMTcgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvY2hy
b21pdW0vQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMTggQEAKKzIwMTItMDItMDkgIERhdmlkIEdyb2dhbiAgPGRncm9nYW5AY2hyb21p
dW0ub3JnPgorCisgICAgICAgIERpc2FibGUgSW5kZXhlZERCIHBlcm1pc3Npb25zIGNoZWNrIGZy
b20gc2hhcmVkIHdvcmtlcnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTc4MzE2CisKKyAgICAgICAgV2Ugd2FudCB0byB0ZW1wb3JhcmlseSBieXBhc3Mg
dGhpcyBjaGVjayBiZWNhdXNlIGl0IGNhdXNlcyBhIGNyYXNoCisgICAgICAgIHdoZW4gY2FsbGVk
IGZyb20gYSBzaGFyZWQgd29ya2VyLiAgV2UnbGwgaGF2ZSB0byBmaXggaXQgYmVmb3JlCisgICAg
ICAgIGluY2x1ZGluZyBpZGIgb24gc2hhcmVkIHdvcmtlcnMgaW4gYSByZWxlYXNlIGJ1dCBkb24n
dCB3YW50IHRvIHN0b3AKKyAgICAgICAgZGV2ZWxvcGVycyBmcm9tIHRlc3Rpbmcgb3V0IGlkYiBv
biBzaGFyZWQgd29ya2Vycy4KKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K
KworICAgICAgICAqIHNyYy9JREJGYWN0b3J5QmFja2VuZFByb3h5LmNwcDoKKyAgICAgICAgKFdl
YktpdDo6SURCRmFjdG9yeUJhY2tlbmRQcm94eTo6YWxsb3dJREJGcm9tV29ya2VyVGhyZWFkKToK
KwogMjAxMi0wMi0wOCAgUmF5bW9uZCBMaXUgIDxyYXltb25kLmxpdUBpbnRlbC5jb20+CiAKICAg
ICAgICAgRml4IGEgdHlwbyBmb3IgcjEwNzEyNQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9j
aHJvbWl1bS9zcmMvSURCRmFjdG9yeUJhY2tlbmRQcm94eS5jcHAgYi9Tb3VyY2UvV2ViS2l0L2No
cm9taXVtL3NyYy9JREJGYWN0b3J5QmFja2VuZFByb3h5LmNwcAppbmRleCA5OWEwYWZhZTRhN2Nj
MDIxOGZmZGU0ZTM1NTU4NjQ2NGNiZTRmODc1Li40Mjk1MzQwMjY3MmZjMzMzOTRmZjI5YjFjNjI0
NGFiMDlkYTgyZjcyIDEwMDc1NQotLS0gYS9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJG
YWN0b3J5QmFja2VuZFByb3h5LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9J
REJGYWN0b3J5QmFja2VuZFByb3h5LmNwcApAQCAtMTU3LDYgKzE1NywxMiBAQCBwcml2YXRlOgog
CiBib29sIElEQkZhY3RvcnlCYWNrZW5kUHJveHk6OmFsbG93SURCRnJvbVdvcmtlclRocmVhZChX
b3JrZXJDb250ZXh0KiB3b3JrZXJDb250ZXh0LCBjb25zdCBTdHJpbmcmIG5hbWUsIGNvbnN0IFdl
YlNlY3VyaXR5T3JpZ2luJikKIHsKKyAgICAvLyBGSVhNRTogQnlwYXNzIGNoZWNraW5nIGZvciBw
ZXJtaXNzaW9uIHNvIGFzIG5vdCB0byBibG9jayBzaGFyZWQgd29ya2VyCisgICAgLy8gdGVzdGlu
ZyB1bnRpbCBhIHBlcm1pc3Npb25zIGNoZWNrIGlzIGltcGxlbWVudGVkLiBUaGlzIGhhcyB0byBi
ZSBmaXhlZAorICAgIC8vIGJlZm9yZSBtMTkgZ29lcyB0byBiZXRhLiBodHRwOi8vY3JidWcuY29t
LzExMjg1NQorICAgIGlmICh3b3JrZXJDb250ZXh0LT5pc1NoYXJlZFdvcmtlckNvbnRleHQoKSkK
KyAgICAgICAgcmV0dXJuIHRydWU7CisKICAgICBXZWJXb3JrZXJDbGllbnRJbXBsKiB3ZWJXb3Jr
ZXJDbGllbnRJbXBsID0gc3RhdGljX2Nhc3Q8V2ViV29ya2VyQ2xpZW50SW1wbCo+KCZ3b3JrZXJD
b250ZXh0LT50aHJlYWQoKS0+d29ya2VyTG9hZGVyUHJveHkoKSk7CiAgICAgV29ya2VyUnVuTG9v
cCYgcnVuTG9vcCA9IHdvcmtlckNvbnRleHQtPnRocmVhZCgpLT5ydW5Mb29wKCk7CiAK
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>126591</attachid>
            <date>2012-02-10 15:12:20 -0800</date>
            <delta_ts>2012-02-10 17:20:51 -0800</delta_ts>
            <desc>Patch for landing</desc>
            <filename>bug-78316-20120210151220.patch</filename>
            <type>text/plain</type>
            <size>2088</size>
            <attacher name="David Grogan">dgrogan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTA3MjE3CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0L2No
cm9taXVtL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCmluZGV4
IGMzODM3ZDFjZmZjNGY1Yjg1MDQ0MDFjY2IwNDJiZDE5NWNiMzM0OWUuLjhhMTA1MWViODBkNzA0
YzczYTRhZDRhMzI1NjZhMjc3OTQ1NWJiMWMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQvY2hy
b21pdW0vQ2hhbmdlTG9nCisrKyBiL1NvdXJjZS9XZWJLaXQvY2hyb21pdW0vQ2hhbmdlTG9nCkBA
IC0xLDMgKzEsMjEgQEAKKzIwMTItMDItMDkgIERhdmlkIEdyb2dhbiAgPGRncm9nYW5AY2hyb21p
dW0ub3JnPgorCisgICAgICAgIERpc2FibGUgSW5kZXhlZERCIHBlcm1pc3Npb25zIGNoZWNrIGZy
b20gc2hhcmVkIHdvcmtlcnMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTc4MzE2CisKKyAgICAgICAgV2Ugd2FudCB0byB0ZW1wb3JhcmlseSBieXBhc3Mg
dGhpcyBjaGVjayBiZWNhdXNlIGl0IGNhdXNlcyBhIGNyYXNoCisgICAgICAgIHdoZW4gY2FsbGVk
IGZyb20gYSBzaGFyZWQgd29ya2VyLiAgV2UnbGwgaGF2ZSB0byBmaXggaXQgYmVmb3JlCisgICAg
ICAgIGluY2x1ZGluZyBpZGIgb24gc2hhcmVkIHdvcmtlcnMgaW4gYSByZWxlYXNlIGJ1dCBkb24n
dCB3YW50IHRvIHN0b3AKKyAgICAgICAgZGV2ZWxvcGVycyBmcm9tIHRlc3Rpbmcgb3V0IGlkYiBv
biBzaGFyZWQgd29ya2Vycy4KKworICAgICAgICBObyBuZXcgdGVzdHMgYmVjYXVzZSBpdCBhZmZl
Y3RzIHNoYXJlZCB3b3JrZXJzIG9ubHksIHdoaWNoIGFyZW4ndAorICAgICAgICB0ZXN0YWJsZSBi
eSBEUlQuCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgVG9ueSBDaGFuZy4KKworICAgICAgICAqIHNy
Yy9JREJGYWN0b3J5QmFja2VuZFByb3h5LmNwcDoKKyAgICAgICAgKFdlYktpdDo6SURCRmFjdG9y
eUJhY2tlbmRQcm94eTo6YWxsb3dJREJGcm9tV29ya2VyVGhyZWFkKToKKwogMjAxMi0wMi0wOCAg
UmF5bW9uZCBMaXUgIDxyYXltb25kLmxpdUBpbnRlbC5jb20+CiAKICAgICAgICAgRml4IGEgdHlw
byBmb3IgcjEwNzEyNQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdC9jaHJvbWl1bS9zcmMvSURC
RmFjdG9yeUJhY2tlbmRQcm94eS5jcHAgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJG
YWN0b3J5QmFja2VuZFByb3h5LmNwcAppbmRleCA5OWEwYWZhZTRhN2NjMDIxOGZmZGU0ZTM1NTU4
NjQ2NGNiZTRmODc1Li40Mjk1MzQwMjY3MmZjMzMzOTRmZjI5YjFjNjI0NGFiMDlkYTgyZjcyIDEw
MDc1NQotLS0gYS9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJGYWN0b3J5QmFja2VuZFBy
b3h5LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0L2Nocm9taXVtL3NyYy9JREJGYWN0b3J5QmFja2Vu
ZFByb3h5LmNwcApAQCAtMTU3LDYgKzE1NywxMiBAQCBwcml2YXRlOgogCiBib29sIElEQkZhY3Rv
cnlCYWNrZW5kUHJveHk6OmFsbG93SURCRnJvbVdvcmtlclRocmVhZChXb3JrZXJDb250ZXh0KiB3
b3JrZXJDb250ZXh0LCBjb25zdCBTdHJpbmcmIG5hbWUsIGNvbnN0IFdlYlNlY3VyaXR5T3JpZ2lu
JikKIHsKKyAgICAvLyBGSVhNRTogQnlwYXNzIGNoZWNraW5nIGZvciBwZXJtaXNzaW9uIHNvIGFz
IG5vdCB0byBibG9jayBzaGFyZWQgd29ya2VyCisgICAgLy8gdGVzdGluZyB1bnRpbCBhIHBlcm1p
c3Npb25zIGNoZWNrIGlzIGltcGxlbWVudGVkLiBUaGlzIGhhcyB0byBiZSBmaXhlZAorICAgIC8v
IGJlZm9yZSBtMTkgZ29lcyB0byBiZXRhLiBodHRwOi8vY3JidWcuY29tLzExMjg1NQorICAgIGlm
ICh3b3JrZXJDb250ZXh0LT5pc1NoYXJlZFdvcmtlckNvbnRleHQoKSkKKyAgICAgICAgcmV0dXJu
IHRydWU7CisKICAgICBXZWJXb3JrZXJDbGllbnRJbXBsKiB3ZWJXb3JrZXJDbGllbnRJbXBsID0g
c3RhdGljX2Nhc3Q8V2ViV29ya2VyQ2xpZW50SW1wbCo+KCZ3b3JrZXJDb250ZXh0LT50aHJlYWQo
KS0+d29ya2VyTG9hZGVyUHJveHkoKSk7CiAgICAgV29ya2VyUnVuTG9vcCYgcnVuTG9vcCA9IHdv
cmtlckNvbnRleHQtPnRocmVhZCgpLT5ydW5Mb29wKCk7CiAK
</data>

          </attachment>
      

    </bug>

</bugzilla>