<?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>107860</bug_id>
          
          <creation_ts>2013-01-24 13:57:38 -0800</creation_ts>
          <short_desc>DFG::JITCompiler::getSpeculation() methods are badly named and superfluous</short_desc>
          <delta_ts>2013-01-24 14:00: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>JavaScriptCore</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>
          
          <blocked>106868</blocked>
          <everconfirmed>1</everconfirmed>
          <reporter name="Filip Pizlo">fpizlo</reporter>
          <assigned_to name="Filip Pizlo">fpizlo</assigned_to>
          <cc>barraclough</cc>
    
    <cc>ggaren</cc>
    
    <cc>mark.lam</cc>
    
    <cc>mhahnenberg</cc>
    
    <cc>msaboff</cc>
    
    <cc>oliver</cc>
    
    <cc>sam</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>815804</commentid>
    <comment_count>0</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2013-01-24 13:57:38 -0800</bug_when>
    <thetext>They should be called getPrediction().  But they&apos;re superfluous since it takes less typing to just call Node::prediction() anyway.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>815806</commentid>
    <comment_count>1</comment_count>
      <attachid>184574</attachid>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2013-01-24 13:58:45 -0800</bug_when>
    <thetext>Created attachment 184574
the patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>815810</commentid>
    <comment_count>2</comment_count>
      <attachid>184574</attachid>
    <who name="Mark Hahnenberg">mhahnenberg</who>
    <bug_when>2013-01-24 13:59:44 -0800</bug_when>
    <thetext>Comment on attachment 184574
the patch

r=me</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>815812</commentid>
    <comment_count>3</comment_count>
    <who name="Filip Pizlo">fpizlo</who>
    <bug_when>2013-01-24 14:00:58 -0800</bug_when>
    <thetext>Landed in http://trac.webkit.org/changeset/140719</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>184574</attachid>
            <date>2013-01-24 13:58:45 -0800</date>
            <delta_ts>2013-01-24 13:59:43 -0800</delta_ts>
            <desc>the patch</desc>
            <filename>blah.patch</filename>
            <type>text/plain</type>
            <size>2775</size>
            <attacher name="Filip Pizlo">fpizlo</attacher>
            
              <data encoding="base64">SW5kZXg6IFNvdXJjZS9KYXZhU2NyaXB0Q29yZS9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gU291
