chromium/src
0
Fork 0
Code Activity

[PDF Ink Signatures] Refactor "updateInk2Thumbnail" message code

Browse Source 
Move the code that generates the "updateInk2Thumbnail" message into its
own helper function. The new function will get used again in
https://crrev.com/6290715.

Change-Id: I987ec38ac3ca093f4ce4773900d54cce64fcc528
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6288573
Reviewed-by: Alan Screen <awscreen@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1423377}
This commit is contained in:
Lei Zhang
2025-02-21 13:17:12 -08:00
committed by Chromium LUCI CQ
parent 7f18fb6485
commit d1cd91bf3c
1 changed files with 15 additions and 7 deletions

22
pdf/pdf_ink_module.cc

@ -62,6 +62,19 @@ namespace {
constexpr ink::AffineTransform kIdentityTransform;
base::Value::Dict CreateUpdateThumbnailMessage(
int page_index,
std::vector<uint8_t> image_data,
const gfx::Size& thumbnail_size) {
base::Value::Dict message;
message.Set("type", "updateInk2Thumbnail");
message.Set("pageNumber", page_index + 1);
message.Set("imageData", std::move(image_data));
message.Set("width", thumbnail_size.width());
message.Set("height", thumbnail_size.height());
return message;
}
ink::StrokeInput::ToolType GetToolTypeFromTouchEvent(
const blink::WebTouchEvent& event) {
// Assumes the caller already handled multi-touch events.
@ -189,13 +202,8 @@ void PdfInkModule::GenerateAndSendInkThumbnail(
return;
}
base::Value::Dict message;
message.Set("type", "updateInk2Thumbnail");
message.Set("pageNumber", page_index + 1);
message.Set("imageData", std::move(image_data));
message.Set("width", thumbnail_size.width());
message.Set("height", thumbnail_size.height());
client_->PostMessage(std::move(message));
client_->PostMessage(CreateUpdateThumbnailMessage(
page_index, std::move(image_data), thumbnail_size));
}
void PdfInkModule::GenerateAndSendInkThumbnailInternal(int page_index) {