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

Unified Diff: components/nacl/browser/nacl_process_host.cc

Issue 2762513002: Remove keep-alive impulse IPCs from NaCl modules. (Closed)
Patch Set: Rebase Created 3 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: components/nacl/browser/nacl_process_host.cc
diff --git a/components/nacl/browser/nacl_process_host.cc b/components/nacl/browser/nacl_process_host.cc
index d92d9cc73373844fe04762e91e4a77711b7d998d..01c55c56d185d3e483559a1e2479ac60721a69fc 100644
--- a/components/nacl/browser/nacl_process_host.cc
+++ b/components/nacl/browser/nacl_process_host.cc
@@ -197,9 +197,6 @@ void CloseFile(base::File file) {
} // namespace
-unsigned NaClProcessHost::keepalive_throttle_interval_milliseconds_ =
- ppapi::kKeepaliveThrottleIntervalDefaultMilliseconds;
-
NaClProcessHost::NaClProcessHost(
const GURL& manifest_url,
base::File nexe_file,
@@ -344,12 +341,6 @@ void NaClProcessHost::EarlyZygoteLaunch() {
}
#endif // defined(OS_POSIX) && !defined(OS_MACOSX)
-// static
-void NaClProcessHost::SetPpapiKeepAliveThrottleForTesting(
- unsigned milliseconds) {
- keepalive_throttle_interval_milliseconds_ = milliseconds;
-}
-
void NaClProcessHost::Launch(
NaClHostMessageFilter* nacl_host_message_filter,
IPC::Message* reply_msg,
@@ -930,14 +921,10 @@ bool NaClProcessHost::StartPPAPIProxy(
nacl_host_message_filter_->render_process_id(),
render_view_id_,
profile_directory_));
- ppapi_host_->SetOnKeepaliveCallback(
- NaClBrowser::GetDelegate()->GetOnKeepaliveCallback());
ppapi::PpapiNaClPluginArgs args;
args.off_the_record = nacl_host_message_filter_->off_the_record();
args.permissions = permissions_;
- args.keepalive_throttle_interval_milliseconds =
- keepalive_throttle_interval_milliseconds_;
base::CommandLine* cmdline = base::CommandLine::ForCurrentProcess();
DCHECK(cmdline);
std::string flag_whitelist[] = {
« no previous file with comments | « components/nacl/browser/nacl_browser_delegate.h ('k') | components/nacl/browser/test_nacl_browser_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698