Add check if renderer is already launching (#912)

* If `StartRenderer` has already started launching a renderer, it skips duplicated launches.
* #912

PiperOrigin-RevId: 622224128

Author: hiroyuki-komatsu

src/renderer/BUILD.bazel

@@ -62,6 +62,7 @@ mozc_cc_library(
         "//ipc:named_event",
         "//protocol:renderer_cc_proto",
         "@com_google_absl//absl/base:core_headers",
+        "@com_google_absl//absl/log",
         "@com_google_absl//absl/synchronization",
         "@com_google_absl//absl/time",
     ] + mozc_select(

src/renderer/renderer_client.cc

@@ -37,11 +37,11 @@
 #include <string>
 
 #include "absl/base/thread_annotations.h"
+#include "absl/log/log.h"
 #include "absl/synchronization/mutex.h"
 #include "absl/time/clock.h"
 #include "absl/time/time.h"
 #include "base/clock.h"
-#include "base/logging.h"
 #include "base/process.h"
 #include "base/system_util.h"
 #include "base/thread.h"
@@ -106,6 +106,10 @@ class RendererLauncher : public RendererLauncherInterface {
       const std::string &name, const std::string &path,
       bool disable_renderer_path_check,
       IPCClientFactoryInterface *ipc_client_factory_interface) override {
+    if (Status() == RendererStatus::RENDERER_LAUNCHING) {
+      // Renderer is already launching.
+      return;
+    }
     SetStatus(RendererStatus::RENDERER_LAUNCHING);
     name_ = name;
     path_ = path;