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

Side by Side Diff: chrome/browser/extensions/api/braille_display_private/mock_braille_controller.cc

Issue 2401393003: Add multiline braille support. (Closed)
Patch Set: Addressed comments Created 4 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 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 #include "chrome/browser/extensions/api/braille_display_private/mock_braille_con troller.h" 5 #include "chrome/browser/extensions/api/braille_display_private/mock_braille_con troller.h"
6 6
7 namespace extensions { 7 namespace extensions {
8 namespace api { 8 namespace api {
9 namespace braille_display_private { 9 namespace braille_display_private {
10 10
11 MockBrailleController::MockBrailleController() 11 MockBrailleController::MockBrailleController()
12 : available_(false), observer_(NULL) {} 12 : available_(false), observer_(NULL) {}
13 13
14 std::unique_ptr<DisplayState> MockBrailleController::GetDisplayState() { 14 std::unique_ptr<DisplayState> MockBrailleController::GetDisplayState() {
15 std::unique_ptr<DisplayState> state(new DisplayState()); 15 std::unique_ptr<DisplayState> state(new DisplayState());
16 state->available = available_; 16 state->available = available_;
17 if (available_) 17 if (available_){
Devlin 2016/11/02 15:10:30 nit: need space after end paren here
ultimatedbz 2016/11/03 19:28:54 Thanks!
18 state->text_cell_count.reset(new int(18)); 18 state->text_column_count.reset(new int(18));
19 state->text_row_count.reset(new int(18));
20 }
19 return state; 21 return state;
20 } 22 }
21 23
22 void MockBrailleController::AddObserver(BrailleObserver* observer) { 24 void MockBrailleController::AddObserver(BrailleObserver* observer) {
23 CHECK(observer_ == NULL); 25 CHECK(observer_ == NULL);
24 observer_ = observer; 26 observer_ = observer;
25 } 27 }
26 28
27 void MockBrailleController::RemoveObserver(BrailleObserver* observer) { 29 void MockBrailleController::RemoveObserver(BrailleObserver* observer) {
28 CHECK(observer == observer_); 30 CHECK(observer == observer_);
29 observer_ = NULL; 31 observer_ = NULL;
30 } 32 }
31 33
32 void MockBrailleController::SetAvailable(bool available) { 34 void MockBrailleController::SetAvailable(bool available) {
33 available_ = available; 35 available_ = available;
34 } 36 }
35 37
36 BrailleObserver* MockBrailleController::GetObserver() const { 38 BrailleObserver* MockBrailleController::GetObserver() const {
37 return observer_; 39 return observer_;
38 } 40 }
39 41
40 } // namespace braille_display_private 42 } // namespace braille_display_private
41 } // namespace api 43 } // namespace api
42 } // namespace extensions 44 } // namespace extensions
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698