Change userid from a uint32_t to a string guid

mojo/shell/public/cpp/lib/application_test_base.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 <utility>
 
 #include "base/command_line.h"
 #include "base/logging.h"
 #include "base/strings/utf_string_conversions.h"
 #include "mojo/public/cpp/bindings/binding.h"
 #include "mojo/public/cpp/environment/environment.h"
 #include "mojo/public/cpp/system/message_pipe.h"
 #include "mojo/shell/public/cpp/application_test_base.h"
 #include "mojo/shell/public/cpp/shell_connection.h"
 #include "mojo/shell/public/interfaces/shell_client.mojom.h"
 
 namespace mojo {
 namespace test {
 
 namespace {
 // Share the application name with multiple application tests.
 String g_name;
 uint32_t g_id = shell::mojom::Connector::kInvalidApplicationID;
-uint32_t g_user_id = shell::mojom::Connector::kUserRoot;
+String g_user_id = shell::mojom::kRootUserID;
 
 // ShellClient request handle passed from the shell in MojoMain, stored in
 // between SetUp()/TearDown() so we can (re-)intialize new ShellConnections.
 InterfaceRequest<shell::mojom::ShellClient> g_shell_client_request;
 
 // Connector pointer passed in the initial mojo.ShellClient.Initialize() call,
 // stored in between initial setup and the first test and between SetUp/TearDown
 // calls so we can (re-)initialize new ShellConnections.
 shell::mojom::ConnectorPtr g_connector;
 
 class ShellGrabber : public shell::mojom::ShellClient {
  public:
   explicit ShellGrabber(InterfaceRequest<shell::mojom::ShellClient> request)
       : binding_(this, std::move(request)) {}
 
   void WaitForInitialize() {
     // Initialize is always the first call made on ShellClient.
     CHECK(binding_.WaitForIncomingMethodCall());
   }
 
  private:
   // shell::mojom::ShellClient implementation.
   void Initialize(shell::mojom::ConnectorPtr connector,
-                  const mojo::String& name,
-                  uint32_t id,
-                  uint32_t user_id) override {
+                  const String& name,
+                  const String& user_id,
+                  uint32_t id) override {
     g_name = name;
     g_id = id;
     g_user_id = user_id;
     g_shell_client_request = binding_.Unbind();
     g_connector = std::move(connector);
   }
 
   void AcceptConnection(
       const String& requestor_name,
-      uint32_t requestor_user_id,
+      const String& requestor_user_id,
       uint32_t requestor_id,
       shell::mojom::InterfaceProviderRequest local_interfaces,
       shell::mojom::InterfaceProviderPtr remote_interfaces,
       Array<String> allowed_interfaces,
       const String& name) override {
     CHECK(false);
   }
 
   Binding<ShellClient> binding_;
 };
@@ skipping 41 matching lines @@
   g_connector.reset();
 
   return (result == 0) ? MOJO_RESULT_OK : MOJO_RESULT_UNKNOWN;
 }
 
 TestHelper::TestHelper(ShellClient* client)
     : shell_connection_(new ShellConnection(
           client == nullptr ? &default_shell_client_ : client,
           std::move(g_shell_client_request))),
       name_(g_name),
-      instance_id_(g_id),
-      userid_(g_user_id) {
+      userid_(g_user_id),
+      instance_id_(g_id) {
   // Fake ShellClient initialization.
   shell::mojom::ShellClient* shell_client = shell_connection_.get();
-  shell_client->Initialize(std::move(g_connector), g_name, g_id, g_user_id);
+  shell_client->Initialize(std::move(g_connector), g_name, g_user_id, g_id);
 }
 
 TestHelper::~TestHelper() {
   // We may have supplied a member as the client. Delete |shell_connection_|
   // while still valid.
   shell_connection_.reset();
 }
 
 ApplicationTestBase::ApplicationTestBase() : test_helper_(nullptr) {}
 
@@ skipping 26 matching lines @@
   if (ShouldCreateDefaultRunLoop())
     Environment::DestroyDefaultRunLoop();
 }
 
 bool ApplicationTestBase::ShouldCreateDefaultRunLoop() {
   return true;
 }
 
 }  // namespace test
 }  // namespace mojo