<?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>99360</bug_id>
          
          <creation_ts>2012-10-15 13:44:44 -0700</creation_ts>
          <short_desc>Remove page visibility hidden histograms</short_desc>
          <delta_ts>2012-10-15 17:53:36 -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>Tools / Tests</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="Julien Chaffraix">jchaffraix</reporter>
          <assigned_to name="Julien Chaffraix">jchaffraix</assigned_to>
          <cc>ojan</cc>
    
    <cc>tony</cc>
    
    <cc>webkit.review.bot</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>742375</commentid>
    <comment_count>0</comment_count>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2012-10-15 13:44:44 -0700</bug_when>
    <thetext>Those histograms were added in bug 85537 to investigate a memory optimization which was abandoned.

Discussing this with Ojan, we could just remove them now. I couldn&apos;t find a way of reusing them for other memory tracking purpose as they are very tied to the original optimization so I am going to remove them.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742378</commentid>
    <comment_count>1</comment_count>
      <attachid>168772</attachid>
    <who name="Julien Chaffraix">jchaffraix</who>
    <bug_when>2012-10-15 13:47:48 -0700</bug_when>
    <thetext>Created attachment 168772
Patch discussed with and reviewed by Ojan.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742662</commentid>
    <comment_count>2</comment_count>
      <attachid>168772</attachid>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-15 17:53:33 -0700</bug_when>
    <thetext>Comment on attachment 168772
Patch discussed with and reviewed by Ojan.

Clearing flags on attachment: 168772

Committed r131391: &lt;http://trac.webkit.org/changeset/131391&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>742663</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Review Bot">webkit.review.bot</who>
    <bug_when>2012-10-15 17:53:36 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>168772</attachid>
            <date>2012-10-15 13:47:48 -0700</date>
            <delta_ts>2012-10-15 17:53:33 -0700</delta_ts>
            <desc>Patch discussed with and reviewed by Ojan.</desc>
            <filename>bug-99360-20121015134638.patch</filename>
            <type>text/plain</type>
            <size>2231</size>
            <attacher name="Julien Chaffraix">jchaffraix</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTMxMzIxCmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggZTRlODk0OGZhNWE1Y2Nl
