<?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>31807</bug_id>
          
          <creation_ts>2009-11-23 08:44:17 -0800</creation_ts>
          <short_desc>[Android] Android port lacks platform-specific WTF thread functions</short_desc>
          <delta_ts>2009-11-25 09:48:34 -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>JavaScriptCore</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Android</rep_platform>
          <op_sys>Android</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="Steve Block">steveblock</reporter>
          <assigned_to name="Steve Block">steveblock</assigned_to>
          <cc>android-webkit-unforking</cc>
    
    <cc>ap</cc>
    
    <cc>commit-queue</cc>
    
    <cc>dimich</cc>
    
    <cc>steveblock</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>165863</commentid>
    <comment_count>0</comment_count>
    <who name="Steve Block">steveblock</who>
    <bug_when>2009-11-23 08:44:17 -0800</bug_when>
    <thetext>Android port lacks platform-specific WTF thread functions
- initializeMainThreadPlatform()
- scheduleDispatchFunctionsOnMainThread()</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165866</commentid>
    <comment_count>1</comment_count>
      <attachid>43718</attachid>
    <who name="Steve Block">steveblock</who>
    <bug_when>2009-11-23 08:54:37 -0800</bug_when>
    <thetext>Created attachment 43718
Patch 1 for Bug 31807</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165896</commentid>
    <comment_count>2</comment_count>
      <attachid>43718</attachid>
    <who name="Dmitry Titov">dimich</who>
    <bug_when>2009-11-23 12:06:33 -0800</bug_when>
    <thetext>Comment on attachment 43718
Patch 1 for Bug 31807

r=me.

&gt; +using namespace android;
This seems not needed and can be removed on landing...</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165940</commentid>
    <comment_count>3</comment_count>
    <who name="Steve Block">steveblock</who>
    <bug_when>2009-11-23 14:38:38 -0800</bug_when>
    <thetext>&gt; &gt; +using namespace android;
&gt; This seems not needed and can be removed on landing...
JavaSharedClient will be added to WebKit/android and uses namespace android.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165945</commentid>
    <comment_count>4</comment_count>
      <attachid>43718</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-11-23 14:55:30 -0800</bug_when>
    <thetext>Comment on attachment 43718
Patch 1 for Bug 31807

Clearing flags on attachment: 43718

