[spellcheck] Re-enable Windows tests

components/spellcheck/renderer/spellcheck_unittest.cc

 // Copyright (c) 2012 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/spellcheck/renderer/spellcheck.h"
 
 #include <stddef.h>
 
 #include <memory>
 #include <utility>
@@ skipping 160 matching lines @@
 //     non-English word;
 //   * Tests for the function with an invalid English word with a following
 //     space character;
 //   * Tests for the function with an invalid English word with a following
 //     non-English word, and;
 //   * Tests for the function with two invalid English words concatenated
 //     with space characters or non-English words.
 // A test with a "[ROBUSTNESS]" mark shows it is a robustness test and it uses
 // grammatically incorrect string.
 // TODO(groby): Please feel free to add more tests.
-#if defined(OS_WIN) && !defined(NDEBUG)
-// Test times out on win dbg. crbug.com/678753.
-#define MAYBE_SpellCheckStrings_EN_US DISABLED_SpellCheckStrings_EN_US
-#else
-#define MAYBE_SpellCheckStrings_EN_US SpellCheckStrings_EN_US
-#endif
-TEST_F(SpellCheckTest, MAYBE_SpellCheckStrings_EN_US) {
+TEST_F(SpellCheckTest, SpellCheckStrings_EN_US) {
   static const struct {
     // A string to be tested.
     const wchar_t* input;
     // An expected result for this test case.
     //   * true: the input string does not have any invalid words.
     //   * false: the input string has one or more invalid words.
     bool expected_result;
     // The position and the length of the first invalid word.
     int misspelling_start;
     int misspelling_length;
@@ skipping 290 matching lines @@
         break;
       }
     }
 
     EXPECT_TRUE(suggested_word_is_present);
   }
 }
 
 // This test verifies our spellchecker can split a text into words and check
 // the spelling of each word in the text.
-#if defined(THREAD_SANITIZER) || defined(OS_WIN)
+#if defined(THREAD_SANITIZER)

[groby-ooo-7-16, 2017/01/23 21:08:11]

Once this has landed, could you give this another try under TSAN as well?

 // SpellCheckTest.SpellCheckText fails under ThreadSanitizer v2.
 // See http://crbug.com/217909.
-// Also fails on windows: crbug.com/678300.
 #define MAYBE_SpellCheckText DISABLED_SpellCheckText
 #else
 #define MAYBE_SpellCheckText SpellCheckText
 #endif  // THREAD_SANITIZER
 TEST_F(SpellCheckTest, MAYBE_SpellCheckText) {
   static const struct {
     const char* language;
     const wchar_t* input;
   } kTestCases[] = {
     {
@@ skipping 1058 matching lines @@
   // to be updated accordingly.
   ASSERT_EQ(5, spellcheck::kMaxSuggestions);
   FillSuggestions(suggestions_list, &suggestion_results);
   ASSERT_EQ(5U, suggestion_results.size());
   EXPECT_EQ(base::ASCIIToUTF16("0foo"), suggestion_results[0]);
   EXPECT_EQ(base::ASCIIToUTF16("1foo"), suggestion_results[1]);
   EXPECT_EQ(base::ASCIIToUTF16("2foo"), suggestion_results[2]);
   EXPECT_EQ(base::ASCIIToUTF16("0bar"), suggestion_results[3]);
   EXPECT_EQ(base::ASCIIToUTF16("1bar"), suggestion_results[4]);
 }