<?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>132805</bug_id>
          
          <creation_ts>2014-05-11 13:01:52 -0700</creation_ts>
          <short_desc>Need function to copy favicon data without image conversion.</short_desc>
          <delta_ts>2014-05-12 00:53:06 -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>WebKit2</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>P1</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Gordon Sheridan">gordon_sheridan</reporter>
          <assigned_to name="Gordon Sheridan">gordon_sheridan</assigned_to>
          <cc>commit-queue</cc>
    
    <cc>darin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1008441</commentid>
    <comment_count>0</comment_count>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 13:01:52 -0700</bug_when>
    <thetext>With WKIconDatabaseSetIconDataForIconURL(), we have a function that can *set* the data for an icon in the destination database.

In order to copy a favicon between icon databases, without potential image conversion, we need a function that can get a copy of the data for an icon in the source database.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008442</commentid>
    <comment_count>1</comment_count>
      <attachid>231264</attachid>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 13:25:01 -0700</bug_when>
    <thetext>Created attachment 231264
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008444</commentid>
    <comment_count>2</comment_count>
      <attachid>231264</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-05-11 13:29:16 -0700</bug_when>
    <thetext>Comment on attachment 231264
Patch

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

&gt; Source/WebKit2/UIProcess/API/C/WKIconDatabase.cpp:77
&gt; +    WebCore::Image *image = toImpl(iconDatabaseRef)-&gt;imageForPageURL(toWTFString(pageURL));

Formatted wrong. Should be Image* rather than Image * with a space.

&gt; Source/WebKit2/UIProcess/API/C/WKIconDatabase.cpp:79
&gt; +        return 0;

Should be nullptr.

&gt; Source/WebKit2/UIProcess/API/C/WKIconDatabase.cpp:81
&gt; +    WebCore::SharedBuffer *iconData = image-&gt;data();

Formatted wrong. Should be SharedBuffer* rather than SharedBuffer * with a space.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008446</commentid>
    <comment_count>3</comment_count>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 13:43:05 -0700</bug_when>
    <thetext>Thanks Darin!

I&apos;ll update it and post a new patch.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008448</commentid>
    <comment_count>4</comment_count>
      <attachid>231265</attachid>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 13:50:07 -0700</bug_when>
    <thetext>Created attachment 231265
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008454</commentid>
    <comment_count>5</comment_count>
      <attachid>231265</attachid>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 15:00:43 -0700</bug_when>
    <thetext>Comment on attachment 231265
Patch

I need to update the patch handle when there is no image data.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008459</commentid>
    <comment_count>6</comment_count>
      <attachid>231268</attachid>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 15:27:20 -0700</bug_when>
    <thetext>Created attachment 231268
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008485</commentid>
    <comment_count>7</comment_count>
      <attachid>231268</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-05-11 23:31:09 -0700</bug_when>
    <thetext>Comment on attachment 231268
Patch

Clearing flags on attachment: 231268

Committed r168609: &lt;http://trac.webkit.org/changeset/168609&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008486</commentid>
    <comment_count>8</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2014-05-11 23:31:12 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008487</commentid>
    <comment_count>9</comment_count>
      <attachid>231268</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-05-11 23:39:24 -0700</bug_when>
    <thetext>Comment on attachment 231268
Patch

There is one thing wrong with this patch that I did not notice until after it was landed.

Generally, API files are not supposed to have substantive code in them. They just have bindings.

