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

Side by Side Diff: mojo/apps/js/application_delegate_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_APPS_JS_CONTENT_HANDLER_H_ 5 #ifndef MOJO_APPS_JS_CONTENT_HANDLER_H_
6 #define MOJO_APPS_JS_CONTENT_HANDLER_H_ 6 #define MOJO_APPS_JS_CONTENT_HANDLER_H_
7 7
8 #include "base/memory/scoped_ptr.h" 8 #include "base/memory/scoped_ptr.h"
9 #include "base/memory/scoped_vector.h" 9 #include "base/memory/scoped_vector.h"
10 #include "mojo/public/cpp/application/application_delegate.h" 10 #include "mojo/public/cpp/application/application_delegate.h"
(...skipping 11 matching lines...) Expand all
22 // Manages the JSApps started by this content handler. This class owns the one 22 // Manages the JSApps started by this content handler. This class owns the one
23 // reference to the Mojo shell. JSApps post a task to the content handler's 23 // reference to the Mojo shell. JSApps post a task to the content handler's
24 // thread to connect to a service or to quit. 24 // thread to connect to a service or to quit.
25 // 25 //
26 // The lifetime each JSApp is defined by its entry in AppVector. When the entry 26 // The lifetime each JSApp is defined by its entry in AppVector. When the entry
27 // is removed ("erased") by QuitJSApp(), the JSApp is destroyed. 27 // is removed ("erased") by QuitJSApp(), the JSApp is destroyed.
28 28
29 class ApplicationDelegateImpl : public ApplicationDelegate { 29 class ApplicationDelegateImpl : public ApplicationDelegate {
30 public: 30 public:
31 ApplicationDelegateImpl(); 31 ApplicationDelegateImpl();
32 virtual ~ApplicationDelegateImpl(); 32 ~ApplicationDelegateImpl() override;
33 33
34 // Add app to the AppVector and call its Start() method. 34 // Add app to the AppVector and call its Start() method.
35 void StartJSApp(scoped_ptr<JSApp> app); 35 void StartJSApp(scoped_ptr<JSApp> app);
36 36
37 // Remove app from the AppVector; destroys the app. 37 // Remove app from the AppVector; destroys the app.
38 void QuitJSApp(JSApp *app); 38 void QuitJSApp(JSApp *app);
39 39
40 void ConnectToService(ScopedMessagePipeHandle pipe_handle, 40 void ConnectToService(ScopedMessagePipeHandle pipe_handle,
41 const std::string& application_url, 41 const std::string& application_url,
42 const std::string& interface_name); 42 const std::string& interface_name);
43 43
44 protected: 44 protected:
45 // ApplicationDelegate: 45 // ApplicationDelegate:
46 virtual void Initialize(ApplicationImpl* app) override; 46 void Initialize(ApplicationImpl* app) override;
47 47
48 private: 48 private:
49 typedef ScopedVector<JSApp> AppVector; 49 typedef ScopedVector<JSApp> AppVector;
50 ApplicationImpl* application_impl_; 50 ApplicationImpl* application_impl_;
51 AppVector app_vector_; 51 AppVector app_vector_;
52 }; 52 };
53 53
54 } // namespace apps 54 } // namespace apps
55 } // namespace mojo 55 } // namespace mojo
56 56
57 #endif // MOJO_APPS_JS_CONTENT_HANDLER_H_ 57 #endif // MOJO_APPS_JS_CONTENT_HANDLER_H_
OLDNEW
« no previous file with comments | « mojo/application_manager/background_shell_application_loader_unittest.cc ('k') | mojo/apps/js/bindings/gl/context.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698