Commit e70b8510 authored by Jose Dapena Paz's avatar Jose Dapena Paz Committed by Commit Bot

Add noexcept to move constructor and assign operator of mojom::InlinedStructPtr

blink::TransferableMessage was failing to build, as it had its move operator and
assignment operators set as noexcept, but their default implementation depended
on mojo::UserActivationSnapshotPtr to have noexcept too. This missed it
because its type was mojom::InlinedStructPtr, that was missing noexcept.

Bug: 819294
Change-Id: I0aaaf179a9f5240c4706cca5128568df45c0de25
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1768516Reviewed-by: default avatarScott Violet <sky@chromium.org>
Commit-Queue: José Dapena Paz <jose.dapena@lge.com>
Cr-Commit-Position: refs/heads/master@{#692160}
parent fc2c6b59
......@@ -127,8 +127,8 @@ class InlinedStructPtr {
return *this;
}
InlinedStructPtr(InlinedStructPtr&& other) { Take(&other); }
InlinedStructPtr& operator=(InlinedStructPtr&& other) {
InlinedStructPtr(InlinedStructPtr&& other) noexcept { Take(&other); }
InlinedStructPtr& operator=(InlinedStructPtr&& other) noexcept {
Take(&other);
return *this;
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment