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

Unified Diff: components/spellcheck/renderer/spellcheck_provider.h

Issue 2799923003: Change SpellCheckProvider into a RenderFrameObserver (Closed)
Patch Set: Tue Apr 11 17:50:52 PDT 2017 Created 3 years, 8 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
« no previous file with comments | « components/spellcheck/renderer/spellcheck.cc ('k') | components/spellcheck/renderer/spellcheck_provider.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/spellcheck/renderer/spellcheck_provider.h
diff --git a/components/spellcheck/renderer/spellcheck_provider.h b/components/spellcheck/renderer/spellcheck_provider.h
index 450385299e2b6eac815f8239234dfcce46db5ee5..cead10758dccd52ad8765b733c2a450fd2270c8d 100644
--- a/components/spellcheck/renderer/spellcheck_provider.h
+++ b/components/spellcheck/renderer/spellcheck_provider.h
@@ -13,8 +13,8 @@
#include "base/id_map.h"
#include "base/macros.h"
#include "components/spellcheck/spellcheck_build_features.h"
-#include "content/public/renderer/render_view_observer.h"
-#include "content/public/renderer/render_view_observer_tracker.h"
+#include "content/public/renderer/render_frame_observer.h"
+#include "content/public/renderer/render_frame_observer_tracker.h"
#include "third_party/WebKit/public/web/WebTextCheckClient.h"
class SpellCheck;
@@ -25,17 +25,16 @@ class WebTextCheckingCompletion;
struct WebTextCheckingResult;
}
-// TODO(xiaochengh): Make SpellCheckProvider a RenderFrameObserver.
// This class deals with invoking browser-side spellcheck mechanism
// which is done asynchronously.
class SpellCheckProvider
- : public content::RenderViewObserver,
- public content::RenderViewObserverTracker<SpellCheckProvider>,
+ : public content::RenderFrameObserver,
+ public content::RenderFrameObserverTracker<SpellCheckProvider>,
public blink::WebTextCheckClient {
public:
using WebTextCheckCompletions = IDMap<blink::WebTextCheckingCompletion*>;
- SpellCheckProvider(content::RenderView* render_view,
+ SpellCheckProvider(content::RenderFrame* render_frame,
SpellCheck* spellcheck);
~SpellCheckProvider() override;
@@ -57,7 +56,7 @@ class SpellCheckProvider
// Enables document-wide spellchecking.
void EnableSpellcheck(bool enabled);
- // RenderViewObserver implementation.
+ // RenderFrameObserver implementation.
bool OnMessageReceived(const IPC::Message& message) override;
void FocusedNodeChanged(const blink::WebNode& node) override;
@@ -70,7 +69,7 @@ class SpellCheckProvider
bool SatisfyRequestFromCache(const base::string16& text,
blink::WebTextCheckingCompletion* completion);
- // RenderViewObserver implementation.
+ // RenderFrameObserver implementation.
void OnDestruct() override;
// blink::WebTextCheckClient implementation.
« no previous file with comments | « components/spellcheck/renderer/spellcheck.cc ('k') | components/spellcheck/renderer/spellcheck_provider.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698