<?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>119663</bug_id>
          
          <creation_ts>2013-08-11 10:26:56 -0700</creation_ts>
          <short_desc>[Qt] [WebKit2] Remove duplicate RAM Size calculation and reuse WTF::ramSize()</short_desc>
          <delta_ts>2013-08-12 10:31:29 -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>WebKit2</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>EasyFix</keywords>
          <priority>P3</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Arunprasad Rajkumar">arurajku</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>abecsi</cc>
    
    <cc>allan.jensen</cc>
    
    <cc>ararunprasad</cc>
    
    <cc>cmarcelo</cc>
    
    <cc>commit-queue</cc>
    
    <cc>hausmann</cc>
    
    <cc>jturcotte</cc>
    
    <cc>menard</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>916129</commentid>
    <comment_count>0</comment_count>
    <who name="Arunprasad Rajkumar">arurajku</who>
    <bug_when>2013-08-11 10:26:56 -0700</bug_when>
    <thetext>WTF already provides function to calculate RAM size, WebProcessQt.cpp also has own implementation of function to get total RAM size.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>916131</commentid>
    <comment_count>1</comment_count>
      <attachid>208498</attachid>
    <who name="Arunprasad Rajkumar">arurajku</who>
    <bug_when>2013-08-11 10:28:49 -0700</bug_when>
    <thetext>Created attachment 208498
Patch</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>916332</commentid>
    <comment_count>2</comment_count>
      <attachid>208498</attachid>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2013-08-12 10:31:27 -0700</bug_when>
    <thetext>Comment on attachment 208498
Patch

Clearing flags on attachment: 208498

Committed r153949: &lt;http://trac.webkit.org/changeset/153949&gt;</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>916333</commentid>
    <comment_count>3</comment_count>
    <who name="WebKit Commit Bot">commit-queue</who>
    <bug_when>2013-08-12 10:31:29 -0700</bug_when>
    <thetext>All reviewed patches have been landed.  Closing bug.</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>208498</attachid>
            <date>2013-08-11 10:28:49 -0700</date>
            <delta_ts>2013-08-12 10:31:27 -0700</delta_ts>
            <desc>Patch</desc>
            <filename>bug-119663-20130811225827.patch</filename>
            <type>text/plain</type>
            <size>2833</size>
            <attacher name="Arunprasad Rajkumar">arurajku</attacher>
            
              <data encoding="base64">U3VidmVyc2lvbiBSZXZpc2lvbjogMTUzODg0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViS2l0Mi9D
aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViS2l0Mi9DaGFuZ2VMb2cKaW5kZXggOWViMjBjNjI1NDE3YjNm
NTNhMGJlNjkwNWFjODYxNDg5NWZhYjhiYy4uYTczYTM4OTZkZDRhNzhjNjJlZGQ1Y2Q1NmNjY2Qy
MjQzZjUyY2M0YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYktpdDIvQ2hhbmdlTG9nCisrKyBiL1Nv
dXJjZS9XZWJLaXQyL0NoYW5nZUxvZwpAQCAtMSwzICsxLDEzIEBACisyMDEzLTA4LTExICBBcnVu
cHJhc2FkIFJhamt1bWFyICA8YXJ1cmFqa3VAY2lzY28uY29tPgorCisgICAgICAgIFtRdF0gW1dl
YktpdDJdIFJlbW92ZSBkdXBsaWNhdGUgUkFNIFNpemUgY2FsY3VsYXRpb24gYW5kIHJldXNlIFdU
Rjo6cmFtU2l6ZSgpCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNn
aT9pZD0xMTk2NjMKKworICAgICAgICBSZXZpZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAg
ICAgICAqIFdlYlByb2Nlc3MvcXQvV2ViUHJvY2Vzc1F0LmNwcDoKKyAgICAgICAgKFdlYktpdDo6
V2ViUHJvY2Vzczo6cGxhdGZvcm1TZXRDYWNoZU1vZGVsKToKKwogMjAxMy0wOC0wOSAgQnJpYW4g
SG9sdCAgPGJyaWFuLmhvbHRAc2Ftc3VuZy5jb20+CiAKICAgICAgICAgW0dUS10gW1dlYktpdDJd
IEFkZCBhbiAnYXV0aGVudGljYXRlJyBzaWduYWwgdG8gV2ViS2l0V2ViVmlldwpkaWZmIC0tZ2l0
IGEvU291cmNlL1dlYktpdDIvV2ViUHJvY2Vzcy9xdC9XZWJQcm9jZXNzUXQuY3BwIGIvU291cmNl
L1dlYktpdDIvV2ViUHJvY2Vzcy9xdC9XZWJQcm9jZXNzUXQuY3BwCmluZGV4IDlhNjdlYmY0MjFh
YjVkNDVjYzRlNTRlNTQ0N2NiMTI1MjNlNzY2ZmMuLjBkY2FmOWQxNzNjYzg3NWIyODFiNzQwNjkz
YjE3MDQwZWUyNGU3OGEgMTAwNjQ0Ci0tLSBhL1NvdXJjZS9XZWJLaXQyL1dlYlByb2Nlc3MvcXQv
V2ViUHJvY2Vzc1F0LmNwcAorKysgYi9Tb3VyY2UvV2ViS2l0Mi9XZWJQcm9jZXNzL3F0L1dlYlBy
b2Nlc3NRdC5jcHAKQEAgLTQyLDYgKzQyLDcgQEAKICNpbmNsdWRlIDxXZWJDb3JlL01lbW9yeUNh
Y2hlLmg+CiAjaW5jbHVkZSA8V2ViQ29yZS9QYWdlQ2FjaGUuaD4KICNpbmNsdWRlIDxXZWJDb3Jl
L1J1bnRpbWVFbmFibGVkRmVhdHVyZXMuaD4KKyNpbmNsdWRlIDx3dGYvUkFNU2l6ZS5oPgogCiAj
aWYgZGVmaW5lZChRX09TX01BQ1gpCiAjaW5jbHVkZSA8ZGlzcGF0Y2gvZGlzcGF0Y2guaD4KQEAg
LTU2LDQyICs1Nyw5IEBAIHVzaW5nIG5hbWVzcGFjZSBXZWJDb3JlOwogCiBuYW1lc3BhY2UgV2Vi
S2l0IHsKIAotc3RhdGljIHVpbnQ2NF90IHBoeXNpY2FsTWVtb3J5U2l6ZUluQnl0ZXMoKQotewot
ICAgIHN0YXRpYyB1aW50NjRfdCBwaHlzaWNhbE1lbW9yeVNpemUgPSAwOwotCi0gICAgaWYgKCFw
aHlzaWNhbE1lbW9yeVNpemUpIHsKLSNpZiBkZWZpbmVkKFFfT1NfTUFDWCkKLSAgICAgICAgaG9z
dF9iYXNpY19pbmZvX2RhdGFfdCBob3N0SW5mbzsKLSAgICAgICAgbWFjaF9wb3J0X3QgaG9zdCA9
IG1hY2hfaG9zdF9zZWxmKCk7Ci0gICAgICAgIG1hY2hfbXNnX3R5cGVfbnVtYmVyX3QgY291bnQg
PSBIT1NUX0JBU0lDX0lORk9fQ09VTlQ7Ci0gICAgICAgIGtlcm5fcmV0dXJuX3QgciA9IGhvc3Rf
aW5mbyhob3N0LCBIT1NUX0JBU0lDX0lORk8sIChob3N0X2luZm9fdCkmaG9zdEluZm8sICZjb3Vu
dCk7Ci0gICAgICAgIG1hY2hfcG9ydF9kZWFsbG9jYXRlKG1hY2hfdGFza19zZWxmKCksIGhvc3Qp
OwotCi0gICAgICAgIGlmIChyID09IEtFUk5fU1VDQ0VTUykKLSAgICAgICAgICAgIHBoeXNpY2Fs
TWVtb3J5U2l6ZSA9IGhvc3RJbmZvLm1heF9tZW07Ci0KLSNlbGlmIGRlZmluZWQoUV9PU19XSU4p
Ci0gICAgICAgIE1FTU9SWVNUQVRVU0VYIHN0YXRleDsKLSAgICAgICAgc3RhdGV4LmR3TGVuZ3Ro
ID0gc2l6ZW9mKHN0YXRleCk7Ci0gICAgICAgIEdsb2JhbE1lbW9yeVN0YXR1c0V4KCZzdGF0ZXgp
OwotICAgICAgICBwaHlzaWNhbE1lbW9yeVNpemUgPSBzdGF0aWNfY2FzdDx1aW50NjRfdD4oc3Rh
dGV4LnVsbFRvdGFsUGh5cyk7Ci0KLSNlbHNlCi0gICAgICAgIGxvbmcgcGFnZVNpemUgPSBzeXNj
b25mKF9TQ19QQUdFU0laRSk7Ci0gICAgICAgIGxvbmcgbnVtYmVyT2ZQYWdlcyA9IHN5c2NvbmYo
X1NDX1BIWVNfUEFHRVMpOwotCi0gICAgICAgIGlmIChwYWdlU2l6ZSA+IDAgJiYgbnVtYmVyT2ZQ
YWdlcyA+IDApCi0gICAgICAgICAgICBwaHlzaWNhbE1lbW9yeVNpemUgPSBzdGF0aWNfY2FzdDx1
aW50NjRfdD4ocGFnZVNpemUpICogc3RhdGljX2Nhc3Q8dWludDY0X3Q+KG51bWJlck9mUGFnZXMp
OwotCi0jZW5kaWYKLSAgICB9Ci0gICAgcmV0dXJuIHBoeXNpY2FsTWVtb3J5U2l6ZTsKLX0KLQog
dm9pZCBXZWJQcm9jZXNzOjpwbGF0Zm9ybVNldENhY2hlTW9kZWwoQ2FjaGVNb2RlbCBjYWNoZU1v
ZGVsKQogewotICAgIHVpbnQ2NF90IHBoeXNpY2FsTWVtb3J5U2l6ZUluTWVnYWJ5dGVzID0gcGh5
c2ljYWxNZW1vcnlTaXplSW5CeXRlcygpIC8gMTAyNCAvIDEwMjQ7CisgICAgdWludDY0X3QgcGh5
c2ljYWxNZW1vcnlTaXplSW5NZWdhYnl0ZXMgPSBXVEY6OnJhbVNpemUoKSAvIDEwMjQgLyAxMDI0
OwogCiAgICAgLy8gVGhlIE1hYyBwb3J0IG9mIFdlYktpdDIgdXNlcyBhIGZ1ZGdlIGZhY3RvciBv
ZiAxMDAwIGhlcmUgdG8gYWNjb3VudCBmb3IgbWlzYWxpZ25tZW50LCBob3dldmVyLAogICAgIC8v
IHRoYXQgdGVuZHMgdG8gb3ZlcmVzdGltYXRlIHRoZSBtZW1vcnkgcXVpdGUgYSBiaXQgKDEgYnl0
ZSBtaXNhbGlnbm1lbnQgfiA0OCBNaUIgbWlzZXN0aW1hdGlvbikuCg==
</data>

          </attachment>
      

    </bug>

</bugzilla>