<?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>95896</bug_id>
          
          <creation_ts>2012-09-05 14:38:31 -0700</creation_ts>
          <short_desc>Remove String::operator+=() from plugin code</short_desc>
          <delta_ts>2012-09-06 01:55:55 -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>Plug-ins</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>All</rep_platform>
          <op_sys>All</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="Patrick R. Gansterer">paroga</reporter>
          <assigned_to name="Patrick R. Gansterer">paroga</assigned_to>
          <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>712545</commentid>
    <comment_count>0</comment_count>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2012-09-05 14:38:31 -0700</bug_when>
    <thetext>Remove String::operator+=() from plugin code</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712550</commentid>
    <comment_count>1</comment_count>
      <attachid>162339</attachid>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2012-09-05 14:41:58 -0700</bug_when>
    <thetext>Created attachment 162339
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>712552</commentid>
    <comment_count>2</comment_count>
      <attachid>162341</attachid>
    <who name="Patrick R. Gansterer">paroga</who>
    <bug_when>2012-09-05 14:44:06 -0700</bug_when>
    <thetext>Created attachment 162341
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>713036</commentid>
    <comment_count>3</comment_count>
      <attachid>162341</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-06 01:55:53 -0700</bug_when>
    <thetext>Comment on attachment 162341
Patch

Clearing flags on attachment: 162341

Committed r127708: &lt;http://trac.webkit.org/changeset/127708&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>713037</commentid>
    <comment_count>4</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-09-06 01:55:55 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="1"
              ispatch="1"
              isprivate="0"
          >
            <attachid>162339</attachid>
            <date>2012-09-05 14:41:58 -0700</date>
            <delta_ts>2012-09-05 14:44:06 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-95896.patch</filename>
            <type>text/plain</type>
            <size>4291</size>
            <attacher name="Patrick R. Gansterer">paroga</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjMWJlYmUzLi5iODI2YzgxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUg
