<?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>43078</bug_id>
          
          <creation_ts>2010-07-27 14:20:00 -0700</creation_ts>
          <short_desc>Enhance FileThreadTask to support return type</short_desc>
          <delta_ts>2010-07-27 14:43:44 -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>WebCore JavaScript</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="Jian Li">jianli</reporter>
          <assigned_to name="Jian Li">jianli</assigned_to>
          <cc>dimich</cc>
    
    <cc>kinuko</cc>
    
    <cc>levin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>256834</commentid>
    <comment_count>0</comment_count>
    <who name="Jian Li">jianli</who>
    <bug_when>2010-07-27 14:20:00 -0700</bug_when>
    <thetext>Enhance FileThreadTask to support return type. This is needed to support synchronous calls of FileStream methods in addition to asynchronous calls via FileThreadTask.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>256836</commentid>
    <comment_count>1</comment_count>
      <attachid>62745</attachid>
    <who name="Jian Li">jianli</who>
    <bug_when>2010-07-27 14:27:01 -0700</bug_when>
    <thetext>Created attachment 62745
Proposed Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>256855</commentid>
    <comment_count>2</comment_count>
    <who name="Jian Li">jianli</who>
    <bug_when>2010-07-27 14:43:44 -0700</bug_when>
    <thetext>Committed as http://trac.webkit.org/changeset/64162.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>62745</attachid>
            <date>2010-07-27 14:27:01 -0700</date>
            <delta_ts>2010-07-27 14:32:49 -0700</delta_ts>
            <desc>Proposed Patch</desc>
            <filename>43078</filename>
            <type>text/plain</type>
            <size>5942</size>
            <attacher name="Jian Li">jianli</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
