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

Side by Side Diff: content/browser/renderer_host/render_process_host_impl.h

Issue 2412113002: Use SharedPersistentMemoryAllocator to share field trial state (Closed)
Patch Set: gclient sync Created 4 years, 1 month 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 (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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 CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 5 #ifndef CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 6 #define CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 559 matching lines...) Expand 10 before | Expand all | Expand 10 after
570 // for UMA. 570 // for UMA.
571 size_t max_worker_count_; 571 size_t max_worker_count_;
572 572
573 // Context shared for each mojom::PermissionService instance created for this 573 // Context shared for each mojom::PermissionService instance created for this
574 // RPH. 574 // RPH.
575 std::unique_ptr<PermissionServiceContext> permission_service_context_; 575 std::unique_ptr<PermissionServiceContext> permission_service_context_;
576 576
577 // The memory allocator, if any, in which the renderer will write its metrics. 577 // The memory allocator, if any, in which the renderer will write its metrics.
578 std::unique_ptr<base::SharedPersistentMemoryAllocator> metrics_allocator_; 578 std::unique_ptr<base::SharedPersistentMemoryAllocator> metrics_allocator_;
579 579
580 // Anonymous shared memory segment to share with subprocess containing list of
581 // field trials (represented as a string).
582 // TODO(crbug.com/653874): Eventually remove this and use single shared memory
583 // object across processes.
584 std::unique_ptr<base::SharedMemory> field_trial_state_;
585
586 bool channel_connected_; 580 bool channel_connected_;
587 bool sent_render_process_ready_; 581 bool sent_render_process_ready_;
588 582
589 #if defined(OS_ANDROID) 583 #if defined(OS_ANDROID)
590 // UI thread is the source of sync IPCs and all shutdown signals. 584 // UI thread is the source of sync IPCs and all shutdown signals.
591 // Therefore a proper shutdown event to unblock the UI thread is not 585 // Therefore a proper shutdown event to unblock the UI thread is not
592 // possible without massive refactoring shutdown code. 586 // possible without massive refactoring shutdown code.
593 // Luckily Android never performs a clean shutdown. So explicitly 587 // Luckily Android never performs a clean shutdown. So explicitly
594 // ignore this problem. 588 // ignore this problem.
595 base::WaitableEvent never_signaled_; 589 base::WaitableEvent never_signaled_;
(...skipping 10 matching lines...) Expand all
606 instance_weak_factory_; 600 instance_weak_factory_;
607 601
608 base::WeakPtrFactory<RenderProcessHostImpl> weak_factory_; 602 base::WeakPtrFactory<RenderProcessHostImpl> weak_factory_;
609 603
610 DISALLOW_COPY_AND_ASSIGN(RenderProcessHostImpl); 604 DISALLOW_COPY_AND_ASSIGN(RenderProcessHostImpl);
611 }; 605 };
612 606
613 } // namespace content 607 } // namespace content
614 608
615 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_ 609 #endif // CONTENT_BROWSER_RENDERER_HOST_RENDER_PROCESS_HOST_IMPL_H_
OLDNEW
« no previous file with comments | « content/browser/ppapi_plugin_process_host.cc ('k') | content/browser/renderer_host/render_process_host_impl.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698