From ed30040dfc78d3f47b515f29ca12cd4b2fa61ada Mon Sep 17 00:00:00 2001 From: Gerrit User 1001562 <1001562@3ce6091f-6c88-37e8-8c75-72f92ae8dfba> Date: Wed, 14 Oct 2026 16:09:21 -0700 Subject: [PATCH] Update patch set 4 Patch Set 4: (2 comments) Patch-set: 4 --- 8dabf898eeee2e08d41666ee24533da35f4dcee7 | 34 ++++++++++++++++++++++++ 1 file changed, 34 insertions(+) diff --git a/8dabf898eeee2e08d41666ee24533da35f4dcee7 b/8dabf898eeee2e08d41666ee24533da35f4dcee7 index 381929448..f80be954b 100644 --- a/8dabf898eeee2e08d41666ee24533da35f4dcee7 +++ b/8dabf898eeee2e08d41666ee24533da35f4dcee7 @@ -1,5 +1,22 @@ { "comments": [ + { + "unresolved": true, + "key": { + "uuid": "df11df10_747fe48c", + "filename": "/COMMIT_MSG", + "patchSetId": 4 + }, + "lineNbr": 31, + "author": { + "id": 1001562 + }, + "writtenOn": "2026-10-14T23:09:21Z", + "side": 1, + "message": "Optional: add the following duplicate issue:\n\n```\nBug: chromium:546841026\n```", + "revId": "8dabf898eeee2e08d41666ee24533da35f4dcee7", + "serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba" + }, { "unresolved": false, "key": { @@ -67,6 +84,23 @@ "message": "Should we replace this line with the following?\n\n```\n dy \u003d 1 \u003c\u003c 16; // 1 as 16.16 fixed point\n```\n\nor\n\n```\n dy \u003d 1 \u003c\u003c 16; // FixedDiv(src_height, src_height)\n```", "revId": "8dabf898eeee2e08d41666ee24533da35f4dcee7", "serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba" + }, + { + "unresolved": false, + "key": { + "uuid": "8e6781ed_772860b1", + "filename": "source/scale_argb.cc", + "patchSetId": 4 + }, + "lineNbr": 909, + "author": { + "id": 1001562 + }, + "writtenOn": "2026-10-14T23:09:21Z", + "side": 1, + "message": "I verified that the `src_width \u003e 32768` check added here fixed [bug chromium:546841026](https://g-issues.chromium.org/issues/546841026).", + "revId": "8dabf898eeee2e08d41666ee24533da35f4dcee7", + "serverId": "3ce6091f-6c88-37e8-8c75-72f92ae8dfba" } ] } \ No newline at end of file