QEAKKzIwMTItMDktMDUgIFBhdHJpY2sgR2Fuc3RlcmVyICA8cGFyb2dhQHdlYmtpdC5vcmc+CisK
KyAgICAgICAgUmVtb3ZlIFN0cmluZzo6b3BlcmF0b3IrPSgpIGZyb20gcGx1Z2luIGNvZGUKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk1ODk2CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVXNlIG9wZXJhdG9y
KygpIHRvIGNvbmNhdGVuYXRlIHN0cmluZ3MgaW5zdGVhZCBvZiBvcGVyYXRvcis9KCkuCisKKyAg
ICAgICAgKiBwbHVnaW5zL1BsdWdpblZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6cGFyc2VS
RkM4MjJIZWFkZXJGaWVsZHMpOgorCiAyMDEyLTA5LTA1ICBCZW5qYW1pbiBQb3VsYWluICA8YnBv
dWxhaW5AYXBwbGUuY29tPgogCiAgICAgICAgIEZpeCB0aGUgdXNlcyBvZiBTdHJpbmc6Om9wZXJh
dG9yKz0oKSBmb3IgTWFjCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdp
blZpZXcuY3BwIGIvU291cmNlL1dlYkNvcmUvcGx1Z2lucy9QbHVnaW5WaWV3LmNwcAppbmRleCBk
ZDIyMGQ4Li45NjcyZTkxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdp
blZpZXcuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAKQEAg
LTExMzMsMTIgKzExMzMsOCBAQCBzdGF0aWMgaW5saW5lIEhUVFBIZWFkZXJNYXAgcGFyc2VSRkM4
MjJIZWFkZXJGaWVsZHMoY29uc3QgVmVjdG9yPGNoYXI+JiBidWZmZXIsCiAgICAgICAgICAgICAg
ICAgICAgIHZhbHVlID0gU3RyaW5nKGNvbG9uLCBlb2wgLSBjb2xvbik7CiAKICAgICAgICAgICAg
ICAgICBTdHJpbmcgb2xkVmFsdWUgPSBoZWFkZXJGaWVsZHMuZ2V0KGxhc3RLZXkpOwotICAgICAg
ICAgICAgICAgIGlmICghb2xkVmFsdWUuaXNOdWxsKCkpIHsKLSAgICAgICAgICAgICAgICAgICAg
U3RyaW5nIHRtcCA9IG9sZFZhbHVlOwotICAgICAgICAgICAgICAgICAgICB0bXAgKz0gIiwgIjsK
LSAgICAgICAgICAgICAgICAgICAgdG1wICs9IHZhbHVlOwotICAgICAgICAgICAgICAgICAgICB2
YWx1ZSA9IHRtcDsKLSAgICAgICAgICAgICAgICB9CisgICAgICAgICAgICAgICAgaWYgKCFvbGRW
YWx1ZS5pc051bGwoKSkKKyAgICAgICAgICAgICAgICAgICAgdmFsdWUgPSBvbGRWYWx1ZSArICIs
ICIgKyB2YWx1ZTsKIAogICAgICAgICAgICAgICAgIGhlYWRlckZpZWxkcy5zZXQobGFzdEtleSwg
dmFsdWUpOwogICAgICAgICAgICAgfQpkaWZmIC0tZ2l0IGEvU291cmNlL1dlYktpdDIvQ2hhbmdl
TG9nIGIvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCmluZGV4IDU1ODhlMTcuLjIxZTVmYWEgMTAw
NjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL0NoYW5nZUxvZworKysgYi9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAxMi0wOS0wNSAgUGF0cmljayBHYW5zdGVyZXIg
IDxwYXJvZ2FAd2Via2l0Lm9yZz4KKworICAgICAgICBSZW1vdmUgU3RyaW5nOjpvcGVyYXRvcis9
KCkgZnJvbSBwbHVnaW4gY29kZQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93
X2J1Zy5jZ2k/aWQ9OTU4OTYKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4K
KworICAgICAgICBVc2Ugb3BlcmF0b3IrKCkgdG8gY29uY2F0ZW5hdGUgc3RyaW5ncyBpbnN0ZWFk
IG9mIG9wZXJhdG9yKz0oKS4KKworICAgICAgICAqIFdlYlByb2Nlc3MvUGx1Z2lucy9OZXRzY2Fw
ZS9OZXRzY2FwZUJyb3dzZXJGdW5jcy5jcHA6CisgICAgICAgIChXZWJLaXQ6OnBhcnNlUkZDODIy
SGVhZGVyRmllbGRzKToKKyAgICAgICAgKFdlYktpdCk6CisKIDIwMTItMDktMDUgIEJhbGF6cyBL
ZWxlbWVuICA8a2JhbGF6c0B3ZWJraXQub3JnPgogCiAgICAgICAgIFtRdF0gUVJhd1dlYlZpZXdQ
cml2YXRlIGRvZXMgbm90IGluaXRpYWxpemUgc29tZSBtZW1iZXJzCmRpZmYgLS1naXQgYS9Tb3Vy
Y2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1BsdWdpbnMvTmV0c2NhcGUvTmV0c2NhcGVCcm93c2VyRnVu
Y3MuY3BwIGIvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9QbHVnaW5zL05ldHNjYXBlL05ldHNj
YXBlQnJvd3NlckZ1bmNzLmNwcAppbmRleCBkNDBkYjc5Li4wM2MyMmVkIDEwMDY0NAotLS0gYS9T
b3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL1BsdWdpbnMvTmV0c2NhcGUvTmV0c2NhcGVCcm93c2Vy
RnVuY3MuY3BwCisrKyBiL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvUGx1Z2lucy9OZXRzY2Fw
ZS9OZXRzY2FwZUJyb3dzZXJGdW5jcy5jcHAKQEAgLTE3MiwxMiArMTcyLDEyIEBAIHN0YXRpYyBI
VFRQSGVhZGVyTWFwIHBhcnNlUkZDODIySGVhZGVyRmllbGRzKGNvbnN0IGNoYXIqIGJ5dGVzLCB1
bnNpZ25lZCBsZW5ndGgpCiAgICAgICAgICAgICBpZiAobGFzdEhlYWRlcktleS5pc051bGwoKSkg
ewogICAgICAgICAgICAgICAgIC8vIG1hbGZvcm1lZCBoZWFkZXI7IGlnbm9yZSBpdCBhbmQgY29u
dGludWUKICAgICAgICAgICAgICAgICBjb250aW51ZTsKLSAgICAgICAgICAgIH0gCi0gICAgICAg
ICAgICAKKyAgICAgICAgICAgIH0KKwogICAgICAgICAgICAgLy8gTWVyZ2UgdGhlIGNvbnRpbnVh
dGlvbiBvZiB0aGUgcHJldmlvdXMgaGVhZGVyCiAgICAgICAgICAgICBTdHJpbmcgY3VycmVudFZh
bHVlID0gaGVhZGVyRmllbGRzLmdldChsYXN0SGVhZGVyS2V5KTsKICAgICAgICAgICAgIFN0cmlu
ZyBuZXdWYWx1ZShsaW5lLCBsaW5lTGVuZ3RoKTsKLSAgICAgICAgICAgIAorCiAgICAgICAgICAg
ICBoZWFkZXJGaWVsZHMuc2V0KGxhc3RIZWFkZXJLZXksIGN1cnJlbnRWYWx1ZSArIG5ld1ZhbHVl
KTsKICAgICAgICAgfSBlbHNlIHsKICAgICAgICAgICAgIC8vIEJyYW5kIG5ldyBoZWFkZXIKQEAg
LTE5MiwzMSArMTkyLDI4IEBAIHN0YXRpYyBIVFRQSGVhZGVyTWFwIHBhcnNlUkZDODIySGVhZGVy
RmllbGRzKGNvbnN0IGNoYXIqIGJ5dGVzLCB1bnNpZ25lZCBsZW5ndGgpCiAKICAgICAgICAgICAg
IGxhc3RIZWFkZXJLZXkgPSBjYXBpdGFsaXplUkZDODIySGVhZGVyRmllbGROYW1lKFN0cmluZyhs
aW5lLCBjb2xvbiAtIGxpbmUpKTsKICAgICAgICAgICAgIFN0cmluZyB2YWx1ZTsKLSAgICAgICAg
ICAgIAorCiAgICAgICAgICAgICBmb3IgKGNvbG9uKys7IGNvbG9uICE9IGVuZE9mTGluZTsgY29s
b24rKykgewogICAgICAgICAgICAgICAgIGlmICgqY29sb24gIT0gJyAnICYmICpjb2xvbiAhPSAn
XHQnKQogICAgICAgICAgICAgICAgICAgICBicmVhazsKICAgICAgICAgICAgIH0KKwogICAgICAg
ICAgICAgaWYgKGNvbG9uID09IGVuZE9mTGluZSkKICAgICAgICAgICAgICAgICB2YWx1ZSA9ICIi
OwogICAgICAgICAgICAgZWxzZQogICAgICAgICAgICAgICAgIHZhbHVlID0gU3RyaW5nKGNvbG9u
LCBlbmRPZkxpbmUgLSBjb2xvbik7Ci0gICAgICAgICAgICAKKwogICAgICAgICAgICAgU3RyaW5n
IG9sZFZhbHVlID0gaGVhZGVyRmllbGRzLmdldChsYXN0SGVhZGVyS2V5KTsKLSAgICAgICAgICAg
IGlmICghb2xkVmFsdWUuaXNOdWxsKCkpIHsKLSAgICAgICAgICAgICAgICBTdHJpbmcgdG1wID0g
b2xkVmFsdWU7Ci0gICAgICAgICAgICAgICAgdG1wICs9ICIsICI7Ci0gICAgICAgICAgICAgICAg
dG1wICs9IHZhbHVlOwotICAgICAgICAgICAgICAgIHZhbHVlID0gdG1wOwotICAgICAgICAgICAg
fQotICAgICAgICAgICAgCisgICAgICAgICAgICBpZiAoIW9sZFZhbHVlLmlzTnVsbCgpKQorICAg
ICAgICAgICAgICAgIHZhbHVlID0gb2xkVmFsdWUgKyAiLCAiICsgdmFsdWU7CisKICAgICAgICAg
ICAgIGhlYWRlckZpZWxkcy5zZXQobGFzdEhlYWRlcktleSwgdmFsdWUpOwogICAgICAgICB9CiAg
ICAgfQogCiAgICAgcmV0dXJuIGhlYWRlckZpZWxkczsKIH0KLSAgICAKKwogc3RhdGljIE5QRXJy
b3IgcGFyc2VQb3N0QnVmZmVyKGJvb2wgaXNGaWxlLCBjb25zdCBjaGFyICpidWZmZXIsIHVpbnQz
Ml90IGxlbmd0aCwgYm9vbCBwYXJzZUhlYWRlcnMsIEhUVFBIZWFkZXJNYXAmIGhlYWRlckZpZWxk
cywgVmVjdG9yPHVpbnQ4X3Q+JiBib2R5RGF0YSkKIHsKICAgICBSZWZQdHI8U2hhcmVkQnVmZmVy
PiBmaWxlQ29udGVudHM7Cg==
</data>

          </attachment>
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>162341</attachid>
            <date>2012-09-05 14:44:06 -0700</date>
            <delta_ts>2012-09-06 01:55:53 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-95896.patch</filename>
            <type>text/plain</type>
            <size>1448</size>
            <attacher name="Patrick R. Gansterer">paroga</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0No
