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

Side by Side Diff: chrome/browser/nacl_host/nacl_process_host.h

Issue 8951014: Change the DidChangeView update to take a new ViewChanged resource. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: New patch Created 9 years 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 | Annotate | Revision Log
OLDNEW
1 // Copyright (c) 2011 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2011 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 CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 5 #ifndef CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
6 #define CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 6 #define CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
7 #pragma once 7 #pragma once
8 8
9 #include "build/build_config.h" 9 #include "build/build_config.h"
10 10
(...skipping 13 matching lines...) Expand all
24 // After that, most of the communication is directly between NaCl plugin 24 // After that, most of the communication is directly between NaCl plugin
25 // running in the renderer and NaCl processes. 25 // running in the renderer and NaCl processes.
26 class NaClProcessHost : public BrowserChildProcessHost { 26 class NaClProcessHost : public BrowserChildProcessHost {
27 public: 27 public:
28 explicit NaClProcessHost(const std::wstring& url); 28 explicit NaClProcessHost(const std::wstring& url);
29 virtual ~NaClProcessHost(); 29 virtual ~NaClProcessHost();
30 30
31 // Do any minimal work that must be done at browser startup. 31 // Do any minimal work that must be done at browser startup.
32 static void EarlyStartup(); 32 static void EarlyStartup();
33 33
34 // Initialize the new NaCl process, returning true on success. 34 // Initialize the new NaCl process, returning true on success. On success,
35 // the NaCl process host will assume responsibility for sending the reply
36 // message. On failure, the reply will not be sent and this is the caller's
37 // responsibility to avoid handling the renderer.
dmichael (off chromium) 2011/12/20 19:01:34 "avoid handling the renderer"? Is it supposed to s
35 bool Launch(ChromeRenderMessageFilter* chrome_render_message_filter, 38 bool Launch(ChromeRenderMessageFilter* chrome_render_message_filter,
36 int socket_count, 39 int socket_count,
37 IPC::Message* reply_msg); 40 IPC::Message* reply_msg);
38 41
39 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE; 42 virtual bool OnMessageReceived(const IPC::Message& msg) OVERRIDE;
40 43
41 void OnProcessLaunchedByBroker(base::ProcessHandle handle); 44 void OnProcessLaunchedByBroker(base::ProcessHandle handle);
42 45
43 protected: 46 protected:
44 virtual void OnProcessCrashed(int exit_code) OVERRIDE; 47 virtual void OnProcessCrashed(int exit_code) OVERRIDE;
(...skipping 10 matching lines...) Expand all
55 virtual void OnProcessLaunched() OVERRIDE; 58 virtual void OnProcessLaunched() OVERRIDE;
56 59
57 void IrtReady(); 60 void IrtReady();
58 void SendStart(base::PlatformFile irt_file); 61 void SendStart(base::PlatformFile irt_file);
59 62
60 private: 63 private:
61 // The ChromeRenderMessageFilter that requested this NaCl process. We use 64 // The ChromeRenderMessageFilter that requested this NaCl process. We use
62 // this for sending the reply once the process has started. 65 // this for sending the reply once the process has started.
63 scoped_refptr<ChromeRenderMessageFilter> chrome_render_message_filter_; 66 scoped_refptr<ChromeRenderMessageFilter> chrome_render_message_filter_;
64 67
65 // The reply message to send. 68 // The reply message to send. We must always send this message when the
69 // sub-process either succeeds or fails to unblock the renderer waiting for
70 // the reply. NULL when there is no reply to send.
66 IPC::Message* reply_msg_; 71 IPC::Message* reply_msg_;
67 72
68 // Socket pairs for the NaCl process and renderer. 73 // Socket pairs for the NaCl process and renderer.
69 scoped_ptr<NaClInternal> internal_; 74 scoped_ptr<NaClInternal> internal_;
70 75
71 base::WeakPtrFactory<NaClProcessHost> weak_factory_; 76 base::WeakPtrFactory<NaClProcessHost> weak_factory_;
72 77
73 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost); 78 DISALLOW_COPY_AND_ASSIGN(NaClProcessHost);
74 }; 79 };
75 80
76 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_ 81 #endif // CHROME_BROWSER_NACL_HOST_NACL_PROCESS_HOST_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/nacl_host/nacl_process_host.cc » ('j') | chrome/browser/nacl_host/nacl_process_host.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698