[Webkit-unassigned] [Bug 146685] [Win] Add memory pressure handler.

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=146685

--- Comment #3 from peavo at outlook.com ---
(In reply to comment #2)
> 

Thanks for reviewing :)

> > Source/WebCore/platform/win/MemoryPressureHandlerWin.cpp:85
> > +    // Only install a memory check timer for Win32, since Win64 has much more memory to play with.
> 
> This is not necessarily true.  64-bit windows can be installed on systems
> with 1GB of memory, for example.  I don't think the memory pressure handler
> should be 32-bit only.
> 

I agree, I will update the patch.

> > Source/WebCore/platform/win/MemoryPressureHandlerWin.cpp:88
> > +    memCheckTimer()->startRepeating(60.0);
> 
> This just checks the memory every minute, right?

Yes.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.webkit.org/pipermail/webkit-unassigned/attachments/20150707/14a88284/attachment-0001.html>