Added histograms on process singleton create when remote process exists and we cannot notify it

chrome/browser/process_singleton.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_PROCESS_SINGLETON_H_
 #define CHROME_BROWSER_PROCESS_SINGLETON_H_
 
 #include "build/build_config.h"
 
 #if defined(OS_WIN)
@@ skipping 31 matching lines @@
 // each other.  It is named according to the user data directory, so
 // we can be sure that no more than one copy of the application can be
 // running at once with a given data directory.
 //
 // Implementation notes:
 // - the Windows implementation uses an invisible global message window;
 // - the Linux implementation uses a Unix domain socket in the user data dir.
 
 class ProcessSingleton : public base::NonThreadSafe {
  public:
+  // Used to send the reason of remote hang process termination as histogram.
+  enum RemoteHungProcessTerminateReason {
+#if defined(OS_WIN)
+    USER_ACCEPTED_TERMINATION = 1,
+    NO_VISIBLE_WINDOW_FOUND = 2,
+#elif defined(OS_POSIX)
+    NOTIFY_ATTEMPTS_EXCEEDED = 3,
+    SOCKET_WRITE_FAILED = 4,
+    SOCKET_READ_FAILED = 5,
+#endif
+    REMOTE_HUNG_PROCESS_TERMINATE_REASON_COUNT = 6
+  };
+
+  // Used to send the result of interaction with remote process as histograms in
+  // case when remote process influences on start.
+  enum RemoteProcessInteractionResult {
+    TERMINATE_SUCCEEDED = 0,
+    TERMINATE_FAILED = 1,
+    REMOTE_PROCESS_NOT_FOUND = 2,
+#if defined(OS_WIN)
+    TERMINATE_WAIT_TIMEOUT = 3,
+    RUNNING_PROCESS_NOTIFY_ERROR = 4,
+#elif defined(OS_POSIX)
+    TERMINATE_NOT_ENOUGH_PERMISSIONS = 5,

[gab, 2017/05/12 18:40:59]

Why are some terminate histograms here instead of under
RemoteHungProcessTerminateReason?

[Alexey Seren, 2017/05/15 08:40:48]

"RemoteProcessInteractionResult" histogram tracks what happened if process
starts in presence of existing browsers.
"RemoteHungProcessTerminateReason" histogram track the reason why we have tried
to terminate remote process.

TERMINATE_NOT_ENOUGH_PERMISSIONS indicated what happened. It means that we have
tried to terminate remote process but have no permissions for it.

+    REMOTE_PROCESS_SHUTTING_DOWN = 6,
+    PROFILE_UNLOCKED = 7,
+    FAILED_TO_EXTRACT_PID = 8,
+    INVALID_LOCK_FILE = 9,
+    ORPHANED_LOCK_FILE = 10,
+    SAME_BROWSER_INSTANCE = 11,
+#endif
+    REMOTE_PROCESS_INTERACTION_RESULT_COUNT = 12
+  };
+
   // Logged as histograms, do not modify these values.
   enum NotifyResult {
     PROCESS_NONE = 0,
     PROCESS_NOTIFIED = 1,
     PROFILE_IN_USE = 2,
     LOCK_ERROR = 3,
     LAST_VALUE = LOCK_ERROR
   };
 
   static constexpr int kNumNotifyResults = LAST_VALUE + 1;
@@ skipping 116 matching lines @@
   // Helper class for linux specific messages.  LinuxWatcher is ref counted
   // because it posts messages between threads.
   class LinuxWatcher;
   scoped_refptr<LinuxWatcher> watcher_;
 #endif
 
   DISALLOW_COPY_AND_ASSIGN(ProcessSingleton);
 };
 
 #endif  // CHROME_BROWSER_PROCESS_SINGLETON_H_