So this patch should have added a copyIconDataForPageURL member function to WebIconDatabase, rather than putting the code into the bindings file itself. Then the bindings file would just have the binding.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008488</commentid>
    <comment_count>10</comment_count>
    <who name="Gordon Sheridan">gordon_sheridan</who>
    <bug_when>2014-05-11 23:41:25 -0700</bug_when>
    <thetext>I can submit a new patch. Can this bug be reopened for that, or should I create a new bug?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1008501</commentid>
    <comment_count>11</comment_count>
    <who name="Darin Adler">darin</who>
    <bug_when>2014-05-12 00:53:06 -0700</bug_when>
    <thetext>(In reply to comment #10)
&gt; I can submit a new patch. Can this bug be reopened for that, or should I create a new bug?

I took care of it and did some other icon database improvements. It’s in bug 132812.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>231264</attachid>
            <date>2014-05-11 13:25:01 -0700</date>
            <delta_ts>2014-05-11 13:50:05 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-132805-20140511132438.patch</filename>
            <type>text/plain</type>
            <size>3080</size>
            <attacher name="Gordon Sheridan">gordon_sheridan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTY4NTk5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYzY2N2FjNTZmYmYzMzA2
NjBjY2NjMzdjMmM5OWRjNDNjNzVjY2JlNC4uOTFhZTJjZmY1MjYyZGZlYmZhZjgyZTQyNWQ5ZThk
MjAzNjVjY2ViNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE0LTA1LTExICBHb3Jk
b24gU2hlcmlkYW4gIDxnb3Jkb25fc2hlcmlkYW5AYXBwbGUuY29tPgorCisgICAgICAgIEV4cG9z
ZSBhIGZ1bmN0aW9uIHRvIGNvcHkgZmF2aWNvbiBkYXRhIHdpdGhvdXQgaW1hZ2UgY29udmVyc2lv
bi4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMjgw
NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVUlQ
cm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmNwcDoKKyAgICAgICAgKFdLSWNvbkRhdGFiYXNl
Q29weUljb25EYXRhRm9yUGFnZVVSTCk6CisgICAgICAgIEFkZCBhIGZ1bmN0aW9uIHRvIGNvcHkg
dGhlIGRhdGEgZnJvbSB0aGUgaW1hZ2Ugb2YgYSBmYXZpY29uIGZvciBhbiBhc3NvY2lhdGVkIHBh
Z2UgdXJsLgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmg6Cisg
ICAgICAgIERlY2xhcmUgV0tJY29uRGF0YWJhc2VDb3B5SWNvbkRhdGFGb3JQYWdlVVJMLgorCiAy
MDE0LTA1LTEwICBDc2FiYSBPc3p0cm9nb27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CiAKICAgICAg
ICAgW1dLMl0gVVJUQkYgYWZ0ZXIgcjE2ODU4NS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25EYXRhYmFzZS5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0MvV0tJY29uRGF0YWJhc2UuY3BwCmluZGV4IDE0MmQxMmRhNjJkOGU0OGJiZmRk
OGRjM2UyNjI2MGQyMmQ1ZjA2MDguLjk5NDQ4Y2E1ZGIwZDc2MGI5M2YxMWMwZTM3OGI3YWY0MDNm
ZjY1NTIgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25E
YXRhYmFzZS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRh
dGFiYXNlLmNwcApAQCAtMjksNiArMjksOSBAQAogI2luY2x1ZGUgIkFQSURhdGEuaCIKICNpbmNs
dWRlICJXS0FQSUNhc3QuaCIKICNpbmNsdWRlICJXZWJJY29uRGF0YWJhc2UuaCIKKyNpbmNsdWRl
IDxXZWJDb3JlL0ltYWdlLmg+CisjaW5jbHVkZSA8V2ViQ29yZS9TaGFyZWRCdWZmZXIuaD4KKyNp
bmNsdWRlIDxXZWJLaXQvV0tEYXRhLmg+CiAKIHVzaW5nIG5hbWVzcGFjZSBXZWJLaXQ7CiAKQEAg
LTY5LDYgKzcyLDE2IEBAIFdLVVJMUmVmIFdLSWNvbkRhdGFiYXNlQ29weUljb25VUkxGb3JQYWdl
VVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZVJlZiwKICAgICByZXR1cm4gdG9Db3Bp
ZWRVUkxBUEkoaWNvblVSTFN0cmluZyk7CiB9CiAKK1dLRGF0YVJlZiBXS0ljb25EYXRhYmFzZUNv
cHlJY29uRGF0YUZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmLCBX
S1VSTFJlZiBwYWdlVVJMKQoreworICAgIFdlYkNvcmU6OkltYWdlICppbWFnZSA9IHRvSW1wbChp
Y29uRGF0YWJhc2VSZWYpLT5pbWFnZUZvclBhZ2VVUkwodG9XVEZTdHJpbmcocGFnZVVSTCkpOwor
ICAgIGlmICghaW1hZ2UpCisgICAgICAgIHJldHVybiAwOworCisgICAgV2ViQ29yZTo6U2hhcmVk
QnVmZmVyICppY29uRGF0YSA9IGltYWdlLT5kYXRhKCk7CisgICAgcmV0dXJuIFdLRGF0YUNyZWF0
ZShyZWludGVycHJldF9jYXN0PGNvbnN0IHVuc2lnbmVkIGNoYXIqPihpY29uRGF0YS0+ZGF0YSgp
KSwgaWNvbkRhdGEtPnNpemUoKSk7Cit9CisKIHZvaWQgV0tJY29uRGF0YWJhc2VFbmFibGVEYXRh
YmFzZUNsZWFudXAoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmKQogewogICAgIHRv
SW1wbChpY29uRGF0YWJhc2VSZWYpLT5lbmFibGVEYXRhYmFzZUNsZWFudXAoKTsKZGlmZiAtLWdp
dCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25EYXRhYmFzZS5oIGIvU291
cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmgKaW5kZXggZDViNzI5
MmI0ZDliNGEzNTA5M2UyMzk4Y2I1ODE4Yzc5Yzg3NTg2ZC4uOTdkYzQxYzZkODA2OGFiMDdhMWY4
ZDE3ZGIzMDY5MTJmOGQ5Mjk0NCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQcm9jZXNz
L0FQSS9DL1dLSWNvbkRhdGFiYXNlLmgKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQ
SS9DL1dLSWNvbkRhdGFiYXNlLmgKQEAgLTgzLDYgKzgzLDcgQEAgV0tfRVhQT1JUIHZvaWQgV0tJ
Y29uRGF0YWJhc2VSZWxlYXNlSWNvbkZvclVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJh
c2UsIFcKIFdLX0VYUE9SVCB2b2lkIFdLSWNvbkRhdGFiYXNlU2V0SWNvbkRhdGFGb3JJY29uVVJM
KFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZSwgV0tEYXRhUmVmIGljb25EYXRhLCBXS1VS
TFJlZiBpY29uVVJMKTsKIFdLX0VYUE9SVCB2b2lkIFdLSWNvbkRhdGFiYXNlU2V0SWNvblVSTEZv
clBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmLCBXS1VSTFJlZiBpY29u
VVJMUmVmLCBXS1VSTFJlZiBwYWdlVVJMUmVmKTsKIFdLX0VYUE9SVCBXS1VSTFJlZiBXS0ljb25E
YXRhYmFzZUNvcHlJY29uVVJMRm9yUGFnZVVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJh
c2VSZWYsIFdLVVJMUmVmIHBhZ2VVUkxSZWYpOworV0tfRVhQT1JUIFdLRGF0YVJlZiBXS0ljb25E
YXRhYmFzZUNvcHlJY29uRGF0YUZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFi
YXNlUmVmLCBXS1VSTFJlZiBwYWdlVVJMKTsKIAogV0tfRVhQT1JUIHZvaWQgV0tJY29uRGF0YWJh
c2VFbmFibGVEYXRhYmFzZUNsZWFudXAoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlKTsK
IAo=
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>231265</attachid>
            <date>2014-05-11 13:50:07 -0700</date>
            <delta_ts>2014-05-11 15:27:18 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-132805-20140511134944.patch</filename>
            <type>text/plain</type>
            <size>3086</size>
            <attacher name="Gordon Sheridan">gordon_sheridan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTY4NTk5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYzY2N2FjNTZmYmYzMzA2
NjBjY2NjMzdjMmM5OWRjNDNjNzVjY2JlNC4uOTFhZTJjZmY1MjYyZGZlYmZhZjgyZTQyNWQ5ZThk
MjAzNjVjY2ViNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE0LTA1LTExICBHb3Jk
b24gU2hlcmlkYW4gIDxnb3Jkb25fc2hlcmlkYW5AYXBwbGUuY29tPgorCisgICAgICAgIEV4cG9z
ZSBhIGZ1bmN0aW9uIHRvIGNvcHkgZmF2aWNvbiBkYXRhIHdpdGhvdXQgaW1hZ2UgY29udmVyc2lv
bi4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMjgw
NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVUlQ
cm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmNwcDoKKyAgICAgICAgKFdLSWNvbkRhdGFiYXNl
Q29weUljb25EYXRhRm9yUGFnZVVSTCk6CisgICAgICAgIEFkZCBhIGZ1bmN0aW9uIHRvIGNvcHkg
dGhlIGRhdGEgZnJvbSB0aGUgaW1hZ2Ugb2YgYSBmYXZpY29uIGZvciBhbiBhc3NvY2lhdGVkIHBh
Z2UgdXJsLgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmg6Cisg
ICAgICAgIERlY2xhcmUgV0tJY29uRGF0YWJhc2VDb3B5SWNvbkRhdGFGb3JQYWdlVVJMLgorCiAy
MDE0LTA1LTEwICBDc2FiYSBPc3p0cm9nb27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CiAKICAgICAg
ICAgW1dLMl0gVVJUQkYgYWZ0ZXIgcjE2ODU4NS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25EYXRhYmFzZS5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0MvV0tJY29uRGF0YWJhc2UuY3BwCmluZGV4IDE0MmQxMmRhNjJkOGU0OGJiZmRk
OGRjM2UyNjI2MGQyMmQ1ZjA2MDguLmNkMjNjNWRiNDk1ZTE3OWMyNmIxZWYzZmY1MjUzNGMwODIx
MGM2ZDMgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25E
YXRhYmFzZS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRh
dGFiYXNlLmNwcApAQCAtMjksNiArMjksOSBAQAogI2luY2x1ZGUgIkFQSURhdGEuaCIKICNpbmNs
dWRlICJXS0FQSUNhc3QuaCIKICNpbmNsdWRlICJXZWJJY29uRGF0YWJhc2UuaCIKKyNpbmNsdWRl
IDxXZWJDb3JlL0ltYWdlLmg+CisjaW5jbHVkZSA8V2ViQ29yZS9TaGFyZWRCdWZmZXIuaD4KKyNp
bmNsdWRlIDxXZWJLaXQvV0tEYXRhLmg+CiAKIHVzaW5nIG5hbWVzcGFjZSBXZWJLaXQ7CiAKQEAg
LTY5LDYgKzcyLDE2IEBAIFdLVVJMUmVmIFdLSWNvbkRhdGFiYXNlQ29weUljb25VUkxGb3JQYWdl
VVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZVJlZiwKICAgICByZXR1cm4gdG9Db3Bp
ZWRVUkxBUEkoaWNvblVSTFN0cmluZyk7CiB9CiAKK1dLRGF0YVJlZiBXS0ljb25EYXRhYmFzZUNv
cHlJY29uRGF0YUZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmLCBX
S1VSTFJlZiBwYWdlVVJMKQoreworICAgIFdlYkNvcmU6OkltYWdlKiBpbWFnZSA9IHRvSW1wbChp
Y29uRGF0YWJhc2VSZWYpLT5pbWFnZUZvclBhZ2VVUkwodG9XVEZTdHJpbmcocGFnZVVSTCkpOwor
ICAgIGlmICghaW1hZ2UpCisgICAgICAgIHJldHVybiBudWxscHRyOworCisgICAgV2ViQ29yZTo6
U2hhcmVkQnVmZmVyKiBpY29uRGF0YSA9IGltYWdlLT5kYXRhKCk7CisgICAgcmV0dXJuIFdLRGF0
YUNyZWF0ZShyZWludGVycHJldF9jYXN0PGNvbnN0IHVuc2lnbmVkIGNoYXIqPihpY29uRGF0YS0+
ZGF0YSgpKSwgaWNvbkRhdGEtPnNpemUoKSk7Cit9CisKIHZvaWQgV0tJY29uRGF0YWJhc2VFbmFi
bGVEYXRhYmFzZUNsZWFudXAoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmKQogewog
ICAgIHRvSW1wbChpY29uRGF0YWJhc2VSZWYpLT5lbmFibGVEYXRhYmFzZUNsZWFudXAoKTsKZGlm
ZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25EYXRhYmFzZS5o
IGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmgKaW5kZXgg
ZDViNzI5MmI0ZDliNGEzNTA5M2UyMzk4Y2I1ODE4Yzc5Yzg3NTg2ZC4uOTdkYzQxYzZkODA2OGFi
MDdhMWY4ZDE3ZGIzMDY5MTJmOGQ5Mjk0NCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvVUlQ
cm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmgKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9j
ZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmgKQEAgLTgzLDYgKzgzLDcgQEAgV0tfRVhQT1JUIHZv
aWQgV0tJY29uRGF0YWJhc2VSZWxlYXNlSWNvbkZvclVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29u
RGF0YWJhc2UsIFcKIFdLX0VYUE9SVCB2b2lkIFdLSWNvbkRhdGFiYXNlU2V0SWNvbkRhdGFGb3JJ
Y29uVVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZSwgV0tEYXRhUmVmIGljb25EYXRh
LCBXS1VSTFJlZiBpY29uVVJMKTsKIFdLX0VYUE9SVCB2b2lkIFdLSWNvbkRhdGFiYXNlU2V0SWNv
blVSTEZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmLCBXS1VSTFJl
ZiBpY29uVVJMUmVmLCBXS1VSTFJlZiBwYWdlVVJMUmVmKTsKIFdLX0VYUE9SVCBXS1VSTFJlZiBX
S0ljb25EYXRhYmFzZUNvcHlJY29uVVJMRm9yUGFnZVVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29u
RGF0YWJhc2VSZWYsIFdLVVJMUmVmIHBhZ2VVUkxSZWYpOworV0tfRVhQT1JUIFdLRGF0YVJlZiBX
S0ljb25EYXRhYmFzZUNvcHlJY29uRGF0YUZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNv
bkRhdGFiYXNlUmVmLCBXS1VSTFJlZiBwYWdlVVJMKTsKIAogV0tfRVhQT1JUIHZvaWQgV0tJY29u
RGF0YWJhc2VFbmFibGVEYXRhYmFzZUNsZWFudXAoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFi
YXNlKTsKIAo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>231268</attachid>
            <date>2014-05-11 15:27:20 -0700</date>
            <delta_ts>2014-05-11 23:39:24 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-132805-20140511152657.patch</filename>
            <type>text/plain</type>
            <size>3133</size>
            <attacher name="Gordon Sheridan">gordon_sheridan</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTY4NTk5CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggYzY2N2FjNTZmYmYzMzA2
NjBjY2NjMzdjMmM5OWRjNDNjNzVjY2JlNC4uOTFhZTJjZmY1MjYyZGZlYmZhZjgyZTQyNWQ5ZThk
MjAzNjVjY2ViNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDE0LTA1LTExICBHb3Jk
b24gU2hlcmlkYW4gIDxnb3Jkb25fc2hlcmlkYW5AYXBwbGUuY29tPgorCisgICAgICAgIEV4cG9z
ZSBhIGZ1bmN0aW9uIHRvIGNvcHkgZmF2aWNvbiBkYXRhIHdpdGhvdXQgaW1hZ2UgY29udmVyc2lv
bi4KKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzMjgw
NQorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogVUlQ
cm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmNwcDoKKyAgICAgICAgKFdLSWNvbkRhdGFiYXNl
Q29weUljb25EYXRhRm9yUGFnZVVSTCk6CisgICAgICAgIEFkZCBhIGZ1bmN0aW9uIHRvIGNvcHkg
dGhlIGRhdGEgZnJvbSB0aGUgaW1hZ2Ugb2YgYSBmYXZpY29uIGZvciBhbiBhc3NvY2lhdGVkIHBh
Z2UgdXJsLgorCisgICAgICAgICogVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmg6Cisg
ICAgICAgIERlY2xhcmUgV0tJY29uRGF0YWJhc2VDb3B5SWNvbkRhdGFGb3JQYWdlVVJMLgorCiAy
MDE0LTA1LTEwICBDc2FiYSBPc3p0cm9nb27DoWMgIDxvc3N5QHdlYmtpdC5vcmc+CiAKICAgICAg
ICAgW1dLMl0gVVJUQkYgYWZ0ZXIgcjE2ODU4NS4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJLaXQy
L1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25EYXRhYmFzZS5jcHAgYi9Tb3VyY2UvV2ViS2l0Mi9VSVBy
b2Nlc3MvQVBJL0MvV0tJY29uRGF0YWJhc2UuY3BwCmluZGV4IDE0MmQxMmRhNjJkOGU0OGJiZmRk
OGRjM2UyNjI2MGQyMmQ1ZjA2MDguLmE3Y2Y2MWU5OTRiOGE3MjlmZTQwMTc5NTdlMjk5ZDIzYzM1
YmNjY2UgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1VJUHJvY2Vzcy9BUEkvQy9XS0ljb25E
YXRhYmFzZS5jcHAKKysrIGIvU291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRh
dGFiYXNlLmNwcApAQCAtMjksNiArMjksOSBAQAogI2luY2x1ZGUgIkFQSURhdGEuaCIKICNpbmNs
dWRlICJXS0FQSUNhc3QuaCIKICNpbmNsdWRlICJXZWJJY29uRGF0YWJhc2UuaCIKKyNpbmNsdWRl
IDxXZWJDb3JlL0ltYWdlLmg+CisjaW5jbHVkZSA8V2ViQ29yZS9TaGFyZWRCdWZmZXIuaD4KKyNp
bmNsdWRlIDxXZWJLaXQvV0tEYXRhLmg+CiAKIHVzaW5nIG5hbWVzcGFjZSBXZWJLaXQ7CiAKQEAg
LTY5LDYgKzcyLDE5IEBAIFdLVVJMUmVmIFdLSWNvbkRhdGFiYXNlQ29weUljb25VUkxGb3JQYWdl
VVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZVJlZiwKICAgICByZXR1cm4gdG9Db3Bp
ZWRVUkxBUEkoaWNvblVSTFN0cmluZyk7CiB9CiAKK1dLRGF0YVJlZiBXS0ljb25EYXRhYmFzZUNv
cHlJY29uRGF0YUZvclBhZ2VVUkwoV0tJY29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlUmVmLCBX
S1VSTFJlZiBwYWdlVVJMKQoreworICAgIFdlYkNvcmU6OkltYWdlKiBpbWFnZSA9IHRvSW1wbChp
Y29uRGF0YWJhc2VSZWYpLT5pbWFnZUZvclBhZ2VVUkwodG9XVEZTdHJpbmcocGFnZVVSTCkpOwor
ICAgIGlmICghaW1hZ2UpCisgICAgICAgIHJldHVybiBudWxscHRyOworCisgICAgV2ViQ29yZTo6
U2hhcmVkQnVmZmVyKiBpY29uRGF0YSA9IGltYWdlLT5kYXRhKCk7CisgICAgaWYgKCFpY29uRGF0
YSkKKyAgICAgICAgcmV0dXJuIG51bGxwdHI7CisKKyAgICByZXR1cm4gV0tEYXRhQ3JlYXRlKHJl
aW50ZXJwcmV0X2Nhc3Q8Y29uc3QgdW5zaWduZWQgY2hhcio+KGljb25EYXRhLT5kYXRhKCkpLCBp
Y29uRGF0YS0+c2l6ZSgpKTsKK30KKwogdm9pZCBXS0ljb25EYXRhYmFzZUVuYWJsZURhdGFiYXNl
Q2xlYW51cChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJhc2VSZWYpCiB7CiAgICAgdG9JbXBs
KGljb25EYXRhYmFzZVJlZiktPmVuYWJsZURhdGFiYXNlQ2xlYW51cCgpOwpkaWZmIC0tZ2l0IGEv
U291cmNlL1dlYktpdDIvVUlQcm9jZXNzL0FQSS9DL1dLSWNvbkRhdGFiYXNlLmggYi9Tb3VyY2Uv
V2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0MvV0tJY29uRGF0YWJhc2UuaAppbmRleCBkNWI3MjkyYjRk
OWI0YTM1MDkzZTIzOThjYjU4MThjNzljODc1ODZkLi45N2RjNDFjNmQ4MDY4YWIwN2ExZjhkMTdk
YjMwNjkxMmY4ZDkyOTQ0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJ
L0MvV0tJY29uRGF0YWJhc2UuaAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9VSVByb2Nlc3MvQVBJL0Mv
V0tJY29uRGF0YWJhc2UuaApAQCAtODMsNiArODMsNyBAQCBXS19FWFBPUlQgdm9pZCBXS0ljb25E
YXRhYmFzZVJlbGVhc2VJY29uRm9yVVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZSwg
VwogV0tfRVhQT1JUIHZvaWQgV0tJY29uRGF0YWJhc2VTZXRJY29uRGF0YUZvckljb25VUkwoV0tJ
Y29uRGF0YWJhc2VSZWYgaWNvbkRhdGFiYXNlLCBXS0RhdGFSZWYgaWNvbkRhdGEsIFdLVVJMUmVm
IGljb25VUkwpOwogV0tfRVhQT1JUIHZvaWQgV0tJY29uRGF0YWJhc2VTZXRJY29uVVJMRm9yUGFn
ZVVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJhc2VSZWYsIFdLVVJMUmVmIGljb25VUkxS
ZWYsIFdLVVJMUmVmIHBhZ2VVUkxSZWYpOwogV0tfRVhQT1JUIFdLVVJMUmVmIFdLSWNvbkRhdGFi
YXNlQ29weUljb25VUkxGb3JQYWdlVVJMKFdLSWNvbkRhdGFiYXNlUmVmIGljb25EYXRhYmFzZVJl
ZiwgV0tVUkxSZWYgcGFnZVVSTFJlZik7CitXS19FWFBPUlQgV0tEYXRhUmVmIFdLSWNvbkRhdGFi
YXNlQ29weUljb25EYXRhRm9yUGFnZVVSTChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJhc2VS
ZWYsIFdLVVJMUmVmIHBhZ2VVUkwpOwogCiBXS19FWFBPUlQgdm9pZCBXS0ljb25EYXRhYmFzZUVu
YWJsZURhdGFiYXNlQ2xlYW51cChXS0ljb25EYXRhYmFzZVJlZiBpY29uRGF0YWJhc2UpOwogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>