{"push_id":2363058453,"size":5,"distinct_size":5,"ref":"refs/heads/master","head":"7ed96438b9889036d89fbd2bdb81e0d4306eeb44","before":"51465ffac6b1f9cd1b728ebd20eec0566c91fafe","commits":[{"sha":"efaea08574338007491bc243eba4ccb266941987","author":{"name":"Dominic Mazzoni","email":"7653c1987ba969fdae060080c081c47b9c70a7e1@chromium.org"},"message":"Introduce Ignored state in ui/accessibility\n\nThere are times when a node needs to be ignored, meaning it\nshouldn't be exposed as part of the accessibility tree, but\nits subtree should. A common example is role=none/presentation,\nbut we also sometimes do this for extra wrapper divs and spans.\n\nPreviously these ignored nodes were just never exposed by Blink,\nbut that has led to problems when a node's ignored status changes.\n\nInstead, let's introduce the concept of ignored to AXNodeData\nand add support for walking the \"unignored\" tree to AXNode.\nWhen a node's ignored state changes, we should fire a Children\nChanged event on the first unignored parent.\n\nThis change introduces a new state and some helper functions\nbut doesn't actually use them. A follow-up change will start\nmaking use of this code.\n\nBug: 651614\n\nCq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation\nChange-Id: Ide93a65d5721c00e8c9d1780c8b524341b837f05\nTbr: tsepez@chromium.org\nReviewed-on: https://chromium-review.googlesource.com/936430\nCommit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>\nReviewed-by: Michael Giuffrida <michaelpg@chromium.org>\nReviewed-by: Nektarios Paisios <nektar@chromium.org>\nCr-Commit-Position: refs/heads/master@{#539508}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/efaea08574338007491bc243eba4ccb266941987"},{"sha":"2a4de97d98d814d010155ca0a9ee5fad035eaec4","author":{"name":"Stephen McGruer","email":"289d5796e44446e007d0bca53156071f82b097de@chromium.org"},"message":"Allow null elements in EffectInput::Convert\n\nBy the spec, a KeyframeEffect may be created with a null target,\nalthough unfortunately the spec doesn't then say what Document to use to\nresolve style/etc. This CL adds null-target handling, falling back to\nthe Document extracted from the ScriptState if there is no target.\n\nIn terms of changes to tests:\n  - Three WPT tests no longer timeout (woo), but unfortunately they\n    currently crash on a DCHECK for setting transform-box to 'border-box'\n    and so are still disabled. See http://crbug.com/816534\n  - A number of tests for processing a keyframes argument start failing\n    due to a bug in that code; previously this went unseen because the\n    tests pass null for the element.\n\nBug: 772014, 816534\nChange-Id: I01502c82bf758a796fe83c87ed395c6b9f6c4e11\nReviewed-on: https://chromium-review.googlesource.com/934625\nCommit-Queue: Stephen McGruer <smcgruer@chromium.org>\nReviewed-by: Robert Flack <flackr@chromium.org>\nCr-Commit-Position: refs/heads/master@{#539509}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/2a4de97d98d814d010155ca0a9ee5fad035eaec4"},{"sha":"57c37da2b9117f8c1c3d94a39f70029d284de335","author":{"name":"Ken Rockot","email":"55a8b4123e533ae3d93e9af5362dbcf23cbea7cc@chromium.org"},"message":"Fix safe browsing interface exposure to renderers\n\nThe safe_browsing::mojom::PhishingDetectorClient interface appears to\nhave been added to the wrong interface_provider_spec. This moves it to\nthe correct one.\n\nBug: 816892\nChange-Id: If6421f57920a36fa9c2be351c5b451cedfd024cc\nReviewed-on: https://chromium-review.googlesource.com/939538\nReviewed-by: Daniel Cheng <dcheng@chromium.org>\nCommit-Queue: Ken Rockot <rockot@chromium.org>\nCr-Commit-Position: refs/heads/master@{#539510}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/57c37da2b9117f8c1c3d94a39f70029d284de335"},{"sha":"7f35c78466bb21586a66893b090583f71463e597","author":{"name":"Emil A Eklund","email":"9441b25ef5f12187e693ab9f841441787ce3c897@chromium.org"},"message":"Check for font in LayoutTextCombine::TransformToInlineCoordinates\n\nCheck for PrimaryFont in LayoutTextCombine::TransformToInlineCoordinates\nas it may be NULL in which case no coordinate transform is required.\n\nBug: 814734\nChange-Id: I9d637bc9956a3129cd294398e407023cc45e32ea\nReviewed-on: https://chromium-review.googlesource.com/938661\nCommit-Queue: Emil A Eklund <eae@chromium.org>\nReviewed-by: Koji Ishii <kojii@chromium.org>\nCr-Commit-Position: refs/heads/master@{#539511}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/7f35c78466bb21586a66893b090583f71463e597"},{"sha":"7ed96438b9889036d89fbd2bdb81e0d4306eeb44","author":{"name":"Kevin Marshall","email":"37450fd2480317c56d08a75aa9a7415346f94a64@chromium.org"},"message":"Fuchsia: switch new runner scripts to use FVM disks instead of bootfs.\n\nSwitching to FVM will give us a functioning blobstore, a prerequisite\nfor installing packages, and allows the Fuchsia team to follow through\nwith bootfs deprecation.\n\nAlso added a missing package reference to local-sdk.py that caused it\nto skip building the Fuchsia userspace.\n\nBug: 802331\nChange-Id: I6cfc45c51dae97bef3c9f7a202f74fff483d5c50\nReviewed-on: https://chromium-review.googlesource.com/919266\nCommit-Queue: Kevin Marshall <kmarshall@chromium.org>\nReviewed-by: Scott Graham <scottmg@chromium.org>\nCr-Commit-Position: refs/heads/master@{#539512}","distinct":true,"url":"https://api.github.com/repos/chromium/chromium/commits/7ed96438b9889036d89fbd2bdb81e0d4306eeb44"}]}
{"actor":{"display_login":"aarongable"}}