NDM5YTk5Zi4uMDIyNDNmOCAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxNiBAQAorMjAxMC0wNy0yNyAgSmlhbiBMaSAgPGpp
YW5saUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgRW5oYW5jZSBGaWxlVGhyZWFkVGFzayB0byBzdXBwb3J0IHJldHVybiB0eXBl
LgorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9NDMwNzgK
KworICAgICAgICBUaGlzIGlzIG5lZWRlZCB0byBzdXBwb3J0IHN5bmNocm9ub3VzIGNhbGxzIG9m
IEZpbGVTdHJlYW0gbWV0aG9kcyBpbgorICAgICAgICBhZGRpdGlvbiB0byBhc3luY2hyb25vdXMg
Y2FsbHMgdmlhIEZpbGVUaHJlYWRUYXNrLgorCisgICAgICAgICogaHRtbC9GaWxlVGhyZWFkVGFz
ay5oOgorICAgICAgICAoV2ViQ29yZTo6Y3JlYXRlRmlsZVRocmVhZFRhc2spOgorCiAyMDEwLTA3
LTI3ICBTaW1vbiBGcmFzZXIgIDxzaW1vbi5mcmFzZXJAYXBwbGUuY29tPgogCiAgICAgICAgIFJl
dmlld2VkIGJ5IERhcmluIEFkbGVyLgpkaWZmIC0tZ2l0IGEvV2ViQ29yZS9odG1sL0ZpbGVUaHJl
YWRUYXNrLmggYi9XZWJDb3JlL2h0bWwvRmlsZVRocmVhZFRhc2suaAppbmRleCBiYWYwODg4Li5k
ZTExNzk4IDEwMDY0NAotLS0gYS9XZWJDb3JlL2h0bWwvRmlsZVRocmVhZFRhc2suaAorKysgYi9X
ZWJDb3JlL2h0bWwvRmlsZVRocmVhZFRhc2suaApAQCAtNTcsMTAgKzU3LDExIEBAIHRlbXBsYXRl
PHR5cGVuYW1lIFQ+IHN0cnVjdCBGaWxlVGhyZWFkVGFza1RyYWl0czxQYXNzT3duUHRyPFQ+ID4g
ewogICAgIHR5cGVkZWYgUGFzc093blB0cjxUPiBQYXJhbVR5cGU7CiB9OwogCit0ZW1wbGF0ZTx0
eXBlbmFtZSBSPgogY2xhc3MgRmlsZVRocmVhZFRhc2swIDogcHVibGljIEZpbGVUaHJlYWQ6OlRh
c2sgewogcHVibGljOgotICAgIHR5cGVkZWYgdm9pZCAoRmlsZVN0cmVhbTo6Kk1ldGhvZCkoKTsK
LSAgICB0eXBlZGVmIEZpbGVUaHJlYWRUYXNrMCBGaWxlVGhyZWFkVGFzazsKKyAgICB0eXBlZGVm
IFIgKEZpbGVTdHJlYW06OipNZXRob2QpKCk7CisgICAgdHlwZWRlZiBGaWxlVGhyZWFkVGFzazA8
Uj4gRmlsZVRocmVhZFRhc2s7CiAKICAgICBzdGF0aWMgUGFzc093blB0cjxGaWxlVGhyZWFkVGFz
az4gY3JlYXRlKEZpbGVTdHJlYW0qIHN0cmVhbSwgTWV0aG9kIG1ldGhvZCkKICAgICB7CkBAIC04
MywxMSArODQsMTEgQEAgcHJpdmF0ZToKICAgICBNZXRob2QgbV9tZXRob2Q7CiB9OwogCi10ZW1w
bGF0ZTx0eXBlbmFtZSBQMSwgdHlwZW5hbWUgTVAxPgordGVtcGxhdGU8dHlwZW5hbWUgUiwgdHlw
ZW5hbWUgUDEsIHR5cGVuYW1lIE1QMT4KIGNsYXNzIEZpbGVUaHJlYWRUYXNrMSA6IHB1YmxpYyBG
aWxlVGhyZWFkOjpUYXNrIHsKIHB1YmxpYzoKLSAgICB0eXBlZGVmIHZvaWQgKEZpbGVTdHJlYW06
OipNZXRob2QpKE1QMSk7Ci0gICAgdHlwZWRlZiBGaWxlVGhyZWFkVGFzazE8UDEsIE1QMT4gRmls
ZVRocmVhZFRhc2s7CisgICAgdHlwZWRlZiBSIChGaWxlU3RyZWFtOjoqTWV0aG9kKShNUDEpOwor
ICAgIHR5cGVkZWYgRmlsZVRocmVhZFRhc2sxPFIsIFAxLCBNUDE+IEZpbGVUaHJlYWRUYXNrOwog
ICAgIHR5cGVkZWYgdHlwZW5hbWUgRmlsZVRocmVhZFRhc2tUcmFpdHM8UDE+OjpQYXJhbVR5cGUg
UGFyYW0xOwogCiAgICAgc3RhdGljIFBhc3NPd25QdHI8RmlsZVRocmVhZFRhc2s+IGNyZWF0ZShG
aWxlU3RyZWFtKiBzdHJlYW0sIE1ldGhvZCBtZXRob2QsIFBhcmFtMSBwYXJhbWV0ZXIxKQpAQCAt
MTEzLDExICsxMTQsMTEgQEAgcHJpdmF0ZToKICAgICBQMSBtX3BhcmFtZXRlcjE7CiB9OwogCi10
ZW1wbGF0ZTx0eXBlbmFtZSBQMSwgdHlwZW5hbWUgTVAxLCB0eXBlbmFtZSBQMiwgdHlwZW5hbWUg
TVAyPgordGVtcGxhdGU8dHlwZW5hbWUgUiwgdHlwZW5hbWUgUDEsIHR5cGVuYW1lIE1QMSwgdHlw
ZW5hbWUgUDIsIHR5cGVuYW1lIE1QMj4KIGNsYXNzIEZpbGVUaHJlYWRUYXNrMiA6IHB1YmxpYyBG
aWxlVGhyZWFkOjpUYXNrIHsKIHB1YmxpYzoKLSAgICB0eXBlZGVmIHZvaWQgKEZpbGVTdHJlYW06
OipNZXRob2QpKE1QMSwgTVAyKTsKLSAgICB0eXBlZGVmIEZpbGVUaHJlYWRUYXNrMjxQMSwgTVAx
LCBQMiwgTVAyPiBGaWxlVGhyZWFkVGFzazsKKyAgICB0eXBlZGVmIFIgKEZpbGVTdHJlYW06OipN
ZXRob2QpKE1QMSwgTVAyKTsKKyAgICB0eXBlZGVmIEZpbGVUaHJlYWRUYXNrMjxSLCBQMSwgTVAx
LCBQMiwgTVAyPiBGaWxlVGhyZWFkVGFzazsKICAgICB0eXBlZGVmIHR5cGVuYW1lIEZpbGVUaHJl
YWRUYXNrVHJhaXRzPFAxPjo6UGFyYW1UeXBlIFBhcmFtMTsKICAgICB0eXBlZGVmIHR5cGVuYW1l
IEZpbGVUaHJlYWRUYXNrVHJhaXRzPFAyPjo6UGFyYW1UeXBlIFBhcmFtMjsKIApAQCAtMTQ2LDEx
ICsxNDcsMTEgQEAgcHJpdmF0ZToKICAgICBQMiBtX3BhcmFtZXRlcjI7CiB9OwogCi10ZW1wbGF0
ZTx0eXBlbmFtZSBQMSwgdHlwZW5hbWUgTVAxLCB0eXBlbmFtZSBQMiwgdHlwZW5hbWUgTVAyLCB0
eXBlbmFtZSBQMywgdHlwZW5hbWUgTVAzPgordGVtcGxhdGU8dHlwZW5hbWUgUiwgdHlwZW5hbWUg
UDEsIHR5cGVuYW1lIE1QMSwgdHlwZW5hbWUgUDIsIHR5cGVuYW1lIE1QMiwgdHlwZW5hbWUgUDMs
IHR5cGVuYW1lIE1QMz4KIGNsYXNzIEZpbGVUaHJlYWRUYXNrMyA6IHB1YmxpYyBGaWxlVGhyZWFk
OjpUYXNrIHsKIHB1YmxpYzoKLSAgICB0eXBlZGVmIHZvaWQgKEZpbGVTdHJlYW06OipNZXRob2Qp
KE1QMSwgTVAyLCBNUDMpOwotICAgIHR5cGVkZWYgRmlsZVRocmVhZFRhc2szPFAxLCBNUDEsIFAy
LCBNUDIsIFAzLCBNUDM+IEZpbGVUaHJlYWRUYXNrOworICAgIHR5cGVkZWYgUiAoRmlsZVN0cmVh
bTo6Kk1ldGhvZCkoTVAxLCBNUDIsIE1QMyk7CisgICAgdHlwZWRlZiBGaWxlVGhyZWFkVGFzazM8
UiwgUDEsIE1QMSwgUDIsIE1QMiwgUDMsIE1QMz4gRmlsZVRocmVhZFRhc2s7CiAgICAgdHlwZWRl
ZiB0eXBlbmFtZSBGaWxlVGhyZWFkVGFza1RyYWl0czxQMT46OlBhcmFtVHlwZSBQYXJhbTE7CiAg
ICAgdHlwZWRlZiB0eXBlbmFtZSBGaWxlVGhyZWFkVGFza1RyYWl0czxQMj46OlBhcmFtVHlwZSBQ
YXJhbTI7CiAgICAgdHlwZWRlZiB0eXBlbmFtZSBGaWxlVGhyZWFkVGFza1RyYWl0czxQMz46OlBh
cmFtVHlwZSBQYXJhbTM7CkBAIC0xODIsNTQgKzE4Myw1NiBAQCBwcml2YXRlOgogICAgIFAzIG1f
cGFyYW1ldGVyMzsKIH07CiAKK3RlbXBsYXRlPHR5cGVuYW1lIFI+CiBQYXNzT3duUHRyPEZpbGVU
aHJlYWQ6OlRhc2s+IGNyZWF0ZUZpbGVUaHJlYWRUYXNrKAogICAgIEZpbGVTdHJlYW0qIGNvbnN0
IGNhbGxlZSwKLSAgICB2b2lkIChGaWxlU3RyZWFtOjoqbWV0aG9kKSgpKTsKKyAgICBSIChGaWxl
U3RyZWFtOjoqbWV0aG9kKSgpKTsKIAordGVtcGxhdGU8dHlwZW5hbWUgUj4KIFBhc3NPd25QdHI8
RmlsZVRocmVhZDo6VGFzaz4gY3JlYXRlRmlsZVRocmVhZFRhc2soCiAgICAgRmlsZVN0cmVhbSog
Y29uc3QgY2FsbGVlLAotICAgIHZvaWQgKEZpbGVTdHJlYW06OiptZXRob2QpKCkpCisgICAgUiAo
RmlsZVN0cmVhbTo6Km1ldGhvZCkoKSkKIHsKLSAgICByZXR1cm4gRmlsZVRocmVhZFRhc2swOjpj
cmVhdGUoCisgICAgcmV0dXJuIEZpbGVUaHJlYWRUYXNrMDxSPjo6Y3JlYXRlKAogICAgICAgICBj
YWxsZWUsCiAgICAgICAgIG1ldGhvZCk7CiB9CiAKLXRlbXBsYXRlPHR5cGVuYW1lIFAxLCB0eXBl
bmFtZSBNUDE+Cit0ZW1wbGF0ZTx0eXBlbmFtZSBSLCB0eXBlbmFtZSBQMSwgdHlwZW5hbWUgTVAx
PgogUGFzc093blB0cjxGaWxlVGhyZWFkOjpUYXNrPiBjcmVhdGVGaWxlVGhyZWFkVGFzaygKICAg
ICBGaWxlU3RyZWFtKiBjb25zdCBjYWxsZWUsCi0gICAgdm9pZCAoRmlsZVN0cmVhbTo6Km1ldGhv
ZCkoTVAxKSwKKyAgICBSIChGaWxlU3RyZWFtOjoqbWV0aG9kKShNUDEpLAogICAgIGNvbnN0IFAx
JiBwYXJhbWV0ZXIxKQogewotICAgIHJldHVybiBGaWxlVGhyZWFkVGFzazE8dHlwZW5hbWUgQ3Jv
c3NUaHJlYWRDb3BpZXI8UDE+OjpUeXBlLCBNUDE+OjpjcmVhdGUoCisgICAgcmV0dXJuIEZpbGVU
aHJlYWRUYXNrMTxSLCB0eXBlbmFtZSBDcm9zc1RocmVhZENvcGllcjxQMT46OlR5cGUsIE1QMT46
OmNyZWF0ZSgKICAgICAgICAgY2FsbGVlLAogICAgICAgICBtZXRob2QsCiAgICAgICAgIENyb3Nz
VGhyZWFkQ29waWVyPFAxPjo6Y29weShwYXJhbWV0ZXIxKSk7CiB9CiAKLXRlbXBsYXRlPHR5cGVu
YW1lIFAxLCB0eXBlbmFtZSBNUDEsIHR5cGVuYW1lIFAyLCB0eXBlbmFtZSBNUDI+Cit0ZW1wbGF0
ZTx0eXBlbmFtZSBSLCB0eXBlbmFtZSBQMSwgdHlwZW5hbWUgTVAxLCB0eXBlbmFtZSBQMiwgdHlw
ZW5hbWUgTVAyPgogUGFzc093blB0cjxGaWxlVGhyZWFkOjpUYXNrPiBjcmVhdGVGaWxlVGhyZWFk
VGFzaygKICAgICBGaWxlU3RyZWFtKiBjb25zdCBjYWxsZWUsCi0gICAgdm9pZCAoRmlsZVN0cmVh
bTo6Km1ldGhvZCkoTVAxLCBNUDIpLAorICAgIFIgKEZpbGVTdHJlYW06OiptZXRob2QpKE1QMSwg
TVAyKSwKICAgICBjb25zdCBQMSYgcGFyYW1ldGVyMSwKICAgICBjb25zdCBQMiYgcGFyYW1ldGVy
MikKIHsKLSAgICByZXR1cm4gRmlsZVRocmVhZFRhc2syPHR5cGVuYW1lIENyb3NzVGhyZWFkQ29w
aWVyPFAxPjo6VHlwZSwgTVAxLCB0eXBlbmFtZSBDcm9zc1RocmVhZENvcGllcjxQMj46OlR5cGUs
IE1QMj46OmNyZWF0ZSgKKyAgICByZXR1cm4gRmlsZVRocmVhZFRhc2syPFIsIHR5cGVuYW1lIENy
b3NzVGhyZWFkQ29waWVyPFAxPjo6VHlwZSwgTVAxLCB0eXBlbmFtZSBDcm9zc1RocmVhZENvcGll
cjxQMj46OlR5cGUsIE1QMj46OmNyZWF0ZSgKICAgICAgICAgY2FsbGVlLAogICAgICAgICBtZXRo
b2QsCiAgICAgICAgIENyb3NzVGhyZWFkQ29waWVyPFAxPjo6Y29weShwYXJhbWV0ZXIxKSwKICAg
ICAgICAgQ3Jvc3NUaHJlYWRDb3BpZXI8UDI+Ojpjb3B5KHBhcmFtZXRlcjIpKTsKIH0KIAotdGVt
cGxhdGU8dHlwZW5hbWUgUDEsIHR5cGVuYW1lIE1QMSwgdHlwZW5hbWUgUDIsIHR5cGVuYW1lIE1Q
MiwgdHlwZW5hbWUgUDMsIHR5cGVuYW1lIE1QMz4KK3RlbXBsYXRlPHR5cGVuYW1lIFIsIHR5cGVu
YW1lIFAxLCB0eXBlbmFtZSBNUDEsIHR5cGVuYW1lIFAyLCB0eXBlbmFtZSBNUDIsIHR5cGVuYW1l
IFAzLCB0eXBlbmFtZSBNUDM+CiBQYXNzT3duUHRyPEZpbGVUaHJlYWQ6OlRhc2s+IGNyZWF0ZUZp
bGVUaHJlYWRUYXNrKAogICAgIEZpbGVTdHJlYW0qIGNvbnN0IGNhbGxlZSwKLSAgICB2b2lkIChG
aWxlU3RyZWFtOjoqbWV0aG9kKShNUDEsIE1QMiwgTVAzKSwKKyAgICBSIChGaWxlU3RyZWFtOjoq
bWV0aG9kKShNUDEsIE1QMiwgTVAzKSwKICAgICBjb25zdCBQMSYgcGFyYW1ldGVyMSwKICAgICBj
b25zdCBQMiYgcGFyYW1ldGVyMiwKICAgICBjb25zdCBQMyYgcGFyYW1ldGVyMykKIHsKLSAgICBy
ZXR1cm4gRmlsZVRocmVhZFRhc2szPHR5cGVuYW1lIENyb3NzVGhyZWFkQ29waWVyPFAxPjo6VHlw
ZSwgTVAxLCB0eXBlbmFtZSBDcm9zc1RocmVhZENvcGllcjxQMj46OlR5cGUsIE1QMiwgdHlwZW5h
bWUgQ3Jvc3NUaHJlYWRDb3BpZXI8UDM+OjpUeXBlLCBNUDM+OjpjcmVhdGUoCisgICAgcmV0dXJu
IEZpbGVUaHJlYWRUYXNrMzxSLCB0eXBlbmFtZSBDcm9zc1RocmVhZENvcGllcjxQMT46OlR5cGUs
IE1QMSwgdHlwZW5hbWUgQ3Jvc3NUaHJlYWRDb3BpZXI8UDI+OjpUeXBlLCBNUDIsIHR5cGVuYW1l
IENyb3NzVGhyZWFkQ29waWVyPFAzPjo6VHlwZSwgTVAzPjo6Y3JlYXRlKAogICAgICAgICBjYWxs
ZWUsCiAgICAgICAgIG1ldGhvZCwKICAgICAgICAgQ3Jvc3NUaHJlYWRDb3BpZXI8UDE+Ojpjb3B5
KHBhcmFtZXRlcjEpLAo=
</data>
<flag name="review"
          id="50983"
          type_id="1"
          status="+"
          setter="levin"
    />
    <flag name="commit-queue"
          id="50984"
          type_id="3"
          status="-"
          setter="jianli"
    />
          </attachment>
      

    </bug>

</bugzilla>