cmNlL0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkocmV2aXNpb24gMTQwNzE4KQorKysgU291cmNl
L0phdmFTY3JpcHRDb3JlL0NoYW5nZUxvZwkod29ya2luZyBjb3B5KQpAQCAtMSwzICsxLDE2IEBA
CisyMDEzLTAxLTI0ICBGaWxpcCBQaXpsbyAgPGZwaXpsb0BhcHBsZS5jb20+CisKKyAgICAgICAg
REZHOjpKSVRDb21waWxlcjo6Z2V0U3BlY3VsYXRpb24oKSBtZXRob2RzIGFyZSBiYWRseSBuYW1l
ZCBhbmQgc3VwZXJmbHVvdXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19i
dWcuY2dpP2lkPTEwNzg2MAorCisgICAgICAgIFJldmlld2VkIGJ5IE5PQk9EWSAoT09QUyEpLgor
CisgICAgICAgICogZGZnL0RGR0pJVENvbXBpbGVyLmg6CisgICAgICAgIChKSVRDb21waWxlcik6
CisgICAgICAgICogZGZnL0RGR1NwZWN1bGF0aXZlSklUNjQuY3BwOgorICAgICAgICAoSlNDOjpE
Rkc6OlNwZWN1bGF0aXZlSklUOjpjb21waWxlTG9naWNhbE5vdCk6CisgICAgICAgIChKU0M6OkRG
Rzo6U3BlY3VsYXRpdmVKSVQ6OmVtaXRCcmFuY2gpOgorCiAyMDEzLTAxLTI0ICBNYXJrIEhhaG5l
bmJlcmcgIDxtaGFobmVuYmVyZ0BhcHBsZS5jb20+CiAKICAgICAgICAgT2JqZWN0aXZlLUMgQVBJ
OiBSZW5hbWUgSlNWYWx1ZS5oL0FQSUpTVmFsdWUuaCB0byBKU0NKU1ZhbHVlLmgvSlNWYWx1ZS5o
CkluZGV4OiBTb3VyY2UvSmF2YVNjcmlwdENvcmUvZGZnL0RGR0pJVENvbXBpbGVyLmgKPT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PQotLS0gU291cmNlL0phdmFTY3JpcHRDb3JlL2RmZy9ERkdKSVRDb21waWxlci5oCShyZXZp
c2lvbiAxNDA0OTMpCisrKyBTb3VyY2UvSmF2YVNjcmlwdENvcmUvZGZnL0RGR0pJVENvbXBpbGVy
LmgJKHdvcmtpbmcgY29weSkKQEAgLTM0MCwxMSArMzQwLDYgQEAgcHVibGljOgogICAgICAgICBt
X2V4Y2VwdGlvbkNoZWNrcy5hcHBlbmQoQ2FsbEV4Y2VwdGlvblJlY29yZChmdW5jdGlvbkNhbGws
IGV4Y2VwdGlvbkNoZWNrLCBjb2RlT3JpZ2luKSk7CiAgICAgfQogICAgIAotICAgIC8vIEhlbHBl
ciBtZXRob2RzIHRvIGdldCBwcmVkaWN0aW9ucwotICAgIFNwZWN1bGF0ZWRUeXBlIGdldFNwZWN1
bGF0aW9uKE5vZGUmIG5vZGUpIHsgcmV0dXJuIG5vZGUucHJlZGljdGlvbigpOyB9Ci0gICAgU3Bl
Y3VsYXRlZFR5cGUgZ2V0U3BlY3VsYXRpb24oTm9kZUluZGV4IG5vZGVJbmRleCkgeyByZXR1cm4g
Z2V0U3BlY3VsYXRpb24oZ3JhcGgoKVtub2RlSW5kZXhdKTsgfQotICAgIFNwZWN1bGF0ZWRUeXBl
IGdldFNwZWN1bGF0aW9uKEVkZ2Ugbm9kZVVzZSkgeyByZXR1cm4gZ2V0U3BlY3VsYXRpb24obm9k
ZVVzZS5pbmRleCgpKTsgfQotCiAgICAgdm9pZCBhcHBlbmRFeGl0SW5mbyhNYWNyb0Fzc2VtYmxl
cjo6SnVtcExpc3QganVtcHNUb0ZhaWwgPSBNYWNyb0Fzc2VtYmxlcjo6SnVtcExpc3QoKSkKICAg
ICB7CiAgICAgICAgIE9TUkV4aXRDb21waWxhdGlvbkluZm8gaW5mbzsKSW5kZXg6IFNvdXJjZS9K
YXZhU2NyaXB0Q29yZS9kZmcvREZHU3BlY3VsYXRpdmVKSVQ2NC5jcHAKPT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0g
U291cmNlL0phdmFTY3JpcHRDb3JlL2RmZy9ERkdTcGVjdWxhdGl2ZUpJVDY0LmNwcAkocmV2aXNp
b24gMTQwNDkzKQorKysgU291cmNlL0phdmFTY3JpcHRDb3JlL2RmZy9ERkdTcGVjdWxhdGl2ZUpJ
VDY0LmNwcAkod29ya2luZyBjb3B5KQpAQCAtMTkzNCw3ICsxOTM0LDcgQEAgdm9pZCBTcGVjdWxh
dGl2ZUpJVDo6Y29tcGlsZUxvZ2ljYWxOb3QoTgogICAgICAgICByZXR1cm47CiAgICAgfQogICAg
IAotICAgIFNwZWN1bGF0ZWRUeXBlIHByZWRpY3Rpb24gPSBtX2ppdC5nZXRTcGVjdWxhdGlvbihu
b2RlLmNoaWxkMSgpKTsKKyAgICBTcGVjdWxhdGVkVHlwZSBwcmVkaWN0aW9uID0gYXQobm9kZS5j
aGlsZDEoKSkucHJlZGljdGlvbigpOwogICAgIGlmIChpc0Jvb2xlYW5TcGVjdWxhdGlvbihwcmVk
aWN0aW9uKSkgewogICAgICAgICBpZiAoaXNCb29sZWFuU3BlY3VsYXRpb24obV9zdGF0ZS5mb3JO
b2RlKG5vZGUuY2hpbGQxKCkpLm1fdHlwZSkpIHsKICAgICAgICAgICAgIFNwZWN1bGF0ZUJvb2xl
YW5PcGVyYW5kIHZhbHVlKHRoaXMsIG5vZGUuY2hpbGQxKCkpOwpAQCAtMjA2Niw3ICsyMDY2LDcg
QEAgdm9pZCBTcGVjdWxhdGl2ZUpJVDo6ZW1pdEJyYW5jaChOb2RlJiBubwogICAgICAgICBKU1Zh
bHVlT3BlcmFuZCB2YWx1ZSh0aGlzLCBub2RlLmNoaWxkMSgpKTsKICAgICAgICAgR1BSUmVnIHZh
bHVlR1BSID0gdmFsdWUuZ3ByKCk7CiAgICAgICAgIAotICAgICAgICBib29sIHByZWRpY3RCb29s
ZWFuID0gaXNCb29sZWFuU3BlY3VsYXRpb24obV9qaXQuZ2V0U3BlY3VsYXRpb24obm9kZS5jaGls
ZDEoKSkpOworICAgICAgICBib29sIHByZWRpY3RCb29sZWFuID0gaXNCb29sZWFuU3BlY3VsYXRp
b24oYXQobm9kZS5jaGlsZDEoKSkucHJlZGljdGlvbigpKTsKICAgICAKICAgICAgICAgaWYgKHBy
ZWRpY3RCb29sZWFuKSB7CiAgICAgICAgICAgICBpZiAoaXNCb29sZWFuU3BlY3VsYXRpb24obV9z
dGF0ZS5mb3JOb2RlKG5vZGUuY2hpbGQxKCkpLm1fdHlwZSkpIHsK
</data>
<flag name="review"
          id="203342"
          type_id="1"
          status="+"
          setter="mhahnenberg"
    />
          </attachment>
      

    </bug>

</bugzilla>