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

Unified Diff: chrome/renderer/spellchecker/spellcheck_provider.cc

Issue 2226833003: Make WebSpellCheckClient aware of leak detection (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@SpellCheckerDebugInfo
Patch Set: Rename to cancelAllPendingRequests Created 4 years, 4 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/spellchecker/spellcheck_provider.cc
diff --git a/chrome/renderer/spellchecker/spellcheck_provider.cc b/chrome/renderer/spellchecker/spellcheck_provider.cc
index 53b30a9584f392be8d85377d1a85b8d619030859..8211fc94c39a4f2a3b1516e7bb16cd6476616486 100644
--- a/chrome/renderer/spellchecker/spellcheck_provider.cc
+++ b/chrome/renderer/spellchecker/spellcheck_provider.cc
@@ -175,6 +175,13 @@ void SpellCheckProvider::requestCheckingOfText(
UMA_HISTOGRAM_COUNTS("SpellCheck.api.async", text.length());
}
+void SpellCheckProvider::cancelAllPendingRequests() {
+ for (WebTextCheckCompletions::iterator iter(&text_check_completions_);
+ !iter.IsAtEnd(); iter.Advance())
yosin_UTC9 2016/08/10 01:34:34 nit: We should have brace "{}" for loop body since
Xiaocheng 2016/08/10 01:45:33 Done.
+ iter.GetCurrentValue()->didCancelCheckingText();
+ text_check_completions_.Clear();
+}
+
void SpellCheckProvider::showSpellingUI(bool show) {
#if defined(USE_BROWSER_SPELLCHECKER)
UMA_HISTOGRAM_BOOLEAN("SpellCheck.api.showUI", show);

Powered by Google App Engine
This is Rietveld 408576698