Drop extensions use of legacy IPC serialization for SkBitmap, use mojo
This converts to using the skia::mojom::Bitmap serialize functions to
turn an SkBitmap into a string and back for putting into a base::Value
and sending over IPC.
This is a reland as the original CL would fail to serialize correctly
for large bitmaps, due to Bitmap mojoms trying to use a shared memory
handle, which does not encode/decode correctly as a string. Switched to
using InlineBitmap which will always serialize to bytes in the string
itself, and added a unit test for this.
Also a bonus browser test for the setIcon path from the extension
process.
[email protected]
[email protected]
Bug: 1144462
Change-Id: Idc2cfb0d70335557464be2abdeaa0bad11d38e9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2527518
Commit-Queue: danakj <[email protected]>
Reviewed-by: Reilly Grant <[email protected]>
Cr-Commit-Position: refs/heads/master@{#826091}
diff --git a/extensions/DEPS b/extensions/DEPS
index c37c9ba3..4d8fbad 100644
--- a/extensions/DEPS
+++ b/extensions/DEPS
@@ -18,6 +18,7 @@
"+extensions/test",
"+mojo/public",
"+services/service_manager/public",
+ "+skia/public/mojom",
"+testing",
"+third_party/blink/public/common/loader/url_loader_throttle.h",
"+third_party/skia/include",