<?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>170828</bug_id>
          
          <creation_ts>2017-04-13 14:41:24 -0700</creation_ts>
          <short_desc>Remove unused SharedBuffer constructor</short_desc>
          <delta_ts>2017-04-15 23:31:09 -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>WebKit 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="Alex Christensen">achristensen</reporter>
          <assigned_to name="Alex Christensen">achristensen</assigned_to>
          <cc>beidson</cc>
    
    <cc>buildbot</cc>
    
    <cc>darin</cc>
    
    <cc>mmaxfield</cc>
    
    <cc>ryanhaddad</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1297139</commentid>
    <comment_count>0</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-13 14:41:24 -0700</bug_when>
    <thetext>Remove unused SharedBuffer constructor</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297141</commentid>
    <comment_count>1</comment_count>
      <attachid>307027</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-13 14:42:08 -0700</bug_when>
    <thetext>Created attachment 307027
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297147</commentid>
    <comment_count>2</comment_count>
      <attachid>307029</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-13 14:52:03 -0700</bug_when>
    <thetext>Created attachment 307029
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297207</commentid>
    <comment_count>3</comment_count>
      <attachid>307029</attachid>
    <who name="Brady Eidson">beidson</who>
    <bug_when>2017-04-13 16:48:46 -0700</bug_when>
    <thetext>Comment on attachment 307029
Patch

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

&gt; Source/WebCore/platform/SharedBuffer.cpp:230
&gt; +    // This copies data unnecessarily right now. We should just use the Vector instead of always copying it.

I don&apos;t find any value-add from this comment.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297213</commentid>
    <comment_count>4</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-13 16:54:29 -0700</bug_when>
    <thetext>http://trac.webkit.org/r215350</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297246</commentid>
    <comment_count>5</comment_count>
      <attachid>307029</attachid>
    <who name="Darin Adler">darin</who>
    <bug_when>2017-04-13 18:22:14 -0700</bug_when>
    <thetext>Comment on attachment 307029
Patch

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

&gt;&gt; Source/WebCore/platform/SharedBuffer.cpp:230
&gt;&gt; +    // This copies data unnecessarily right now. We should just use the Vector instead of always copying it.
&gt; 
&gt; I don&apos;t find any value-add from this comment.

