Merge lp:~raof/mir/actually-call-input-receiver-report into lp:mir
- actually-call-input-receiver-report
- Merge into development-branch
Status: | Merged |
---|---|
Approved by: | Brandon Schaefer |
Approved revision: | no longer in the source branch. |
Merged at revision: | 4279 |
Proposed branch: | lp:~raof/mir/actually-call-input-receiver-report |
Merge into: | lp:mir |
Diff against target: |
279 lines (+45/-2) 9 files modified
src/client/default_connection_configuration.cpp (+1/-1) src/client/rpc/make_rpc_channel.h (+6/-0) src/client/rpc/make_socket_rpc_channel.cpp (+2/-1) src/client/rpc/mir_protobuf_rpc_channel.cpp (+6/-0) src/client/rpc/mir_protobuf_rpc_channel.h (+10/-0) tests/include/mir/test/test_protobuf_client.h (+8/-0) tests/mir_test_doubles/test_protobuf_client.cpp (+2/-0) tests/unit-tests/client/test_protobuf_rpc_channel.cpp (+6/-0) tests/unit-tests/frontend/stress_protobuf_communicator.cpp (+4/-0) |
To merge this branch: | bzr merge lp:~raof/mir/actually-call-input-receiver-report |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mir CI Bot | continuous-integration | Approve | |
Alan Griffiths | Approve | ||
Review via email: mp+331587@code.launchpad.net |
Commit message
input: Make the client InputReceiverReport work
By actually calling it during event processing"
Description of the change
Mir CI Bot (mir-ci-bot) wrote : | # |
PASSED: Continuous integration, rev:4276
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
Brandon Schaefer (brandontschaefer) wrote : | # |
Was stuck for 16 hours... on some strange jenkins issues. Restarting
Preview Diff
1 | === modified file 'src/client/default_connection_configuration.cpp' | |||
2 | --- src/client/default_connection_configuration.cpp 2017-07-28 17:00:43 +0000 | |||
3 | +++ src/client/default_connection_configuration.cpp 2017-09-29 16:27:00 +0000 | |||
4 | @@ -71,7 +71,7 @@ | |||
5 | 71 | return mcl::rpc::make_rpc_channel( | 71 | return mcl::rpc::make_rpc_channel( |
6 | 72 | the_socket_file(), the_surface_map(), the_buffer_factory(), | 72 | the_socket_file(), the_surface_map(), the_buffer_factory(), |
7 | 73 | the_display_configuration(), the_input_devices(), the_rpc_report(), | 73 | the_display_configuration(), the_input_devices(), the_rpc_report(), |
9 | 74 | the_lifecycle_control(), the_ping_handler(), | 74 | the_input_receiver_report(), the_lifecycle_control(), the_ping_handler(), |
10 | 75 | the_error_handler(), the_event_sink()); | 75 | the_error_handler(), the_event_sink()); |
11 | 76 | }); | 76 | }); |
12 | 77 | } | 77 | } |
13 | 78 | 78 | ||
14 | === modified file 'src/client/rpc/make_rpc_channel.h' | |||
15 | --- src/client/rpc/make_rpc_channel.h 2017-07-28 17:00:43 +0000 | |||
16 | +++ src/client/rpc/make_rpc_channel.h 2017-09-29 16:27:00 +0000 | |||
17 | @@ -29,6 +29,11 @@ | |||
18 | 29 | namespace input | 29 | namespace input |
19 | 30 | { | 30 | { |
20 | 31 | class InputDevices; | 31 | class InputDevices; |
21 | 32 | |||
22 | 33 | namespace receiver | ||
23 | 34 | { | ||
24 | 35 | class InputReceiverReport; | ||
25 | 36 | } | ||
26 | 32 | } | 37 | } |
27 | 33 | namespace client | 38 | namespace client |
28 | 34 | { | 39 | { |
29 | @@ -50,6 +55,7 @@ | |||
30 | 50 | std::shared_ptr<DisplayConfiguration> const& disp_conf, | 55 | std::shared_ptr<DisplayConfiguration> const& disp_conf, |
31 | 51 | std::shared_ptr<input::InputDevices> const& input_devices, | 56 | std::shared_ptr<input::InputDevices> const& input_devices, |
32 | 52 | std::shared_ptr<RpcReport> const& rpc_report, | 57 | std::shared_ptr<RpcReport> const& rpc_report, |
33 | 58 | std::shared_ptr<input::receiver::InputReceiverReport> const& input_report, | ||
34 | 53 | std::shared_ptr<LifecycleControl> const& lifecycle_control, | 59 | std::shared_ptr<LifecycleControl> const& lifecycle_control, |
35 | 54 | std::shared_ptr<PingHandler> const& ping_handler, | 60 | std::shared_ptr<PingHandler> const& ping_handler, |
36 | 55 | std::shared_ptr<ErrorHandler> const& error_handler, | 61 | std::shared_ptr<ErrorHandler> const& error_handler, |
37 | 56 | 62 | ||
38 | === modified file 'src/client/rpc/make_socket_rpc_channel.cpp' | |||
39 | --- src/client/rpc/make_socket_rpc_channel.cpp 2017-07-28 17:00:43 +0000 | |||
40 | +++ src/client/rpc/make_socket_rpc_channel.cpp 2017-09-29 16:27:00 +0000 | |||
41 | @@ -48,6 +48,7 @@ | |||
42 | 48 | std::shared_ptr<mcl::DisplayConfiguration> const& disp_conf, | 48 | std::shared_ptr<mcl::DisplayConfiguration> const& disp_conf, |
43 | 49 | std::shared_ptr<input::InputDevices> const& input_devices, | 49 | std::shared_ptr<input::InputDevices> const& input_devices, |
44 | 50 | std::shared_ptr<RpcReport> const& rpc_report, | 50 | std::shared_ptr<RpcReport> const& rpc_report, |
45 | 51 | std::shared_ptr<input::receiver::InputReceiverReport> const& input_report, | ||
46 | 51 | std::shared_ptr<mcl::LifecycleControl> const& lifecycle_control, | 52 | std::shared_ptr<mcl::LifecycleControl> const& lifecycle_control, |
47 | 52 | std::shared_ptr<mcl::PingHandler> const& ping_handler, | 53 | std::shared_ptr<mcl::PingHandler> const& ping_handler, |
48 | 53 | std::shared_ptr<mcl::ErrorHandler> const& error_handler, | 54 | std::shared_ptr<mcl::ErrorHandler> const& error_handler, |
49 | @@ -65,6 +66,6 @@ | |||
50 | 65 | } | 66 | } |
51 | 66 | return std::make_shared<MirProtobufRpcChannel>( | 67 | return std::make_shared<MirProtobufRpcChannel>( |
52 | 67 | std::move(transport), map, buffer_factory, disp_conf, | 68 | std::move(transport), map, buffer_factory, disp_conf, |
54 | 68 | input_devices, rpc_report, lifecycle_control, ping_handler, | 69 | input_devices, rpc_report, input_report, lifecycle_control, ping_handler, |
55 | 69 | error_handler, event_sink); | 70 | error_handler, event_sink); |
56 | 70 | } | 71 | } |
57 | 71 | 72 | ||
58 | === modified file 'src/client/rpc/mir_protobuf_rpc_channel.cpp' | |||
59 | --- src/client/rpc/mir_protobuf_rpc_channel.cpp 2017-07-28 17:00:43 +0000 | |||
60 | +++ src/client/rpc/mir_protobuf_rpc_channel.cpp 2017-09-29 16:27:00 +0000 | |||
61 | @@ -18,6 +18,7 @@ | |||
62 | 18 | 18 | ||
63 | 19 | #include "mir_protobuf_rpc_channel.h" | 19 | #include "mir_protobuf_rpc_channel.h" |
64 | 20 | #include "rpc_report.h" | 20 | #include "rpc_report.h" |
65 | 21 | #include "mir/input/input_receiver_report.h" | ||
66 | 21 | 22 | ||
67 | 22 | #include "mir/client/surface_map.h" | 23 | #include "mir/client/surface_map.h" |
68 | 23 | #include "../buffer.h" | 24 | #include "../buffer.h" |
69 | @@ -61,12 +62,14 @@ | |||
70 | 61 | std::shared_ptr<DisplayConfiguration> const& disp_config, | 62 | std::shared_ptr<DisplayConfiguration> const& disp_config, |
71 | 62 | std::shared_ptr<input::InputDevices> const& input_devices, | 63 | std::shared_ptr<input::InputDevices> const& input_devices, |
72 | 63 | std::shared_ptr<RpcReport> const& rpc_report, | 64 | std::shared_ptr<RpcReport> const& rpc_report, |
73 | 65 | std::shared_ptr<input::receiver::InputReceiverReport> const& input_report, | ||
74 | 64 | std::shared_ptr<LifecycleControl> const& lifecycle_control, | 66 | std::shared_ptr<LifecycleControl> const& lifecycle_control, |
75 | 65 | std::shared_ptr<PingHandler> const& ping_handler, | 67 | std::shared_ptr<PingHandler> const& ping_handler, |
76 | 66 | std::shared_ptr<ErrorHandler> const& error_handler, | 68 | std::shared_ptr<ErrorHandler> const& error_handler, |
77 | 67 | std::shared_ptr<EventSink> const& event_sink) : | 69 | std::shared_ptr<EventSink> const& event_sink) : |
78 | 68 | rpc_report(rpc_report), | 70 | rpc_report(rpc_report), |
79 | 69 | pending_calls(rpc_report), | 71 | pending_calls(rpc_report), |
80 | 72 | input_report(input_report), | ||
81 | 70 | surface_map(surface_map), | 73 | surface_map(surface_map), |
82 | 71 | buffer_factory(buffer_factory), | 74 | buffer_factory(buffer_factory), |
83 | 72 | display_configuration(disp_config), | 75 | display_configuration(disp_config), |
84 | @@ -426,6 +429,7 @@ | |||
85 | 426 | window_id = e->to_close_window()->surface_id(); | 429 | window_id = e->to_close_window()->surface_id(); |
86 | 427 | break; | 430 | break; |
87 | 428 | case mir_event_type_keymap: | 431 | case mir_event_type_keymap: |
88 | 432 | input_report->received_event(*e); | ||
89 | 429 | window_id = e->to_keymap()->surface_id(); | 433 | window_id = e->to_keymap()->surface_id(); |
90 | 430 | break; | 434 | break; |
91 | 431 | case mir_event_type_window_output: | 435 | case mir_event_type_window_output: |
92 | @@ -435,9 +439,11 @@ | |||
93 | 435 | window_id = e->to_window_placement()->id(); | 439 | window_id = e->to_window_placement()->id(); |
94 | 436 | break; | 440 | break; |
95 | 437 | case mir_event_type_input: | 441 | case mir_event_type_input: |
96 | 442 | input_report->received_event(*e); | ||
97 | 438 | window_id = e->to_input()->window_id(); | 443 | window_id = e->to_input()->window_id(); |
98 | 439 | break; | 444 | break; |
99 | 440 | case mir_event_type_input_device_state: | 445 | case mir_event_type_input_device_state: |
100 | 446 | input_report->received_event(*e); | ||
101 | 441 | window_id = e->to_input_device_state()->window_id(); | 447 | window_id = e->to_input_device_state()->window_id(); |
102 | 442 | break; | 448 | break; |
103 | 443 | default: | 449 | default: |
104 | 444 | 450 | ||
105 | === modified file 'src/client/rpc/mir_protobuf_rpc_channel.h' | |||
106 | --- src/client/rpc/mir_protobuf_rpc_channel.h 2017-07-28 17:00:43 +0000 | |||
107 | +++ src/client/rpc/mir_protobuf_rpc_channel.h 2017-09-29 16:27:00 +0000 | |||
108 | @@ -39,6 +39,12 @@ | |||
109 | 39 | namespace input | 39 | namespace input |
110 | 40 | { | 40 | { |
111 | 41 | class InputDevices; | 41 | class InputDevices; |
112 | 42 | |||
113 | 43 | namespace receiver | ||
114 | 44 | { | ||
115 | 45 | class InputReceiverReport; | ||
116 | 46 | } | ||
117 | 47 | |||
118 | 42 | } | 48 | } |
119 | 43 | namespace client | 49 | namespace client |
120 | 44 | { | 50 | { |
121 | @@ -46,6 +52,7 @@ | |||
122 | 46 | class SurfaceMap; | 52 | class SurfaceMap; |
123 | 47 | class EventSink; | 53 | class EventSink; |
124 | 48 | class AsyncBufferFactory; | 54 | class AsyncBufferFactory; |
125 | 55 | |||
126 | 49 | namespace rpc | 56 | namespace rpc |
127 | 50 | { | 57 | { |
128 | 51 | 58 | ||
129 | @@ -63,6 +70,7 @@ | |||
130 | 63 | std::shared_ptr<DisplayConfiguration> const& disp_config, | 70 | std::shared_ptr<DisplayConfiguration> const& disp_config, |
131 | 64 | std::shared_ptr<input::InputDevices> const& input_devices, | 71 | std::shared_ptr<input::InputDevices> const& input_devices, |
132 | 65 | std::shared_ptr<RpcReport> const& rpc_report, | 72 | std::shared_ptr<RpcReport> const& rpc_report, |
133 | 73 | std::shared_ptr<input::receiver::InputReceiverReport> const& input_report, | ||
134 | 66 | std::shared_ptr<LifecycleControl> const& lifecycle_control, | 74 | std::shared_ptr<LifecycleControl> const& lifecycle_control, |
135 | 67 | std::shared_ptr<PingHandler> const& ping_handler, | 75 | std::shared_ptr<PingHandler> const& ping_handler, |
136 | 68 | std::shared_ptr<ErrorHandler> const& error_handler, | 76 | std::shared_ptr<ErrorHandler> const& error_handler, |
137 | @@ -105,6 +113,8 @@ | |||
138 | 105 | std::shared_ptr<RpcReport> const rpc_report; | 113 | std::shared_ptr<RpcReport> const rpc_report; |
139 | 106 | detail::PendingCallCache pending_calls; | 114 | detail::PendingCallCache pending_calls; |
140 | 107 | 115 | ||
141 | 116 | std::shared_ptr<input::receiver::InputReceiverReport> const input_report; | ||
142 | 117 | |||
143 | 108 | std::mutex discard_mutex; | 118 | std::mutex discard_mutex; |
144 | 109 | bool discard{false}; | 119 | bool discard{false}; |
145 | 110 | 120 | ||
146 | 111 | 121 | ||
147 | === modified file 'tests/include/mir/test/test_protobuf_client.h' | |||
148 | --- tests/include/mir/test/test_protobuf_client.h 2017-07-28 17:00:43 +0000 | |||
149 | +++ tests/include/mir/test/test_protobuf_client.h 2017-09-29 16:27:00 +0000 | |||
150 | @@ -33,6 +33,13 @@ | |||
151 | 33 | 33 | ||
152 | 34 | namespace mir | 34 | namespace mir |
153 | 35 | { | 35 | { |
154 | 36 | namespace input | ||
155 | 37 | { | ||
156 | 38 | namespace receiver | ||
157 | 39 | { | ||
158 | 40 | class NullInputReceiverReport; | ||
159 | 41 | } | ||
160 | 42 | } | ||
161 | 36 | namespace client | 43 | namespace client |
162 | 37 | { | 44 | { |
163 | 38 | class SurfaceMap; | 45 | class SurfaceMap; |
164 | @@ -53,6 +60,7 @@ | |||
165 | 53 | TestProtobufClient(std::string socket_file, int timeout_ms); | 60 | TestProtobufClient(std::string socket_file, int timeout_ms); |
166 | 54 | 61 | ||
167 | 55 | std::shared_ptr<doubles::MockRpcReport> rpc_report; | 62 | std::shared_ptr<doubles::MockRpcReport> rpc_report; |
168 | 63 | std::shared_ptr<input::receiver::NullInputReceiverReport> input_report; | ||
169 | 56 | std::shared_ptr<mir::client::SurfaceMap> surface_map; | 64 | std::shared_ptr<mir::client::SurfaceMap> surface_map; |
170 | 57 | std::shared_ptr<mir::client::rpc::MirBasicRpcChannel> channel; | 65 | std::shared_ptr<mir::client::rpc::MirBasicRpcChannel> channel; |
171 | 58 | std::shared_ptr<dispatch::ThreadedDispatcher> eventloop; | 66 | std::shared_ptr<dispatch::ThreadedDispatcher> eventloop; |
172 | 59 | 67 | ||
173 | === modified file 'tests/mir_test_doubles/test_protobuf_client.cpp' | |||
174 | --- tests/mir_test_doubles/test_protobuf_client.cpp 2017-07-28 17:00:43 +0000 | |||
175 | +++ tests/mir_test_doubles/test_protobuf_client.cpp 2017-09-29 16:27:00 +0000 | |||
176 | @@ -18,6 +18,7 @@ | |||
177 | 18 | 18 | ||
178 | 19 | #include "mir/test/test_protobuf_client.h" | 19 | #include "mir/test/test_protobuf_client.h" |
179 | 20 | #include "mir/test/doubles/mock_rpc_report.h" | 20 | #include "mir/test/doubles/mock_rpc_report.h" |
180 | 21 | #include "mir/input/null_input_receiver_report.h" | ||
181 | 21 | #include "mir/test/doubles/null_client_event_sink.h" | 22 | #include "mir/test/doubles/null_client_event_sink.h" |
182 | 22 | 23 | ||
183 | 23 | #include "src/client/connection_surface_map.h" | 24 | #include "src/client/connection_surface_map.h" |
184 | @@ -50,6 +51,7 @@ | |||
185 | 50 | std::make_shared<mir::client::DisplayConfiguration>(), | 51 | std::make_shared<mir::client::DisplayConfiguration>(), |
186 | 51 | std::make_shared<mir::input::InputDevices>(surface_map), | 52 | std::make_shared<mir::input::InputDevices>(surface_map), |
187 | 52 | rpc_report, | 53 | rpc_report, |
188 | 54 | input_report, | ||
189 | 53 | std::make_shared<mir::client::LifecycleControl>(), | 55 | std::make_shared<mir::client::LifecycleControl>(), |
190 | 54 | std::make_shared<mir::client::AtomicCallback<int32_t>>(), | 56 | std::make_shared<mir::client::AtomicCallback<int32_t>>(), |
191 | 55 | std::make_shared<mir::client::AtomicCallback<MirError const*>>(), | 57 | std::make_shared<mir::client::AtomicCallback<MirError const*>>(), |
192 | 56 | 58 | ||
193 | === modified file 'tests/unit-tests/client/test_protobuf_rpc_channel.cpp' | |||
194 | --- tests/unit-tests/client/test_protobuf_rpc_channel.cpp 2017-07-28 17:00:43 +0000 | |||
195 | +++ tests/unit-tests/client/test_protobuf_rpc_channel.cpp 2017-09-29 16:27:00 +0000 | |||
196 | @@ -31,6 +31,7 @@ | |||
197 | 31 | #include "mir/client/surface_map.h" | 31 | #include "mir/client/surface_map.h" |
198 | 32 | #include "mir/input/input_devices.h" | 32 | #include "mir/input/input_devices.h" |
199 | 33 | 33 | ||
200 | 34 | #include "mir/input/null_input_receiver_report.h" | ||
201 | 34 | #include "mir/test/doubles/null_client_event_sink.h" | 35 | #include "mir/test/doubles/null_client_event_sink.h" |
202 | 35 | #include "mir/test/doubles/mock_mir_buffer_stream.h" | 36 | #include "mir/test/doubles/mock_mir_buffer_stream.h" |
203 | 36 | #include "mir/test/doubles/mock_client_buffer.h" | 37 | #include "mir/test/doubles/mock_client_buffer.h" |
204 | @@ -248,6 +249,7 @@ | |||
205 | 248 | std::make_shared<mcl::DisplayConfiguration>(), | 249 | std::make_shared<mcl::DisplayConfiguration>(), |
206 | 249 | std::make_shared<mir::input::InputDevices>(surface_map), | 250 | std::make_shared<mir::input::InputDevices>(surface_map), |
207 | 250 | std::make_shared<mclr::NullRpcReport>(), | 251 | std::make_shared<mclr::NullRpcReport>(), |
208 | 252 | std::make_shared<mir::input::receiver::NullInputReceiverReport>(), | ||
209 | 251 | lifecycle, | 253 | lifecycle, |
210 | 252 | std::make_shared<mir::client::PingHandler>(), | 254 | std::make_shared<mir::client::PingHandler>(), |
211 | 253 | std::make_shared<mir::client::ErrorHandler>(), | 255 | std::make_shared<mir::client::ErrorHandler>(), |
212 | @@ -404,6 +406,7 @@ | |||
213 | 404 | std::make_shared<mcl::DisplayConfiguration>(), | 406 | std::make_shared<mcl::DisplayConfiguration>(), |
214 | 405 | std::make_shared<mir::input::InputDevices>(stream_map), | 407 | std::make_shared<mir::input::InputDevices>(stream_map), |
215 | 406 | std::make_shared<mclr::NullRpcReport>(), | 408 | std::make_shared<mclr::NullRpcReport>(), |
216 | 409 | std::make_shared<mir::input::receiver::NullInputReceiverReport>(), | ||
217 | 407 | lifecycle, | 410 | lifecycle, |
218 | 408 | std::make_shared<mir::client::PingHandler>(), | 411 | std::make_shared<mir::client::PingHandler>(), |
219 | 409 | std::make_shared<mir::client::ErrorHandler>(), | 412 | std::make_shared<mir::client::ErrorHandler>(), |
220 | @@ -760,6 +763,7 @@ | |||
221 | 760 | std::make_shared<mcl::DisplayConfiguration>(), | 763 | std::make_shared<mcl::DisplayConfiguration>(), |
222 | 761 | std::make_shared<mir::input::InputDevices>(stream_map), | 764 | std::make_shared<mir::input::InputDevices>(stream_map), |
223 | 762 | std::make_shared<mclr::NullRpcReport>(), | 765 | std::make_shared<mclr::NullRpcReport>(), |
224 | 766 | std::make_shared<mir::input::receiver::NullInputReceiverReport>(), | ||
225 | 763 | lifecycle, | 767 | lifecycle, |
226 | 764 | std::make_shared<mir::client::PingHandler>(), | 768 | std::make_shared<mir::client::PingHandler>(), |
227 | 765 | std::make_shared<mir::client::ErrorHandler>(), | 769 | std::make_shared<mir::client::ErrorHandler>(), |
228 | @@ -798,6 +802,7 @@ | |||
229 | 798 | std::make_shared<mcl::DisplayConfiguration>(), | 802 | std::make_shared<mcl::DisplayConfiguration>(), |
230 | 799 | std::make_shared<mir::input::InputDevices>(stream_map), | 803 | std::make_shared<mir::input::InputDevices>(stream_map), |
231 | 800 | std::make_shared<mclr::NullRpcReport>(), | 804 | std::make_shared<mclr::NullRpcReport>(), |
232 | 805 | std::make_shared<mir::input::receiver::NullInputReceiverReport>(), | ||
233 | 801 | lifecycle, | 806 | lifecycle, |
234 | 802 | std::make_shared<mir::client::PingHandler>(), | 807 | std::make_shared<mir::client::PingHandler>(), |
235 | 803 | std::make_shared<mir::client::ErrorHandler>(), | 808 | std::make_shared<mir::client::ErrorHandler>(), |
236 | @@ -834,6 +839,7 @@ | |||
237 | 834 | std::make_shared<mcl::DisplayConfiguration>(), | 839 | std::make_shared<mcl::DisplayConfiguration>(), |
238 | 835 | std::make_shared<mir::input::InputDevices>(stream_map), | 840 | std::make_shared<mir::input::InputDevices>(stream_map), |
239 | 836 | std::make_shared<mclr::NullRpcReport>(), | 841 | std::make_shared<mclr::NullRpcReport>(), |
240 | 842 | std::make_shared<mir::input::receiver::NullInputReceiverReport>(), | ||
241 | 837 | lifecycle, | 843 | lifecycle, |
242 | 838 | std::make_shared<mir::client::PingHandler>(), | 844 | std::make_shared<mir::client::PingHandler>(), |
243 | 839 | std::make_shared<mir::client::ErrorHandler>(), | 845 | std::make_shared<mir::client::ErrorHandler>(), |
244 | 840 | 846 | ||
245 | === modified file 'tests/unit-tests/frontend/stress_protobuf_communicator.cpp' | |||
246 | --- tests/unit-tests/frontend/stress_protobuf_communicator.cpp 2017-07-28 17:00:43 +0000 | |||
247 | +++ tests/unit-tests/frontend/stress_protobuf_communicator.cpp 2017-09-29 16:27:00 +0000 | |||
248 | @@ -37,6 +37,7 @@ | |||
249 | 37 | #include "src/client/rpc/mir_display_server.h" | 37 | #include "src/client/rpc/mir_display_server.h" |
250 | 38 | #include "src/client/buffer_factory.h" | 38 | #include "src/client/buffer_factory.h" |
251 | 39 | #include "mir/input/input_devices.h" | 39 | #include "mir/input/input_devices.h" |
252 | 40 | #include "mir/input/null_input_receiver_report.h" | ||
253 | 40 | 41 | ||
254 | 41 | #include <gtest/gtest.h> | 42 | #include <gtest/gtest.h> |
255 | 42 | #include <gmock/gmock.h> | 43 | #include <gmock/gmock.h> |
256 | @@ -59,6 +60,7 @@ | |||
257 | 59 | StubProtobufClient(std::string socket_file, int timeout_ms); | 60 | StubProtobufClient(std::string socket_file, int timeout_ms); |
258 | 60 | 61 | ||
259 | 61 | std::shared_ptr<mir::client::rpc::RpcReport> rpc_report; | 62 | std::shared_ptr<mir::client::rpc::RpcReport> rpc_report; |
260 | 63 | std::shared_ptr<mir::input::receiver::InputReceiverReport> input_report; | ||
261 | 62 | std::shared_ptr<mir::client::SurfaceMap> surface_map; | 64 | std::shared_ptr<mir::client::SurfaceMap> surface_map; |
262 | 63 | std::shared_ptr<mir::client::rpc::MirBasicRpcChannel> channel; | 65 | std::shared_ptr<mir::client::rpc::MirBasicRpcChannel> channel; |
263 | 64 | mir::client::rpc::DisplayServer display_server; | 66 | mir::client::rpc::DisplayServer display_server; |
264 | @@ -163,6 +165,7 @@ | |||
265 | 163 | std::string socket_file, | 165 | std::string socket_file, |
266 | 164 | int timeout_ms) : | 166 | int timeout_ms) : |
267 | 165 | rpc_report(std::make_shared<mir::client::rpc::NullRpcReport>()), | 167 | rpc_report(std::make_shared<mir::client::rpc::NullRpcReport>()), |
268 | 168 | input_report(std::make_shared<mir::input::receiver::NullInputReceiverReport>()), | ||
269 | 166 | surface_map(std::make_shared<mir::client::ConnectionSurfaceMap>()), | 169 | surface_map(std::make_shared<mir::client::ConnectionSurfaceMap>()), |
270 | 167 | channel(mir::client::rpc::make_rpc_channel( | 170 | channel(mir::client::rpc::make_rpc_channel( |
271 | 168 | socket_file, | 171 | socket_file, |
272 | @@ -171,6 +174,7 @@ | |||
273 | 171 | std::make_shared<mir::client::DisplayConfiguration>(), | 174 | std::make_shared<mir::client::DisplayConfiguration>(), |
274 | 172 | std::make_shared<mir::input::InputDevices>(surface_map), | 175 | std::make_shared<mir::input::InputDevices>(surface_map), |
275 | 173 | rpc_report, | 176 | rpc_report, |
276 | 177 | input_report, | ||
277 | 174 | std::make_shared<mir::client::LifecycleControl>(), | 178 | std::make_shared<mir::client::LifecycleControl>(), |
278 | 175 | std::make_shared<mir::client::PingHandler>(), | 179 | std::make_shared<mir::client::PingHandler>(), |
279 | 176 | std::make_shared<mir::client::ErrorHandler>(), | 180 | std::make_shared<mir::client::ErrorHandler>(), |
LGTM