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

Unified Diff: components/test_runner/mock_grammar_check.cc

Issue 2211813002: Revert removal of grammar checking and marking code (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Minor fix 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
« no previous file with comments | « components/test_runner/mock_grammar_check.h ('k') | components/test_runner/spell_check_client.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/test_runner/mock_grammar_check.cc
diff --git a/components/test_runner/mock_grammar_check.cc b/components/test_runner/mock_grammar_check.cc
new file mode 100644
index 0000000000000000000000000000000000000000..912576624d7e2e1e88ab7007f6ad650b9c3ea208
--- /dev/null
+++ b/components/test_runner/mock_grammar_check.cc
@@ -0,0 +1,65 @@
+// Copyright 2013 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#include "components/test_runner/mock_grammar_check.h"
+
+#include <stddef.h>
+
+#include <algorithm>
+
+#include "base/logging.h"
+#include "base/macros.h"
+#include "components/test_runner/test_common.h"
+#include "third_party/WebKit/public/platform/WebString.h"
+#include "third_party/WebKit/public/web/WebTextCheckingResult.h"
+
+namespace test_runner {
+
+bool MockGrammarCheck::CheckGrammarOfString(
+ const blink::WebString& text,
+ std::vector<blink::WebTextCheckingResult>* results) {
+ DCHECK(results);
+ base::string16 string_text = text;
+ if (std::find_if(string_text.begin(), string_text.end(), IsASCIIAlpha) ==
+ string_text.end())
+ return true;
+
+ // Find matching grammatical errors from known ones. This function has to
+ // check all errors because the given text may consist of two or more
+ // sentences that have grammatical errors.
+ static const struct {
+ const char* text;
+ int location;
+ int length;
+ } kGrammarErrors[] = {
+ {"I have a issue.", 7, 1},
+ {"I have an grape.", 7, 2},
+ {"I have an kiwi.", 7, 2},
+ {"I have an muscat.", 7, 2},
+ {"You has the right.", 4, 3},
+ {"apple orange zz.", 0, 16},
+ {"apple zz orange.", 0, 16},
+ {"apple,zz,orange.", 0, 16},
+ {"orange,zz,apple.", 0, 16},
+ {"the the adlj adaasj sdklj. there there", 4, 3},
+ {"the the adlj adaasj sdklj. there there", 33, 5},
+ {"zz apple orange.", 0, 16},
+ };
+ for (size_t i = 0; i < arraysize(kGrammarErrors); ++i) {
+ size_t offset = 0;
+ base::string16 error(
+ kGrammarErrors[i].text,
+ kGrammarErrors[i].text + strlen(kGrammarErrors[i].text));
+ while ((offset = string_text.find(error, offset)) != base::string16::npos) {
+ results->push_back(
+ blink::WebTextCheckingResult(blink::WebTextDecorationTypeGrammar,
+ offset + kGrammarErrors[i].location,
+ kGrammarErrors[i].length));
+ offset += kGrammarErrors[i].length;
+ }
+ }
+ return false;
+}
+
+} // namespace test_runner
« no previous file with comments | « components/test_runner/mock_grammar_check.h ('k') | components/test_runner/spell_check_client.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698