{"push_id":2803431589,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"5f80ff49403645dc574296bb8b29fe44a2193c3c","before":"b6e3d231ef2c28df1e5149cb16ad7046ebb0714e","commits":[{"sha":"3e360453cded5f2f435195923ede0935f6847194","author":{"name":"Tom Sepez","email":"47f022c5217a84c904816c5375403e20a45c37f6@chromium.org"},"message":"Remove optional prepend argument from matrix transformations.\n\nCallers always want one form or the other, so split into separate\nmethods. This may save some branching.\n\nRemove forms that are not used.\nPut more helpful helper function in .cpp file.\n\nChange-Id: I4e025de7f69ce3f323cd290a5dc8202dd4f8ca07\nReviewed-on: https://pdfium-review.googlesource.com/40270\nCommit-Queue: Tom Sepez <tsepez@chromium.org>\nReviewed-by: Lei Zhang <thestig@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/3e360453cded5f2f435195923ede0935f6847194"},{"sha":"5f4cd74d2693e23b27237b935f28a87611e25e6b","author":{"name":"Henrique Nakashima","email":"c5741d2503c578c951245eae5dc1c73799e5b063@chromium.org"},"message":"Do not set barcode module width and height in unittest.\n\nSetting these to anything outside of [1, 10] is non-op. Not sure why\nthe test did that, but these lines have no effect.\n\nBug: pdfium:1135\nChange-Id: I175d07137674ab1b6b9c86b3bc68b4a128601b9f\nReviewed-on: https://pdfium-review.googlesource.com/40250\nReviewed-by: Tom Sepez <tsepez@chromium.org>\nReviewed-by: Ryan Harrison <rharrison@chromium.org>\nCommit-Queue: Henrique Nakashima <hnakashima@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/5f4cd74d2693e23b27237b935f28a87611e25e6b"},{"sha":"2a3377ce9a39d47d29c95d5db64690ad749d8c94","author":{"name":"Henrique Nakashima","email":"c5741d2503c578c951245eae5dc1c73799e5b063@chromium.org"},"message":"Optimize rendering of two dimensional barcodes: deduplicate regions.\n\nReduce the logical size of the barcode by removing unnecessary\nregion duplication.\n\nAs far as I can tell, the line thickness is useless and the aspect\nratio causes arbitrary changes in rounding, but ultimately the\ndimensions of a barcode are defined by its width and height, rather\nthan by this ratio.\n\nThe improvement with this CL is from ~580ms to ~390ms per barcode,\nso about 1.5x. Combined with\nhttps://pdfium-review.googlesource.com/c/pdfium/+/40010\nthe improvement is to ~15ms, which is about 39x.\n\nThis also fixes the rendering of the barcode in the pixel and\ncorpus tests. You can verify this pointing a barcode reader app at\nthe screen. It does not however fix every case, as the unit test is\nstill unreadable.\n\nBug: 872907, pdfium:1135\nChange-Id: Ic28e60f54719552cfe69ace7ebc3f730c338a129\nReviewed-on: https://pdfium-review.googlesource.com/40030\nCommit-Queue: Henrique Nakashima <hnakashima@chromium.org>\nReviewed-by: Lei Zhang <thestig@chromium.org>\nReviewed-by: Ryan Harrison <rharrison@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/2a3377ce9a39d47d29c95d5db64690ad749d8c94"},{"sha":"048d7f7c1e0c6c42679a5646ae9db5b7d98ed755","author":{"name":"Tom Sepez","email":"47f022c5217a84c904816c5375403e20a45c37f6@chromium.org"},"message":"Remove more optional args in core/\n\nChange-Id: I6a2bd03e00ad4e3d57f6931c0c6cf4ae0c760afb\nReviewed-on: https://pdfium-review.googlesource.com/40290\nReviewed-by: Lei Zhang <thestig@chromium.org>\nCommit-Queue: Tom Sepez <tsepez@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/048d7f7c1e0c6c42679a5646ae9db5b7d98ed755"},{"sha":"f956badf3dd2c7eedba47dfcb787d6e6dfe92cf7","author":{"name":"Henrique Nakashima","email":"c5741d2503c578c951245eae5dc1c73799e5b063@chromium.org"},"message":"Add a way to save a barcode generated bitmap to a .png.\n\nThis is useful for debugging and validation purposes. The hashes that\nare in BarcodeTest are not necessarily for valid outputs.\n\nThis CL refactors the code in embedder_test.png that already does this,\nmoving it to testing/utils where unit tests can access it too.\n\nBug: pdfium:1135\nChange-Id: I6f1d70a4e133f8f04dbe52646087f99c448e95f8\nReviewed-on: https://pdfium-review.googlesource.com/40152\nCommit-Queue: Henrique Nakashima <hnakashima@chromium.org>\nReviewed-by: Ryan Harrison <rharrison@chromium.org>\nReviewed-by: Tom Sepez <tsepez@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/f956badf3dd2c7eedba47dfcb787d6e6dfe92cf7"},{"sha":"5f80ff49403645dc574296bb8b29fe44a2193c3c","author":{"name":"Henrique Nakashima","email":"c5741d2503c578c951245eae5dc1c73799e5b063@chromium.org"},"message":"If rows need to overflow in a PDF417 barcode, keep it to a minimum.\n\nBy using mincols as fallback, the exact opposite is done. Maxcols\nshould be used instead to minimize the size.\n\nThis is important so the next fix\n(https://pdfium-review.googlesource.com/c/pdfium/+/40350)\ndoes not break a test that was working well by coincidence.\n\nBug: pdfium:1135\nChange-Id: I95ddc547654966f655e2556c1796503a5456fb8f\nReviewed-on: https://pdfium-review.googlesource.com/40330\nReviewed-by: Ryan Harrison <rharrison@chromium.org>\nCommit-Queue: Ryan Harrison <rharrison@chromium.org>","distinct":true,"url":"https://api.github.com/repos/ibaoger/pdfium/commits/5f80ff49403645dc574296bb8b29fe44a2193c3c"}]}
{"actor":{"display_login":"ibaoger"}}