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

Side by Side Diff: components/font_service/font_service_app.h

Issue 1674903003: Extract shell methods from ApplicationImpl into a base class, and pass this to Initialize() instead. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@mojom
Patch Set: . Created 4 years, 10 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 #ifndef COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_ 5 #ifndef COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_
6 #define COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_ 6 #define COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include "base/macros.h" 10 #include "base/macros.h"
11 #include "components/font_service/public/interfaces/font_service.mojom.h" 11 #include "components/font_service/public/interfaces/font_service.mojom.h"
12 #include "mojo/common/weak_binding_set.h" 12 #include "mojo/common/weak_binding_set.h"
13 #include "mojo/services/tracing/public/cpp/tracing_impl.h" 13 #include "mojo/services/tracing/public/cpp/tracing_impl.h"
14 #include "mojo/shell/public/cpp/application_delegate.h" 14 #include "mojo/shell/public/cpp/application_delegate.h"
15 #include "mojo/shell/public/cpp/interface_factory.h" 15 #include "mojo/shell/public/cpp/interface_factory.h"
16 #include "skia/ext/skia_utils_base.h" 16 #include "skia/ext/skia_utils_base.h"
17 17
18 namespace font_service { 18 namespace font_service {
19 19
20 class FontServiceApp : public mojo::ApplicationDelegate, 20 class FontServiceApp : public mojo::ApplicationDelegate,
21 public mojo::InterfaceFactory<FontService>, 21 public mojo::InterfaceFactory<FontService>,
22 public FontService { 22 public FontService {
23 public: 23 public:
24 FontServiceApp(); 24 FontServiceApp();
25 ~FontServiceApp() override; 25 ~FontServiceApp() override;
26 26
27 private: 27 private:
28 // ApplicationDelegate: 28 // ApplicationDelegate:
29 void Initialize(mojo::ApplicationImpl* app) override; 29 void Initialize(mojo::Shell* shell, const std::string& url,
30 uint32_t id) override;
30 bool AcceptConnection( 31 bool AcceptConnection(
31 mojo::ApplicationConnection* connection) override; 32 mojo::ApplicationConnection* connection) override;
32 33
33 // mojo::InterfaceFactory<FontService>: 34 // mojo::InterfaceFactory<FontService>:
34 void Create(mojo::ApplicationConnection* connection, 35 void Create(mojo::ApplicationConnection* connection,
35 mojo::InterfaceRequest<FontService> request) override; 36 mojo::InterfaceRequest<FontService> request) override;
36 37
37 // FontService: 38 // FontService:
38 void MatchFamilyName(const mojo::String& family_name, 39 void MatchFamilyName(const mojo::String& family_name,
39 TypefaceStyle requested_style, 40 TypefaceStyle requested_style,
(...skipping 10 matching lines...) Expand all
50 // We don't want to leak paths to our callers; we thus enumerate the paths of 51 // We don't want to leak paths to our callers; we thus enumerate the paths of
51 // fonts. 52 // fonts.
52 SkTDArray<SkString*> paths_; 53 SkTDArray<SkString*> paths_;
53 54
54 DISALLOW_COPY_AND_ASSIGN(FontServiceApp); 55 DISALLOW_COPY_AND_ASSIGN(FontServiceApp);
55 }; 56 };
56 57
57 } // namespace font_service 58 } // namespace font_service
58 59
59 #endif // COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_ 60 #endif // COMPONENTS_FONT_SERVICE_FONT_SERVICE_APP_H_
OLDNEW
« no previous file with comments | « components/filesystem/public/cpp/prefs/pref_service_factory.cc ('k') | components/font_service/font_service_app.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698