Chromium Code Reviews| OLD | NEW |
|---|---|
| (Empty) | |
| 1 // Copyright 2016 The Chromium Authors. All rights reserved. | |
| 2 // Use of this source code is governed by a BSD-style license that can be | |
| 3 // found in the LICENSE file. | |
| 4 | |
| 5 #ifndef COMPONENTS_ARC_TEST_FAKE_ARC_BRIDGE_BOOTSTRAP_H_ | |
| 6 #define COMPONENTS_ARC_TEST_FAKE_ARC_BRIDGE_BOOTSTRAP_H_ | |
| 7 | |
| 8 #include "components/arc/arc_bridge_bootstrap.h" | |
| 9 | |
| 10 namespace arc { | |
| 11 | |
| 12 class FakeArcBridgeInstance; | |
| 13 | |
| 14 // A fake ArcBridgeBootstrap that creates a local connection. | |
| 15 class FakeArcBridgeBootstrap : public ArcBridgeBootstrap { | |
| 16 public: | |
| 17 explicit FakeArcBridgeBootstrap(FakeArcBridgeInstance* instance); | |
| 18 ~FakeArcBridgeBootstrap() override {} | |
|
bartfab (slow)
2016/03/02 14:45:41
Nit: Do we really need to override this?
Polina Bondarenko
2016/03/06 20:22:34
Yes, g_bootstrap_for_testing is deleted in destruc
bartfab (slow)
2016/03/07 13:44:04
I do not understand. Yes, ArcBridgeBootstrap::~Arc
| |
| 19 | |
| 20 void Start() override; | |
|
bartfab (slow)
2016/03/02 14:45:41
Nit: // ArcBridgeBootstrap:
Polina Bondarenko
2016/03/06 20:22:34
Done.
| |
| 21 void Stop() override; | |
| 22 | |
| 23 private: | |
| 24 // Owned by the caller. | |
| 25 FakeArcBridgeInstance* instance_; | |
| 26 DISALLOW_COPY_AND_ASSIGN(FakeArcBridgeBootstrap); | |
|
bartfab (slow)
2016/03/02 14:45:41
Nit 1: Add blank line above.
Nit 2: #include "base
Polina Bondarenko
2016/03/06 20:22:34
Done.
| |
| 27 }; | |
| 28 | |
| 29 } // namespace arc | |
| 30 | |
| 31 #endif // COMPONENTS_ARC_TEST_FAKE_ARC_BRIDGE_BOOTSTRAP_H_ | |
| OLD | NEW |