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

Side by Side Diff: mojo/services/test_service/test_request_tracker_impl.h

Issue 668663006: Standardize usage of virtual/override/final in mojo/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_ 5 #ifndef MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_
6 #define MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_ 6 #define MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_
7 7
8 #include "base/memory/weak_ptr.h" 8 #include "base/memory/weak_ptr.h"
9 #include "mojo/public/cpp/system/macros.h" 9 #include "mojo/public/cpp/system/macros.h"
10 #include "mojo/services/test_service/test_request_tracker.mojom.h" 10 #include "mojo/services/test_service/test_request_tracker.mojom.h"
(...skipping 10 matching lines...) Expand all
21 TrackingContext(); 21 TrackingContext();
22 ~TrackingContext(); 22 ~TrackingContext();
23 AllRecordsMap records; 23 AllRecordsMap records;
24 std::map<uint64_t, std::string> ids_to_names; 24 std::map<uint64_t, std::string> ids_to_names;
25 uint64_t next_id; 25 uint64_t next_id;
26 }; 26 };
27 27
28 class TestRequestTrackerImpl : public InterfaceImpl<TestRequestTracker> { 28 class TestRequestTrackerImpl : public InterfaceImpl<TestRequestTracker> {
29 public: 29 public:
30 explicit TestRequestTrackerImpl(TrackingContext* context); 30 explicit TestRequestTrackerImpl(TrackingContext* context);
31 virtual ~TestRequestTrackerImpl(); 31 ~TestRequestTrackerImpl() override;
32 32
33 // TestRequestTracker. 33 // TestRequestTracker.
34 virtual void RecordStats(uint64_t client_id, ServiceStatsPtr stats) override; 34 void RecordStats(uint64_t client_id, ServiceStatsPtr stats) override;
35 35
36 // InterfaceImpl override. 36 // InterfaceImpl override.
37 virtual void OnConnectionEstablished() override; 37 void OnConnectionEstablished() override;
38 38
39 private: 39 private:
40 void UploaderNameCallback(uint64_t id, const mojo::String& name); 40 void UploaderNameCallback(uint64_t id, const mojo::String& name);
41 TrackingContext* context_; 41 TrackingContext* context_;
42 base::WeakPtrFactory<TestRequestTrackerImpl> weak_factory_; 42 base::WeakPtrFactory<TestRequestTrackerImpl> weak_factory_;
43 MOJO_DISALLOW_COPY_AND_ASSIGN(TestRequestTrackerImpl); 43 MOJO_DISALLOW_COPY_AND_ASSIGN(TestRequestTrackerImpl);
44 }; 44 };
45 45
46 class TestTrackedRequestServiceImpl 46 class TestTrackedRequestServiceImpl
47 : public InterfaceImpl<TestTrackedRequestService> { 47 : public InterfaceImpl<TestTrackedRequestService> {
48 public: 48 public:
49 explicit TestTrackedRequestServiceImpl(TrackingContext* context); 49 explicit TestTrackedRequestServiceImpl(TrackingContext* context);
50 virtual ~TestTrackedRequestServiceImpl(); 50 ~TestTrackedRequestServiceImpl() override;
51 51
52 // |TestTrackedRequestService| implementation. 52 // |TestTrackedRequestService| implementation.
53 virtual void GetReport( 53 void GetReport(const mojo::Callback<void(mojo::Array<ServiceReportPtr>)>&
54 const mojo::Callback<void(mojo::Array<ServiceReportPtr>)>& callback) 54 callback) override;
55 override;
56 55
57 private: 56 private:
58 TrackingContext* context_; 57 TrackingContext* context_;
59 MOJO_DISALLOW_COPY_AND_ASSIGN(TestTrackedRequestServiceImpl); 58 MOJO_DISALLOW_COPY_AND_ASSIGN(TestTrackedRequestServiceImpl);
60 }; 59 };
61 60
62 } // namespace test 61 } // namespace test
63 } // namespace mojo 62 } // namespace mojo
64 63
65 #endif // MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_ 64 #endif // MOJO_SERVICES_TEST_SERVICE_TEST_REQUEST_TRACKER_IMPL_H_
OLDNEW
« no previous file with comments | « mojo/services/test_service/test_request_tracker_client_impl.h ('k') | mojo/services/test_service/test_service_application.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698