Re-enable tests under mac-asan.

third_party/WebKit/Source/bindings/core/v8/ScriptStreamerTest.cpp

 // Copyright 2014 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 "bindings/core/v8/ScriptStreamer.h"
 
 #include "bindings/core/v8/ScriptSourceCode.h"
 #include "bindings/core/v8/ScriptStreamerThread.h"
 #include "bindings/core/v8/V8Binding.h"
@@ skipping 91 matching lines @@
 
     void notifyFinished(Resource*) override { m_finished = true; }
     String debugName() const override { return "TestScriptResourceClient"; }
 
     bool finished() const { return m_finished; }
 
 private:
     bool m_finished;
 };
 
-#if OS(MACOSX) && defined(ADDRESS_SANITIZER)
-// TODO(marja): Fix this test, http://crbug.com/572987
-#define MAYBE_CompilingStreamedScript DISABLED_CompilingStreamedScript
-#else
-#define MAYBE_CompilingStreamedScript CompilingStreamedScript
-#endif
-TEST_F(ScriptStreamingTest, MAYBE_CompilingStreamedScript)
+TEST_F(ScriptStreamingTest, CompilingStreamedScript)
 {
     // Test that we can successfully compile a streamed script.
     V8TestingScope scope;
     ScriptStreamer::startStreaming(getPendingScript(), ScriptStreamer::ParsingBlocking, m_settings.get(), scope.getScriptState(), m_loadingTaskRunner);
     TestScriptResourceClient* client = new TestScriptResourceClient;
     getPendingScript()->watchForLoad(client);
 
     appendData("function foo() {");
     appendPadding();
     appendData("return 5; }");
@@ skipping 148 matching lines @@
     // The finished notification should arrive immediately and not be cycled
     // through a background thread.
     EXPECT_TRUE(client->finished());
 
     bool errorOccurred = false;
     ScriptSourceCode sourceCode = getPendingScript()->getSource(KURL(), errorOccurred);
     EXPECT_FALSE(errorOccurred);
     EXPECT_FALSE(sourceCode.streamer());
 }
 
-#if OS(MACOSX) && defined(ADDRESS_SANITIZER)
-// TODO(marja): Fix this test, http://crbug.com/572987
-#define MAYBE_ScriptsWithSmallFirstChunk DISABLED_ScriptsWithSmallFirstChunk
-#else
-#define MAYBE_ScriptsWithSmallFirstChunk ScriptsWithSmallFirstChunk
-#endif
-TEST_F(ScriptStreamingTest, MAYBE_ScriptsWithSmallFirstChunk)
+TEST_F(ScriptStreamingTest, ScriptsWithSmallFirstChunk)
 {
     // If a script is long enough, if should be streamed, even if the first data
     // chunk is small.
     V8TestingScope scope;
     ScriptStreamer::setSmallScriptThresholdForTesting(100);
 
     ScriptStreamer::startStreaming(getPendingScript(), ScriptStreamer::ParsingBlocking, m_settings.get(), scope.getScriptState(), m_loadingTaskRunner);
     TestScriptResourceClient* client = new TestScriptResourceClient;
     getPendingScript()->watchForLoad(client);
 
@@ skipping 10 matching lines @@
     bool errorOccurred = false;
     ScriptSourceCode sourceCode = getPendingScript()->getSource(KURL(), errorOccurred);
     EXPECT_FALSE(errorOccurred);
     EXPECT_TRUE(sourceCode.streamer());
     v8::TryCatch tryCatch(scope.isolate());
     v8::Local<v8::Script> script;
     EXPECT_TRUE(V8ScriptRunner::compileScript(sourceCode, scope.isolate()).ToLocal(&script));
     EXPECT_FALSE(tryCatch.HasCaught());
 }
 
-#if OS(MACOSX) && defined(ADDRESS_SANITIZER)
-// TODO(marja): Fix this test, http://crbug.com/572987
-#define MAYBE_EncodingChanges DISABLED_EncodingChanges
-#else
-#define MAYBE_EncodingChanges EncodingChanges
-#endif
-TEST_F(ScriptStreamingTest, MAYBE_EncodingChanges)
+TEST_F(ScriptStreamingTest, EncodingChanges)
 {
     // It's possible that the encoding of the Resource changes after we start
     // loading it.
     V8TestingScope scope;
     m_resource->setEncoding("windows-1252");
 
     ScriptStreamer::startStreaming(getPendingScript(), ScriptStreamer::ParsingBlocking, m_settings.get(), scope.getScriptState(), m_loadingTaskRunner);
     TestScriptResourceClient* client = new TestScriptResourceClient;
     getPendingScript()->watchForLoad(client);
 
     m_resource->setEncoding("UTF-8");
     // \xec\x92\x81 are the raw bytes for \uc481.
     appendData("function foo() { var foob\xec\x92\x81r = 13; return foob\xec\x92\x81r; } foo();");
 
     finish();
 
     processTasksUntilStreamingComplete();
     EXPECT_TRUE(client->finished());
     bool errorOccurred = false;
     ScriptSourceCode sourceCode = getPendingScript()->getSource(KURL(), errorOccurred);
     EXPECT_FALSE(errorOccurred);
     EXPECT_TRUE(sourceCode.streamer());
     v8::TryCatch tryCatch(scope.isolate());
     v8::Local<v8::Script> script;
     EXPECT_TRUE(V8ScriptRunner::compileScript(sourceCode, scope.isolate()).ToLocal(&script));
     EXPECT_FALSE(tryCatch.HasCaught());
 }
 
 
-#if OS(MACOSX) && defined(ADDRESS_SANITIZER)
-// TODO(marja): Fix this test, http://crbug.com/572987
-#define MAYBE_EncodingFromBOM DISABLED_EncodingFromBOM
-#else
-#define MAYBE_EncodingFromBOM EncodingFromBOM
-#endif
-TEST_F(ScriptStreamingTest, MAYBE_EncodingFromBOM)
+TEST_F(ScriptStreamingTest, EncodingFromBOM)
 {
     // Byte order marks should be removed before giving the data to V8. They
     // will also affect encoding detection.
     V8TestingScope scope;
     m_resource->setEncoding("windows-1252"); // This encoding is wrong on purpose.
 
     ScriptStreamer::startStreaming(getPendingScript(), ScriptStreamer::ParsingBlocking, m_settings.get(), scope.getScriptState(), m_loadingTaskRunner);
     TestScriptResourceClient* client = new TestScriptResourceClient;
     getPendingScript()->watchForLoad(client);
 
@@ skipping 10 matching lines @@
     EXPECT_TRUE(sourceCode.streamer());
     v8::TryCatch tryCatch(scope.isolate());
     v8::Local<v8::Script> script;
     EXPECT_TRUE(V8ScriptRunner::compileScript(sourceCode, scope.isolate()).ToLocal(&script));
     EXPECT_FALSE(tryCatch.HasCaught());
 }
 
 } // namespace
 
 } // namespace blink