Address review comments

Author: bleroux

shell/platform/common/client_wrapper/basic_message_channel_unittests.cc

@@ -25,9 +25,8 @@ class TestBinaryMessenger : public BinaryMessenger {
     send_called_ = true;
     int length = static_cast<int>(message_size);
     last_message_size_ = length;
-    std::vector<uint8_t> last_message(length);
-    memcpy(&last_message[0], &message[0], length * sizeof(uint8_t));
-    last_message_ = last_message;
+    last_message_ =
+        std::vector<uint8_t>(message, message + length * sizeof(uint8_t));
   }
 
   void SetMessageHandler(const std::string& channel,

shell/platform/common/client_wrapper/include/flutter/basic_message_channel.h

@@ -12,7 +12,7 @@
 #include "binary_messenger.h"
 #include "message_codec.h"
 #include "method_channel.h"
-#include "standard_method_codec.h"
+
 namespace flutter {
 
 class EncodableValue;

shell/platform/common/client_wrapper/include/flutter/method_channel.h

@@ -15,12 +15,12 @@
 #include "method_result.h"
 #include "standard_method_codec.h"
 
+namespace flutter {
+
 static constexpr char kControlChannelName[] = "dev.flutter/channel-buffers";
 static constexpr char kResizeMethod[] = "resize";
 static constexpr char kOverflowMethod[] = "overflow";
 
-namespace flutter {
-
 class EncodableValue;
 
 // A handler for receiving a method call from the Flutter engine.