{"payload":{"feedbackUrl":"https://github.com/orgs/community/discussions/53140","repo":{"id":221047297,"defaultBranch":"main","name":"pdfium","ownerLogin":"chromium","currentUserCanPush":false,"isFork":false,"isEmpty":false,"createdAt":"2019-11-11T18:46:42.000Z","ownerAvatar":"https://avatars.githubusercontent.com/u/30044?v=4","public":true,"private":false,"isOrgOwned":true},"refInfo":{"name":"","listCacheKey":"v0:1716520010.0","currentOid":""},"activityList":{"items":[{"before":null,"after":"151ec02bc547f4d0a52a2386b12e0bb17e89dd4d","ref":"refs/heads/chromium/6498","pushedAt":"2024-05-24T03:06:50.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Roll base/allocator/partition_allocator/ 0e650666e..17c0ead1e0f8 (14 commits)\n\nhttps://chromium.googlesource.com/chromium/src/base/allocator/partition_allocator.git/+log/0e650666e776..17c0ead1e0f8\n\nPartitionAlloc needs to stop depending on chromium's //build in order to\nbe used in Skia. In particular, this means to stop depending on\n`BUILDFLAG`. It will becomes `PA_BUILDFLAG`.\n\nBug: 41481467\nChange-Id: Ib2221088f47e172499f6f8ffb13618c9ec28a41f\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119550\nReviewed-by: Lei Zhang \nAuto-Submit: Arthur Sonzogni \nCommit-Queue: Arthur Sonzogni ","shortMessageHtmlLink":"Roll base/allocator/partition_allocator/ 0e650666e..17c0ead1e0f8 (14 …"}},{"before":"1187f90253249bad1b9f2601314af5f76c82378e","after":"74245ead79a4b9aa40010d2b84710000f1a767c9","ref":"refs/heads/main","pushedAt":"2024-05-24T03:06:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Mark unsafe regions in core/fpdfapi files.\n\nOne file remains in this directory which is suppressed at file level,\nsince it is not straightforward to fix.\n\nBug: pdfium: 42271175\nChange-Id: I33c833aab44f7210482c4bcd6b4fc640e47aaaba\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119572\nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Mark unsafe regions in core/fpdfapi files."}},{"before":"14a61610bdd2af26d1cce44761b1a99b3d6b5b39","after":"1187f90253249bad1b9f2601314af5f76c82378e","ref":"refs/heads/main","pushedAt":"2024-05-24T00:30:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Spanify methods in BC_OneDimWriter\n\nBug: pdfium:2155\nChange-Id: If1812cbf546137b08166b73e367f9ca5b0679748\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119511\nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Spanify methods in BC_OneDimWriter"}},{"before":"29b9bb023e3b12e208d8babf4ae2df28bfa6d2b1","after":"14a61610bdd2af26d1cce44761b1a99b3d6b5b39","ref":"refs/heads/main","pushedAt":"2024-05-23T20:10:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Use fxcrt:ToArray<>() in more places.\n\nFirst, make fxcrt::ToArray<>() more similar to std::to_array<>() by\nremoving the right-hand side type parameter and using the left-hand\nside type parameter througout. This allows usage against structured\ndata without having to specify the exact type.\n\nThen, use it in two places as useful.\n\nChange-Id: Iea5ee7930b864c24094df6b56e87419c2024a063\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119570\nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Use fxcrt:ToArray<>() in more places."}},{"before":"151ec02bc547f4d0a52a2386b12e0bb17e89dd4d","after":"29b9bb023e3b12e208d8babf4ae2df28bfa6d2b1","ref":"refs/heads/main","pushedAt":"2024-05-23T18:26:52.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Rename CPDF_String variables\n\n- Follow Google C++ style guide.\n- Clarify what \"m_bHex\" does.\n\nChange-Id: I6017ba3ad2571f6e796630081ab041334063a9b2\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119431\nCommit-Queue: Lei Zhang \nReviewed-by: Tom Sepez \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Rename CPDF_String variables"}},{"before":null,"after":"d4b014a1b48e326ac526877cd8697da283aaa479","ref":"refs/heads/chromium/6497","pushedAt":"2024-05-23T17:08:52.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()\n\nInstead of instantiating a CPDF_String to ultimately call\nPDF_DecodeText(), just call PDF_DecodeText() directly.\n\nChange-Id: I9d97927b96561f1239c42c18d5ee21cb79acd93c\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119411\nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()"}},{"before":"59f088efc51f9cb831e8278263c90dfe3935207d","after":"151ec02bc547f4d0a52a2386b12e0bb17e89dd4d","ref":"refs/heads/main","pushedAt":"2024-05-23T16:42:52.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Roll base/allocator/partition_allocator/ 0e650666e..17c0ead1e0f8 (14 commits)\n\nhttps://chromium.googlesource.com/chromium/src/base/allocator/partition_allocator.git/+log/0e650666e776..17c0ead1e0f8\n\nPartitionAlloc needs to stop depending on chromium's //build in order to\nbe used in Skia. In particular, this means to stop depending on\n`BUILDFLAG`. It will becomes `PA_BUILDFLAG`.\n\nBug: 41481467\nChange-Id: Ib2221088f47e172499f6f8ffb13618c9ec28a41f\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119550\nReviewed-by: Lei Zhang \nAuto-Submit: Arthur Sonzogni \nCommit-Queue: Arthur Sonzogni ","shortMessageHtmlLink":"Roll base/allocator/partition_allocator/ 0e650666e..17c0ead1e0f8 (14 …"}},{"before":"ddc8f9c5331090e1e9665ee42f0cd45c3e63633e","after":"59f088efc51f9cb831e8278263c90dfe3935207d","ref":"refs/heads/main","pushedAt":"2024-05-23T16:16:53.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Fix a regression in linearized PDFs that have been edited\n\nUnlike regular PDFs, linearized PDFs can have a XRefStm section in the\nfirst page trailer. The spec is not clear. This code was removed in\nhttps://pdfium-review.googlesource.com/118571. Undo that specific change\nand add it back.\n\nBug: 341973428\nChange-Id: I48b8bdff191f4c7313f961850dc5023d48ac19e5\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119530\nCommit-Queue: Lei Zhang \nReviewed-by: Tom Sepez \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Fix a regression in linearized PDFs that have been edited"}},{"before":null,"after":"d4b014a1b48e326ac526877cd8697da283aaa479","ref":"refs/heads/chromium/6496","pushedAt":"2024-05-23T03:16:50.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()\n\nInstead of instantiating a CPDF_String to ultimately call\nPDF_DecodeText(), just call PDF_DecodeText() directly.\n\nChange-Id: I9d97927b96561f1239c42c18d5ee21cb79acd93c\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119411\nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()"}},{"before":"b79109a34b24a017a32969c8f62d1837458c1cac","after":"ddc8f9c5331090e1e9665ee42f0cd45c3e63633e","ref":"refs/heads/main","pushedAt":"2024-05-23T02:50:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Fix unsafe usage in CPDF_StreamContentParser\n\nConvert one local to std::array<>.\n\nBug: pdfium:2155\nChange-Id: I1dfc606bf03454c68a5f87ceba7f550ae3003eaf\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119510\nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Fix unsafe usage in CPDF_StreamContentParser"}},{"before":"58ac43157113d4b68b452c8be41c75d3633e1ecd","after":"b79109a34b24a017a32969c8f62d1837458c1cac","ref":"refs/heads/main","pushedAt":"2024-05-22T23:48:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Mark unsafe_buffers regions in core/fpdfapi/font files.\n\nBug: pdfium:2154\nChange-Id: Idd5e91bf8f1141820d9cc916b2889ffd5893e2eb\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119470\nReviewed-by: Thomas Sepez \nReviewed-by: Lei Zhang \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Mark unsafe_buffers regions in core/fpdfapi/font files."}},{"before":"6dc6a01137ef87bbcb1bd082e80f3cf6469afdcf","after":"58ac43157113d4b68b452c8be41c75d3633e1ecd","ref":"refs/heads/main","pushedAt":"2024-05-22T21:38:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Mark unsafe_buffers sections in core/fpdfdoc/ non-test files\n\nBug: pdfium:2154\nChange-Id: Ied9b68d6f179b39c7ae7ba389d1f078358eebfbe\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119451\nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Mark unsafe_buffers sections in core/fpdfdoc/ non-test files"}},{"before":"d4b014a1b48e326ac526877cd8697da283aaa479","after":"6dc6a01137ef87bbcb1bd082e80f3cf6469afdcf","ref":"refs/heads/main","pushedAt":"2024-05-22T21:12:51.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Mark unsafe regions in CFGAS_Decimal implementation\n\nBug: pdfium:2154\nChange-Id: Icbc7dd24e59d3ed9d9cbd857037f49c7041ac1cc\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119450\nReviewed-by: Thomas Sepez \nReviewed-by: Lei Zhang \nCommit-Queue: Tom Sepez ","shortMessageHtmlLink":"Mark unsafe regions in CFGAS_Decimal implementation"}},{"before":null,"after":"8007d6ed5cd165d13c5655f9a230b58cf10195f0","ref":"refs/heads/chromium/6495","pushedAt":"2024-05-22T15:08:53.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Relax pdfium_test assertion for not depending on gtest\n\nOnly check pdfium_test does not depend on gtest when build_with_chromium\nis set to false. In Chromium's build environment, pdfium_test has an\nindirect dependency on gtest.\n\nChange-Id: I602acab0373616d24a40ac44002e85090124d75b\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119413\nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Relax pdfium_test assertion for not depending on gtest"}},{"before":null,"after":"8007d6ed5cd165d13c5655f9a230b58cf10195f0","ref":"refs/heads/chromium/6494","pushedAt":"2024-05-22T03:26:50.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Relax pdfium_test assertion for not depending on gtest\n\nOnly check pdfium_test does not depend on gtest when build_with_chromium\nis set to false. In Chromium's build environment, pdfium_test has an\nindirect dependency on gtest.\n\nChange-Id: I602acab0373616d24a40ac44002e85090124d75b\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119413\nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Relax pdfium_test assertion for not depending on gtest"}},{"before":"3d82f31bff680789d1e428c70875b1490ab8530b","after":"d4b014a1b48e326ac526877cd8697da283aaa479","ref":"refs/heads/main","pushedAt":"2024-05-22T02:34:50.000Z","pushType":"push","commitsCount":3,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()\n\nInstead of instantiating a CPDF_String to ultimately call\nPDF_DecodeText(), just call PDF_DecodeText() directly.\n\nChange-Id: I9d97927b96561f1239c42c18d5ee21cb79acd93c\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119411\nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Avoid CPDF_String instantiation in FPDFAttachment_GetStringValue()"}},{"before":"7d870c71d7612b560391be35f0dccecb4680101f","after":"3d82f31bff680789d1e428c70875b1490ab8530b","ref":"refs/heads/main","pushedAt":"2024-05-22T00:24:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Fix regression in CFX_AggDeviceDriver::SetClipMask()\n\nIn https://pdfium-review.googlesource.com/115412, SetClipMask()\naccidentally returned early and failed to do clipping when the clip box\nis empty. Adjust the code to restore the original behavior, so\nCFX_ClipRgn::IntersectMaskF() still gets called. Then change\nIntersectMaskF() to tolerate empty bitmaps.\n\nAdd yet another rectangles.in variant as a pixel test to exercise this\ncase. With a 0x0 clip box, the PDF should render blank.\n\nBug: 341357062\nChange-Id: I05d1b096d9652668aa338ca95ab20de91ccfeace\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119430\nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Fix regression in CFX_AggDeviceDriver::SetClipMask()"}},{"before":null,"after":"94407286f475918c68195af1f5e01847b2d6f6ce","ref":"refs/heads/chromium/6422_76","pushedAt":"2024-05-21T20:30:51.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"M125: Fix one more edited PDF regression\n\nRestore some of the CPDF_Parser::LoadAllSecondaryCrossRefStreams() logic\nthat https://pdfium-review.googlesource.com/118571 deleted. Since the\nCPDF_Parser code changed quite a bit, incorporate that logic into\nFindAllCrossReferenceTablesAndStream().\n\nWith this fix in place, unsuppress the bug_2152.pdf pixel test.\n\nBug: 335309995, pdfium:2152\nChange-Id: Ie4cf79cddca6e0f1f4a331ceea5676f62c7d4a6e\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/118756\nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \n(cherry picked from commit 9783ba4860001b8891bb68f9fab4ee746d027ea3)\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119091","shortMessageHtmlLink":"M125: Fix one more edited PDF regression"}},{"before":null,"after":"94407286f475918c68195af1f5e01847b2d6f6ce","ref":"refs/heads/chromium/6422_53","pushedAt":"2024-05-21T18:20:52.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"M125: Fix one more edited PDF regression\n\nRestore some of the CPDF_Parser::LoadAllSecondaryCrossRefStreams() logic\nthat https://pdfium-review.googlesource.com/118571 deleted. Since the\nCPDF_Parser code changed quite a bit, incorporate that logic into\nFindAllCrossReferenceTablesAndStream().\n\nWith this fix in place, unsuppress the bug_2152.pdf pixel test.\n\nBug: 335309995, pdfium:2152\nChange-Id: Ie4cf79cddca6e0f1f4a331ceea5676f62c7d4a6e\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/118756\nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez \n(cherry picked from commit 9783ba4860001b8891bb68f9fab4ee746d027ea3)\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119091","shortMessageHtmlLink":"M125: Fix one more edited PDF regression"}},{"before":"4937f86f6646d045433e97ca94fe9391baed8937","after":"7d870c71d7612b560391be35f0dccecb4680101f","ref":"refs/heads/main","pushedAt":"2024-05-21T17:02:52.000Z","pushType":"push","commitsCount":2,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Add a 2-parameter CPDF_String ctor\n\nPrepare for removing the 3-parameter CPDF_String ctor by adding a\n2-param ctor that covers the majority of the use cases, where the `bHex`\nargument is false.\n\nChange-Id: I6b4f99c87201e71712303b8c9be2f2406597edb1\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119391\nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez \nReviewed-by: Tom Sepez ","shortMessageHtmlLink":"Add a 2-parameter CPDF_String ctor"}},{"before":"8007d6ed5cd165d13c5655f9a230b58cf10195f0","after":"4937f86f6646d045433e97ca94fe9391baed8937","ref":"refs/heads/main","pushedAt":"2024-05-21T16:36:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Make CFX_CodecMemory XFA-only\n\nIt is only used by XFA-specific decoders.\n\nChange-Id: I6129755f700445ddb77ebc040486b15f64b70f36\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119412\nReviewed-by: Thomas Sepez \nCommit-Queue: Lei Zhang \nReviewed-by: Tom Sepez ","shortMessageHtmlLink":"Make CFX_CodecMemory XFA-only"}},{"before":"58884c2bdb5e7d7d3b823d9c3e2916a825d22205","after":"8007d6ed5cd165d13c5655f9a230b58cf10195f0","ref":"refs/heads/main","pushedAt":"2024-05-21T15:44:53.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Relax pdfium_test assertion for not depending on gtest\n\nOnly check pdfium_test does not depend on gtest when build_with_chromium\nis set to false. In Chromium's build environment, pdfium_test has an\nindirect dependency on gtest.\n\nChange-Id: I602acab0373616d24a40ac44002e85090124d75b\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119413\nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Relax pdfium_test assertion for not depending on gtest"}},{"before":null,"after":"49db42fb84e011f3367f30872266ab3a9c621489","ref":"refs/heads/chromium/6493","pushedAt":"2024-05-21T15:18:53.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Roll Catapult from 5fbda23b3a86 to 1bce0121e31e (52 revisions)\n\nhttps://chromium.googlesource.com/catapult.git/+log/5fbda23b3a86..1bce0121e31e\n\nIf this roll has caused a breakage, revert this CL and stop the roller\nusing the controls here:\nhttps://autoroll.skia.org/r/catapult-pdfium\nPlease CC dhoss@chromium.org,pdfium-reviews+autoroller@googlegroups.com,thestig@chromium.org on the revert to ensure that a human\nis aware of the problem.\n\nTo report a problem with the AutoRoller itself, please file a bug:\nhttps://issues.skia.org/issues/new?component=1389291&template=1850622\n\nDocumentation for the AutoRoller is here:\nhttps://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md\n\nBug: None\nTbr: pdfium-reviews+autoroller@googlegroups.com\nChange-Id: I6d4bae2821ad5f7543b562b99b8ea065d31abce1\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119291\nCommit-Queue: pdfium-autoroll \nBot-Commit: pdfium-autoroll ","shortMessageHtmlLink":"Roll Catapult from 5fbda23b3a86 to 1bce0121e31e (52 revisions)"}},{"before":"4bf2bb0063e71375d6a6ea3d03883e1b324e17a5","after":"58884c2bdb5e7d7d3b823d9c3e2916a825d22205","ref":"refs/heads/main","pushedAt":"2024-05-21T05:46:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Remove unneeded core/fxcrt/fx_memory_wrappers.h includes\n\nFiles that do not directly use code from fx_memory_wrappers.h can omit\nthis include. This is likely because they stopped using FxFreeDeleter\nand switched to DataVector instead.\n\nChange-Id: I9543609e37bf4a645fe90812a4b49f6ee96fcf5b\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119326\nReviewed-by: Tom Sepez \nReviewed-by: Thomas Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Remove unneeded core/fxcrt/fx_memory_wrappers.h includes"}},{"before":"387d0ea98ed5d33a25db982eac3baa989a808339","after":"4bf2bb0063e71375d6a6ea3d03883e1b324e17a5","ref":"refs/heads/main","pushedAt":"2024-05-21T05:20:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Rename DataVectorAndBytesConsumed to DataAndBytesConsumed\n\nNow that the original DataAndBytesConsumed is gone,\nDataVectorAndBytesConsumed can take its place and use its name.\n\nBug: pdfium:1872\nChange-Id: Idbe09215d3c6eb7ddb27ff424d67acadf9ed866f\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119325\nReviewed-by: Tom Sepez \nReviewed-by: Thomas Sepez \nCommit-Queue: Lei Zhang ","shortMessageHtmlLink":"Rename DataVectorAndBytesConsumed to DataAndBytesConsumed"}},{"before":"32811dd30b152166c04e19b3ae4d758a36caf094","after":"387d0ea98ed5d33a25db982eac3baa989a808339","ref":"refs/heads/main","pushedAt":"2024-05-21T03:36:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Convert FlateUncompress() to return DataVectorAndBytesConsumed instead\n\nConvert FlateUncompress(), the last DataAndBytesConsumed user, to return\nDataVectorAndBytesConsumed instead. Then remove DataAndBytesConsumed.\n\nBug: pdfium:1872\nChange-Id: I68977817dc85c8bbaefc1107519c484e3ab71e36\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119324\nReviewed-by: Tom Sepez \nCommit-Queue: Lei Zhang \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Convert FlateUncompress() to return DataVectorAndBytesConsumed instead"}},{"before":null,"after":"49db42fb84e011f3367f30872266ab3a9c621489","ref":"refs/heads/chromium/6492","pushedAt":"2024-05-21T03:10:51.000Z","pushType":"branch_creation","commitsCount":0,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Roll Catapult from 5fbda23b3a86 to 1bce0121e31e (52 revisions)\n\nhttps://chromium.googlesource.com/catapult.git/+log/5fbda23b3a86..1bce0121e31e\n\nIf this roll has caused a breakage, revert this CL and stop the roller\nusing the controls here:\nhttps://autoroll.skia.org/r/catapult-pdfium\nPlease CC dhoss@chromium.org,pdfium-reviews+autoroller@googlegroups.com,thestig@chromium.org on the revert to ensure that a human\nis aware of the problem.\n\nTo report a problem with the AutoRoller itself, please file a bug:\nhttps://issues.skia.org/issues/new?component=1389291&template=1850622\n\nDocumentation for the AutoRoller is here:\nhttps://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md\n\nBug: None\nTbr: pdfium-reviews+autoroller@googlegroups.com\nChange-Id: I6d4bae2821ad5f7543b562b99b8ea065d31abce1\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119291\nCommit-Queue: pdfium-autoroll \nBot-Commit: pdfium-autoroll ","shortMessageHtmlLink":"Roll Catapult from 5fbda23b3a86 to 1bce0121e31e (52 revisions)"}},{"before":"ead67c5e719a61603476a114076af36900293754","after":"32811dd30b152166c04e19b3ae4d758a36caf094","ref":"refs/heads/main","pushedAt":"2024-05-21T02:44:51.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Change CFX_FolderFontInfo to use FixedSizeDataVector\n\nSwitch from std::unique_ptr to FixedSizeDataVector.\n\nBug: pdfium:1872\nChange-Id: I9c5f27964603510f9ef93929bcc5e184b5efac4c\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119320\nCommit-Queue: Lei Zhang \nReviewed-by: Tom Sepez \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Change CFX_FolderFontInfo to use FixedSizeDataVector"}},{"before":"392e86f2e94ccf6f44b2ee1a7cffadb7fe6d6a2d","after":"ead67c5e719a61603476a114076af36900293754","ref":"refs/heads/main","pushedAt":"2024-05-21T02:18:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Fix unsafe buffer usage in core/fxge/dib/cfx_cmyk_to_srgb.cpp.\n\n-- Convert to std::array<>\n-- Use struct instead of uint8_t[3]\n-- Tidy index calculation with helper function.\n\nChange-Id: Iab2cce4803c5209ec8ed18d1735a916faf2ebcb8\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119390\nReviewed-by: Lei Zhang \nCommit-Queue: Tom Sepez \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Fix unsafe buffer usage in core/fxge/dib/cfx_cmyk_to_srgb.cpp."}},{"before":"15e66cefa0db503d1bc2522b75c7bcb321b84311","after":"392e86f2e94ccf6f44b2ee1a7cffadb7fe6d6a2d","ref":"refs/heads/main","pushedAt":"2024-05-21T01:52:50.000Z","pushType":"push","commitsCount":1,"pusher":{"login":"hubot","name":"Hubot","path":"/hubot","primaryAvatarUrl":"https://avatars.githubusercontent.com/u/480938?s=80&v=4"},"commit":{"message":"Introduce more helpful FX_*_STRUCT classes.\n\nAvoid buf[3]-style indexing and unsafe buffer warning suppression as\na result in CPDF_DeviceCS.\n\n-- avoid some else-clauses while at it.\n-- provide a truncating_reinterpret_span<>() function as a fudge\n for callers that have fixed-size buffers which may not be\n multiples of the cast-to type's size.\n-- Fix CPDF_ColorSpace to know actual number of components it holds\n to avoid length assertion of size 16 modulo 3 (size of RGB) during\n reinterpret_cast<>.\n\nChange-Id: I1ba92c5ff4be56403a12c707f9bf729e8f0b9486\nReviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/119253\nCommit-Queue: Tom Sepez \nReviewed-by: Lei Zhang \nReviewed-by: Thomas Sepez ","shortMessageHtmlLink":"Introduce more helpful FX_*_STRUCT classes."}}],"hasNextPage":true,"hasPreviousPage":false,"activityType":"all","actor":null,"timePeriod":"all","sort":"DESC","perPage":30,"cursor":"djE6ks8AAAAEUp7i4wA","startCursor":null,"endCursor":null}},"title":"Activity · chromium/pdfium"}