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

Unified Diff: sky/tools/tester/test_observer_impl.cc

Issue 715123002: Rename sky::TestObserver to sky::TestHarness (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years, 1 month 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 | « sky/tools/tester/test_observer_impl.h ('k') | sky/tools/tester/test_runner.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sky/tools/tester/test_observer_impl.cc
diff --git a/sky/tools/tester/test_observer_impl.cc b/sky/tools/tester/test_observer_impl.cc
deleted file mode 100644
index 0037d507e7fb16743f5fcfaa80b7054df688bc08..0000000000000000000000000000000000000000
--- a/sky/tools/tester/test_observer_impl.cc
+++ /dev/null
@@ -1,29 +0,0 @@
-// 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 "sky/tools/tester/test_observer_impl.h"
-
-#include "sky/tools/tester/test_runner.h"
-
-namespace sky {
-namespace tester {
-
-TestObserverImpl::TestObserverImpl(TestRunner* test_runner)
- : test_runner_(test_runner->GetWeakPtr()) {
- // FIXME: This is technically when the V8 context gets created and
- // not when the test is started. An error before we instantiated
- // the V8 context would show up before the #BEGIN line for this test.
- test_runner->OnTestStart();
-}
-
-TestObserverImpl::~TestObserverImpl() {
-}
-
-void TestObserverImpl::OnTestComplete(const mojo::String& test_result) {
- if (test_runner_)
- test_runner_->OnTestComplete(test_result);
-}
-
-} // namespace tester
-} // namespace sky
« no previous file with comments | « sky/tools/tester/test_observer_impl.h ('k') | sky/tools/tester/test_runner.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698