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

Unified Diff: services/test_service/test_request_tracker_impl.h

Issue 901463002: De-client TestRequestTracker (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 11 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
Index: services/test_service/test_request_tracker_impl.h
diff --git a/services/test_service/test_request_tracker_impl.h b/services/test_service/test_request_tracker_impl.h
index ed45c861ac032818d508c7c82933b55e01fbf44d..8856dcd1a4c290316dec5b16ffe7c81e3317b597 100644
--- a/services/test_service/test_request_tracker_impl.h
+++ b/services/test_service/test_request_tracker_impl.h
@@ -6,6 +6,7 @@
#define SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_
#include "base/memory/weak_ptr.h"
+#include "mojo/public/cpp/bindings/strong_binding.h"
#include "mojo/public/cpp/system/macros.h"
#include "services/test_service/test_request_tracker.mojom.h"
@@ -25,19 +26,21 @@ struct TrackingContext {
uint64_t next_id;
};
-class TestRequestTrackerImpl : public InterfaceImpl<TestRequestTracker> {
+class TestRequestTrackerImpl : public TestRequestTracker {
public:
- explicit TestRequestTrackerImpl(TrackingContext* context);
+ TestRequestTrackerImpl(InterfaceRequest<TestRequestTracker> request,
+ TrackingContext* context);
~TestRequestTrackerImpl() override;
// TestRequestTracker.
+ void SetNameAndReturnId(const String& service_name,
+ const Callback<void(uint64_t id)>& callback) override;
void RecordStats(uint64_t client_id, ServiceStatsPtr stats) override;
- void OnConnectionEstablished();
-
private:
void UploaderNameCallback(uint64_t id, const mojo::String& name);
TrackingContext* context_;
+ StrongBinding<TestRequestTracker> binding_;
base::WeakPtrFactory<TestRequestTrackerImpl> weak_factory_;
MOJO_DISALLOW_COPY_AND_ASSIGN(TestRequestTrackerImpl);
};

Powered by Google App Engine
This is Rietveld 408576698