{"push_id":2529115057,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"35f7133120bbe69c2f68512d3d3d82c6a39ba35e","before":"7803ea9309d2270c583e0a3dc7027aed2253da5a","commits":[{"sha":"11ab394e56836cf983a58de8626c8ec63e2198ec","author":{"name":"Matt Giuca","email":"d1760e71d9d2f2003c419afef791fb2187292ae1@chromium.org"},"message":"Revert \"Handle error return from zygote recvmsg call.\"\n\nThis reverts commit 86ed44e053ece378ccbb0c2425e4163c65164f9f.\n\nReason for revert: Blocking revert of r554902.\n\nOriginal change's description:\n> Handle error return from zygote recvmsg call.\n> \n> This trips a check far away from the actual issue. The CHECK is too\n> strong and prevents executing the error handling path beneath it.\n> \n> Shuffle a few blocks and comments around so that the flow makes\n> more sense and is easier to read, and initial fork failures are\n> caught earlier.\n> \n> Bug: 835778\n> Change-Id: I46b960f9fd1095f5fc5224bf9cc80703e8fc325f\n> Reviewed-on: https://chromium-review.googlesource.com/1030876\n> Commit-Queue: Tom Sepez <tsepez@chromium.org>\n> Reviewed-by: Daniel Cheng <dcheng@chromium.org>\n> Cr-Commit-Position: refs/heads/master@{#554949}\n\nTBR=dcheng@chromium.org,jln@chromium.org,jam@chromium.org,tsepez@chromium.org\n\nChange-Id: I877d3610bd75eaa1176189319923bcc75b794781\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nBug: 835778, 838498\nReviewed-on: https://chromium-review.googlesource.com/1036983\nReviewed-by: Matt Giuca <mgiuca@chromium.org>\nCommit-Queue: Matt Giuca <mgiuca@chromium.org>\nCr-Commit-Position: refs/heads/master@{#554999}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/11ab394e56836cf983a58de8626c8ec63e2198ec"},{"sha":"82fd02342d240c3f6d728b0cbd35d7841338d0d0","author":{"name":"Peter Collingbourne","email":"0a9c5350547d97e930a976072b24aa4bbddb3981@chromium.org"},"message":"clang: Install blacklist files to the correct location.\n\nLLVM r322451 moved these files to a share/ directory in the\ninstallation directory, so we must do the same in our package.\n\nAmazingly, the move didn't cause breakage in our packaging script\nbecause of a combination of two bugs: 1) the filter regular expression\nfor blacklist files also matched paths beginning with share/ and 2)\na bug in the CopyDirectoryContents function meant that we ended\nup copying the blacklist files to the directory where the files\npreviously lived.\n\nThe bug revealed that we only ever use this function to copy\ndirectories that only contain files, so I removed the tree walking\ncode from this function. I also removed the filtering feature because\nit was only being used to copy blacklists and we can now simply copy\nthe entire share/ directory (which just contains blacklist files).\n\nChange-Id: I78dca1a6d4cb94d9a8a929f26c0e159d2794efbb\nReviewed-on: https://chromium-review.googlesource.com/1036450\nReviewed-by: Nico Weber <thakis@chromium.org>\nCommit-Queue: Peter Collingbourne <pcc@chromium.org>\nCr-Commit-Position: refs/heads/master@{#555000}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/82fd02342d240c3f6d728b0cbd35d7841338d0d0"},{"sha":"35f7133120bbe69c2f68512d3d3d82c6a39ba35e","author":{"name":"Matt Giuca","email":"d1760e71d9d2f2003c419afef791fb2187292ae1@chromium.org"},"message":"Revert \"Move zygote code to service_manager\"\n\nThis reverts commit dbe3973e8ff5eb249dd35474b69f9316c15ca1f0.\n\nReason for revert: Broke compile on LKGR and local builds.\nSee https://crbug.com/838498.\n\nOriginal change's description:\n> Move zygote code to service_manager\n> \n> This CL moves the zygote code from content/ to\n> services/service_manager/zygote. This involves moving some switches from\n> content_switches.h to services/service_manager/embedder/switches.h and\n> moving some codes from content/public/common/result_codes.h to the new\n> services/service_manager/embedder/result_codes.h.\n> \n> Bug: 831846\n> Change-Id: I9776a30412ea43c2c7f511967d2b7d05f18aa841\n> Reviewed-on: https://chromium-review.googlesource.com/1012194\n> Reviewed-by: Avi Drissman <avi@chromium.org>\n> Reviewed-by: Scott Violet <sky@chromium.org>\n> Reviewed-by: Scott Graham <scottmg@chromium.org>\n> Reviewed-by: Ken Rockot <rockot@chromium.org>\n> Reviewed-by: Stephen Lanham <slan@chromium.org>\n> Reviewed-by: Eric Seckler <eseckler@chromium.org>\n> Reviewed-by: Robert Sesek <rsesek@chromium.org>\n> Commit-Queue: Jay Civelli <jcivelli@chromium.org>\n> Cr-Commit-Position: refs/heads/master@{#554902}\n\nTBR=avi@chromium.org,sky@chromium.org,dschuff@chromium.org,jcivelli@chromium.org,rockot@chromium.org,slan@chromium.org,scottmg@chromium.org,rsesek@chromium.org,eseckler@chromium.org\n\nChange-Id: I318b3d8d74520b1fa7eda3d6d612d54e9dc2f040\nNo-Presubmit: true\nNo-Tree-Checks: true\nNo-Try: true\nBug: 831846, 838498\nReviewed-on: https://chromium-review.googlesource.com/1036706\nCommit-Queue: Matt Giuca <mgiuca@chromium.org>\nReviewed-by: Matt Giuca <mgiuca@chromium.org>\nCr-Commit-Position: refs/heads/master@{#555001}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/35f7133120bbe69c2f68512d3d3d82c6a39ba35e"}]}
{"actor":{"display_login":"aarongable"}}