Bug 173264 – Call _sqlite3_purgeEligiblePagerCacheMemory() on memory pressure only if sqlite is initialized

• Status: RESOLVED
• Resolution: FIXED
• Component: Platform

[Chris Dumez]

Call _sqlite3_purgeEligiblePagerCacheMemory() on memory pressure only if sqlite is initialized.

[Chris Dumez]

Created attachment 312672 [details]
Patch

[Chris Dumez]

<rdar://problem/30780050>

[Ryosuke Niwa]

Comment on attachment 312672 [details]
Patch

Okay. Let's try this fix.

[Chris Dumez]

Comment on attachment 312672 [details]
Patch

Clearing flags on attachment: 312672

Committed r218108: <http://trac.webkit.org/changeset/218108>

[Chris Dumez]

All reviewed patches have been landed.  Closing bug.

[Darin Adler]

Comment on attachment 312672 [details]
Patch

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

> Source/WebCore/page/cocoa/MemoryReleaseCocoa.mm:43
> +static bool isSQLiteMemoryPressureHandlerRegisted { false };

Typo here: "registed" instead of "registered".

[Chris Dumez]

(In reply to Darin Adler from comment #6)
> Comment on attachment 312672 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=312672&action=review
> 
> > Source/WebCore/page/cocoa/MemoryReleaseCocoa.mm:43
> > +static bool isSQLiteMemoryPressureHandlerRegisted { false };
> 
> Typo here: "registed" instead of "registered".

Thanks, I have fixed the typo in <http://trac.webkit.org/changeset/218178>.