<?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>96307</bug_id>
          
          <creation_ts>2012-09-10 13:20:18 -0700</creation_ts>
          <short_desc>Silence a warning when running update-webkit-localizable-strings</short_desc>
          <delta_ts>2012-09-10 13:30:53 -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>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="Anders Carlsson">andersca</reporter>
          <assigned_to name="Anders Carlsson">andersca</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>716416</commentid>
    <comment_count>0</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2012-09-10 13:20:18 -0700</bug_when>
    <thetext>Silence a warning when running update-webkit-localizable-strings</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>716422</commentid>
    <comment_count>1</comment_count>
      <attachid>163191</attachid>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2012-09-10 13:21:50 -0700</bug_when>
    <thetext>Created attachment 163191
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>716429</commentid>
    <comment_count>2</comment_count>
      <attachid>163191</attachid>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2012-09-10 13:23:52 -0700</bug_when>
    <thetext>Comment on attachment 163191
Patch

The other way to do this would be to make the two instances have identical localizer comments, as long as we are certain that there’s no chance that the two strings would be localized differently (taking context into account) in any language.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>716431</commentid>
    <comment_count>3</comment_count>
      <attachid>163191</attachid>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2012-09-10 13:25:27 -0700</bug_when>
    <thetext>Comment on attachment 163191
Patch

Oops, you also need to update Localizable.strings in this patch (you wouldn’t need to if you made the localizer comments the same).</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>716439</commentid>
    <comment_count>4</comment_count>
      <attachid>163191</attachid>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2012-09-10 13:28:25 -0700</bug_when>
    <thetext>Comment on attachment 163191
Patch

r+ assuming you also check in the updated Localizable.strings</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>716443</commentid>
    <comment_count>5</comment_count>
    <who name="Anders Carlsson">andersca</who>
    <bug_when>2012-09-10 13:30:53 -0700</bug_when>
    <thetext>Committed r128098: &lt;http://trac.webkit.org/changeset/128098&gt;</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>163191</attachid>
            <date>2012-09-10 13:21:50 -0700</date>
            <delta_ts>2012-09-10 13:28:23 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-96307-20120910132130.patch</filename>
            <type>text/plain</type>
            <size>2131</size>
            <attacher name="Anders Carlsson">andersca</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTI4MDcwCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZGRlNmU3NTc4YjkwOTFk
