{"push_id":2664192496,"size":18,"distinct_size":18,"ref":"refs/heads/master","head":"4d251274996d6d3289715236a8306cfd8e831ade","before":"5f173cb15baf27e7ba461814cdfdc7194e937ea2","commits":[{"sha":"68185c7d664ec847c608a13e4224e85b11c17859","author":{"name":"Lu Chen","email":"7154a6c5b2891653aea2aaf2b5a85a7053db0929@chromium.org"},"message":"Fix overflow issue in blink::AnchorElementMetrics::RecordMetrics.\n\nDetail: In order to get finer resolution when recording metrics,\nthe program multiples the area/distance metrics by 100 before sending\nto UMA, which can cause overflow.\n\nFix: For area metric, the result is limited to 1.0; for distance\nmetric, the result is limited to the maximum expected value (10000)\nby UMA before sending.\n\nBug: 853435\nChange-Id: Ie8d9751c86bd64f38bd79c3e12d502f71a551522\nReviewed-on: https://chromium-review.googlesource.com/1105506\nCommit-Queue: Lu Chen <chelu@chromium.org>\nReviewed-by: Kouhei Ueno <kouhei@chromium.org>\nReviewed-by: Ryan Sturm <ryansturm@chromium.org>\nReviewed-by: Tarun Bansal <tbansal@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569390}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/68185c7d664ec847c608a13e4224e85b11c17859"},{"sha":"3d0c4bcca8ab7b7bd0e642511e2f6d7f0166f2e7","author":{"name":"Scott Violet","email":"5737ef08a3ec16a337ac79a1d719fb91acba20a4@chromium.org"},"message":"chromeos: fix id sent to parent in OnFrameSinkIdAllocated()\n\nIt was passing in the FrameSinkId, when it should have passed in the window\nid.\n\nBUG=837686\nTEST=covered by test\n\nChange-Id: I2f6920528403452696d358110605aa85519a0b2d\nReviewed-on: https://chromium-review.googlesource.com/1110748\nReviewed-by: Michael Wasserman <msw@chromium.org>\nCommit-Queue: Scott Violet <sky@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569391}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/3d0c4bcca8ab7b7bd0e642511e2f6d7f0166f2e7"},{"sha":"7302d41b0c998bd48c2e000c0a5d9fbeaf1d9e3f","author":{"name":"Luke Sorenson","email":"b6b05af2f101144e13f8def34bafa2d22b5b7cb5@chromium.org"},"message":"Uprev min env version for rtanalytics-light to 4.1\n\nminijail0 security policy was updated to support new features.\nNew components that use these new syscalls must use this new\nhigher env version.\n\nBug: 850956\nChange-Id: I53252a4c25bc1b037b89e81960b9a0f164aab351\nReviewed-on: https://chromium-review.googlesource.com/1106007\nReviewed-by: Joshua Pawlicki <waffles@chromium.org>\nCommit-Queue: Luke Sorenson <lasoren@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569392}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/7302d41b0c998bd48c2e000c0a5d9fbeaf1d9e3f"},{"sha":"0a6b1c3e92f57af7fe80e42a366fda8a8efd7992","author":{"name":"Caleb Rouleau","email":"58bdb6276c01e3ed53e660bf706aa902204feb0d@chromium.org"},"message":"[ChromeDriver] Improve desktop launch errors and report them faster.\n\n1. Enforce our lack of support for a custom remote-debugging-port\nby throwing an error if someone uses it.\n\n2. If we can't parse a devtools port from the file, then check\nto see if Chrome has crashed and use that information to improve\nthe error message. Do this right away instead of waiting 60 seconds.\n\n3. Write tests for these errors!\n\nBug: chromedriver:2474\nChange-Id: Ie649f980745e5241ec462e7527a24d58ced9af02\nReviewed-on: https://chromium-review.googlesource.com/1106769\nCommit-Queue: Caleb Rouleau <crouleau@chromium.org>\nReviewed-by: John Chen <johnchen@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569393}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/0a6b1c3e92f57af7fe80e42a366fda8a8efd7992"},{"sha":"c2e22fc81bd398b836bc2c567107868c7422364f","author":{"name":"Theresa","email":"56132c141164a8d5c6784677328c2d837b727813@chromium.org"},"message":"Disable FeedNewTabPage tests on M 64-bit bots\n\nBUG=854866\n\nChange-Id: Iff9021221210078e2d013b4c54628b6667323eee\nReviewed-on: https://chromium-review.googlesource.com/1110676\nReviewed-by: Becky Zhou <huayinz@chromium.org>\nCommit-Queue: Theresa <twellington@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569394}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/c2e22fc81bd398b836bc2c567107868c7422364f"},{"sha":"68497996df4bde8b8ee71857c9f160657279b5a8","author":{"name":"Ted Choc","email":"fdab362c6eab457746484b519392cd01a4f71737@google.com"},"message":"Fix compositor sizing of OnePlus in fullscreen mode.\n\nBUG=854109\n\nChange-Id: Iac8a8b5c947d890a17b187f1cded75332b3dba27\nReviewed-on: https://chromium-review.googlesource.com/1110517\nReviewed-by: Matthew Jones <mdjones@chromium.org>\nCommit-Queue: Ted Choc <tedchoc@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569395}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/68497996df4bde8b8ee71857c9f160657279b5a8"},{"sha":"d02f85c18e9118863a5bc5aff0632bdbe037cd22","author":{"name":"Rayan Kanso","email":"8467f9d4d02d7700f9bd1ddda04a874d839c2a48@chromium.org"},"message":"[Background Fetch] Replace IconDefinition w/ManifestImageResource\n\nTBR=jbroman@chromium.org\n\nBug: 817445\nChange-Id: I2336b16d2e10a524c176997b75471b8bc8310460\nReviewed-on: https://chromium-review.googlesource.com/1097482\nCommit-Queue: Rayan Kanso <rayankans@chromium.org>\nReviewed-by: Jeremy Roman <jbroman@chromium.org>\nReviewed-by: Tom Sepez <tsepez@chromium.org>\nReviewed-by: Peter Beverloo <peter@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569396}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/d02f85c18e9118863a5bc5aff0632bdbe037cd22"},{"sha":"3fba85fe486b739275c78ff452d3a6a9454a2e20","author":{"name":"Yue Li","email":"577dfa8baebdde8a76bd4903b43825e48c0b8002@chromium.org"},"message":"Add confirm reject screen and ready screen for Assistant OptIn flow\n\n- Add two more screens for the flow.\n- Update proto definitions\n\nBug: b/78190629\nTest: Manual\nCq-Include-Trybots: luci.chromium.try:closure_compilation\nChange-Id: Ic9a849e3184ea5b09e0f0f338a018caf2193d829\nReviewed-on: https://chromium-review.googlesource.com/1105469\nCommit-Queue: Yue Li <updowndota@chromium.org>\nReviewed-by: Xiaohui Chen <xiaohuic@chromium.org>\nReviewed-by: Xiyuan Xia <xiyuan@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569397}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/3fba85fe486b739275c78ff452d3a6a9454a2e20"},{"sha":"d95a02e69592a62dc8ea668e90ed1897b7c24aa9","author":{"name":"David Tseng","email":"57d6711e15f60434c9e159ea3db9eda3f784ef2f@chromium.org"},"message":"Adds jump_to_details message string\n\nTest: ChromeVox menus (Search+.) works again. It was breaking the page due to this missing message id.\nCq-Include-Trybots: luci.chromium.try:closure_compilation\nChange-Id: Id2a5539d29d46343472829b5e4b5d8deefe06e94\nReviewed-on: https://chromium-review.googlesource.com/1107165\nReviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>\nCommit-Queue: David Tseng <dtseng@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569398}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/d95a02e69592a62dc8ea668e90ed1897b7c24aa9"},{"sha":"70148c9c56c02f2da7c59fe22131e5f307ec86dd","author":{"name":"pdfium-chromium-autoroll","email":"4f5c2aa046d0ac4bebd4d4a0b24dc16beac157a1@skia-buildbots.google.com.iam.gserviceaccount.com"},"message":"Roll src/third_party/pdfium bcd66f54de12..3d8131535e6b (3 commits)\n\nhttps://pdfium.googlesource.com/pdfium.git/+log/bcd66f54de12..3d8131535e6b\n\n\ngit log bcd66f54de12..3d8131535e6b --date=short --no-merges --format='%ad %ae %s'\n2018-06-21 tsepez@chromium.org Pass output parameters as pointers in ijs_event_context\n2018-06-21 tsepez@chromium.org Remove WideString::c_str() calls where possible.\n2018-06-21 tsepez@chromium.org Avoid const-refs to implicitly constructed strings.\n\n\nCreated with:\n  gclient setdep -r src/third_party/pdfium@3d8131535e6b\n\nThe AutoRoll server is located here: https://pdfium-roll.skia.org\n\nDocumentation for the AutoRoller is here:\nhttps://skia.googlesource.com/buildbot/+/master/autoroll/README.md\n\nIf the roll is causing failures, please contact the current sheriff, who should\nbe CC'd on the roll, and stop the roller if necessary.\n\n\nTBR=dsinclair@chromium.org\n\nChange-Id: Ic8232f5102b620e309927cb2b89866ff19f7251b\nReviewed-on: https://chromium-review.googlesource.com/1110457\nReviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>\nCommit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>\nCr-Commit-Position: refs/heads/master@{#569399}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/70148c9c56c02f2da7c59fe22131e5f307ec86dd"},{"sha":"6ef585dfb4f360df022f391af121bdfb2929cb83","author":{"name":"Nico Weber","email":"c0f02d98506190746d1cdcb01be0f37b51e27077@chromium.org"},"message":"Remove chromium_mac_fyi_gtests suite, run regular chromium_mac_gtests suite on the mac FYI bots.\n\nThis makes the fyi bots run viz_gtests instead of viz_fyi_gtests which removes\na few test binaries, but since those weren't run on any other desktop bot,\nit probably wasn't intentional that they run on mac anyways.\n\nBug: 843511\nChange-Id: Ieb132304f3e6b3869c1c2238ec509ed5a95174ab\nReviewed-on: https://chromium-review.googlesource.com/1108360\nReviewed-by: Jonathan Ross <jonross@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569400}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/6ef585dfb4f360df022f391af121bdfb2929cb83"},{"sha":"f68571e38febb3a1222da1439050fd990929b274","author":{"name":"Robert Ma","email":"637a4f31a3f061b08257f7846726e5a1e9d8683d@chromium.org"},"message":"Fix two testharness.js tests with harness errors due to stray asserts\n\nAll asserts need to be in a step func. Unfortunately this makes the\ntests more verbose.\n\nFound in https://github.com/web-platform-tests/wpt/pull/8748#issuecomment-398954260.\n\nChange-Id: Ib15270d6093ea094b0f8c387cc4370f5154cf5dd\nReviewed-on: https://chromium-review.googlesource.com/1109689\nCommit-Queue: Robert Ma <robertma@chromium.org>\nReviewed-by: Robert Ma <robertma@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569401}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/f68571e38febb3a1222da1439050fd990929b274"},{"sha":"3045a6e26dbfb23a21c79cf122b2b1cb786332fd","author":{"name":"Robert Liao","email":"527ca1f25a3d0c4f4c1a492442dbd447e5cafc8e@chromium.org"},"message":"Set a ConstrainedDialogWebView Minimum Size of 1x1 on Mac\n\nMacOS does not handle zero-sized windows well.\n\nBUG=854267\n\nChange-Id: I3da67cef7abdc035ca6378d5cb31dc68a097ee67\nReviewed-on: https://chromium-review.googlesource.com/1109200\nReviewed-by: Scott Violet <sky@chromium.org>\nCommit-Queue: Robert Liao <robliao@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569402}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/3045a6e26dbfb23a21c79cf122b2b1cb786332fd"},{"sha":"74ddb9c6f8d69d7ef4fffd4f07325b1a98532872","author":{"name":"Daniele Castagna","email":"bf401ac9f4607f1574de95f28428a77f41d45b82@chromium.org"},"message":"Add chrome://flags entry to select overlay strategies\n\nThis patch adds a multi-value entry in about://flags that will\nhelp enabling/disabling HW overlay strategies used to promote quads\nto HW overlays.\n\nBug: 854677\nChange-Id: Ibaedb02084a0b6aea490637bd6c4456caa63e29d\nReviewed-on: https://chromium-review.googlesource.com/1108520\nCommit-Queue: Daniele Castagna <dcastagna@chromium.org>\nReviewed-by: David Reveman <reveman@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569403}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/74ddb9c6f8d69d7ef4fffd4f07325b1a98532872"},{"sha":"3c83d2f6a054af909d340cc9036e2b7d98adc4c6","author":{"name":"Sarah Hu","email":"2e33c596da65d385f0bcceeefe87ceb1f2ce8a5c@chromium.org"},"message":"Always position the gaia dialog in the center of the screen.\n\nWe should re-position the dialog when tablet mode is changed, and when\ndisplay metrics is changed.\n\nBug: 841074, 837005\nChange-Id: I11299070a947e2fc4f4f494d2bbac4c94c27746e\nReviewed-on: https://chromium-review.googlesource.com/1110487\nReviewed-by: Jacob Dufault <jdufault@chromium.org>\nCommit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569404}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/3c83d2f6a054af909d340cc9036e2b7d98adc4c6"},{"sha":"6e4c5386ae51b24383d6f42f8c1be08356c40ed6","author":{"name":"Frank Kastenholz","email":"63868783f757b92e7dcea14895f91a5c9844ec30@chromium.org"},"message":"Landing Recent QUIC changes\n\nUpdate flags\nhttps://chromium-review.googlesource.com/c/1108077\n\n--------------------------------------------------\n\nFix include guard name\nhttps://chromium-review.googlesource.com/c/1106672/\n\n--------------------------------------------------\n\nAdd net/third_party/quic/core/quic-debug_info_provider_interface.h\n\nAdds this file into the git commit history\nhttps://chromium-review.googlesource.com/c/1106672/\n\n--------------------------------------------------\n\nDo not set send alarm if write blocked.\n\nDo not set send alarm in the end of quic packet processing if it's\nwrite blocked. Protected by\nFLAGS_quic_reloadable_flag_quic_no_send_alarm_in_process_packet_if_write_blocked.\n\nMerge internal change: 200989386\nhttps://chromium-review.googlesource.com/c/1106902/\n\n--------------------------------------------------\n\nValidate largest observed field in ACK frames.\n\nIn QUIC, validate largest observed in received ack frame by comparing\nwith largest sent packet instead of largest created packet. Protected\nby FLAGS_quic_reloadable_flag_quic_validate_ack_largest_observed.\n\nMerge internal change: 200744974\nhttps://chromium-review.googlesource.com/c/1106899/\n\n--------------------------------------------------\n\nAdd two flag protected CHECKs for oob access into sent packet manager's unacked_packets.\n\nProtected by FLAGS_quic_reloadable_flag_quic_extra_checks_in_ack_processing.\n\nSince bandaid does not have logs or core dumps, CHECK failure is the\nonly way we can think of to get more information(from deathrattle\nreport) to debug b/110029150.\n\nDo not intend to verify this flag, we only need to turn it on on\nbandaid from liveflag.\n\nMerge internal change: 200621124\nhttps://chromium-review.googlesource.com/c/1106672/\n\n--------------------------------------------------\n\nClean up QUIC Frame Generation and Parsing\n\nClean up left-over stuff in the generation and parsing of QUIC Frames\n- Remove the IETF_TEMP_ frame types that were allocated to the IETF\nframer for testing purposes,\n- Remove code that uses/references/expects the IETF_TEMP types\n- Close connection (and/or do a QUIC_BUG) if we attempt to\ngenerate a GO AWAY or STOP WAITING in version 99. These\nframes are not valid in Version 99\n- Close connection (and/or do a QUIC_BUG) if we attempt to\ngenerate an APPLICATION_CLOSE, NEW_CONNECTION_ID, MAX_STREAM_ID,\nSTREAM_ID_BLOCKED, PATH_RESPONSE, PATH_CHALLENGE, or STOP_SENDING\nframe while NOT in version 99. These frames are IETF only.\n\nN/A - affected code all protected by the version 99 flag.\n\nMerge internal change: 200443164\nhttps://chromium-review.googlesource.com/c/1106669/\n\n--------------------------------------------------\n\nFix the flaky QuartcConnectionTest.DisconnectClosesRemote\n\n1) We were removing packet writer before the connection was destructed\n(ownership was probably moved at some point in time).\n\n2) QuartcThreadSafeSession::CloseConnectionSync was not closing a session,\nwhich led to the destruction of QuicConnection on the main thread, which led to\na race condition. Reinstituted session_ = nullptr and added checks in other\ndelegates.\nAlso added a check in a destructor.\n\n3) Changed \"CancelImpl\" to \"Cancel\" in the QuartcAlarm so that the deadline is resetted.\n\nFix ordering of destructors in QuartcSession.\n\nMerge internal change: 200439184\nhttps://chromium-review.googlesource.com/c/1106666/\n\n--------------------------------------------------\n\nAdd underflow checks in IETF ACK frame processing.\n\nAdd checks for underflows when stepping through the additional ack\nblocks for IETF ACK frames\n\nN/A Code is executed only if version-99 negotiated, which does not happen.\n\nThis addresses issues found in Chromium Fuzz Testing - numbers 850476, 850554\n\nMerge internal change: 200395378\nhttps://chromium-review.googlesource.com/c/1106662/\n\n--------------------------------------------------\n\nIncrease the default stream/session flow control limits.\n\nThe default limit of 16KB causes blocking on sender and prevents us\nfrom even reaching BWE! This change fixes the delay and dropped frames\nin Quartc, and should have a significant impact on video quality.\n\nIncrease the default stream & session flow control limits for quartc\nquic connection.\n\nMerge internal change: 200269081\nhttps://chromium-review.googlesource.com/c/1106661/\n\n--------------------------------------------------\n\nAllow parsing/serializing a CryptoHandshakeMessage in QUIC's TransportParameters\n\nThe CryptoHandshakeMessage in TransportParameters carries 3 values from\nQuicConfig: silent close, initial roundtrip time, and connection options.\nIt also contains UAID. This provides support for gQUIC specific parameters\nin the TLS handshake as a stopgap until they either get added to IETF QUIC\nor removed from gQUIC.\n\nMore Transport Parameters in QUIC; proected by --quic_supports_tls_handshake\n\nMerge internal change: 200258885\nhttps://chromium-review.googlesource.com/c/1106660/\n\n--------------------------------------------------\n\nDeprecate FLAGS_quic_reloadable_flag_quic_no_mspc.\n\nMerge internal change: 200126339\nhttps://chromium-review.googlesource.com/c/1106658/\n\n--------------------------------------------------\n\nSet bytes for QUIC connection id to 0.\n\nThe ICE transport provides unique 5-tuples for each connection, so it's possible\nto omit the QUIC connection id from packet headers.  This should save overhead.\n\nn/a (quartc only)\n\nMerge internal change: 200118909\nhttps://chromium-review.googlesource.com/c/1106520/\n\n--------------------------------------------------\n\nRemove third_party/quic/http/ as this code is now unused.\n\nn/a - unused code\n\nMerge internal change: 200099282\nhttps://chromium-review.googlesource.com/c/1106305/\n\n--------------------------------------------------\nDeprecate FLAGS_quic_reloadable_flag_quic_enable_server_proxy2.\n\nMerge internal change: 200091805\nhttps://chromium-review.googlesource.com/c/1106297/\n\n--------------------------------------------------\n\nDeprecate FLAGS_quic_reloadable_flag_quic_enable_h2_deframer.\n\nMerge internal change: 200089857\nhttps://chromium-review.googlesource.com/c/1106219/\n\n--------------------------------------------------\n\nDeprecate FLAGS_quic_reloadable_flag_quic_version_manager_dedupe_transport_versions\n\nMerge internal change: 200085914\nhttps://chromium-review.googlesource.com/c/1106004/\n\n--------------------------------------------------\n\nDeprecate FLAGS_quic_reloadable_flag_quic_incremental_loss_detection.\n\nMerge internal change: 200057466\nhttps://chromium-review.googlesource.com/c/1106002/\n\n--------------------------------------------------\n\nDeprecate FLAGS_quic_reloadable_flag_quic_respect_ietf_header.\n\nMerge internal change: 200055473\nhttps://chromium-review.googlesource.com/c/1106000/\n\n--------------------------------------------------\n\nReplace flag quic_reloadable_flag_quic_enable_pcc with ..._pcc2\n\nReplace the old pcc enable flag with a new one, so that we can\ncheck that the new codes are indeed running in experiment.\n\nChanges flag quic_reloadable_flag_quic_enable_pcc to\nquic_reloadable_flag_quic_enable_pcc2\n\nMerge internal change: 200046441\nhttps://chromium-review.googlesource.com/c/1105999/\n\n--------------------------------------------------\n\nIn QUIC, move time wait list manager ProcessPacket from\nOnUnauthenticatedHeader to OnUnauthenticatedPublicHeader because\npacket number is not needed for public reset anymore. No function\nchange expected. Not protected.\n\nAlso remove unused version fields.\n\nMerge internal change: 200037571\n\nR=rch@chromium.org\n\nChange-Id: I5e76d4b8627c0c2810c7468dad38086f16f82211\nhttps: //chromium-review.googlesource.com/c/1105997/\nReviewed-on: https://chromium-review.googlesource.com/1108373\nCommit-Queue: Frank Kastenholz <fkastenholz@chromium.org>\nReviewed-by: Ryan Hamilton <rch@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569405}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/6e4c5386ae51b24383d6f42f8c1be08356c40ed6"},{"sha":"6ca67282f7e406dd83b31a526d601423f4f026e7","author":{"name":"Fady Samuel","email":"8326626c65f191a538cf9e58be478cfe3430c95b@chromium.org"},"message":"Surface Synchronization: Reset deadline on invalid SurfaceId\n\nIf a SurfaceLayer is given an invalid SurfaceId then it should not block\non that invalid SurfaceId and the specified deadline should play no part\nin the deadline of the CompositorFrame.\n\nBug: 672962\nCq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel\nChange-Id: I71c31315ff0dbcd0ab9697e98c5f58aa3691bf67\nReviewed-on: https://chromium-review.googlesource.com/1110028\nReviewed-by: danakj <danakj@chromium.org>\nReviewed-by: kylechar <kylechar@chromium.org>\nCommit-Queue: Fady Samuel <fsamuel@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569406}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/6ca67282f7e406dd83b31a526d601423f4f026e7"},{"sha":"4d251274996d6d3289715236a8306cfd8e831ade","author":{"name":"Pedro Amaral","email":"d91b0943b8212e51d0af4dfeff78f33979be3c22@chromium.org"},"message":"Add null check to BottomToolbarMediator#destroy()\n\nIf the BottomToolbar is destroyed before native is initialized there\nwill be a null pointer exception. This CL adds a null check to\nprevent this.\n\nBug: 854342\nChange-Id: Ic0a82574e87bce7f46b20d3621796fad09c119a5\nReviewed-on: https://chromium-review.googlesource.com/1107053\nCommit-Queue: Pedro Amaral <amaralp@chromium.org>\nReviewed-by: Matthew Jones <mdjones@chromium.org>\nCr-Commit-Position: refs/heads/master@{#569407}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/4d251274996d6d3289715236a8306cfd8e831ade"}]}
{"actor":{"display_login":"aarongable"}}