Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(6301)

Unified Diff: chrome/renderer/chrome_content_renderer_client.h

Issue 7222023: Forward handling of low-mem message to the content renderer client. Add APIs to handle this case. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Get rid of notification. Forward call from RPO to client. Created 9 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: chrome/renderer/chrome_content_renderer_client.h
diff --git a/chrome/renderer/chrome_content_renderer_client.h b/chrome/renderer/chrome_content_renderer_client.h
index ad629bd7abeee5c0bcd6e2bc7b33dd97373138fc..8612c8aa7be0f255e1acbeab6eec55534e01df0a 100644
--- a/chrome/renderer/chrome_content_renderer_client.h
+++ b/chrome/renderer/chrome_content_renderer_client.h
@@ -15,6 +15,7 @@ class ExtensionDispatcher;
class RendererHistogramSnapshots;
class RendererNetPredictor;
class SpellCheck;
+class SpellCheckProvider;
class VisitedLinkSlave;
namespace safe_browsing {
@@ -73,6 +74,10 @@ class ChromeContentRendererClient : public content::ContentRendererClient {
// For testing.
void SetExtensionDispatcher(ExtensionDispatcher* extension_dispatcher);
+ // Called in low-memory conditions to dump the memory used by the spellchecker
jam 2011/07/22 16:31:50 nit: why add two methods? i.e. why not just have t
Greg Billock 2011/07/22 17:02:07 Good point. I liked having the caller call OnPurge
+ // and start over.
+ void OnPurgeMemory();
+
private:
WebKit::WebPlugin* CreatePluginPlaceholder(
RenderView* render_view,
@@ -88,11 +93,14 @@ class ChromeContentRendererClient : public content::ContentRendererClient {
// extension app's extent.
bool CrossesExtensionExtents(WebKit::WebFrame* frame, const GURL& new_url);
+ void ResetSpellCheck();
+
scoped_ptr<ChromeRenderProcessObserver> chrome_observer_;
scoped_ptr<ExtensionDispatcher> extension_dispatcher_;
scoped_ptr<RendererHistogramSnapshots> histogram_snapshots_;
scoped_ptr<RendererNetPredictor> net_predictor_;
scoped_ptr<SpellCheck> spellcheck_;
+ scoped_ptr<SpellCheckProvider> spellcheck_provider_;
scoped_ptr<VisitedLinkSlave> visited_link_slave_;
scoped_ptr<safe_browsing::PhishingClassifierFilter> phishing_classifier_;
};

Powered by Google App Engine
This is Rietveld 408576698