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

Unified Diff: third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h

Issue 1715203002: Stop async spellchecker before running the leak detector. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase + parameterize WebLeakDetector over WebFrames instead Created 4 years, 10 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/editing/spellcheck/SpellChecker.h
diff --git a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h
index 13dd1c0888b897d5d555ae571da74f57849a65bc..40bcdf9a82f4d6f61fa7a03418e88df383d163df 100644
--- a/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h
+++ b/third_party/WebKit/Source/core/editing/spellcheck/SpellChecker.h
@@ -90,6 +90,14 @@ public:
// Exposed for testing only
SpellCheckRequester& spellCheckRequester() const { return *m_spellCheckRequester; }
+ // The leak detector will report leaks should queued requests be posted
+ // while it GCs repeatedly, as the requests keep their associated element
+ // alive.
+ //
+ // Hence allow the leak detector to effectively stop the spell checker to
+ // ensure leak reporting stability.
+ void prepareForLeakDetection();
+
private:
explicit SpellChecker(LocalFrame&);

Powered by Google App Engine
This is Rietveld 408576698