419918 - add memory pressure release valve to gtk embedding widget test harness

Status: RESOLVED FIXED

(Core Graveyard :: Embedding: GTK Widget, defect)

Description

[Christopher Blizzard (:blizzard)]

I've got a bit of code that lets you tell the memory pressure observers to release memory.  Good for doing testing in memory-constrained environments.  Taken from Stuart's memback extension.

Comment 1

[Christopher Blizzard (:blizzard)]

Attachment: memory pressure addition to TestGtkEmbed.cpp

Simple patch to fire the memory pressure observers from the widget when the menu item is selected.

Comment 2

[Doug Turner (:dougt)]

Comment on attachment 306091 [details] [diff] [review]
memory pressure addition to TestGtkEmbed.cpp

do you really want the:
g_print("menu_memory_cb\n");

Comment 3

[Christopher Blizzard (:blizzard)]

Yeah, all of the callbacks print stuff out.  It's a test harness, of course.

Comment 4

[Christopher Blizzard (:blizzard)]

Comment on attachment 306091 [details] [diff] [review]
memory pressure addition to TestGtkEmbed.cpp

Looking for post-b4 approval.  Should not affect the browser at all.  It's the embedding test harness.

Comment 5

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 306091 [details] [diff] [review]
memory pressure addition to TestGtkEmbed.cpp

a1.9=beltzner

Comment 6

[Christopher Blizzard (:blizzard)]

Checked in.  Thanks, guys!