inspector_protocol serializable extra copy

[Hunter Laux]

Any idea why this isn't being moved?

https://chromium.googlesource.com/v8/v8.git/+/refs/heads/master/third_party/inspector_protocol/crdtp/serializable.cc#21

- explicit PreSerialized(std::vector<uint8_t> bytes) : bytes_(bytes) {}

+ explicit PreSerialized(std::vector<uint8_t> bytes) : bytes_(std::move(bytes)) {}

[Andreas Haas]

Hi,

Thank you for the hint. It seems like we did not roll the latest version of the inspector protocol into V8.

Cheers, Andreas

--
--
v8-dev mailing list
v8-...@googlegroups.com
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to v8-dev+un...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/v8-dev/4ab46791-489e-42ce-aca2-0d1333878474n%40googlegroups.com.

[Yang Guo]

+Andrey Kosyakov 

To view this discussion on the web visit https://groups.google.com/d/msgid/v8-dev/CAELSTvdPVwf%3DyQ%3DEHdDS6OS_G77f5g_NYPY%2B%3Dztqni7BK6NMww%40mail.gmail.com.

[Andrey Kosyakov]

Thanks for head-up! Here goes the roll: https://chromium-review.googlesource.com/c/v8/v8/+/2892842