- 11 Oct, 2019 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/88c57b498266..af9c6ce85491 git log 88c57b498266..af9c6ce85491 --date=short --no-merges --format='%ad %ae %s' 2019-10-11 vapier@chromium.org run_tests: clear python compiled caches Created with: gclient setdep -r src/third_party/chromite@af9c6ce85491 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:chromeos-kevin-rel TBR=chrome-os-gardeners@google.com Bug: None Change-Id: Iaa5e11086227696e3a7379bbe7373a7af74579f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855473Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#705127}
-
Thomas Tangl authored
... and wait for SyncTransportActive during the setup. Bug: 995720, 1012167 Change-Id: Ib0523c68738f226a29171fabc64264cf36c03f37 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1856061 Commit-Queue: Thomas Tangl <tangltom@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#705126}
-
Yi Gu authored
When a remote frame has an local ancestor with pointer-events: none, the frame and all its descendants should be excluded in hit testing. See https://docs.google.com/document/d/1xN1bRigzahOgUM4cEuyHGhe069TRc5-0rXz1JZsgCP8/edit?usp=sharing Bug: 1002245 Change-Id: I9e2f86b1d1e6727a66499639994d269d301f1a34 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1836954Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Reviewed-by:
Robert Flack <flackr@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#705125}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/8e57b4bc55c0..e3703bb8b8af git log 8e57b4bc55c0..e3703bb8b8af --date=short --no-merges --format='%ad %ae %s' 2019-10-11 tandrii@chromium.org Release newest bb tool. Created with: gclient setdep -r src/third_party/depot_tools@e3703bb8b8af If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll Please CC agable@chromium.org on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=agable@chromium.org Bug: chromium:1004140 Change-Id: Iacfa4e7d4df76e52271c441442643bf0680d1db6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855472Reviewed-by:
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/51bf20029462..5740f3e2b8fd git log 51bf20029462..5740f3e2b8fd --date=short --no-merges --format='%ad %ae %s' 2019-10-11 danilchap@webrtc.org Clarify expectation on GlobalLock 2019-10-11 gustaf@webrtc.org Fix bypass of unnecessary resampling Created with: gclient setdep -r src/third_party/webrtc@5740f3e2b8fd If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/webrtc-chromium-autoroll Please CC webrtc-chromium-sheriffs-robots@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=webrtc-chromium-sheriffs-robots@google.com Bug: chromium:None,chromium:b/130016532 Change-Id: Ide7481d7dc71f8be9efe552a36d4d825b052f8e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855471Reviewed-by:
-
Dave Tapuska authored
When this code was changed to not use the RevocableInterfacePtr it was assumed that the calls into the mojo bindings only occurred while the object's associated context was still alive. It is possible to have a reference to a cache storage object from a frame that has been destroyed and make calls to it. In this case the RevocableInterfacePtr previously dropped them because the calls would fail. Now we will return a promise that is rejected. BUG=1012781,1012358 Change-Id: I9504af8245be13137db82e39e3dc5a702c89e7d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854184 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
Ben Kelly <wanderview@chromium.org> Cr-Commit-Position: refs/heads/master@{#705122}
-
Sylvain Defresne authored
base::Bind/base::Callback are deprecated in favor of either base::BindOnce/base::OnceCallback or base::BindRepeating/ base::RepeatingCallback (depending on whether the callback is invoked once or multiple time). Convert some uses of base::Bind/base::Callback (this is a multipart change) in components/prefs and services/prefs to the recommended methods/types. Bug: 1007719, 1007831 Change-Id: Ib83da5603c9888c2704de80b4d40ea27fdcc3654 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1844998 Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Cr-Commit-Position: refs/heads/master@{#705121}
-
Aaron Colwell authored
This fixes WebView crashes introduced by http://crrev.com/c/1837093 . This change introduces an exception for the origin consistency check because WebView doesn't generate opaque origins for data: URLs. This is a temporary workaround to avoid crashing while we figure out how to make data: origin behavior more consistent with other platforms. Bug: 1013171, 991607 Change-Id: I304fef3d5eda5bd0d31495e62e4292641392575f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852421 Commit-Queue: Aaron Colwell <acolwell@chromium.org> Commit-Queue: Bo <boliu@chromium.org> Auto-Submit: Aaron Colwell <acolwell@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#705120}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/04988676..ddd5171b Please follow these instructions for assigning/CC'ing issues: https://v8.dev/docs/triage-issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux-blink-rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:linux_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:mac_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:win_optional_gpu_tests_rel CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I93cfb1eb9c15337945d887d07bee02eefdea2959 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855663Reviewed-by:
v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: v8-ci-autoroll-builder <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#705119}
-
Andrew Grieve authored
And add step for ErrorProne. Change-Id: Ic6054026e5c8d7b47a0f212499076b8f8d5ff8dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855470Reviewed-by:
Tibor Goldschwendt <tiborg@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#705118}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/ae022faf53b9..1c462ebc904c git log ae022faf53b9..1c462ebc904c --date=short --no-merges --format='%ad %ae %s' 2019-10-11 sugoi@google.com Stencil buffer texture sampling Created with: gclient setdep -r src/third_party/swiftshader@1c462ebc904c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_chromium_msan_rel_ng;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=swiftshader-team+autoroll@google.com Bug: chromium:b/142385547 Change-Id: I57e70d89ac9abdfad47046120bde8b4c98ee3e91 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855469Reviewed-by:
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/df640e6d14a5..106e86516480 git log df640e6d14a5..106e86516480 --date=short --no-merges --format='%ad %ae %s' 2019-10-11 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 5d9c4ee328bc..67486ec24551 (18 commits) Created with: gclient setdep -r src/third_party/skia@106e86516480 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC kjlubick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=kjlubick@google.com Bug: chromium:None Change-Id: If6dbe676f8a573b7a4c32552e966980460dde145 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855468Reviewed-by:
-
Sylvain Defresne authored
base::Bind/base::Callback are deprecated in favor of either base::BindOnce/base::OnceCallback or base::BindRepeating/ base::RepeatingCallback (depending on whether the callback is invoked once or multiple time). Convert all uses of base::Bind/base::Callback in components/zoom to the recommended methods/types. Bug: 1007758 Change-Id: I00b967cd051ff169824c9438e4d4a218482230d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1832271 Auto-Submit: Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
James MacLean <wjmaclean@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#705115}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 7dbbaa34. With Chromium commits locally applied on WPT: 23748e60 "Change text directive to text=" 2f7c5562 "Fetch Metadata: Split `sec-fetch-dest` out from other headers' tests." d52a9d07 "Port IndexedDB Blob Web Tests to web-platform-tests." 6386d9d8 "Update service_worker_unregister_and_register()" 1644eff7 "Update scroll to text feature detectability API" Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: I1cca5fe80c476e62cd6fca3d425a3fd3df0ac605 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855662Reviewed-by:
WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Commit-Queue: WPT Autoroller <wpt-autoroller@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#705114}
-
Adam Rice authored
Add a comment to note that creating a WritableStream with a high_water_mark of 0 will result in piping not working. Change-Id: Iacf6769fa017b56f192789b02fb1ce13df4e4986 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852347Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Commit-Queue: Adam Rice <ricea@chromium.org> Cr-Commit-Position: refs/heads/master@{#705113}
-
Albert Chaulk authored
An exo surface is collapsed into a single CompositorFrame during presentation; if there is an aura window displaying webcontents it will be rendered on top of all exo content due to the actual viz layer structure This replaces the contents of an exo layer with the web contents by emitting a SurfaceDrawQuad instead, drawing the web contents in the correct position As a consequence the aura window corresponding to the web contents is hidden as we do not want it drawn in the usual aura manner Bug: b/141980352 Test: desktop build w/ webview Change-Id: If45aa68e80f115f36ef0edee34f03c09f06dfcf1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1850366Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Reviewed-by:
Daniele Castagna <dcastagna@chromium.org> Reviewed-by:
Alex Sakhartchouk <alexst@chromium.org> Commit-Queue: Albert Chaulk <achaulk@chromium.org> Cr-Commit-Position: refs/heads/master@{#705112}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/67486ec24551..a21362f5a6ae git log 67486ec24551..a21362f5a6ae --date=short --no-merges --format='%ad %ae %s' 2019-10-11 jonahr@google.com GL: Missing check for texture format sample counts Created with: gclient setdep -r src/third_party/angle@a21362f5a6ae If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/angle-chromium-autoroll Please CC geofflang@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_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 TBR=geofflang@google.com Bug: chromium:976382 Change-Id: I346854d57499fb898649506005630542af3d5d78 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855467Reviewed-by:
-
Andy Paicu authored
These are supporting metrics for the experiment to be run around quieter notification prompts. Bug: 986737 Change-Id: Ia1220ef5d8e4aa2980c18d7a1ebc91d19bdaaf43 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1833826 Commit-Queue: Andy Paicu <andypaicu@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Reviewed-by:
Peter Beverloo <peter@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Cr-Commit-Position: refs/heads/master@{#705110}
-
Brian White authored
Adds expires_after attribute for 148 histograms that do not already have one. The date will get pushed out automatically if significant use is detected. Change-Id: I576b8cd4d503eb34586dfcd89a54542fd23008a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854599Reviewed-by:
-
chrome://sync-internalsPeter Wen authored
This reverts commit 711a5e45. Reason for revert: https://crbug.com/1013522 Original change's description: > [DevUI DFM] Add chrome://sync-internals resources > > Moves resources used in chrome://sync-internals out of > components_resources.grd into its own GRD file. Also move these > resources into the Dev UI Dynamic Feature Module (DFM) to save binary > size on Chrome for Android. > > Update iOS dependency to include the new resource target. > > Bug: 927131 > Change-Id: Ibad6e27d004cc22138b68b615b517a171969c5ae > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849418 > Commit-Queue: Peter Wen <wnwen@chromium.org> > Commit-Queue: Colin Blundell <blundell@chromium.org> > Reviewed-by: Samuel Huang <huangs@chromium.org> > Reviewed-by: Colin Blundell <blundell@chromium.org> > Reviewed-by: Marc Treib <treib@chromium.org> > Reviewed-by: Dan Beam <dbeam@chromium.org> > Auto-Submit: Peter Wen <wnwen@chromium.org> > Cr-Commit-Position: refs/heads/master@{#704639} TBR=huangs@chromium.org,dbeam@chromium.org,wnwen@chromium.org,blundell@chromium.org,treib@chromium.org,vitaliii@chromium.org Change-Id: I87c92143986fef0bbf8853d50d7cefcc1db92af7 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 927131, 1013522 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854764Reviewed-by:
Peter Wen <wnwen@chromium.org> Commit-Queue: Peter Wen <wnwen@chromium.org> Cr-Commit-Position: refs/heads/master@{#705108}
-
Lei Zhang authored
This reverts commit 9db0a32a. Reason for revert: Breaking Deterministic {Linux,Windows} bots. Original change's description: > Add builder for opus test in prevision of rolls. > > This CL introduces an ad-hoc builder for roll of opus into chromium. > 1/ Used as pres-submit validation. > 2/ Runs dedicated tests and benchmarks for a fixed Android model. > > The point 2/ shall be completed in follow-up CLs. > > Bug: chromium:1002973 > Change-Id: Idb5c8b46ab68d8a9d63a864594396ac021a22746 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1816555 > Commit-Queue: Yves Gerey <yvesg@google.com> > Reviewed-by: John Budorick <jbudorick@chromium.org> > Cr-Commit-Position: refs/heads/master@{#704847} TBR=agable@chromium.org,jbudorick@chromium.org,yvesg@google.com Change-Id: Ie6631651d93f83780bbf81b8291f3a9e3fd2bab5 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: chromium:1002973 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854686Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#705107}
-
Nasko Oskov authored
The WebUI tests in render_frame_host_manager_unittest.cc have some code that is no longer necessary or can be streamlined a bit. There is also a test that no longer matches reality, so I'm removing it as well as some accidental copy/paste duplicate expectations. Bug: 713313, 1002276 Change-Id: I29a0b3c0dea69bbef3eaff5df1dce08127333a00 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1849439 Commit-Queue: Nasko Oskov <nasko@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/d0458b338358..8d99c8fdb68b Created with: gclient setdep -r src-internal@8d99c8fdb68b If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/src-internal-chromium-autoroll Please CC jbudorick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chrome.try:linux-chromeos-chrome TBR=jbudorick@google.com Bug: chromium:None Change-Id: I256f8295f578986c77189f5603603b7990f5faa3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1856198Reviewed-by:
chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#705105}
-
Arthur Hemery authored
Default period content can be cached is 15 seconds with the BackForwardCache enabled. Long tests with timeouts of more than 15 seconds can technically pass tests by waiting for 15 seconds and getting the usual notifications from the RenderFrameHost being destroyed. We want to identify these tests, so we add a content feature that removes all time limit to how long we can keep something in the cache. Change-Id: Ib0736c9ba6f0718fbc9ba729a1e3742d61721a65 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852448Reviewed-by:
Alexander Timin <altimin@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Reviewed-by:
-
David Roger authored
Disables Mac Debug: virtual/disable-deferred-rendering/fast/canvas/color-space/canvas-drawImage-offscreenCanvas.html virtual/disable-deferred-rendering/fast/canvas/OffscreenCanvas-placeholder-createImageBitmap.html TBR=fserb Bug: 1010472, 1012146 Change-Id: I8e1d2c28ee1710267293e5576a968617a45f1c12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1856164 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
-
Torne (Richard Coles) authored
WebViewDelegate.getPackageId sometimes throws an exception complaining that it's unable to find the WebView package in the resource table. This looks like it might be some kind of obscure race caused by doing it on a background thread at the same time as WebView initialisation (which in some cases can also be manipulating the resource table). The goal of the background thread is to parallelize work for a startup time benefit, but the call to getPackageId is actually very fast (the expensive part is calling R.onResourcesLoaded). So, just do the package ID lookup on the main thread first and pass the ID in to the background task, in the hope that this avoids this hypothetical race. Bug: 995400 Change-Id: Id4479cdda54ee7b1aec273a491fa0f0f64ed348c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854466Reviewed-by:
Changwan Ryu <changwan@chromium.org> Commit-Queue: Richard Coles <torne@chromium.org> Cr-Commit-Position: refs/heads/master@{#705102}
-
Dominique Fauteux-Chapleau authored
Added fields per event type: "dangerousDownloadEvent": "contentType", "contentSize", and "trigger" "sensitiveDataEvent": "contentType", "contentSize", and "trigger" "largeUnscannedFileEvent": "contentType", "contentSize", and "trigger" Bug: 1006938 Change-Id: Ib20e1cee7eb339d15374b30880d161f55510a4ad Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1847799 Commit-Queue: Dominique Fauteux-Chapleau <domfc@chromium.org> Reviewed-by:Roger Tawa <rogerta@chromium.org> Reviewed-by:
Daniel Rubery <drubery@chromium.org> Cr-Commit-Position: refs/heads/master@{#705101}
-
Vasilii Sukhanov authored
The promo should be shown again 3 times starting from M79. Bug: 1013181 Change-Id: I3bcf46096f18aefc8d8894d34597aea24aa24601 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855981 Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#705100}
-
Yi Gu authored
In v2, if a hit test region is marked as kHitTestIgnore, this region and its descendants should be excluded from hit testing. However, in v1 this flag may be set in other scenarios [1]. When v2 is enabled by default, some tests will fail due to the inconsistent use of the flag. Using parameterized test to cover this issue until v1 is removed. [1] https://cs.chromium.org/chromium/src/components/viz/common/quads/surface_draw_quad.h?type=cs&g=0&l=50 Bug: None Change-Id: I8ffe49c291f827b83e52766ab549fcb6f245d3a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1839014Reviewed-by:
Sadrul Chowdhury <sadrul@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#705099}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/58a3ebb1bb9f..f5952b718415 git log 58a3ebb1bb9f..f5952b718415 --date=short --no-merges --format='%ad %ae %s' 2019-10-11 tneda@google.com Merge "Adding support in flamegraph API for hovering over callsites and expanding them by clicking." 2019-10-11 treehugger-gerrit@google.com Merge "Fix VulkanMemoryAllocation typo in proto_parser" Created with: gclient setdep -r src/third_party/perfetto@f5952b718415 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md TBR=perfetto-bugs@google.com Bug: None Change-Id: I90a417d1f3fb2cab1a5064af8ef084323b44a080 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1855466Reviewed-by:
-
Antonio Gomes authored
These files need to be moved to Blink as part of the phase 8 on the design document [1]. [1] https://docs.google.com/document/d/1AJKVA5U4nDkyDB9p4ROrggWXadCxyy-grKaE9KS5vOU/edit?ts=5d5c1757 Since they can get moved independently, this CL does so, so reviewing the main CL is eased. BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: I23f65a6187f2eb1402170ebf182552d054ad3fdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1839091 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#705097}
-
Mario Bianucci authored
run-swarmed.py was failing on Windows for two reasons: 1. Windows expects 'python' before calling a python script 2. Tasks on Windows Swarming bots are invoked via vpython, which isn't installed by default on them. This change adds 'python' to the list of arguments for any subprocess, and adds includes vpython with the isolates uploaded to Swarming via --cipd-package and --env-prefix args when the target OS is Windows. Bug: 1013382 Change-Id: I9b8edf9ebc2341cb72f8034ccec044f951745496 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854462 Commit-Queue: Mario Bianucci <mabian@microsoft.com> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#705096}
-
Antonio Gomes authored
These testing classes are needed for peer_connection_tracker_unittest.cc, that will be Onion soup'ed next, together with various other classes and tests. For simplicity, this particular pair of files are moved independently, so the core CL is easier to review. Note that the classes got renamed to FakeRtcRtp{XXX}Impl, to mirror their respective non-fake classes, moved to Blink in [1], [2] and [3]. [1] https://crrev.com/c/1819801 [2] https://crrev.com/c/1819811 [3] https://crrev.com/c/1820060 BUG=787254 R=guidou@chromium.org, haraken@chromium.org Change-Id: I3b8de656d96ef4148f6a08afb23080803a52255f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1835799 Commit-Queue: Antonio Gomes <tonikitoo@igalia.com> Reviewed-by: -
Kevin Ellis authored
The implementation of cancel was aligned with the web-animations spec in: https://chromium-review.googlesource.com/c/chromium/src/+/1824058 Resolving the ready promise after the reject and reset is not to spec [1]. After the above patch we no longer resolve the ready promise after a cancel, so it is possible to already have a pending promise when we enter the "pending" state. This introduced a DCHECK failure when transitioning from an idle state to a pending state. Per above this DCHECK is incorrect so we remove it. Previously, when an animation was cancelled, the destructor for PlayStateUpdateScope will not only reject and reset the ready promise, but it will then resolve this new promise. When transitioning from idle to pending, an additional reset is performed with a check that we do not already have a pending promise. Spec: https://drafts.csswg.org/web-animations/#canceling-an-animation-section Bug: 1012384 Change-Id: Id40fc094f599463a5e3ab67715edde02bbec6720 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1853164 Commit-Queue: Majid Valipour <majidvp@chromium.org> Reviewed-by:
Majid Valipour <majidvp@chromium.org> Cr-Commit-Position: refs/heads/master@{#705094}
-
Daniel Vogelheim authored
Reason for removal: Origin Trials are time-limited, and this one has run out. We're removing the trial itself, as well as the test that tested feature initialization when performed via an origin trial token (977099). Bug: 739170 Change-Id: Ibfdf902c9125fe58111de82b48622f45db60c9b7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1827285 Commit-Queue: Daniel Vogelheim <vogelheim@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Reviewed-by:
Mike West <mkwst@chromium.org> Cr-Commit-Position: refs/heads/master@{#705093}
-
Dominique Fauteux-Chapleau authored
Adds pattern matching to DomainsToNotCheckComplianceOfUploadedContent, DomainsToCheckComplianceOfDownloadedContent and DomainsToCheckForMalwareOfUploadedContent policies. This makes those policies more efficient by not needing to enumerate exact domains and also allows to match against more granular url properties, for instance by using patterns like 'http://*', 'https://server:8080/path' or '*'. The pattern matching used is the same one used by the URLBlacklist policy. To achieve this, this cl includes a refactor that puts that pattern matching code in a common util file. It is worth noting that this changes what the policies actually do and so DomainsToCheck* is no longer an appropriate name, they will be renamed in a different cl. Bug: 1008486 Change-Id: I71410849403f2fc54cefe34d94b0f64256f3e18a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1841772Reviewed-by:
Owen Min <zmin@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
-
Kuang-che Wu authored
Reduce time complexity from O(n^2) to O(n) Bug: chromium:855272, chromium:1009098 Change-Id: I9365c7ae7135ec694a5b83ef47674461fc7f353e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854104Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Commit-Queue: Kuang-che Wu <kcwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#705091}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/6c2b2bb02402..df640e6d14a5 git log 6c2b2bb02402..df640e6d14a5 --date=short --no-merges --format='%ad %ae %s' 2019-10-11 borenet@google.com Revert "Roll recipe dependencies (trivial)." Created with: gclient setdep -r src/third_party/skia@df640e6d14a5 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/skia-autoroll Please CC kjlubick@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md CQ_INCLUDE_TRYBOTS=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-blink-rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel TBR=kjlubick@google.com Bug: None Change-Id: I340772d3fbb8b53714d23b2bebcdf293b99b929c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1854814Reviewed-by:
-
Francois Beaufort authored
This CL makes sure a mediaType is required only if recordType is "json" or "opaque" in a NDEFRecord as described at https://w3c.github.io/web-nfc/#data-mapping Bug: 1013167 Change-Id: I22a6508b66e5e5e7f69ca295fd5281b02dcb2b6d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1852215 Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Reviewed-by:
Rijubrata Bhaumik <rijubrata.bhaumik@intel.com> Cr-Commit-Position: refs/heads/master@{#705089}
-
David Roger authored
TBR=akihiroota Bug: 981100, 1013551 Change-Id: I50890923e35a6d6196a14dcdd48d40367d166f52 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1856059Reviewed-by:
-
