Bug 91094 - [Qt] Implement MemoryUsageSupport
Summary: [Qt] Implement MemoryUsageSupport
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 420+
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Allan Sandfeld Jensen
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-07-12 07:47 PDT by Allan Sandfeld Jensen
Modified: 2012-07-12 10:08 PDT (History)
7 users (show)

See Also:


Attachments
Patch (10.09 KB, patch)
2012-07-12 07:54 PDT, Allan Sandfeld Jensen
no flags Details | Formatted Diff | Diff
Patch (10.33 KB, patch)
2012-07-12 08:12 PDT, Allan Sandfeld Jensen
abarth: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Allan Sandfeld Jensen 2012-07-12 07:47:19 PDT
We should support the MemoryUsageSupport class, it is currently only used by the V8 garbage-collector, but it can be usefull for us to measure run-time memory-usage more accurately and to implement a memory pressure handler.
Comment 1 Allan Sandfeld Jensen 2012-07-12 07:54:26 PDT
Created attachment 151952 [details]
Patch
Comment 2 WebKit Review Bot 2012-07-12 08:00:40 PDT
Comment on attachment 151952 [details]
Patch

Attachment 151952 [details] did not pass chromium-ews (chromium-xvfb):
Output: http://queues.webkit.org/results/13180834
Comment 3 Allan Sandfeld Jensen 2012-07-12 08:12:00 PDT
Created attachment 151957 [details]
Patch
Comment 4 Adam Barth 2012-07-12 08:23:02 PDT
Comment on attachment 151957 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=151957&action=review

> Source/WebCore/platform/MemoryUsageSupport.h:36
> +// MemoryUsageSupport is currently only used by the V8 garbage collector.

I wouldn't add this sort of comment.  It's likely to get out of date quickly.
Comment 5 Allan Sandfeld Jensen 2012-07-12 08:27:49 PDT
(In reply to comment #4)
> (From update of attachment 151957 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=151957&action=review
> 
> > Source/WebCore/platform/MemoryUsageSupport.h:36
> > +// MemoryUsageSupport is currently only used by the V8 garbage collector.
> 
> I wouldn't add this sort of comment.  It's likely to get out of date quickly.

Okay.
Comment 6 Allan Sandfeld Jensen 2012-07-12 08:34:53 PDT
Committed r122462: <http://trac.webkit.org/changeset/122462>
Comment 7 Csaba Osztrogonác 2012-07-12 09:15:34 PDT
(In reply to comment #6)
> Committed r122462: <http://trac.webkit.org/changeset/122462>

It broke the 4.8 builds, could you fix it, please?
/ramdisk/qt-linux-release/build/Source/WebCore/platform/qt/MemoryUsageSupportQt.cpp:25:45: error: JavaScriptCore/MemoryStatistics.h: No such file or directory
Comment 8 Allan Sandfeld Jensen 2012-07-12 10:08:07 PDT
(In reply to comment #7)
> (In reply to comment #6)
> > Committed r122462: <http://trac.webkit.org/changeset/122462>
> 
> It broke the 4.8 builds, could you fix it, please?
> /ramdisk/qt-linux-release/build/Source/WebCore/platform/qt/MemoryUsageSupportQt.cpp:25:45: error: JavaScriptCore/MemoryStatistics.h: No such file or directory
Fixed in r122474.