Committed r51323: &lt;http://trac.webkit.org/changeset/51323&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165946</commentid>
    <comment_count>5</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2009-11-23 14:55:35 -0800</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>165954</commentid>
    <comment_count>6</comment_count>
    <who name="Alexey Proskuryakov">ap</who>
    <bug_when>2009-11-23 15:52:16 -0800</bug_when>
    <thetext>Should we delete Android-specific code from ThreadingPthreads.cpp now? I can&apos;t wait to get rid of layering violations in runThreadWithRegistration().</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>166493</commentid>
    <comment_count>7</comment_count>
    <who name="Steve Block">steveblock</who>
    <bug_when>2009-11-25 09:48:34 -0800</bug_when>
    <thetext>(In reply to comment #6)
&gt; Should we delete Android-specific code from ThreadingPthreads.cpp now? I can&apos;t
&gt; wait to get rid of layering violations in runThreadWithRegistration().
No, I&apos;m afraid this is still required.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>43718</attachid>
            <date>2009-11-23 08:54:37 -0800</date>
            <delta_ts>2009-11-23 14:55:30 -0800</delta_ts>
            <desc>Patch 1 for Bug 31807</desc>
            <filename>addMainThreadAndroid.txt</filename>
            <type>text/plain</type>
            <size>2830</size>
            <attacher name="Steve Block">steveblock</attacher>
            
              <data encoding="base64">SW5kZXg6IEphdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwo9PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0
Q29yZS9DaGFuZ2VMb2cJKHJldmlzaW9uIDUxMzEwKQorKysgSmF2YVNjcmlwdENvcmUvQ2hhbmdl
TG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTYgQEAKKzIwMDktMTEtMjMgIFN0ZXZlIEJs
b2NrICA8c3RldmVibG9ja0Bnb29nbGUuY29tPgorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9E
WSAoT09QUyEpLgorCisgICAgICAgIEFkZHMgTWFpblRocmVhZEFuZHJvaWQuY3BwIHdpdGggQW5k
cm9pZC1zcGVjaWZpYyBXVEYgdGhyZWFkaW5nIGZ1bmN0aW9ucy4KKyAgICAgICAgaHR0cHM6Ly9i
dWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTMxODA3CisKKyAgICAgICAgKiB3dGYvYW5k
cm9pZDogQWRkZWQuCisgICAgICAgICogd3RmL2FuZHJvaWQvTWFpblRocmVhZEFuZHJvaWQuY3Bw
OiBBZGRlZC4KKyAgICAgICAgKFdURjo6dGltZW91dEZpcmVkKToKKyAgICAgICAgKFdURjo6aW5p
dGlhbGl6ZU1haW5UaHJlYWRQbGF0Zm9ybSk6CisgICAgICAgIChXVEY6OnNjaGVkdWxlRGlzcGF0
Y2hGdW5jdGlvbnNPbk1haW5UaHJlYWQpOgorCiAyMDA5LTExLTIzICBMYXN6bG8gR29tYm9zICA8
bGFzemxvLjEuZ29tYm9zQG5va2lhLmNvbT4KIAogICAgICAgICBSZXZpZXdlZCBieSBLZW5uZXRo
IFJvaGRlIENocmlzdGlhbnNlbi4KSW5kZXg6IEphdmFTY3JpcHRDb3JlL3d0Zi9hbmRyb2lkL01h
aW5UaHJlYWRBbmRyb2lkLmNwcAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBKYXZhU2NyaXB0Q29yZS93dGYvYW5k
cm9pZC9NYWluVGhyZWFkQW5kcm9pZC5jcHAJKHJldmlzaW9uIDApCisrKyBKYXZhU2NyaXB0Q29y
ZS93dGYvYW5kcm9pZC9NYWluVGhyZWFkQW5kcm9pZC5jcHAJKHJldmlzaW9uIDApCkBAIC0wLDAg
KzEsNTAgQEAKKy8qCisgKiBDb3B5cmlnaHQgMjAwOSwgVGhlIEFuZHJvaWQgT3BlbiBTb3VyY2Ug
UHJvamVjdAorICoKKyAqIFJlZGlzdHJpYnV0aW9uIGFuZCB1c2UgaW4gc291cmNlIGFuZCBiaW5h
cnkgZm9ybXMsIHdpdGggb3Igd2l0aG91dAorICogbW9kaWZpY2F0aW9uLCBhcmUgcGVybWl0dGVk
IHByb3ZpZGVkIHRoYXQgdGhlIGZvbGxvd2luZyBjb25kaXRpb25zCisgKiBhcmUgbWV0OgorICog
ICogUmVkaXN0cmlidXRpb25zIG9mIHNvdXJjZSBjb2RlIG11c3QgcmV0YWluIHRoZSBhYm92ZSBj
b3B5cmlnaHQKKyAqICAgIG5vdGljZSwgdGhpcyBsaXN0IG9mIGNvbmRpdGlvbnMgYW5kIHRoZSBm
b2xsb3dpbmcgZGlzY2xhaW1lci4KKyAqICAqIFJlZGlzdHJpYnV0aW9ucyBpbiBiaW5hcnkgZm9y
bSBtdXN0IHJlcHJvZHVjZSB0aGUgYWJvdmUgY29weXJpZ2h0CisgKiAgICBub3RpY2UsIHRoaXMg
bGlzdCBvZiBjb25kaXRpb25zIGFuZCB0aGUgZm9sbG93aW5nIGRpc2NsYWltZXIgaW4gdGhlCisg
KiAgICBkb2N1bWVudGF0aW9uIGFuZC9vciBvdGhlciBtYXRlcmlhbHMgcHJvdmlkZWQgd2l0aCB0
aGUgZGlzdHJpYnV0aW9uLgorICoKKyAqIFRISVMgU09GVFdBUkUgSVMgUFJPVklERUQgQlkgVEhF
IENPUFlSSUdIVCBIT0xERVJTIGBgQVMgSVMnJyBBTkQgQU5ZCisgKiBFWFBSRVNTIE9SIElNUExJ
RUQgV0FSUkFOVElFUywgSU5DTFVESU5HLCBCVVQgTk9UIExJTUlURUQgVE8sIFRIRQorICogSU1Q
TElFRCBXQVJSQU5USUVTIE9GIE1FUkNIQU5UQUJJTElUWSBBTkQgRklUTkVTUyBGT1IgQSBQQVJU
SUNVTEFSCisgKiBQVVJQT1NFIEFSRSBESVNDTEFJTUVELiAgSU4gTk8gRVZFTlQgU0hBTEwgVEhF
IENPUFlSSUdIVCBPV05FUiBPUgorICogQ09OVFJJQlVUT1JTIEJFIExJQUJMRSBGT1IgQU5ZIERJ
UkVDVCwgSU5ESVJFQ1QsIElOQ0lERU5UQUwsIFNQRUNJQUwsCisgKiBFWEVNUExBUlksIE9SIENP
TlNFUVVFTlRJQUwgREFNQUdFUyAoSU5DTFVESU5HLCBCVVQgTk9UIExJTUlURUQgVE8sCisgKiBQ
Uk9DVVJFTUVOVCBPRiBTVUJTVElUVVRFIEdPT0RTIE9SIFNFUlZJQ0VTOyBMT1NTIE9GIFVTRSwg
REFUQSwgT1IKKyAqIFBST0ZJVFM7IE9SIEJVU0lORVNTIElOVEVSUlVQVElPTikgSE9XRVZFUiBD
QVVTRUQgQU5EIE9OIEFOWSBUSEVPUlkKKyAqIE9GIExJQUJJTElUWSwgV0hFVEhFUiBJTiBDT05U
UkFDVCwgU1RSSUNUIExJQUJJTElUWSwgT1IgVE9SVAorICogKElOQ0xVRElORyBORUdMSUdFTkNF
IE9SIE9USEVSV0lTRSkgQVJJU0lORyBJTiBBTlkgV0FZIE9VVCBPRiBUSEUgVVNFCisgKiBPRiBU
SElTIFNPRlRXQVJFLCBFVkVOIElGIEFEVklTRUQgT0YgVEhFIFBPU1NJQklMSVRZIE9GIFNVQ0gg
REFNQUdFLgorICovCisKKyNpbmNsdWRlICJjb25maWcuaCIKKyNpbmNsdWRlICJNYWluVGhyZWFk
LmgiCisKKyNpbmNsdWRlICJKYXZhU2hhcmVkQ2xpZW50LmgiCisKK3VzaW5nIG5hbWVzcGFjZSBh
bmRyb2lkOworCituYW1lc3BhY2UgV1RGIHsKKworLy8gQ2FsbGJhY2sgaW4gdGhlIG1haW4gdGhy
ZWFkLgorc3RhdGljIHZvaWQgdGltZW91dEZpcmVkKHZvaWQqKQoreworICAgIGRpc3BhdGNoRnVu
Y3Rpb25zRnJvbU1haW5UaHJlYWQoKTsKK30KKwordm9pZCBpbml0aWFsaXplTWFpblRocmVhZFBs
YXRmb3JtKCkKK3sKK30KKwordm9pZCBzY2hlZHVsZURpc3BhdGNoRnVuY3Rpb25zT25NYWluVGhy
ZWFkKCkKK3sKKyAgICBKYXZhU2hhcmVkQ2xpZW50OjpFbnF1ZXVlRnVuY3Rpb25QdHIodGltZW91
dEZpcmVkLCAwKTsKK30KKworfSAvLyBuYW1lc3BhY2UgV1RGCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>