Demote Mojo shell errors to warnings

mojo/shell/shell.cc

 // Copyright 2014 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.
 
 #include "mojo/shell/shell.h"
 
 #include <stdint.h>
 
 #include <utility>
 
@@ skipping 281 matching lines @@
     }
     return true;
   }
 
   bool ValidateClientProcessConnection(
       mojom::ClientProcessConnectionPtr* client_process_connection,
       const Identity& target,
       const ConnectCallback& callback) {
     if (!client_process_connection->is_null()) {
       if (!HasClass(capability_spec_, kCapabilityClass_ClientProcess)) {
-        LOG(ERROR) << "Error: Instance: " << identity_.name() << " attempting "
-                   << "to register an instance for a process it created for "
-                   << "target: " << target.name() << " without the "
-                   << "mojo:shell{client_process} capability class.";
+        LOG(WARNING) << "Instance: " << identity_.name() << " attempting "
+                     << "to register an instance for a process it created for "
+                     << "target: " << target.name() << " without the "
+                     << "mojo:shell{client_process} capability class.";
         callback.Run(mojom::ConnectResult::ACCESS_DENIED,
                      mojom::kInheritUserID, mojom::kInvalidInstanceID);
         return false;
       }
 
       if (!(*client_process_connection)->shell_client.is_valid() ||
           !(*client_process_connection)->pid_receiver_request.is_valid()) {
-        LOG(ERROR) << "Error: must supply both shell_client AND "
-                   << "pid_receiver_request when sending "
-                   << "client_process_connection.";
+        LOG(WARNING) << "Must supply both shell_client AND "
+                     << "pid_receiver_request when sending "
+                     << "client_process_connection.";
         callback.Run(mojom::ConnectResult::INVALID_ARGUMENT,
                      mojom::kInheritUserID, mojom::kInvalidInstanceID);
         return false;
       }
       if (shell_->GetExistingInstance(target)) {
-        LOG(ERROR) << "Error: Cannot client process matching existing identity:"
-                   << "Name: " << target.name() << " User: " << target.user_id()
-                   << " Instance: " << target.instance();
+        LOG(WARNING) << "Cannot client process matching existing identity:"
+                     << "Name: " << target.name() << " User: "
+                     << target.user_id() << " Instance: " << target.instance();
         callback.Run(mojom::ConnectResult::INVALID_ARGUMENT,
                      mojom::kInheritUserID, mojom::kInvalidInstanceID);
         return false;
       }
     }
     return true;
   }
 
   bool ValidateCapabilities(const Identity& target,
                             const ConnectCallback& callback) {
@@ skipping 445 matching lines @@
   for (auto it = native_runners_.begin(); it != native_runners_.end(); ++it) {
     if (it->get() == runner) {
       native_runners_.erase(it);
       return;
     }
   }
 }
 
 }  // namespace shell
 }  // namespace mojo