{"push_id":2891169413,"size":3,"distinct_size":3,"ref":"refs/heads/master","head":"0fc143447e455d5258e43b1736a8dfcb2ebaa976","before":"16c10f2b1f5d4b703f192d8452837213b560fbf3","commits":[{"sha":"2c99b53dc248ac4c1f50d084ef9d8779bb491862","author":{"name":"Derek Cheng","email":"84c8702ffc13fff978d08b9103b84192e2b126b1@chromium.org"},"message":"[Presentation API] Close PresentationConnection on contextDestroyed().\n\nBy calling close() during contextDestroyed(), the PresentationConnection\nin Blink will notify the other end of the mojo message pipe via\nRequestClose(). Native MRPs holding the other end of the pipe\n(e.g. Cast MRP 2-UA mode) can then use this as a signal to clean up the\npresentation, instead of going through MediaRouteProvider::DetachRoute()\n\nThis also fixes a bug where in 1-UA mode, the receiver connection does\nnot receive state change to CLOSED when the controller frame navigates\naway.\n\nNote that PresentationService::CloseConnection() is still being called\nfrom Blink. This is still required for 1-UA mode where the other end of\nthe pipe is not in the browser.\n\nAlso, PresentationServiceImpl still performs a Reset() on frame\nnavigation to clean up other internal states.\n\nBug: 882690\nChange-Id: I9b4ecb862c39c737b36b2d5435286fb59a3ff752\nReviewed-on: https://chromium-review.googlesource.com/1222679\nCommit-Queue: Derek Cheng <imcheng@chromium.org>\nReviewed-by: Daniel Cheng <dcheng@chromium.org>\nReviewed-by: mark a. foltz <mfoltz@chromium.org>\nCr-Commit-Position: refs/heads/master@{#592552}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/2c99b53dc248ac4c1f50d084ef9d8779bb491862"},{"sha":"8415014ca0243afd14ea58aa27ca5006f8d0bddf","author":{"name":"Wenzhao Zang","email":"3bfe43dc09754df5a1fe9feb46624ad5cbab1aea@chromium.org"},"message":"cros: Whitelist additional demo mode Screensaver apps\n\nThere're two other screensaver apps that need whitelisting.\n\nTBR=rdevlin.cronin@chromium.org\n\nBug: 870851\nChange-Id: I78de824892311edd28b3a0d71e05360304c5d562\nReviewed-on: https://chromium-review.googlesource.com/1220053\nCommit-Queue: Wenzhao (Colin) Zang <wzang@chromium.org>\nReviewed-by: Michael Giuffrida <michaelpg@chromium.org>\nCr-Commit-Position: refs/heads/master@{#592553}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/8415014ca0243afd14ea58aa27ca5006f8d0bddf"},{"sha":"0fc143447e455d5258e43b1736a8dfcb2ebaa976","author":{"name":"David Black","email":"9b6aa4010fa524c8170409d62db62be4fcff1d37@google.com"},"message":"Create Chrome Settings deep link.\n\nUsed to launch a specific Chrome Settings page, or top level Settings,\nin a new browser tab.\n\nBug: b:115675454\nChange-Id: I7bc6bf0e9b5e950e23c1ec2f4ad8b801d7b54958\nReviewed-on: https://chromium-review.googlesource.com/1226206\nCommit-Queue: David Black <dmblack@google.com>\nReviewed-by: Xiaohui Chen <xiaohuic@chromium.org>\nCr-Commit-Position: refs/heads/master@{#592554}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/0fc143447e455d5258e43b1736a8dfcb2ebaa976"}]}
{"actor":{"display_login":"aarongable"}}