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

Unified Diff: third_party/WebKit/Source/core/loader/EmptyClients.h

Issue 2795113002: Move ownership of TextCheckerClientImpl to WebLocalFrameImpl (Closed)
Patch Set: Rebased 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
Index: third_party/WebKit/Source/core/loader/EmptyClients.h
diff --git a/third_party/WebKit/Source/core/loader/EmptyClients.h b/third_party/WebKit/Source/core/loader/EmptyClients.h
index 81fa4ed2bf6923acd2888aef65da3f9efcc70e66..3ccd5789b7f3da06e0a2b08e6c92f224f77da43d 100644
--- a/third_party/WebKit/Source/core/loader/EmptyClients.h
+++ b/third_party/WebKit/Source/core/loader/EmptyClients.h
@@ -369,15 +369,18 @@ class CORE_EXPORT EmptyLocalFrameClient : public LocalFrameClient {
std::unique_ptr<WebApplicationCacheHost> createApplicationCacheHost(
WebApplicationCacheHostClient*) override;
+ TextCheckerClient& textCheckerClient() const override;
+
protected:
EmptyLocalFrameClient() {}
};
class CORE_EXPORT EmptyTextCheckerClient : public TextCheckerClient {
- DISALLOW_NEW();
+ WTF_MAKE_NONCOPYABLE(EmptyTextCheckerClient);
+ USING_FAST_MALLOC(EmptyTextCheckerClient);
public:
- ~EmptyTextCheckerClient() {}
+ EmptyTextCheckerClient() {}
void checkSpellingOfString(const String&, int*, int*) override {}
void requestCheckingOfString(TextCheckingRequest*) override;
@@ -394,15 +397,9 @@ class EmptySpellCheckerClient : public SpellCheckerClient {
bool isSpellCheckingEnabled() override { return false; }
void toggleSpellCheckingEnabled() override {}
-
- TextCheckerClient& textChecker() override { return m_textCheckerClient; }
-
void updateSpellingUIWithMisspelledWord(const String&) override {}
void showSpellingUI(bool) override {}
bool spellingUIIsShowing() override { return false; }
-
- private:
- EmptyTextCheckerClient m_textCheckerClient;
};
class EmptyEditorClient final : public EditorClient {
« no previous file with comments | « third_party/WebKit/Source/core/frame/LocalFrameClient.h ('k') | third_party/WebKit/Source/core/loader/EmptyClients.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698