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

Side by Side Diff: base/metrics/field_trial.h

Issue 2530573002: Share field trial allocator on zygote-using Linuxes (Closed)
Patch Set: add check back Created 4 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
« no previous file with comments | « no previous file | base/metrics/field_trial.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 // FieldTrial is a class for handling details of statistical experiments 5 // FieldTrial is a class for handling details of statistical experiments
6 // performed by actual users in the field (i.e., in a shipped or beta product). 6 // performed by actual users in the field (i.e., in a shipped or beta product).
7 // All code is called exclusively on the UI thread currently. 7 // All code is called exclusively on the UI thread currently.
8 // 8 //
9 // The simplest example is an experiment to see whether one of two options 9 // The simplest example is an experiment to see whether one of two options
10 // produces "better" results across our user population. In that scenario, UMA 10 // produces "better" results across our user population. In that scenario, UMA
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 #include <stdint.h> 58 #include <stdint.h>
59 59
60 #include <map> 60 #include <map>
61 #include <memory> 61 #include <memory>
62 #include <set> 62 #include <set>
63 #include <string> 63 #include <string>
64 #include <vector> 64 #include <vector>
65 65
66 #include "base/base_export.h" 66 #include "base/base_export.h"
67 #include "base/command_line.h" 67 #include "base/command_line.h"
68 #include "base/files/file.h"
68 #include "base/gtest_prod_util.h" 69 #include "base/gtest_prod_util.h"
69 #include "base/macros.h" 70 #include "base/macros.h"
70 #include "base/memory/ref_counted.h" 71 #include "base/memory/ref_counted.h"
71 #include "base/memory/shared_memory.h" 72 #include "base/memory/shared_memory.h"
72 #include "base/metrics/persistent_memory_allocator.h" 73 #include "base/metrics/persistent_memory_allocator.h"
73 #include "base/observer_list_threadsafe.h" 74 #include "base/observer_list_threadsafe.h"
74 #include "base/process/launch.h" 75 #include "base/process/launch.h"
75 #include "base/strings/string_piece.h" 76 #include "base/strings/string_piece.h"
76 #include "base/synchronization/lock.h" 77 #include "base/synchronization/lock.h"
77 #include "base/time/time.h" 78 #include "base/time/time.h"
(...skipping 411 matching lines...) Expand 10 before | Expand all | Expand 10 after
489 // through a command line argument to the browser process. Created field 490 // through a command line argument to the browser process. Created field
490 // trials will be marked "used" for the purposes of active trial reporting 491 // trials will be marked "used" for the purposes of active trial reporting
491 // if they are prefixed with |kActivationMarker|. Trial names in 492 // if they are prefixed with |kActivationMarker|. Trial names in
492 // |ignored_trial_names| are ignored when parsing |trials_string|. 493 // |ignored_trial_names| are ignored when parsing |trials_string|.
493 static bool CreateTrialsFromString( 494 static bool CreateTrialsFromString(
494 const std::string& trials_string, 495 const std::string& trials_string,
495 const std::set<std::string>& ignored_trial_names); 496 const std::set<std::string>& ignored_trial_names);
496 497
497 // Achieves the same thing as CreateTrialsFromString, except wraps the logic 498 // Achieves the same thing as CreateTrialsFromString, except wraps the logic
498 // by taking in the trials from the command line, either via shared memory 499 // by taking in the trials from the command line, either via shared memory
499 // handle or command line argument. 500 // handle or command line argument. A bit of a misnomer since on POSIX we
500 // If using shared memory to pass around the list of field trials, then 501 // simply get the trials from opening |fd_key| if using shared memory. On
501 // expects |field_trial_handle_switch| command line argument to 502 // Windows, we expect the |cmd_line| switch for |field_trial_handle_switch| to
502 // contain the shared memory handle. 503 // contain the shared memory handle that contains the field trial allocator.
503 // If not, then create the trials as before (using the kForceFieldTrials 504 // We need the |field_trial_handle_switch| and |fd_key| arguments to be passed
504 // switch). Needs the |field_trial_handle_switch| argument to be passed in 505 // in since base/ can't depend on content/.
505 // since base/ can't depend on content/. 506 static void CreateTrialsFromCommandLine(const base::CommandLine& cmd_line,
506 static void CreateTrialsFromCommandLine( 507 const char* field_trial_handle_switch,
507 const base::CommandLine& cmd_line, 508 int fd_key);
508 const char* field_trial_handle_switch);
509 509
510 #if defined(OS_WIN) 510 #if defined(OS_WIN)
511 // On Windows, we need to explicitly pass down any handles to be inherited. 511 // On Windows, we need to explicitly pass down any handles to be inherited.
512 // This function adds the shared memory handle to field trial state to the 512 // This function adds the shared memory handle to field trial state to the
513 // list of handles to be inherited. 513 // list of handles to be inherited.
514 static void AppendFieldTrialHandleIfNeeded( 514 static void AppendFieldTrialHandleIfNeeded(
515 base::HandlesToInheritVector* handles); 515 base::HandlesToInheritVector* handles);
516 #elif defined(OS_POSIX)
517 // On POSIX, we also need to explicitly pass down this file descriptor that
518 // should be shared with the child process. Returns kInvalidPlatformFile if no
519 // handle exists or was not initialized properly.
520 static PlatformFile GetFieldTrialHandle();
516 #endif 521 #endif
517 522
518 // Adds a switch to the command line containing the field trial state as a 523 // Adds a switch to the command line containing the field trial state as a
519 // string (if not using shared memory to share field trial state), or the 524 // string (if not using shared memory to share field trial state), or the
520 // shared memory handle + length. 525 // shared memory handle + length.
521 // Needs the |field_trial_handle_switch| argument to be passed in since base/ 526 // Needs the |field_trial_handle_switch| argument to be passed in since base/
522 // can't depend on content/. 527 // can't depend on content/.
523 static void CopyFieldTrialStateToFlags(const char* field_trial_handle_switch, 528 static void CopyFieldTrialStateToFlags(const char* field_trial_handle_switch,
524 base::CommandLine* cmd_line); 529 base::CommandLine* cmd_line);
525 530
(...skipping 25 matching lines...) Expand all
551 static size_t GetFieldTrialCount(); 556 static size_t GetFieldTrialCount();
552 557
553 private: 558 private:
554 // Allow tests to access our innards for testing purposes. 559 // Allow tests to access our innards for testing purposes.
555 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest, InstantiateAllocator); 560 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest, InstantiateAllocator);
556 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest, AddTrialsToAllocator); 561 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest, AddTrialsToAllocator);
557 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest, 562 FRIEND_TEST_ALL_PREFIXES(FieldTrialListTest,
558 DoNotAddSimulatedFieldTrialsToAllocator); 563 DoNotAddSimulatedFieldTrialsToAllocator);
559 564
560 #if defined(OS_WIN) 565 #if defined(OS_WIN)
561 // Takes in |handle| that should have been retrieved from the command line and 566 // Takes in |handle_switch| from the command line which represents the shared
562 // creates a SharedMemoryHandle from it, and then calls 567 // memory handle for field trials, parses it, and creates the field trials.
563 // CreateTrialsFromSharedMemory(). Returns true on success, false on failure. 568 // Returns true on success, false on failure.
564 static bool CreateTrialsFromWindowsHandle(HANDLE handle); 569 static bool CreateTrialsFromHandleSwitch(const std::string& handle_switch);
565 #endif 570 #endif
566 571
572 #if defined(OS_POSIX) && !defined(OS_NACL) && !defined(OS_MACOSX) && \
573 !defined(OS_ANDROID)
574 // On POSIX systems that use the zygote, we look up the correct fd that backs
575 // the shared memory segment containing the field trials by looking it up via
576 // an fd key in GlobalDescriptors. Returns true on success, false on failure.
577 static bool CreateTrialsFromDescriptor(int fd_key);
578 #endif
579
580 // Takes an unmapped SharedMemoryHandle, creates a SharedMemory object from it
581 // and maps it with the correct size.
582 static bool CreateTrialsFromSharedMemoryHandle(SharedMemoryHandle shm_handle);
583
567 // Expects a mapped piece of shared memory |shm| that was created from the 584 // Expects a mapped piece of shared memory |shm| that was created from the
568 // browser process's field_trial_allocator and shared via the command line. 585 // browser process's field_trial_allocator and shared via the command line.
569 // This function recreates the allocator, iterates through all the field 586 // This function recreates the allocator, iterates through all the field
570 // trials in it, and creates them via CreateFieldTrial(). Returns true if 587 // trials in it, and creates them via CreateFieldTrial(). Returns true if
571 // successful and false otherwise. 588 // successful and false otherwise.
572 static bool CreateTrialsFromSharedMemory( 589 static bool CreateTrialsFromSharedMemory(
573 std::unique_ptr<base::SharedMemory> shm); 590 std::unique_ptr<base::SharedMemory> shm);
574 591
575 // Instantiate the field trial allocator, add all existing field trials to it, 592 // Instantiate the field trial allocator, add all existing field trials to it,
576 // and duplicates its handle to a read-only handle, which gets stored in 593 // and duplicates its handle to a read-only handle, which gets stored in
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
620 637
621 // List of observers to be notified when a group is selected for a FieldTrial. 638 // List of observers to be notified when a group is selected for a FieldTrial.
622 scoped_refptr<ObserverListThreadSafe<Observer> > observer_list_; 639 scoped_refptr<ObserverListThreadSafe<Observer> > observer_list_;
623 640
624 // Allocator in shared memory containing field trial data. Used in both 641 // Allocator in shared memory containing field trial data. Used in both
625 // browser and child processes, but readonly in the child. 642 // browser and child processes, but readonly in the child.
626 // In the future, we may want to move this to a more generic place if we want 643 // In the future, we may want to move this to a more generic place if we want
627 // to start passing more data other than field trials. 644 // to start passing more data other than field trials.
628 std::unique_ptr<FieldTrialAllocator> field_trial_allocator_ = nullptr; 645 std::unique_ptr<FieldTrialAllocator> field_trial_allocator_ = nullptr;
629 646
630 #if defined(OS_WIN)
631 // Readonly copy of the handle to the allocator. Needs to be a member variable 647 // Readonly copy of the handle to the allocator. Needs to be a member variable
632 // because it's needed from both CopyFieldTrialStateToFlags() and 648 // because it's needed from both CopyFieldTrialStateToFlags() and
633 // AppendFieldTrialHandleIfNeeded(). 649 // AppendFieldTrialHandleIfNeeded().
634 HANDLE readonly_allocator_handle_ = nullptr; 650 PlatformFile readonly_allocator_handle_ = kInvalidPlatformFile;
635 #endif
636 651
637 // Tracks whether CreateTrialsFromCommandLine() has been called. 652 // Tracks whether CreateTrialsFromCommandLine() has been called.
638 bool create_trials_from_command_line_called_ = false; 653 bool create_trials_from_command_line_called_ = false;
639 654
640 DISALLOW_COPY_AND_ASSIGN(FieldTrialList); 655 DISALLOW_COPY_AND_ASSIGN(FieldTrialList);
641 }; 656 };
642 657
643 } // namespace base 658 } // namespace base
644 659
645 #endif // BASE_METRICS_FIELD_TRIAL_H_ 660 #endif // BASE_METRICS_FIELD_TRIAL_H_
OLDNEW
« no previous file with comments | « no previous file | base/metrics/field_trial.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698