Mojo C++ bindings: switch services/ui/public/interfaces mojom target to use STL types.

services/ui/public/cpp/tests/window_tree_client_unittest.cc

 // Copyright 2015 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 "services/ui/public/cpp/window_tree_client.h"
 
 #include <stdint.h>
 
 #include "base/logging.h"
 #include "base/macros.h"
@@ skipping 22 matching lines @@
 namespace {
 
 void DoNothingWithEventResult(mojom::EventResult result) {}
 
 Id server_id(ui::Window* window) {
   return WindowPrivate(window).server_id();
 }
 
 }  // namespace
 
-mojo::Array<uint8_t> Int32ToPropertyTransportValue(int32_t value) {
-  const std::vector<uint8_t> bytes =
-      mojo::ConvertTo<std::vector<uint8_t>>(value);
-  mojo::Array<uint8_t> transport_value;
-  transport_value.resize(bytes.size());
-  memcpy(&transport_value.front(), &(bytes.front()), bytes.size());
-  return transport_value;
+std::vector<uint8_t> Int32ToPropertyTransportValue(int32_t value) {
+  return mojo::ConvertTo<std::vector<uint8_t>>(value);
 }
 
 class TestWindowTreeClientDelegate : public WindowTreeClientDelegate {
  public:
   TestWindowTreeClientDelegate() {}
   ~TestWindowTreeClientDelegate() override {}
 
   ui::PointerEvent* last_event_observed() { return last_event_observed_.get(); }
 
   void Reset() { last_event_observed_.reset(); }
@@ skipping 712 matching lines @@
   uint32_t property_in_flight_change_id;
   ASSERT_TRUE(
       setup.window_tree()->GetAndClearChangeId(&property_in_flight_change_id));
   EXPECT_NE(bounds_in_flight_change_id, property_in_flight_change_id);
 
   // Ack the new window top level window. Vis and bounds shouldn't change.
   mojom::WindowDataPtr data = mojom::WindowData::New();
   data->window_id = server_id(root2);
   data->bounds.SetRect(1, 2, 3, 4);
   data->visible = true;
-  data->properties["xx"] = mojo::Array<uint8_t>::From(std::string("server_xx"));
-  data->properties["yy"] = mojo::Array<uint8_t>::From(std::string("server_yy"));
+  data->properties["xx"] =
+      mojo::Array<uint8_t>::From(std::string("server_xx")).PassStorage();
+  data->properties["yy"] =
+      mojo::Array<uint8_t>::From(std::string("server_yy")).PassStorage();
   const int64_t display_id = 1;
   setup.window_tree_client()->OnTopLevelCreated(
       new_window_in_flight_change_id, std::move(data), display_id, true);
 
   // The only value that should take effect is the property for 'yy' as it was
   // not in flight.
   EXPECT_TRUE(WindowPrivate(root2).parent_drawn());
   EXPECT_FALSE(root2->visible());
   EXPECT_EQ(1, root2->display_id());
   EXPECT_EQ(gfx::Rect(5, 6, 7, 8), root2->bounds());
@@ skipping 289 matching lines @@
 
   // Changes originating from server should notify observers too.
   WindowTreeClientPrivate(setup.client()).CallOnCaptureChanged(child2, nullptr);
   EXPECT_EQ(1, capture_recorder.capture_changed_count());
   EXPECT_EQ(child2_id, capture_recorder.last_gained_capture_window_id());
   EXPECT_EQ(0, capture_recorder.last_lost_capture_window_id());
   capture_recorder.reset_capture_captured_count();
 }
 
 }  // namespace ui