YW5nZUxvZwppbmRleCBjMWJlYmUzLi5iODI2YzgxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29y
ZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCkBAIC0xLDMgKzEsMTUg
QEAKKzIwMTItMDktMDUgIFBhdHJpY2sgR2Fuc3RlcmVyICA8cGFyb2dhQHdlYmtpdC5vcmc+CisK
KyAgICAgICAgUmVtb3ZlIFN0cmluZzo6b3BlcmF0b3IrPSgpIGZyb20gcGx1Z2luIGNvZGUKKyAg
ICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk1ODk2CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisKKyAgICAgICAgVXNlIG9wZXJhdG9y
KygpIHRvIGNvbmNhdGVuYXRlIHN0cmluZ3MgaW5zdGVhZCBvZiBvcGVyYXRvcis9KCkuCisKKyAg
ICAgICAgKiBwbHVnaW5zL1BsdWdpblZpZXcuY3BwOgorICAgICAgICAoV2ViQ29yZTo6cGFyc2VS
RkM4MjJIZWFkZXJGaWVsZHMpOgorCiAyMDEyLTA5LTA1ICBCZW5qYW1pbiBQb3VsYWluICA8YnBv
dWxhaW5AYXBwbGUuY29tPgogCiAgICAgICAgIEZpeCB0aGUgdXNlcyBvZiBTdHJpbmc6Om9wZXJh
dG9yKz0oKSBmb3IgTWFjCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdp
blZpZXcuY3BwIGIvU291cmNlL1dlYkNvcmUvcGx1Z2lucy9QbHVnaW5WaWV3LmNwcAppbmRleCBk
ZDIyMGQ4Li45NjcyZTkxIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbHVnaW5zL1BsdWdp
blZpZXcuY3BwCisrKyBiL1NvdXJjZS9XZWJDb3JlL3BsdWdpbnMvUGx1Z2luVmlldy5jcHAKQEAg
LTExMzMsMTIgKzExMzMsOCBAQCBzdGF0aWMgaW5saW5lIEhUVFBIZWFkZXJNYXAgcGFyc2VSRkM4
MjJIZWFkZXJGaWVsZHMoY29uc3QgVmVjdG9yPGNoYXI+JiBidWZmZXIsCiAgICAgICAgICAgICAg
ICAgICAgIHZhbHVlID0gU3RyaW5nKGNvbG9uLCBlb2wgLSBjb2xvbik7CiAKICAgICAgICAgICAg
ICAgICBTdHJpbmcgb2xkVmFsdWUgPSBoZWFkZXJGaWVsZHMuZ2V0KGxhc3RLZXkpOwotICAgICAg
ICAgICAgICAgIGlmICghb2xkVmFsdWUuaXNOdWxsKCkpIHsKLSAgICAgICAgICAgICAgICAgICAg
U3RyaW5nIHRtcCA9IG9sZFZhbHVlOwotICAgICAgICAgICAgICAgICAgICB0bXAgKz0gIiwgIjsK
LSAgICAgICAgICAgICAgICAgICAgdG1wICs9IHZhbHVlOwotICAgICAgICAgICAgICAgICAgICB2
YWx1ZSA9IHRtcDsKLSAgICAgICAgICAgICAgICB9CisgICAgICAgICAgICAgICAgaWYgKCFvbGRW
YWx1ZS5pc051bGwoKSkKKyAgICAgICAgICAgICAgICAgICAgdmFsdWUgPSBvbGRWYWx1ZSArICIs
ICIgKyB2YWx1ZTsKIAogICAgICAgICAgICAgICAgIGhlYWRlckZpZWxkcy5zZXQobGFzdEtleSwg
dmFsdWUpOwogICAgICAgICAgICAgfQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>