<?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>38517</bug_id>
          
          <creation_ts>2010-05-04 07:09:31 -0700</creation_ts>
          <short_desc>Commit and Review policy should encourage unofficial reviews</short_desc>
          <delta_ts>2010-05-05 10:53:58 -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>WebKit Website</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="Shinichiro Hamaji">hamaji</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>220577</commentid>
    <comment_count>0</comment_count>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-05-04 07:09:31 -0700</bug_when>
    <thetext>It would be better to encourage unofficial reviews in

http://webkit.org/coding/commit-review-policy.html</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>220579</commentid>
    <comment_count>1</comment_count>
      <attachid>55017</attachid>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-05-04 07:10:39 -0700</bug_when>
    <thetext>Created attachment 55017
Patch v1</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221167</commentid>
    <comment_count>2</comment_count>
      <attachid>55017</attachid>
    <who name="Adam Barth">abarth</who>
    <bug_when>2010-05-05 10:12:06 -0700</bug_when>
    <thetext>Comment on attachment 55017
Patch v1

LGTM.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221193</commentid>
    <comment_count>3</comment_count>
      <attachid>55017</attachid>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-05-05 10:53:51 -0700</bug_when>
    <thetext>Comment on attachment 55017
Patch v1

Clearing flags on attachment: 55017

Committed r58827: &lt;http://trac.webkit.org/changeset/58827&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>221194</commentid>
    <comment_count>4</comment_count>
    <who name="Shinichiro Hamaji">hamaji</who>
    <bug_when>2010-05-05 10:53:58 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>55017</attachid>
            <date>2010-05-04 07:10:39 -0700</date>
            <delta_ts>2010-05-05 10:53:51 -0700</delta_ts>
            <desc>Patch v1</desc>
            <filename>bug-38517-20100504231037.patch</filename>
            <type>text/plain</type>
            <size>1384</size>
            <attacher name="Shinichiro Hamaji">hamaji</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYktpdFNpdGUvQ2hhbmdlTG9nIGIvV2ViS2l0U2l0ZS9DaGFuZ2VMb2cK
aW5kZXggNmJiM2IyODUzMWY3MmNhMTA3ZTkwNjJkZmU1NzZjZmQ1NWUxZWFjMS4uYzFkNmZhYmFh
M2QwOWM5YTJjMGFhMDI0ZWRlZDA0ZTQ1ZDY3YTY0NyAxMDA2NDQKLS0tIGEvV2ViS2l0U2l0ZS9D
aGFuZ2VMb2cKKysrIGIvV2ViS2l0U2l0ZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMiBAQAorMjAx
MC0wNS0wNCAgU2hpbmljaGlybyBIYW1hamkgIDxoYW1hamlAY2hyb21pdW0ub3JnPgorCisgICAg
ICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIENvbW1pdCBhbmQgUmV2
aWV3IHBvbGljeSBzaG91bGQgZW5jb3VyYWdlIHVub2ZmaWNpYWwgcmV2aWV3cworICAgICAgICBo
dHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9Mzg1MTcKKworICAgICAgICAq
IGNvZGluZy9jb21taXQtcmV2aWV3LXBvbGljeS5odG1sOgorCiAyMDEwLTA1LTAyICBEaXJrIFNj
aHVsemUgIDxrcml0QHdlYmtpdC5vcmc+CiAKICAgICAgICAgUmV2aWV3ZWQgYnkgTmlrb2xhcyBa
aW1tZXJtYW5uLgpkaWZmIC0tZ2l0IGEvV2ViS2l0U2l0ZS9jb2RpbmcvY29tbWl0LXJldmlldy1w
b2xpY3kuaHRtbCBiL1dlYktpdFNpdGUvY29kaW5nL2NvbW1pdC1yZXZpZXctcG9saWN5Lmh0bWwK
aW5kZXggMWQxODY4OGU5ZmZmNDQzZTUzODk1ZWQxNzNmMTRmM2ExMTEzZjQ5YS4uYzc4YjdkYWMy
ZjkwZGYwZDRhOTEyNTdiMjE2MDA3NDMzZTk0MjVjNiAxMDA2NDQKLS0tIGEvV2ViS2l0U2l0ZS9j
b2RpbmcvY29tbWl0LXJldmlldy1wb2xpY3kuaHRtbAorKysgYi9XZWJLaXRTaXRlL2NvZGluZy9j
b21taXQtcmV2aWV3LXBvbGljeS5odG1sCkBAIC04Nyw3ICs4NywxMCBAQCBhcmVhcy48L3A+CiAK
IDxwPkEgcGVyc29uIHdobyBzdWJtaXRzIG1hbnkgcGF0Y2hlcyBidXQgZG9lcyBub3Qgc2hvdyBn
b29kCiBjb2xsYWJvcmF0aW9uIHNraWxscywgY29kZSB1bmRlcnN0YW5kaW5nIG9yIHVuZGVyc3Rh
bmRpbmcgb2YgcHJvamVjdAotcG9saWNpZXMgbWF5IG5ldmVyIGJlIG5vbWluYXRlZC48L3A+Citw
b2xpY2llcyBtYXkgbmV2ZXIgYmUgbm9taW5hdGVkLiBNYWtpbmcgdW5vZmZpY2lhbCByZXZpZXdz
IGJlZm9yZSB5b3UKK2JlY29tZSBhIHJldmlld2VyIGlzIGVuY291cmFnZWQuIFRoaXMgaXMgYW4g
ZXhjZWxsZW50IHdheSB0byBzaG93IHlvdXIKK3NraWxscy4gTm90ZSB0aGF0IHlvdSBzaG91bGQg
bm90IHB1dCByKyBub3Igci0gb24gcGF0Y2hlcyBpbiBzdWNoCit1bm9mZmljaWFsIHJldmlld3Mu
PC9wPgogCiA8cD5Gb3IgUmV2aWV3ZXIgc3RhdHVzLCB0aGVyZSBpcyBubyBzdXBlcnZpc2lvbiBl
eGNlcHRpb24uPC9wPgogCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>