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

Side by Side Diff: mash/browser_driver/browser_driver_application_delegate.cc

Issue 1527183003: Change mojo enums to be scoped enums in the generated C++ bindings. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mojo-binding-equals
Patch Set: Created 4 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "mash/browser_driver/browser_driver_application_delegate.h" 5 #include "mash/browser_driver/browser_driver_application_delegate.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "components/mus/public/cpp/event_matcher.h" 10 #include "components/mus/public/cpp/event_matcher.h"
11 #include "mojo/application/public/cpp/application_connection.h" 11 #include "mojo/application/public/cpp/application_connection.h"
12 #include "mojo/application/public/cpp/application_impl.h" 12 #include "mojo/application/public/cpp/application_impl.h"
13 13
14 namespace mash { 14 namespace mash {
15 namespace browser_driver { 15 namespace browser_driver {
16 namespace { 16 namespace {
17 17
18 enum class Accelerator : uint32_t { 18 enum class Accelerator : uint32_t {
19 NewWindow, 19 NewWindow,
20 NewTab, 20 NewTab,
21 NewIncognitoWindow, 21 NewIncognitoWindow,
22 }; 22 };
23 23
24 struct AcceleratorSpec { 24 struct AcceleratorSpec {
25 Accelerator id; 25 Accelerator id;
26 mus::mojom::KeyboardCode keyboard_code; 26 mus::mojom::KeyboardCode keyboard_code;
27 mus::mojom::EventFlags event_flags; 27 int event_flags;
yzshen1 2016/01/07 19:24:52 Please add comment.
Sam McNally 2016/01/08 05:12:51 Done.
28 }; 28 };
29 29
30 AcceleratorSpec g_spec[] = { 30 AcceleratorSpec g_spec[] = {
31 { Accelerator::NewWindow, 31 {Accelerator::NewWindow, mus::mojom::KeyboardCode::N,
32 mus::mojom::KEYBOARD_CODE_N, 32 mus::mojom::kEventFlagControlDown},
33 mus::mojom::EVENT_FLAGS_CONTROL_DOWN }, 33 {Accelerator::NewTab, mus::mojom::KeyboardCode::T,
34 { Accelerator::NewTab, 34 mus::mojom::kEventFlagControlDown},
35 mus::mojom::KEYBOARD_CODE_T, 35 {Accelerator::NewIncognitoWindow, mus::mojom::KeyboardCode::N,
36 mus::mojom::EVENT_FLAGS_CONTROL_DOWN }, 36 mus::mojom::kEventFlagControlDown | mus::mojom::kEventFlagShiftDown},
37 { Accelerator::NewIncognitoWindow,
38 mus::mojom::KEYBOARD_CODE_N,
39 static_cast<mus::mojom::EventFlags>(mus::mojom::EVENT_FLAGS_CONTROL_DOWN |
40 mus::mojom::EVENT_FLAGS_SHIFT_DOWN) },
41 }; 37 };
42 38
43 void AssertTrue(bool success) { 39 void AssertTrue(bool success) {
44 DCHECK(success); 40 DCHECK(success);
45 } 41 }
46 42
47 } // namespace 43 } // namespace
48 44
49 BrowserDriverApplicationDelegate::BrowserDriverApplicationDelegate() 45 BrowserDriverApplicationDelegate::BrowserDriverApplicationDelegate()
50 : app_(nullptr), 46 : app_(nullptr),
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
98 for (const AcceleratorSpec& spec : g_spec) { 94 for (const AcceleratorSpec& spec : g_spec) {
99 registrar->AddAccelerator( 95 registrar->AddAccelerator(
100 static_cast<uint32_t>(spec.id), 96 static_cast<uint32_t>(spec.id),
101 mus::CreateKeyMatcher(spec.keyboard_code, spec.event_flags), 97 mus::CreateKeyMatcher(spec.keyboard_code, spec.event_flags),
102 base::Bind(&AssertTrue)); 98 base::Bind(&AssertTrue));
103 } 99 }
104 } 100 }
105 101
106 } // namespace browser_driver 102 } // namespace browser_driver
107 } // namespace main 103 } // namespace main
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698