<?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>23891</bug_id>
          
          <creation_ts>2009-02-10 22:14:07 -0800</creation_ts>
          <short_desc>[WebHistoryItem _setVisitCount:] is unused and should be removed</short_desc>
          <delta_ts>2009-02-10 22:28:53 -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>History</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>Mac</rep_platform>
          <op_sys>OS X 10.5</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="John Sullivan">sullivan</reporter>
          <assigned_to name="John Sullivan">sullivan</assigned_to>
          
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>109249</commentid>
    <comment_count>0</comment_count>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2009-02-10 22:14:07 -0800</bug_when>
    <thetext>The method -[WebHistoryItem _setVisitCount:] is a synonym for the longstanding public API -[WebHistoryItem setVisitCount:]. It was added recently as a WebKit-internal method, but soon thereafter abandoned. It&apos;s just clogging up the series of tubes, and should be deleted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>109251</commentid>
    <comment_count>1</comment_count>
      <attachid>27553</attachid>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2009-02-10 22:18:45 -0800</bug_when>
    <thetext>Created attachment 27553
Patch to remove unused method.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>109255</commentid>
    <comment_count>2</comment_count>
    <who name="John Sullivan">sullivan</who>
    <bug_when>2009-02-10 22:28:53 -0800</bug_when>
    <thetext>Committed revision 40850.
</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>27553</attachid>
            <date>2009-02-10 22:18:45 -0800</date>
            <delta_ts>2009-02-10 22:20:49 -0800</delta_ts>
            <desc>Patch to remove unused method.</desc>
            <filename>23891_patch.txt</filename>
            <type>text/plain</type>
            <size>1750</size>
            <attacher name="John Sullivan">sullivan</attacher>
            
              <data encoding="base64">SW5kZXg6IG1hYy9DaGFuZ2VMb2cKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gbWFjL0NoYW5nZUxvZwkocmV2aXNp
b24gNDA4NDgpCisrKyBtYWMvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBAIC0xLDMgKzEsMTgg
QEAKKzIwMDktMDItMTAgIEpvaG4gU3VsbGl2YW4gIDxzdWxsaXZhbkBhcHBsZS5jb20+CisKKyAg
ICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZIChPT1BTISkuCisgICAgICAgIAorICAgICAgICA8aHR0
cHM6Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTIzODkxPgorICAgICAgICBbV2Vi
SGlzdG9yeUl0ZW0gX3NldFZpc2l0Q291bnQ6XSBpcyB1bnVzZWQgYW5kIHNob3VsZCBiZSByZW1v
dmVkCisKKyAgICAgICAgKiBIaXN0b3J5L1dlYkhpc3RvcnlJdGVtLm1tOgorICAgICAgICAoLVtX
ZWJIaXN0b3J5SXRlbSBfc2V0VmlzaXRDb3VudDpdKToKKyAgICAgICAgcmVtb3ZlZCB0aGlzIHVu
dXNlZCBtZXRob2QsIHdoaWNoIGlzIGEgc3lub255bSBmb3Igc2V0VmlzaXRDb3VudDogdGhhdCB3
YXMgaW50cm9kdWNlZAorICAgICAgICByZWNlbnRseSBhbmQgYWJhbmRvbmVkIGV2ZW4gbW9yZSBy
ZWNlbnRseQorCisgICAgICAgICogSGlzdG9yeS9XZWJIaXN0b3J5SXRlbUludGVybmFsLmg6Cisg
ICAgICAgIHJlbW92ZWQgZGVjbGFyYXRpb24gb2YgX3NldFZpc2l0Q291bnQ6CisKIDIwMDktMDIt
MTAgIEFuZGVycyBDYXJsc3NvbiAgPGFuZGVyc2NhQGFwcGxlLmNvbT4KIAogICAgICAgICBSZXZp
ZXdlZCBieSBNYXJrIFJvd2UuCkluZGV4OiBtYWMvSGlzdG9yeS9XZWJIaXN0b3J5SXRlbUludGVy
bmFsLmgKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQotLS0gbWFjL0hpc3RvcnkvV2ViSGlzdG9yeUl0ZW1JbnRlcm5hbC5o
CShyZXZpc2lvbiA0MDgyOSkKKysrIG1hYy9IaXN0b3J5L1dlYkhpc3RvcnlJdGVtSW50ZXJuYWwu
aAkod29ya2luZyBjb3B5KQpAQCAtNTIsNyArNTIsNiBAQAogLSAodm9pZClfbWVyZ2VBdXRvQ29t
cGxldGVIaW50czooV2ViSGlzdG9yeUl0ZW0gKilvdGhlckl0ZW07CiAtICh2b2lkKXNldFRpdGxl
OihOU1N0cmluZyAqKXRpdGxlOwogLSAodm9pZClfdmlzaXRlZFdpdGhUaXRsZTooTlNTdHJpbmcg
Kil0aXRsZTsKLS0gKHZvaWQpX3NldFZpc2l0Q291bnQ6KGludCljb3VudDsKIC0gKHZvaWQpX3Jl
Y29yZEluaXRpYWxWaXNpdDsKIAogQGVuZApJbmRleDogbWFjL0hpc3RvcnkvV2ViSGlzdG9yeUl0
ZW0ubW0KPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PQotLS0gbWFjL0hpc3RvcnkvV2ViSGlzdG9yeUl0ZW0ubW0JKHJldmlz
aW9uIDQwODI5KQorKysgbWFjL0hpc3RvcnkvV2ViSGlzdG9yeUl0ZW0ubW0JKHdvcmtpbmcgY29w
eSkKQEAgLTQxMywxMSArNDIyLDYgQEAKICAgICBjb3JlKF9wcml2YXRlKS0+dmlzaXRlZCh0aXRs
ZSwgW05TRGF0ZSB0aW1lSW50ZXJ2YWxTaW5jZVJlZmVyZW5jZURhdGVdKTsKIH0KIAotLSAodm9p
ZClfc2V0VmlzaXRDb3VudDooaW50KWNvdW50Ci17Ci0gICAgY29yZShfcHJpdmF0ZSktPnNldFZp
c2l0Q291bnQoY291bnQpOwotfQotCiAtICh2b2lkKV9yZWNvcmRJbml0aWFsVmlzaXQKIHsKICAg
ICBjb3JlKF9wcml2YXRlKS0+cmVjb3JkSW5pdGlhbFZpc2l0KCk7Cg==
</data>
<flag name="review"
          id="13346"
          type_id="1"
          status="+"
          setter="mitz"
    />
          </attachment>
      

    </bug>

</bugzilla>