<?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>24145</bug_id>
          
          <creation_ts>2009-02-24 18:03:00 -0800</creation_ts>
          <short_desc>Remove &quot;#if ENABLE(WORKERS)&quot; wrap from CrossThreadCopier files.</short_desc>
          <delta_ts>2009-02-25 10:07:58 -0800</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 Misc.</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>0</everconfirmed>
          <reporter name="Jian Li">jianli</reporter>
          <assigned_to name="David Levin">levin</assigned_to>
          <cc>ap</cc>
    
    <cc>dimich</cc>
    
    <cc>jianli</cc>
    
    <cc>levin</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>110870</commentid>
    <comment_count>0</comment_count>
    <who name="Jian Li">jianli</who>
    <bug_when>2009-02-24 18:03:00 -0800</bug_when>
    <thetext>Need to remove &quot;#if ENABLE(WORKERS)&quot; wrap from CrossThreadCopier files.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110874</commentid>
    <comment_count>1</comment_count>
      <attachid>27946</attachid>
    <who name="Jian Li">jianli</who>
    <bug_when>2009-02-24 18:07:54 -0800</bug_when>
    <thetext>Created attachment 27946
Proposed Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110929</commentid>
    <comment_count>2</comment_count>
      <attachid>27946</attachid>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-02-25 02:26:58 -0800</bug_when>
    <thetext>Comment on attachment 27946
Proposed Patch

r=me

Please mention the rationale for changes like this (at least in bugzilla), for those who follow along at home. CrossThreadCopier can be used when communicating with non-worker threads, which is why it doesn&apos;t need the ENABLE() guards, and lives in platform/.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110967</commentid>
    <comment_count>3</comment_count>
    <who name="David Levin">levin</who>
    <bug_when>2009-02-25 09:11:57 -0800</bug_when>
    <thetext>Assigned to levin@chromium.org for landing.
</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>110978</commentid>
    <comment_count>4</comment_count>
    <who name="David Levin">levin</who>
    <bug_when>2009-02-25 10:07:58 -0800</bug_when>
    <thetext>Committed in r41217.

</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>27946</attachid>
            <date>2009-02-24 18:07:54 -0800</date>
            <delta_ts>2009-02-25 02:26:58 -0800</delta_ts>
            <desc>Proposed Patch</desc>
            <filename>Bug24145</filename>
            <type>text/plain</type>
            <size>1543</size>
            <attacher name="Jian Li">jianli</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL1dlYkNvcmUvQ2hhbmdlTG9nIGIvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXgg
MzNkMGMwNC4uMWEyMWNlOSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvV2Vi
Q29yZS9DaGFuZ2VMb2cKQEAgLTEsMyArMSwxMyBAQAorMjAwOS0wMi0yNCAgSmlhbiBMaSAgPGpp
YW5saUBjaHJvbWl1bS5vcmc+CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISku
CisKKyAgICAgICAgUmVtb3ZlICIjaWYgRU5BQkxFKFdPUktFUlMpIiB3cmFwIGZyb20gQ3Jvc3NU
aHJlYWRDb3BpZXIgZmlsZXMuCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3df
YnVnLmNnaT9pZD0yNDE0NQorCisgICAgICAgICogcGxhdGZvcm0vQ3Jvc3NUaHJlYWRDb3BpZXIu
Y3BwOgorICAgICAgICAqIHBsYXRmb3JtL0Nyb3NzVGhyZWFkQ29waWVyLmg6CisKIDIwMDktMDIt
MjQgIERhdmlkIExldmluICA8bGV2aW5AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFJldmlld2Vk
IGJ5IE5PQk9EWSAoYnVpbGQgZml4KS4KZGlmZiAtLWdpdCBhL1dlYkNvcmUvcGxhdGZvcm0vQ3Jv
c3NUaHJlYWRDb3BpZXIuY3BwIGIvV2ViQ29yZS9wbGF0Zm9ybS9Dcm9zc1RocmVhZENvcGllci5j
cHAKaW5kZXggMjdhZGYzYi4uOWNhNjI2ZiAxMDA2NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9D
cm9zc1RocmVhZENvcGllci5jcHAKKysrIGIvV2ViQ29yZS9wbGF0Zm9ybS9Dcm9zc1RocmVhZENv
cGllci5jcHAKQEAgLTMwLDggKzMwLDYgQEAKIAogI2luY2x1ZGUgImNvbmZpZy5oIgogCi0jaWYg
RU5BQkxFKFdPUktFUlMpCi0KICNpbmNsdWRlICJDcm9zc1RocmVhZENvcGllci5oIgogCiAjaW5j
bHVkZSAiUGxhdGZvcm1TdHJpbmcuaCIKQEAgLTYyLDUgKzYwLDMgQEAgQ3Jvc3NUaHJlYWRDb3Bp
ZXJCYXNlPGZhbHNlLCBSZXNvdXJjZVJlc3BvbnNlPjo6VHlwZSBDcm9zc1RocmVhZENvcGllckJh
c2U8ZmFsc2UKIH0KIAogfSAvLyBuYW1lc3BhY2UgV2ViQ29yZQotCi0jZW5kaWYgLy8gRU5BQkxF
KFdPUktFUlMpCmRpZmYgLS1naXQgYS9XZWJDb3JlL3BsYXRmb3JtL0Nyb3NzVGhyZWFkQ29waWVy
LmggYi9XZWJDb3JlL3BsYXRmb3JtL0Nyb3NzVGhyZWFkQ29waWVyLmgKaW5kZXggZDhjMWMyNS4u
ZjAxMmE1ZSAxMDA2NDQKLS0tIGEvV2ViQ29yZS9wbGF0Zm9ybS9Dcm9zc1RocmVhZENvcGllci5o
CisrKyBiL1dlYkNvcmUvcGxhdGZvcm0vQ3Jvc3NUaHJlYWRDb3BpZXIuaApAQCAtMzEsOCArMzEs
NiBAQAogI2lmbmRlZiBDcm9zc1RocmVhZENvcGllcl9oCiAjZGVmaW5lIENyb3NzVGhyZWFkQ29w
aWVyX2gKIAotI2lmIEVOQUJMRShXT1JLRVJTKQotCiAjaW5jbHVkZSA8bWVtb3J5PgogI2luY2x1
ZGUgPHd0Zi9QYXNzUmVmUHRyLmg+CiAjaW5jbHVkZSA8d3RmL1JlZlB0ci5oPgpAQCAtMTA4LDYg
KzEwNiw0IEBAIG5hbWVzcGFjZSBXZWJDb3JlIHsKIAogfSAvLyBuYW1lc3BhY2UgV2ViQ29yZQog
Ci0jZW5kaWYgLy8gRU5BQkxFKFdPUktFUlMpCi0KICNlbmRpZiAvLyBDcm9zc1RocmVhZENvcGll
cl9oCg==
</data>
<flag name="review"
          id="13611"
          type_id="1"
          status="+"
          setter="ap"
    />
          </attachment>
      

    </bug>

</bugzilla>