OWQ4YjVjZWNkMDM1NWM1ZmI4NDBhNTRjYS4uN2NkNGI1MTQ2ZDI4MmQ1NzA5MjJlMTE2YzU0MjFl
NmQ2NzdkYzEyMiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE3IEBACisyMDEyLTA5LTEwICBBbmRl
cnMgQ2FybHNzb24gIDxhbmRlcnNjYUBhcHBsZS5jb20+CisKKyAgICAgICAgU2lsZW5jZSBhIHdh
cm5pbmcgd2hlbiBydW5uaW5nIHVwZGF0ZS13ZWJraXQtbG9jYWxpemFibGUtc3RyaW5ncworICAg
ICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9OTYzMDcKKworICAg
ICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBTaW5jZSBib3RoIHRo
ZSBibG9ja2VkUGx1Z2luQnlDb250ZW50U2VjdXJpdHlQb2xpY3lUZXh0IGFuZCBpbnNlY3VyZVBs
dWdpblZlcnNpb25UZXh0IHN0cmluZ3MgdXNlIGV4YWN0IHRoZSBzYW1lIHdvcmRpbmcsCisgICAg
ICAgIHVzZSBXRUJfVUlfU1RSSU5HX0tFWSB0byBpbmRpY2F0ZSB0aGF0IHRoZXkgYXJlIGRpZmZl
cmVudC4KKworICAgICAgICAqIHBsYXRmb3JtL0xvY2FsaXplZFN0cmluZ3MuY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6YmxvY2tlZFBsdWdpbkJ5Q29udGVudFNlY3VyaXR5UG9saWN5VGV4dCk6Cisg
ICAgICAgIChXZWJDb3JlOjppbnNlY3VyZVBsdWdpblZlcnNpb25UZXh0KToKKwogMjAxMi0wOS0x
MCAgTWlrZSBXZXN0ICA8bWt3c3RAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIENsYXJpZnkgdGhl
IGNhdXNlIG9mIGNvbnNvbGUgd2FybmluZ3MgZ2VuZXJhdGVkIGJ5ICJjcm9zcy1vcmlnaW4iIGFj
Y2VzcyB0byBzYW5kYm94ZWQgaWZyYW1lcy4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL3Bs
YXRmb3JtL0xvY2FsaXplZFN0cmluZ3MuY3BwIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vTG9j
YWxpemVkU3RyaW5ncy5jcHAKaW5kZXggMDE3MzRlMWZmYTliNjViNGMwYmQ1MzExZGIzYzkwMTAy
MDUzMjVjYS4uNWE3ZGRmOGJhYmNkY2UwNmE2N2NmNTZmMzgxZDA5MzA3OGUzZWY3ZCAxMDA2NDQK
LS0tIGEvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vTG9jYWxpemVkU3RyaW5ncy5jcHAKKysrIGIv
U291cmNlL1dlYkNvcmUvcGxhdGZvcm0vTG9jYWxpemVkU3RyaW5ncy5jcHAKQEAgLTY3NiwxMiAr
Njc2LDEyIEBAIFN0cmluZyBjcmFzaGVkUGx1Z2luVGV4dCgpCiAKIFN0cmluZyBibG9ja2VkUGx1
Z2luQnlDb250ZW50U2VjdXJpdHlQb2xpY3lUZXh0KCkKIHsKLSAgICByZXR1cm4gV0VCX1VJX1NU
UklORygiQmxvY2tlZCBQbHVnLWluIiwgIkxhYmVsIHRleHQgdG8gYmUgdXNlZCBpZiBwbHVnaW4g
aXMgYmxvY2tlZCBieSBhIHBhZ2UncyBDb250ZW50IFNlY3VyaXR5IFBvbGljeSIpOworICAgIHJl
dHVybiBXRUJfVUlfU1RSSU5HX0tFWSgiQmxvY2tlZCBQbHVnLWluIiwgIkJsb2NrZWQgUGx1Zy1J
biAoQmxvY2tlZCBieSBwYWdlJ3MgQ29udGVudCBTZWN1cml0eSBQb2xpY3kpIiwgIkxhYmVsIHRl
eHQgdG8gYmUgdXNlZCBpZiBwbHVnaW4gaXMgYmxvY2tlZCBieSBhIHBhZ2UncyBDb250ZW50IFNl
Y3VyaXR5IFBvbGljeSIpOwogfQogCiBTdHJpbmcgaW5zZWN1cmVQbHVnaW5WZXJzaW9uVGV4dCgp
CiB7Ci0gICAgcmV0dXJuIFdFQl9VSV9TVFJJTkcoIkJsb2NrZWQgUGx1Zy1pbiIsICJMYWJlbCB0
ZXh0IHRvIGJlIHVzZWQgd2hlbiBhbiBpbnNlY3VyZSBwbHVnLWluIHZlcnNpb24gd2FzIGJsb2Nr
ZWQgZnJvbSBsb2FkaW5nIik7CisgICAgcmV0dXJuIFdFQl9VSV9TVFJJTkdfS0VZKCJCbG9ja2Vk
IFBsdWctaW4iLCAiQmxvY2tlZCBQbHVnLUluIChJbnNlY3VyZSBwbHVnLWluKSIsICJMYWJlbCB0
ZXh0IHRvIGJlIHVzZWQgd2hlbiBhbiBpbnNlY3VyZSBwbHVnLWluIHZlcnNpb24gd2FzIGJsb2Nr
ZWQgZnJvbSBsb2FkaW5nIik7CiB9CiAKIFN0cmluZyBtdWx0aXBsZUZpbGVVcGxvYWRUZXh0KHVu
c2lnbmVkIG51bWJlck9mRmlsZXMpCg==
</data>
<flag name="review"
          id="174553"
          type_id="1"
          status="+"
          setter="sullivan"
    />
          </attachment>
      

    </bug>

</bugzilla>