I asked for a comment here after reviewing another bug. The comment I would write is slightly different. Something like this:

    // This takes its argument as a rvalue reference because we intend to have a future
    // version take ownership of the vector rather than copying.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297407</commentid>
    <comment_count>6</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-04-14 09:53:07 -0700</bug_when>
    <thetext>(In reply to Alex Christensen from comment #4)
&gt; http://trac.webkit.org/r215350

I think this change broke the Windows build:
https://build.webkit.org/builders/Apple%20Win%20Release%20%28Build%29/builds/674

C:\cygwin\home\buildbot\slave\win-release\build\Source\WebCore\platform\graphics\win\FontPlatformDataWin.cpp(81): error C2665: &apos;WebCore::SharedBuffer::create&apos;: none of the 4 overloads could convert all the argument types [C:\cygwin\home\buildbot\slave\win-release\build\WebKitBuild\Release\Source\WebCore\WebCore.vcxproj]</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297424</commentid>
    <comment_count>7</comment_count>
    <who name="Ryan Haddad">ryanhaddad</who>
    <bug_when>2017-04-14 10:31:55 -0700</bug_when>
    <thetext>Reverted r215350 for reason:

This change broke the Windows build.

Committed r215364: &lt;http://trac.webkit.org/changeset/215364&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297701</commentid>
    <comment_count>8</comment_count>
      <attachid>307207</attachid>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-15 17:01:09 -0700</bug_when>
    <thetext>Created attachment 307207
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297703</commentid>
    <comment_count>9</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-15 17:10:16 -0700</bug_when>
    <thetext>http://trac.webkit.org/r215396</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1297738</commentid>
    <comment_count>10</comment_count>
    <who name="Alex Christensen">achristensen</who>
    <bug_when>2017-04-15 23:31:09 -0700</bug_when>
    <thetext>https://trac.webkit.org/changeset/215398/webkit</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>307027</attachid>
            <date>2017-04-13 14:42:08 -0700</date>
            <delta_ts>2017-04-13 14:52:02 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-170828-20170413144207.patch</filename>
            <type>text/plain</type>
            <size>2527</size>
            <attacher name="Alex Christensen">achristensen</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNTMzOCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDE3LTA0LTEzICBBbGV4IENo
cmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmVtb3ZlIHVu
dXNlZCBTaGFyZWRCdWZmZXIgY29uc3RydWN0b3IKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTE3MDgyOAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vU2hhcmVkQnVmZmVyLmNwcDoKKyAgICAg
ICAgKFdlYkNvcmU6OlNoYXJlZEJ1ZmZlcjo6YXBwZW5kKToKKyAgICAgICAgKiBwbGF0Zm9ybS9T
aGFyZWRCdWZmZXIuaDoKKyAgICAgICAgKFdlYkNvcmU6OlNoYXJlZEJ1ZmZlcjo6Y3JlYXRlKToK
KwogMjAxNy0wNC0xMyAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBVcGRhdGUgdGhlIGNvbW1lbnRzIGZvciB0aGUgbnVtYmVyIG9mIGJpdHMgaW4gUmVuZGVyU3R5
bGU6OkluaGVyaXRlZEZsYWdzLgpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2hhcmVk
QnVmZmVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRC
dWZmZXIuY3BwCShyZXZpc2lvbiAyMTUzMjQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9T
aGFyZWRCdWZmZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC02NiwxMiArNjYsNiBAQCBTaGFyZWRC
dWZmZXI6OlNoYXJlZEJ1ZmZlcigpCiB7CiB9CiAKLVNoYXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVy
KHVuc2lnbmVkIHNpemUpCi0gICAgOiBtX3NpemUoc2l6ZSkKLSAgICAsIG1fYnVmZmVyKGFkb3B0
UmVmKCpuZXcgRGF0YUJ1ZmZlcikpCi17Ci19Ci0KIFNoYXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVy
KGNvbnN0IGNoYXIqIGRhdGEsIHVuc2lnbmVkIHNpemUpCiAgICAgOiBtX2J1ZmZlcihhZG9wdFJl
ZigqbmV3IERhdGFCdWZmZXIpKQogewpAQCAtMjMzLDYgKzIyNyw3IEBAIHZvaWQgU2hhcmVkQnVm
ZmVyOjphcHBlbmQoY29uc3QgY2hhciogZGEKIAogdm9pZCBTaGFyZWRCdWZmZXI6OmFwcGVuZChW
ZWN0b3I8Y2hhcj4mJiBkYXRhKQogeworICAgIC8vIFRoaXMgY29waWVzIGRhdGEgdW5uZWNlc3Nh
cmlseSByaWdodCBub3cuIFdlIHNob3VsZCBqdXN0IHVzZSB0aGUgVmVjdG9yIGluc3RlYWQgb2Yg
YWx3YXlzIGNvcHlpbmcgaXQuCiAgICAgYXBwZW5kKGRhdGEuZGF0YSgpLCBkYXRhLnNpemUoKSk7
CiB9CiAKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1ZmZlci5oCj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1ZmZlci5oCShyZXZpc2lv
biAyMTUzMjQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRCdWZmZXIuaAkod29y
a2luZyBjb3B5KQpAQCAtNTIsNyArNTIsNiBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiBjbGFzcyBT
aGFyZWRCdWZmZXIgOiBwdWJsaWMgUmVmQ291bnRlZDxTaGFyZWRCdWZmZXI+IHsKIHB1YmxpYzoK
ICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKCkgeyByZXR1cm4gYWRvcHRSZWYo
Km5ldyBTaGFyZWRCdWZmZXIpOyB9Ci0gICAgc3RhdGljIFJlZjxTaGFyZWRCdWZmZXI+IGNyZWF0
ZSh1bnNpZ25lZCBzaXplKSB7IHJldHVybiBhZG9wdFJlZigqbmV3IFNoYXJlZEJ1ZmZlcihzaXpl
KSk7IH0KICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKGNvbnN0IGNoYXIqIGMs
IHVuc2lnbmVkIGkpIHsgcmV0dXJuIGFkb3B0UmVmKCpuZXcgU2hhcmVkQnVmZmVyKGMsIGkpKTsg
fQogICAgIHN0YXRpYyBSZWY8U2hhcmVkQnVmZmVyPiBjcmVhdGUoY29uc3QgdW5zaWduZWQgY2hh
ciogZGF0YSwgdW5zaWduZWQgc2l6ZSkgeyByZXR1cm4gYWRvcHRSZWYoKm5ldyBTaGFyZWRCdWZm
ZXIoZGF0YSwgc2l6ZSkpOyB9CiAKQEAgLTEyMyw3ICsxMjIsNiBAQCBwdWJsaWM6CiAKIHByaXZh
dGU6CiAgICAgV0VCQ09SRV9FWFBPUlQgU2hhcmVkQnVmZmVyKCk7Ci0gICAgZXhwbGljaXQgU2hh
cmVkQnVmZmVyKHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFyZWRCdWZmZXIoY29u
c3QgY2hhciosIHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFyZWRCdWZmZXIoY29u
c3QgdW5zaWduZWQgY2hhciosIHVuc2lnbmVkKTsKICAgICBleHBsaWNpdCBTaGFyZWRCdWZmZXIo
TWFwcGVkRmlsZURhdGEmJik7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>307029</attachid>
            <date>2017-04-13 14:52:03 -0700</date>
            <delta_ts>2017-04-15 17:01:08 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-170828-20170413145203.patch</filename>
            <type>text/plain</type>
            <size>3656</size>
            <attacher name="Alex Christensen">achristensen</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNTMzOCkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE1IEBACisyMDE3LTA0LTEzICBBbGV4IENo
cmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmVtb3ZlIHVu
dXNlZCBTaGFyZWRCdWZmZXIgY29uc3RydWN0b3IKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTE3MDgyOAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vU2hhcmVkQnVmZmVyLmNwcDoKKyAgICAg
ICAgKFdlYkNvcmU6OlNoYXJlZEJ1ZmZlcjo6YXBwZW5kKToKKyAgICAgICAgKiBwbGF0Zm9ybS9T
aGFyZWRCdWZmZXIuaDoKKyAgICAgICAgKFdlYkNvcmU6OlNoYXJlZEJ1ZmZlcjo6Y3JlYXRlKToK
KwogMjAxNy0wNC0xMyAgUnlvc3VrZSBOaXdhICA8cm5pd2FAd2Via2l0Lm9yZz4KIAogICAgICAg
ICBVcGRhdGUgdGhlIGNvbW1lbnRzIGZvciB0aGUgbnVtYmVyIG9mIGJpdHMgaW4gUmVuZGVyU3R5
bGU6OkluaGVyaXRlZEZsYWdzLgpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vU2hhcmVk
QnVmZmVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRC
dWZmZXIuY3BwCShyZXZpc2lvbiAyMTUzMjQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9T
aGFyZWRCdWZmZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBAIC02NiwxMiArNjYsNiBAQCBTaGFyZWRC
dWZmZXI6OlNoYXJlZEJ1ZmZlcigpCiB7CiB9CiAKLVNoYXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVy
KHVuc2lnbmVkIHNpemUpCi0gICAgOiBtX3NpemUoc2l6ZSkKLSAgICAsIG1fYnVmZmVyKGFkb3B0
UmVmKCpuZXcgRGF0YUJ1ZmZlcikpCi17Ci19Ci0KIFNoYXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVy
KGNvbnN0IGNoYXIqIGRhdGEsIHVuc2lnbmVkIHNpemUpCiAgICAgOiBtX2J1ZmZlcihhZG9wdFJl
ZigqbmV3IERhdGFCdWZmZXIpKQogewpAQCAtMjMzLDYgKzIyNyw3IEBAIHZvaWQgU2hhcmVkQnVm
ZmVyOjphcHBlbmQoY29uc3QgY2hhciogZGEKIAogdm9pZCBTaGFyZWRCdWZmZXI6OmFwcGVuZChW
ZWN0b3I8Y2hhcj4mJiBkYXRhKQogeworICAgIC8vIFRoaXMgY29waWVzIGRhdGEgdW5uZWNlc3Nh
cmlseSByaWdodCBub3cuIFdlIHNob3VsZCBqdXN0IHVzZSB0aGUgVmVjdG9yIGluc3RlYWQgb2Yg
YWx3YXlzIGNvcHlpbmcgaXQuCiAgICAgYXBwZW5kKGRhdGEuZGF0YSgpLCBkYXRhLnNpemUoKSk7
CiB9CiAKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1ZmZlci5oCj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1ZmZlci5oCShyZXZpc2lv
biAyMTUzMjQpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRCdWZmZXIuaAkod29y
a2luZyBjb3B5KQpAQCAtNTIsNyArNTIsNiBAQCBuYW1lc3BhY2UgV2ViQ29yZSB7CiBjbGFzcyBT
aGFyZWRCdWZmZXIgOiBwdWJsaWMgUmVmQ291bnRlZDxTaGFyZWRCdWZmZXI+IHsKIHB1YmxpYzoK
ICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKCkgeyByZXR1cm4gYWRvcHRSZWYo
Km5ldyBTaGFyZWRCdWZmZXIpOyB9Ci0gICAgc3RhdGljIFJlZjxTaGFyZWRCdWZmZXI+IGNyZWF0
ZSh1bnNpZ25lZCBzaXplKSB7IHJldHVybiBhZG9wdFJlZigqbmV3IFNoYXJlZEJ1ZmZlcihzaXpl
KSk7IH0KICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKGNvbnN0IGNoYXIqIGMs
IHVuc2lnbmVkIGkpIHsgcmV0dXJuIGFkb3B0UmVmKCpuZXcgU2hhcmVkQnVmZmVyKGMsIGkpKTsg
fQogICAgIHN0YXRpYyBSZWY8U2hhcmVkQnVmZmVyPiBjcmVhdGUoY29uc3QgdW5zaWduZWQgY2hh
ciogZGF0YSwgdW5zaWduZWQgc2l6ZSkgeyByZXR1cm4gYWRvcHRSZWYoKm5ldyBTaGFyZWRCdWZm
ZXIoZGF0YSwgc2l6ZSkpOyB9CiAKQEAgLTEyMyw3ICsxMjIsNiBAQCBwdWJsaWM6CiAKIHByaXZh
dGU6CiAgICAgV0VCQ09SRV9FWFBPUlQgU2hhcmVkQnVmZmVyKCk7Ci0gICAgZXhwbGljaXQgU2hh
cmVkQnVmZmVyKHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFyZWRCdWZmZXIoY29u
c3QgY2hhciosIHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFyZWRCdWZmZXIoY29u
c3QgdW5zaWduZWQgY2hhciosIHVuc2lnbmVkKTsKICAgICBleHBsaWNpdCBTaGFyZWRCdWZmZXIo
TWFwcGVkRmlsZURhdGEmJik7CkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9ncmFwaGlj
cy9mcmVldHlwZS9Gb250UGxhdGZvcm1EYXRhRnJlZVR5cGUuY3BwCj09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNv
dXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2ZyZWV0eXBlL0ZvbnRQbGF0Zm9ybURhdGFG
cmVlVHlwZS5jcHAJKHJldmlzaW9uIDIxNTMyNCkKKysrIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3Jt
L2dyYXBoaWNzL2ZyZWV0eXBlL0ZvbnRQbGF0Zm9ybURhdGFGcmVlVHlwZS5jcHAJKHdvcmtpbmcg
Y29weSkKQEAgLTM2MiwxNiArMzYyLDEzIEBAIFJlZlB0cjxTaGFyZWRCdWZmZXI+IEZvbnRQbGF0
Zm9ybURhdGE6Om8KICAgICBpZiAoRlRfTG9hZF9TZm50X1RhYmxlKGZyZWVUeXBlRmFjZSwgdGFn
LCAwLCAwLCAmdGFibGVTaXplKSkKICAgICAgICAgcmV0dXJuIG51bGxwdHI7CiAKLSAgICBSZWZQ
dHI8U2hhcmVkQnVmZmVyPiBidWZmZXIgPSBTaGFyZWRCdWZmZXI6OmNyZWF0ZSh0YWJsZVNpemUp
OwotICAgIGlmIChidWZmZXItPnNpemUoKSAhPSB0YWJsZVNpemUpCi0gICAgICAgIHJldHVybiBu
dWxscHRyOwotCisgICAgVmVjdG9yPGNoYXI+IGRhdGEodGFibGVTaXplKTsKICAgICBGVF9VTG9u
ZyBleHBlY3RlZFRhYmxlU2l6ZSA9IHRhYmxlU2l6ZTsKLSAgICBGVF9FcnJvciBlcnJvciA9IEZU
X0xvYWRfU2ZudF9UYWJsZShmcmVlVHlwZUZhY2UsIHRhZywgMCwgcmVpbnRlcnByZXRfY2FzdDxG
VF9CeXRlKj4oY29uc3RfY2FzdDxjaGFyKj4oYnVmZmVyLT5kYXRhKCkpKSwgJnRhYmxlU2l6ZSk7
CisgICAgRlRfRXJyb3IgZXJyb3IgPSBGVF9Mb2FkX1NmbnRfVGFibGUoZnJlZVR5cGVGYWNlLCB0
YWcsIDAsIHJlaW50ZXJwcmV0X2Nhc3Q8RlRfQnl0ZSo+KGRhdGEuZGF0YSgpKSwgJnRhYmxlU2l6
ZSk7CiAgICAgaWYgKGVycm9yIHx8IHRhYmxlU2l6ZSAhPSBleHBlY3RlZFRhYmxlU2l6ZSkKICAg
ICAgICAgcmV0dXJuIG51bGxwdHI7CiAKLSAgICByZXR1cm4gYnVmZmVyOworICAgIHJldHVybiBT
aGFyZWRCdWZmZXI6OmNyZWF0ZShXVEZNb3ZlKGRhdGEpKTsKIH0KIAogfSAvLyBuYW1lc3BhY2Ug
V2ViQ29yZQo=
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>307207</attachid>
            <date>2017-04-15 17:01:09 -0700</date>
            <delta_ts>2017-04-15 17:01:29 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-170828-20170415170108.patch</filename>
            <type>text/plain</type>
            <size>4813</size>
            <attacher name="Alex Christensen">achristensen</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2UvV2Vi
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDIxNTM5NSkKKysrIFNvdXJjZS9XZWJDb3JlL0NoYW5n
ZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE5IEBACisyMDE3LTA0LTE1ICBBbGV4IENo
cmlzdGVuc2VuICA8YWNocmlzdGVuc2VuQHdlYmtpdC5vcmc+CisKKyAgICAgICAgUmVtb3ZlIHVu
dXNlZCBTaGFyZWRCdWZmZXIgY29uc3RydWN0b3IKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtp
dC5vcmcvc2hvd19idWcuY2dpP2lkPTE3MDgyOAorCisgICAgICAgIFJldmlld2VkIGJ5IEJyYWR5
IEVpZHNvbi4KKworICAgICAgICAqIHBsYXRmb3JtL1NoYXJlZEJ1ZmZlci5jcHA6CisgICAgICAg
IChXZWJDb3JlOjpTaGFyZWRCdWZmZXI6OmFwcGVuZCk6CisgICAgICAgICogcGxhdGZvcm0vU2hh
cmVkQnVmZmVyLmg6CisgICAgICAgIChXZWJDb3JlOjpTaGFyZWRCdWZmZXI6OmNyZWF0ZSk6Cisg
ICAgICAgICogcGxhdGZvcm0vZ3JhcGhpY3MvZnJlZXR5cGUvRm9udFBsYXRmb3JtRGF0YUZyZWVU
eXBlLmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZvbnRQbGF0Zm9ybURhdGE6Om9wZW5UeXBlVGFi
bGUpOgorICAgICAgICAqIHBsYXRmb3JtL2dyYXBoaWNzL3dpbi9Gb250UGxhdGZvcm1EYXRhV2lu
LmNwcDoKKyAgICAgICAgKFdlYkNvcmU6OkZvbnRQbGF0Zm9ybURhdGE6Om9wZW5UeXBlVGFibGUp
OgorCiAyMDE3LTA0LTE1ICBDb21taXQgUXVldWUgIDxjb21taXQtcXVldWVAd2Via2l0Lm9yZz4K
IAogICAgICAgICBVbnJldmlld2VkLCByb2xsaW5nIG91dCByMjE1MzkzLgpJbmRleDogU291cmNl
L1dlYkNvcmUvcGxhdGZvcm0vU2hhcmVkQnVmZmVyLmNwcAo9PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBTb3VyY2Uv
V2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRCdWZmZXIuY3BwCShyZXZpc2lvbiAyMTUzOTUpCisrKyBT
b3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRCdWZmZXIuY3BwCSh3b3JraW5nIGNvcHkpCkBA
IC02NiwxMiArNjYsNiBAQCBTaGFyZWRCdWZmZXI6OlNoYXJlZEJ1ZmZlcigpCiB7CiB9CiAKLVNo
YXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVyKHVuc2lnbmVkIHNpemUpCi0gICAgOiBtX3NpemUoc2l6
ZSkKLSAgICAsIG1fYnVmZmVyKGFkb3B0UmVmKCpuZXcgRGF0YUJ1ZmZlcikpCi17Ci19Ci0KIFNo
YXJlZEJ1ZmZlcjo6U2hhcmVkQnVmZmVyKGNvbnN0IGNoYXIqIGRhdGEsIHVuc2lnbmVkIHNpemUp
CiAgICAgOiBtX2J1ZmZlcihhZG9wdFJlZigqbmV3IERhdGFCdWZmZXIpKQogewpAQCAtMjMzLDYg
KzIyNyw4IEBAIHZvaWQgU2hhcmVkQnVmZmVyOjphcHBlbmQoY29uc3QgY2hhciogZGEKIAogdm9p
ZCBTaGFyZWRCdWZmZXI6OmFwcGVuZChWZWN0b3I8Y2hhcj4mJiBkYXRhKQogeworICAgIC8vIFRo
aXMgdGFrZXMgaXRzIGFyZ3VtZW50IGFzIGEgcnZhbHVlIHJlZmVyZW5jZSBiZWNhdXNlIHdlIGlu
dGVuZCB0byBoYXZlIGEgZnV0dXJlCisgICAgLy8gdmVyc2lvbiB0YWtlIG93bmVyc2hpcCBvZiB0
aGUgdmVjdG9yIHJhdGhlciB0aGFuIGNvcHlpbmcuCiAgICAgYXBwZW5kKGRhdGEuZGF0YSgpLCBk
YXRhLnNpemUoKSk7CiB9CiAKSW5kZXg6IFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1
ZmZlci5oCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL1NoYXJlZEJ1ZmZl
ci5oCShyZXZpc2lvbiAyMTUzOTUpCisrKyBTb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9TaGFyZWRC
dWZmZXIuaAkod29ya2luZyBjb3B5KQpAQCAtNTIsNyArNTIsNiBAQCBuYW1lc3BhY2UgV2ViQ29y
ZSB7CiBjbGFzcyBTaGFyZWRCdWZmZXIgOiBwdWJsaWMgUmVmQ291bnRlZDxTaGFyZWRCdWZmZXI+
IHsKIHB1YmxpYzoKICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKCkgeyByZXR1
cm4gYWRvcHRSZWYoKm5ldyBTaGFyZWRCdWZmZXIpOyB9Ci0gICAgc3RhdGljIFJlZjxTaGFyZWRC
dWZmZXI+IGNyZWF0ZSh1bnNpZ25lZCBzaXplKSB7IHJldHVybiBhZG9wdFJlZigqbmV3IFNoYXJl
ZEJ1ZmZlcihzaXplKSk7IH0KICAgICBzdGF0aWMgUmVmPFNoYXJlZEJ1ZmZlcj4gY3JlYXRlKGNv
bnN0IGNoYXIqIGMsIHVuc2lnbmVkIGkpIHsgcmV0dXJuIGFkb3B0UmVmKCpuZXcgU2hhcmVkQnVm
ZmVyKGMsIGkpKTsgfQogICAgIHN0YXRpYyBSZWY8U2hhcmVkQnVmZmVyPiBjcmVhdGUoY29uc3Qg
dW5zaWduZWQgY2hhciogZGF0YSwgdW5zaWduZWQgc2l6ZSkgeyByZXR1cm4gYWRvcHRSZWYoKm5l
dyBTaGFyZWRCdWZmZXIoZGF0YSwgc2l6ZSkpOyB9CiAKQEAgLTEyMyw3ICsxMjIsNiBAQCBwdWJs
aWM6CiAKIHByaXZhdGU6CiAgICAgV0VCQ09SRV9FWFBPUlQgU2hhcmVkQnVmZmVyKCk7Ci0gICAg
ZXhwbGljaXQgU2hhcmVkQnVmZmVyKHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFy
ZWRCdWZmZXIoY29uc3QgY2hhciosIHVuc2lnbmVkKTsKICAgICBXRUJDT1JFX0VYUE9SVCBTaGFy
ZWRCdWZmZXIoY29uc3QgdW5zaWduZWQgY2hhciosIHVuc2lnbmVkKTsKICAgICBleHBsaWNpdCBT
aGFyZWRCdWZmZXIoTWFwcGVkRmlsZURhdGEmJik7CkluZGV4OiBTb3VyY2UvV2ViQ29yZS9wbGF0
Zm9ybS9ncmFwaGljcy9mcmVldHlwZS9Gb250UGxhdGZvcm1EYXRhRnJlZVR5cGUuY3BwCj09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT0KLS0tIFNvdXJjZS9XZWJDb3JlL3BsYXRmb3JtL2dyYXBoaWNzL2ZyZWV0eXBlL0ZvbnRQ
bGF0Zm9ybURhdGFGcmVlVHlwZS5jcHAJKHJldmlzaW9uIDIxNTM5NSkKKysrIFNvdXJjZS9XZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL2ZyZWV0eXBlL0ZvbnRQbGF0Zm9ybURhdGFGcmVlVHlwZS5j
cHAJKHdvcmtpbmcgY29weSkKQEAgLTM2MiwxNiArMzYyLDEzIEBAIFJlZlB0cjxTaGFyZWRCdWZm
ZXI+IEZvbnRQbGF0Zm9ybURhdGE6Om8KICAgICBpZiAoRlRfTG9hZF9TZm50X1RhYmxlKGZyZWVU
eXBlRmFjZSwgdGFnLCAwLCAwLCAmdGFibGVTaXplKSkKICAgICAgICAgcmV0dXJuIG51bGxwdHI7
CiAKLSAgICBSZWZQdHI8U2hhcmVkQnVmZmVyPiBidWZmZXIgPSBTaGFyZWRCdWZmZXI6OmNyZWF0
ZSh0YWJsZVNpemUpOwotICAgIGlmIChidWZmZXItPnNpemUoKSAhPSB0YWJsZVNpemUpCi0gICAg
ICAgIHJldHVybiBudWxscHRyOwotCisgICAgVmVjdG9yPGNoYXI+IGRhdGEodGFibGVTaXplKTsK
ICAgICBGVF9VTG9uZyBleHBlY3RlZFRhYmxlU2l6ZSA9IHRhYmxlU2l6ZTsKLSAgICBGVF9FcnJv
ciBlcnJvciA9IEZUX0xvYWRfU2ZudF9UYWJsZShmcmVlVHlwZUZhY2UsIHRhZywgMCwgcmVpbnRl
cnByZXRfY2FzdDxGVF9CeXRlKj4oY29uc3RfY2FzdDxjaGFyKj4oYnVmZmVyLT5kYXRhKCkpKSwg
JnRhYmxlU2l6ZSk7CisgICAgRlRfRXJyb3IgZXJyb3IgPSBGVF9Mb2FkX1NmbnRfVGFibGUoZnJl
ZVR5cGVGYWNlLCB0YWcsIDAsIHJlaW50ZXJwcmV0X2Nhc3Q8RlRfQnl0ZSo+KGRhdGEuZGF0YSgp
KSwgJnRhYmxlU2l6ZSk7CiAgICAgaWYgKGVycm9yIHx8IHRhYmxlU2l6ZSAhPSBleHBlY3RlZFRh
YmxlU2l6ZSkKICAgICAgICAgcmV0dXJuIG51bGxwdHI7CiAKLSAgICByZXR1cm4gYnVmZmVyOwor
ICAgIHJldHVybiBTaGFyZWRCdWZmZXI6OmNyZWF0ZShXVEZNb3ZlKGRhdGEpKTsKIH0KIAogfSAv
LyBuYW1lc3BhY2UgV2ViQ29yZQpJbmRleDogU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhp
Y3Mvd2luL0ZvbnRQbGF0Zm9ybURhdGFXaW4uY3BwCj09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFNvdXJjZS9XZWJD
b3JlL3BsYXRmb3JtL2dyYXBoaWNzL3dpbi9Gb250UGxhdGZvcm1EYXRhV2luLmNwcAkocmV2aXNp
b24gMjE1Mzk1KQorKysgU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vZ3JhcGhpY3Mvd2luL0ZvbnRQ
bGF0Zm9ybURhdGFXaW4uY3BwCSh3b3JraW5nIGNvcHkpCkBAIC03OCwxMyArNzgsMTQgQEAgUmVm
UHRyPFNoYXJlZEJ1ZmZlcj4gRm9udFBsYXRmb3JtRGF0YTo6bwogICAgIERXT1JEIHNpemUgPSBH
ZXRGb250RGF0YShoZGMsIHRhYmxlLCAwLCAwLCAwKTsKICAgICBSZWZQdHI8U2hhcmVkQnVmZmVy
PiBidWZmZXI7CiAgICAgaWYgKHNpemUgIT0gR0RJX0VSUk9SKSB7Ci0gICAgICAgIGJ1ZmZlciA9
IFNoYXJlZEJ1ZmZlcjo6Y3JlYXRlKHNpemUpOwotICAgICAgICBEV09SRCByZXN1bHQgPSBHZXRG
b250RGF0YShoZGMsIHRhYmxlLCAwLCAoUFZPSUQpYnVmZmVyLT5kYXRhKCksIHNpemUpOworICAg
ICAgICBWZWN0b3I8Y2hhcj4gZGF0YShzaXplKTsKKyAgICAgICAgRFdPUkQgcmVzdWx0ID0gR2V0
Rm9udERhdGEoaGRjLCB0YWJsZSwgMCwgKFBWT0lEKWRhdGEtPmRhdGEoKSwgc2l6ZSk7CiAgICAg
ICAgIEFTU0VSVChyZXN1bHQgPT0gc2l6ZSk7CisgICAgICAgIGJ1ZmZlciA9IFNoYXJlZEJ1ZmZl
cjo6Y3JlYXRlKFdURk1vdmUoZGF0YSkpOwogICAgIH0KIAogICAgIFNlbGVjdE9iamVjdChoZGMs
IG9sZEZvbnQpOwotICAgIHJldHVybiBidWZmZXIucmVsZWFzZSgpOworICAgIHJldHVybiBidWZm
ZXI7CiB9CiAKICNpZm5kZWYgTkRFQlVHCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>