{"push_id":2588593740,"size":6,"distinct_size":6,"ref":"refs/heads/master","head":"55701fa64cadc9e252bb813472bcb65b90d36953","before":"4b9ef810f091b200d82fc6db150099043b132d73","commits":[{"sha":"08877dedc09bb91574d2fec18423767e210a249d","author":{"name":"Dominic Mazzoni","email":"f1ba45afdaeae77f45548dbdc09644eb747dce38@chromium.org"},"message":"Revert \"Avoid serializing the same accessibility node twice in the same message.\"\n\nThis reverts commit 8322b0b5429b545a4533be255e51725d06e719a2.\n\nReason for revert: Renderer crash, http://crbug.com/845778\n\nOriginal change's description:\n> Avoid serializing the same accessibility node twice in the same message.\n> \n> Bug: 651614\n> TBR=alexst@chromium.org\n> \n> Change-Id: I77ef290a193bd3dbb3dafdf0de5670a7deec6c86\n> Reviewed-on: https://chromium-review.googlesource.com/1063007\n> Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>\n> Reviewed-by: David Tseng <dtseng@chromium.org>\n> Cr-Commit-Position: refs/heads/master@{#560165}\n\nTBR=dmazzoni@chromium.org,alexst@chromium.org,dtseng@chromium.org\n\n# Not skipping CQ checks because original CL landed > 1 day ago.\n\nBug: 651614, 845778\nChange-Id: Ic025a09d11845892a0e11a350fe8c5963f053081\nReviewed-on: https://chromium-review.googlesource.com/1069891\nReviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>\nCommit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561246}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/08877dedc09bb91574d2fec18423767e210a249d"},{"sha":"8ca9db2b2cc381525b83c434bedc89834ff614fe","author":{"name":"Xiaocheng Hu","email":"59a3fcd17837bbbb682c8b6364c42a7a2ab279cb@chromium.org"},"message":"Remove offset_ from RangeSelectionAdjuster::RenderedPosition\n\n|RangeSelectionAdjuster::RenderedPosition::offset_| is a redundant\nmember of the class. It is used at only two places and can be replaced:\n\n1. In GetPosition(), where the RenderedPosition must be at bidi boundary,\n   in which case |offset_| is implied by |bidi_boundary_type_|.\n\n\n2. In operator==(). However,\n  - If |inline_box_| or |bidi_boundary_type_| of the two operands\n    differ, we don't need to compare |offset_|\n  - If |bidi_boundary_type_| is not |kNotBoundary|, |offset_| is implied\n    and doesn't need to be compared\n  - If |bidi_boundary_type_| is |kNotBoundary| (for both operands), we\n    can early-stop the bidi adjustment, and still don't need the exact\n    value of |offset_|\n\nHence, this patch removes the member, and modifies the two use sites\naccordingly.\n\nThis patch also helps generalizing |RangeSelectionAdjuster| for\nLayoutNG, where we don't necessarily have an offset on\n|NGPaintFragment|.\n\nBug: 811502\nChange-Id: I14bd6236073dcf262c34f98209d57871665f463c\nReviewed-on: https://chromium-review.googlesource.com/1070471\nCommit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>\nReviewed-by: Emil A Eklund <eae@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561247}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/8ca9db2b2cc381525b83c434bedc89834ff614fe"},{"sha":"564be974ba354505d1fde671c95622aefeae5c87","author":{"name":"Christopher Grant","email":"d6b2bad850a45b5a3004fa1d0baae992d9e6a46b@chromium.org"},"message":"VR: Close omnibox when the keyboard is explicitly dismissed\n\nUsually, the keyboard disappears because the focus requiring it is lost.\nBut for the omnibox, if the keyboard is explicitly dismissed with the\non-keyboard button, and omnibox focus is lost, we should exit omnibox\nedit mode.\n\nBUG=842898\nR=bshe\n\nCq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr\nChange-Id: I36ec30f3a6e430a5b412b680258463afca9b8c06\nReviewed-on: https://chromium-review.googlesource.com/1070528\nReviewed-by: Biao She <bshe@chromium.org>\nCommit-Queue: Christopher Grant <cjgrant@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561248}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/564be974ba354505d1fde671c95622aefeae5c87"},{"sha":"fa67db5c1a6e20836650aa14e15f7a6507a44da0","author":{"name":"Xi Cheng","email":"d9813a3b13a5e185a9ce323c881c6471791f41f6@chromium.org"},"message":"Log notification launch id decode status\n\nWe have ~1% invalid launch ids that are invalid, which needs more investigation.\n\nBug: 839942, 734095\nChange-Id: Ibebb6beeebb6fe4200d49a728dde56d979a795cc\nReviewed-on: https://chromium-review.googlesource.com/1068655\nCommit-Queue: Xi Cheng <chengx@chromium.org>\nReviewed-by: Peter Beverloo <peter@chromium.org>\nReviewed-by: Finnur Thorarinsson <finnur@chromium.org>\nReviewed-by: Brian White <bcwhite@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561249}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/fa67db5c1a6e20836650aa14e15f7a6507a44da0"},{"sha":"3c1035553514e770022274c9c3b3bd207f3d515e","author":{"name":"Jay Civelli","email":"466ae04a4132785762ba94816232f8a2241e1861@google.com"},"message":"Change worker browser tests to use HTTP instead of file URLs\n\nChanging the worker browser tests to use HTTP URLs instead of file URLs\nin preparation for disabling file access in the network service which\nwill cause these tests to fail.\n\nBug: 759230\nChange-Id: I0d7cccfe19698f8c273a04ee44d1b508aaff1149\nReviewed-on: https://chromium-review.googlesource.com/1069743\nReviewed-by: Matt Falkenhagen <falken@chromium.org>\nReviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>\nCommit-Queue: Jay Civelli <jcivelli@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561250}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/3c1035553514e770022274c9c3b3bd207f3d515e"},{"sha":"55701fa64cadc9e252bb813472bcb65b90d36953","author":{"name":"Xiangjun Zhang","email":"656591f89f9368c8286e684e2d900ef849badbc7@chromium.org"},"message":"Mirroring Service: Change cast message data to string type.\n\nChanged the cast message data to JSON format string type. For both\ninbound and outbound messages, the serialization/deserialization are\nboth done by Mirroring Service only.\n\nBug: 734672\nChange-Id: I0d97c07b7d0d4af1012fafcb4972fc552b28968f\nReviewed-on: https://chromium-review.googlesource.com/1069450\nReviewed-by: Yuri Wiitala <miu@chromium.org>\nCommit-Queue: Xiangjun Zhang <xjz@chromium.org>\nCr-Commit-Position: refs/heads/master@{#561251}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/55701fa64cadc9e252bb813472bcb65b90d36953"}]}
{"actor":{"display_login":"aarongable"}}