NWJjNzQ1ZjkzY2MyNzI2MzZmMzMxM2I5YS4uZmUyMGNmMmRiM2EwZWJmYzNiNWUyMDRmODY1MTNl
ODI3MWI4ODFhNCAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE4IEBACisyMDEyLTEwLTE1ICBKdWxp
ZW4gQ2hhZmZyYWl4ICA8amNoYWZmcmFpeEB3ZWJraXQub3JnPgorCisgICAgICAgIFJlbW92ZSBw
YWdlIHZpc2liaWxpdHkgaGlkZGVuIGhpc3RvZ3JhbXMKKyAgICAgICAgaHR0cHM6Ly9idWdzLndl
YmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTk5MzYwCisKKyAgICAgICAgUmV2aWV3ZWQgYnkgT2ph
biBWYWZhaS4KKworICAgICAgICBUaGUgMiBwYWdlIHZpc2liaWxpdHkgaGlkZGVuIGhpc3RvZ3Jh
bXMgd2VyZSBhZGRlZCBpbiBodHRwOi8vd2Via2l0Lm9yZy9iLzg1NTM3IHRvCisgICAgICAgIGlu
dmVzdGlnYXRlIGEgcG90ZW50aWFsIG9wdGltaXphdGlvbiB3aGVyZSB3ZSB3b3VsZCBkZXN0cm95
IHRoZSByZW5kZXIgdHJlZSB3aGVuCisgICAgICAgIGEgdGFiIHdhcyBoaWRkZW4uIFRoZSBvcHRp
bWl6YXRpb24gd2FzIGFiYW5kb25lZCBidXQgdGhlIGhpc3RvZ3JhbXMgd2VyZSBuZXZlcgorICAg
ICAgICByZW1vdmVkLiBUaGV5IGFyZSB0b28gbmFycm93IHRvIGJlIHJldXNlZCBzbyB0aGlzIGNo
YW5nZSByZW1vdmVzIHRoZW0uCisKKyAgICAgICAgKiBwYWdlL1BhZ2UuY3BwOgorICAgICAgICAo
V2ViQ29yZTo6UGFnZTo6c2V0VmlzaWJpbGl0eVN0YXRlKToKKwogMjAxMi0xMC0xNSAgTWlrZSBX
ZXN0ICA8bWt3c3RAY2hyb21pdW0ub3JnPgogCiAgICAgICAgIENTUCBzb3VyY2UgZXhwcmVzc2lv
bnMgc2hvdWxkIHN1cHBvcnQgcGF0aHMgYXQgZmlsZS1sZXZlbCBncmFudWxhcml0eS4KZGlmZiAt
LWdpdCBhL1NvdXJjZS9XZWJDb3JlL3BhZ2UvUGFnZS5jcHAgYi9Tb3VyY2UvV2ViQ29yZS9wYWdl
L1BhZ2UuY3BwCmluZGV4IDNkZjc1ODZlODY5YTJiOTA5NzI3ZmVmYzNiODZlZWY4OWY3NDAyZGUu
LjQ4NGExZWI5OTIwNmVkNTg2NzcwM2ZkNDgzNTM0NzQ1MzU3NDgzYjggMTAwNjQ0Ci0tLSBhL1Nv
dXJjZS9XZWJDb3JlL3BhZ2UvUGFnZS5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGFnZS9QYWdl
LmNwcApAQCAtNDQsNyArNDQsNiBAQAogI2luY2x1ZGUgIkZyYW1lVHJlZS5oIgogI2luY2x1ZGUg
IkZyYW1lVmlldy5oIgogI2luY2x1ZGUgIkhUTUxFbGVtZW50LmgiCi0jaW5jbHVkZSAiSGlzdG9n
cmFtU3VwcG9ydC5oIgogI2luY2x1ZGUgIkhpc3RvcnlJdGVtLmgiCiAjaW5jbHVkZSAiSW5zcGVj
dG9yQ29udHJvbGxlci5oIgogI2luY2x1ZGUgIkluc3BlY3Rvckluc3RydW1lbnRhdGlvbi5oIgpA
QCAtMTEyMSwxNCArMTEyMCw4IEBAIHZvaWQgUGFnZTo6c2V0VmlzaWJpbGl0eVN0YXRlKFBhZ2VW
aXNpYmlsaXR5U3RhdGUgdmlzaWJpbGl0eVN0YXRlLCBib29sIGlzSW5pdGlhCiAgICAgICAgIHJl
dHVybjsKICAgICBtX3Zpc2liaWxpdHlTdGF0ZSA9IHZpc2liaWxpdHlTdGF0ZTsKIAotICAgIGlm
ICghaXNJbml0aWFsU3RhdGUgJiYgbV9tYWluRnJhbWUpIHsKLSAgICAgICAgaWYgKHZpc2liaWxp
dHlTdGF0ZSA9PSBQYWdlVmlzaWJpbGl0eVN0YXRlSGlkZGVuKSB7Ci0gICAgICAgICAgICBBcmVu
YVNpemUgc2l6ZSA9IHJlbmRlclRyZWVTaXplKCk7Ci0gICAgICAgICAgICBIaXN0b2dyYW1TdXBw
b3J0OjpoaXN0b2dyYW1DdXN0b21Db3VudHMoIldlYkNvcmUuUGFnZS5yZW5kZXJUcmVlU2l6ZUJ5
dGVzIiwgc2l6ZS50cmVlU2l6ZSwgMTAwMCwgNTAwMDAwMDAwLCA1MCk7Ci0gICAgICAgICAgICBI
aXN0b2dyYW1TdXBwb3J0OjpoaXN0b2dyYW1DdXN0b21Db3VudHMoIldlYkNvcmUuUGFnZS5yZW5k
ZXJUcmVlQWxsb2NhdGVkQnl0ZXMiLCBzaXplLmFsbG9jYXRlZCwgMTAwMCwgNTAwMDAwMDAwLCA1
MCk7Ci0gICAgICAgIH0KKyAgICBpZiAoIWlzSW5pdGlhbFN0YXRlICYmIG1fbWFpbkZyYW1lKQog
ICAgICAgICBtX21haW5GcmFtZS0+ZGlzcGF0Y2hWaXNpYmlsaXR5U3RhdGVDaGFuZ2VFdmVudCgp
OwotICAgIH0KICNlbmRpZgogCiAjaWYgRU5BQkxFKEhJRERFTl9QQUdFX0RPTV9USU1FUl9USFJP
VFRMSU5HKQo=
</data>

          </attachment>
      

    </bug>

</bugzilla>