- 31 Mar, 2020 40 commits
-
-
Johann authored
Backwards roll to try to fix ycbcr-with-cmyk-color-profile.html failures https://skia.googlesource.com/skia.git/+log/63a4edd314aa..46de36f99362 $ git log 63a4edd31..46de36f99 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/skia TBR=michaelludwig@google.com Change-Id: I64966d7cf3d5e185287bfe5f21d11a370aa91b65 Bug: 1066392 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129454Reviewed-by:
Johann Koenig <johannkoenig@google.com> Reviewed-by:
Michael Ershov <miersh@google.com> Commit-Queue: Johann Koenig <johannkoenig@google.com> Cr-Commit-Position: refs/heads/master@{#754862}
-
Omar Morsi authored
This CL introduces the CompressionAlgorithm ONC OpenVPN field. This field specifies the compression algorithm to be used by OpenVPN. Allowed values are "None", "FramingOnly", "LZ4", "LZ4-V2", "LZO". These values are then translated to the values expected by OpenVPN to be passed directly as a value to the --compress option. system_api related CL:2115616 shill related CL:2116030 are propagated as intended from DM test server to Chrome to Shill to OpenVPN. Bug: 1064197 Test: chromeos_unittests ONC* and manual testing by checking that values Change-Id: I56854edb23fc46a15ba95ca9a3e6d61b4eac23b3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119527Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Steven Bennetts <stevenjb@chromium.org> Commit-Queue: Omar Morsi <omorsi@google.com> Cr-Commit-Position: refs/heads/master@{#754861}
-
Mario Sanchez Prada authored
Migrate this test out of third_party/blink/web_tests/css3/flexbox and into the WPT-specific directory, adding links to the relevant specs and a test assertion describing its purpose. Also, rename its filename to something more descriptive and less Chromium-specific (it currently references crbug.com/604346). Bug: 1063749 Change-Id: I2510df86d6eaa93e8b06665ba9e40521f1cc58fc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127026Reviewed-by:
Robert Ma <robertma@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Commit-Queue: Mario Sanchez Prada <mario@igalia.com> Cr-Commit-Position: refs/heads/master@{#754860}
-
Michael Ershov authored
This reverts commit 6966bd53. Reason for revert: The tests became flaky, see crbug.com/1064500. Original change's description: > [WebLayer] Switch geolocation tests to use permission prompt > > These tests were previously using the fake permission manager which > always grants permissions. Now we can use the real permission manager, > and the fake permission manager can be removed. > > Fixes a strict mode violation in permission dialogs (similar to fixes > from https://crrev.com/c/2108603). > > Bug: 1025625, 1025609 > Change-Id: I8b756c61ee213151e53d68a375ffa4d3f8fd7643 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2111061 > Commit-Queue: Clark DuVall <cduvall@chromium.org> > Reviewed-by: John Abd-El-Malek <jam@chromium.org> > Cr-Commit-Position: refs/heads/master@{#752494} TBR=jam@chromium.org,cduvall@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1025625, 1025609, 1064500 Change-Id: I881f788accb3968955410e1ea413949a72688ecd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128069Reviewed-by:
-
Jiahe Zhang authored
Bug: 976583 Change-Id: I1967e144fba5eca4d2b0d25aab755c0510030ce4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117083 Commit-Queue: Khushal <khushalsagar@chromium.org> Reviewed-by:
Khushal <khushalsagar@chromium.org> Cr-Commit-Position: refs/heads/master@{#754858}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/05fa1a99914a..bf4a734c7e6c git log 05fa1a99914a..bf4a734c7e6c --date=short --first-parent --format='%ad %ae %s' 2020-03-31 angle-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/vulkan-loader/src b0c27ce32864..ba07fa0be82c (1 commits) Created with: gclient setdep -r src/third_party/angle@bf4a734c7e6c 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 cwallez@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-asan;luci.chromium.try:win_optional_gpu_tests_rel Bug: None Tbr: cwallez@google.com Change-Id: I793f13db914b40c30193b63d641c9e672c5f4518 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129404Reviewed-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@{#754857}
-
Mohamed Mansour authored
This was failing because `stat.S_ISDIR` requires a mode not an object, the following stack trace happened: TypeError: unsupported operand type(s) for &: 'nt.stat_result' and 'int' BUG=1066235 Change-Id: Iac03eef706f684f4bcccba0d22f8fc7a4aa23730 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128459 Commit-Queue: Greg Thompson <grt@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#754856}
-
Kent Tamura authored
Bug: 1060022 Change-Id: I397398e767dd3c45aa47d175cf0fd6a7d12edbb9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128005 Commit-Queue: Kent Tamura <tkent@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Cr-Commit-Position: refs/heads/master@{#754855}
-
Alexander Hendrich authored
Whitelisting a batch of extension IDs that will be used in form of our version pinning workaround (see bug for details). Bug: 1065112 Change-Id: Ibde37c10e3026a9340b3607f0b0be817c5648654 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2123667 Commit-Queue: Alexander Hendrich <hendrich@chromium.org> Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#754854}
-
Kenichi Ishibashi authored
This CL adds StoreRegistration() and DeleteRegistration() mojo methods which wrap corresponding ServiceWorkerStorage methods. ServiceWorkerStorage's methods return a deleted version and resource ids that can be purged after storing/deleting a registration. This CL doesn't expose these values as mojo return values. Purging resources should be done only when no browser/embedders use them. The storage service will manage purging resources but the mechanism hasn't been implemented yet. Bug: 1055677 Change-Id: I1bcd8bad4172a3c759e89692f8d7948da33219b5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2126432 Commit-Queue: Kenichi Ishibashi <bashi@chromium.org> Reviewed-by:
Makoto Shimazu <shimazu@chromium.org> Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Cr-Commit-Position: refs/heads/master@{#754853}
-
Anastasiia Nikolaienko authored
Bug: 1054798 Change-Id: I09f616428f0dd123f51505af4afc552619111a5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107630 Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Cr-Commit-Position: refs/heads/master@{#754852}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/17111b3e43c3..d7f02588fc51 git log 17111b3e43c3..d7f02588fc51 --date=short --first-parent --format='%ad %ae %s' 2020-03-31 szuend@chromium.org [formatter] Improve formatting for code containing await expressions Created with: gclient setdep -r src/third_party/devtools-frontend/src@d7f02588fc51 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.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 Bug: chromium:1062617 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: If3217a3b758fa7fdce9882b4dcd304767cde38d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129402Reviewed-by:
-
Anand K. Mistry authored
Bug: 1065271 Change-Id: Ie746af2f801ae4466f7a02dff0755c8a5200d425 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2125911Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#754850}
-
Ewann authored
This CL adds a coordinator to the privacy table view controller. Before that the controller was also the coordinator. Bug: 1064961 Change-Id: I781924cc24b4c8aae0e957e1e504e95ece8953b1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120423 Commit-Queue: Ewann Pellé <ewannpv@chromium.org> Reviewed-by:
Sylvain Defresne <sdefresne@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#754849}
-
Friedrich Horschig authored
The password dropdown keys credentials by username. For two suggestions with the same username (i.e. a locally and a account-stored one), the first one in the list would match the selection of either entry. This CL fixes that issue by checking from which store the selected entry originated from. Therefore, the credential that is second in list can be previewed and filled as expected. (See linked bug for screenshot.) Bug: 1060552 Change-Id: Icb13d4af705fc0cac5609c0a359ac0dc0bbaf8ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119526Reviewed-by:
Christos Froussios <cfroussios@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#754848}
-
Finnur Thorarinsson authored
Apparently, as seen in the crash report from the field, the ContentResolver can be in a weird state when asking for an AssetFileDescriptor for an image. The content URI looks a bit weird, so hopefully this is a one-off problem, and not something systemic for the whole image collection. Bug: 1063578, 656015 Change-Id: I00a46e26f634a080b4d10110ef4d545804bad1f7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127787Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Finnur Thorarinsson <finnur@chromium.org> Cr-Commit-Position: refs/heads/master@{#754847}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/54265a4c5ab7..fddb4d33089e Created with: gclient setdep -r src-internal@fddb4d33089e 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 Bug: None Tbr: jbudorick@google.com Change-Id: Ic318c4abe9a51c90aacee63dbb887c38c1556a9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129401Reviewed-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@{#754846}
-
Nazerke authored
This CL removes unused NonModalAlertCoordinator, NonModalAlertTouchForwarder and NonModalAlertPresentationUpdater. Change-Id: Icb676ce2e1b8a057f30b55336204ab5654d9b6f6 Bug: 1065878, 1029346 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128486 Commit-Queue: Nazerke Kalidolda <nazerke@google.com> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#754845}
-
Hitoshi Yoshida authored
The current IDL compiler generates two sets of files foo/bar.{cc,h} and bindings/{components}/v8/v8_bar.{cc,h} for foo/bar.idl that defines a dictionary, and makes v8_bar.h include bar.h. On the other hand, our new code generator which is enabled behind a flag will generate only v8_bar.{cc,h} and foo/bar.{cc,h} won't be necessary. This CL replaces inclusions of foo/bar.h to v8/v8_bar.h to make them work with both code generators, and has no behavior changes. Bug: 839389 Change-Id: I477e0e9029597cc6a9522463d08d5442a2625bc8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120223Reviewed-by:Kentaro Hara <haraken@chromium.org> Commit-Queue: Hitoshi Yoshida <peria@chromium.org> Cr-Commit-Position: refs/heads/master@{#754844}
-
Mathias Carlen authored
This trybot runs linux browser_tests and unit_tests with the enable_autofill_assistant_api build flag set. Bug: b/152289442 Change-Id: Iee527fe0d0f7813830e2d1d4ec7b27cf08835913 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124253 Commit-Queue: Mathias Carlen <mcarlen@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Cr-Commit-Position: refs/heads/master@{#754843}
-
Roman Sorokin authored
This is a reland of 04e528b0 Original change's description: > cros oobe: s/LoginManagerTest/OobeBaseTest > > Moved DemoSetup* and EnableDebugging* tests to OobeBaseTest > > Bug: 987587 > Change-Id: I1b0860861720d1cbd77be9e2c007c74a7e2f8801 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124608 > Reviewed-by: Denis Kuznetsov [CET] <antrim@chromium.org> > Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> > Cr-Commit-Position: refs/heads/master@{#754456} Bug: 987587 Fixed: 1066120 Change-Id: I91439edc93288427822c1d453d1ef66be6eb472f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2127806 Commit-Queue: Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Denis Kuznetsov [CET] <antrim@chromium.org> Cr-Commit-Position: refs/heads/master@{#754842}
-
dpapad authored
Also removing calls to CallJavascriptFunctionUnsafe() in the process, replacing with FireWebUIListener(). This is in preparation of migrating to Polymer 3. Bug: 1012533 Change-Id: I01bd5afcac01db2a9827846b165be7d36d16b376 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128972 Commit-Queue: David Roger <droger@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#754841}
-
Andres Calderon Jaramillo authored
Problem: CL:2010243 made it so that the GL image for a video frame is created using the visible size. This is correct because we want to make sure that when we composite, we don't sample padding (which would result in unwelcome green lines). However, the coded size of the video frame sent to the client still contained the non-visible area. This caused PaintCanvasVideoRenderer to sometimes issue GL commands with dimensions exceeding those of the GL image backing the texture. These commands would subsequently fail validation resulting in the associated video frame not being rendered on the canvas. Solution: This CL is a generalization of CL:2056285 which solved the same problem but for the VA-API video decode accelerator. The essence of the solution here is to override ProvidePictureBuffersWithVisibleRect() in VdaVideoDecoder. That way, we end up creating the outgoing video frame with a coded size equal to the visible size solving the problem above. Note that there's one caveat: there is an unusual corner case where an H.264 video includes information about a visible rectangle whose origin is not at (0, 0). Therefore some of the top and left parts of the video should not be displayed. However, Chrome does not currently deal well with this case. This CL does not make that situation better, but this could be addressed in a follow-up CL considering that it's a rare case. I tested across many boards. To see the test cases and screenshots of the results, see [1] (a d08cb982 in the filename represents the base commit before this CL). [1] https://drive.google.com/open?id=1h94xbzNLBrlHBlbmaaasHV1P7HOaPBt1 Bug: 1062002 Test: video in bug gets rendered to canvas in kevin successfully. Change-Id: I6daf9df07c8588d82d5396908d8f91c92ce3984d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2122484 Commit-Queue: Andres Calderon Jaramillo <andrescj@chromium.org> Reviewed-by:
Alexandre Courbot <acourbot@chromium.org> Reviewed-by:
Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#754840}
-
Dominic Mazzoni authored
Completely mechanical/automated. Doing it in small chunks and landing with TBR to minimize collisions. Bug: 1063155 Change-Id: If4a545312b17b2b2adfc385cc26ddd246a7ccf30 Tbr: aleventhal@chromium.org Relnotes: N/A Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2125389 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#754839}
-
Takashi Toyoshima authored
Since the situation is still unclear, let's flip the default mode to keep policies activated. Since the Chrome 82 was cancelled, mitigation removal may happen at Chrome 83 via Finch in the best case, but the plan may be postponed through this flag until Chrome 84. Bug: 1053866 Change-Id: Id4020ff8074622bb6965a3dc61a57d97bbf69ebc TBR: emaxx@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128035Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/external/github.com/KhronosGroup/glslang/+log/e7c94f4973bc..e157435c1e77 git log e7c94f4973bc..e157435c1e77 --date=short --first-parent --format='%ad %ae %s' 2020-03-31 cepheus@frii.com Bump revision. Created with: gclient setdep -r src/third_party/glslang/src@e157435c1e77 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/glslang-chromium-autoroll Please CC radial-bots+chrome-roll@google.com,cwallez@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_angle_vk32_deqp_rel_ng;luci.chromium.try:android_angle_vk32_rel_ng;luci.chromium.try:android_angle_vk64_deqp_rel_ng;luci.chromium.try:android_angle_vk64_rel_ng;luci.chromium.try:linux_angle_deqp_rel_ng;luci.chromium.try:linux-angle-rel;luci.chromium.try:win-angle-rel-32;luci.chromium.try:win-angle-rel-64;luci.chromium.try:win-angle-deqp-rel-32;luci.chromium.try:win-angle-deqp-rel-64 Bug: None Tbr: radial-bots+chrome-roll@google.com,cwallez@google.com Change-Id: I26d148ec7e3aacd60f297ccd91e2ced964731d7c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129393Reviewed-by:
-
Clemens Arbesser authored
Bug: b/145043394 Change-Id: I58214e0073f255662c6efca3f3f1e47f1c8b94ed Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124711Reviewed-by:
Mathias Carlen <mcarlen@chromium.org> Commit-Queue: Clemens Arbesser <arbesser@google.com> Cr-Commit-Position: refs/heads/master@{#754836}
-
Hirokazu Honda authored
The client of ImageProcessor feeds ImageProcessor VideoFrame in import mode. The underlying buffer of the fed VideoFrame can have a different stride and size from the expectation of ImageProcessor. The difference is critical in V4L2ImageProcessor. It is necessary to reconfigure the v4l2 image processor driver by VIDIOC_S_FMT with the stride and height of the fed VideoFrame. Bug: 1033799 Test: ip_test --gtest_filter=NV12* on kukui Change-Id: I73050718f6a95a1eaa3d9c7a60eeea1bc791f04e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097857 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
-
Luciano Pacheco authored
This reverts commit aed7a18a. Reason for revert: Disabling for M-83, feature moved to M-84. Original change's description: > Reland "Files app: Enable flag files-ng by default" > > This reverts commit 7545fd48. > > Reason for revert: We want to enable the flag by default on ToT, the revert was intended to disable the flag on M-82, on ToT it should be > enabled. > > Original change's description: > > Revert "Files app: Enable flag files-ng by default" > > > > This reverts commit ea497580. > > > > Reason for revert: Feature moved from M-82 to M-83. > > > > Original change's description: > > > Files app: Enable flag files-ng by default > > > > > > Add files-ng classes and attribute to main.html so files-ng styles are > > > applied during app startup (to avoid a FOUC flash of non-files-ng blue > > > color). > > > > > > Enable file-ng by default in FilesApp and its browser test harness. > > > > > > Bug: 992818 > > > Change-Id: If437271c5604d36b4b0908abb663a84e52b6ae35 > > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087308 > > > Reviewed-by: Alex Danilo <adanilo@chromium.org> > > > Commit-Queue: Alex Danilo <adanilo@chromium.org> > > > Cr-Commit-Position: refs/heads/master@{#747421} > > > > TBR=noel@chromium.org,lucmult@chromium.org,adanilo@chromium.org > > > > Bug: 992818 > > Change-Id: I8b1c770e2c43823eff9a2a9aceae650fcb25a3f8 > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2103992 > > Reviewed-by: Luciano Pacheco <lucmult@chromium.org> > > Reviewed-by: Noel Gordon <noel@chromium.org> > > Reviewed-by: Alex Danilo <adanilo@chromium.org> > > Commit-Queue: Luciano Pacheco <lucmult@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#750448} > > TBR=noel@chromium.org,lucmult@chromium.org,adanilo@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 992818 > Change-Id: I94e63a873d4e2970d201a32dca0944a4e8fa520f > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2116152 > Reviewed-by: Luciano Pacheco <lucmult@chromium.org> > Reviewed-by: Noel Gordon <noel@chromium.org> > Commit-Queue: Luciano Pacheco <lucmult@chromium.org> > Cr-Commit-Position: refs/heads/master@{#752674} TBR=noel@chromium.org,lucmult@chromium.org,adanilo@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 992818 Change-Id: I3e6a3a82ded4953ea6c9757c0b94b500cd21563f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128045Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Alex Danilo <adanilo@chromium.org> Reviewed-by:
-
David Jean authored
Indenting was added by git cl format Bug: 1065322 Change-Id: Id1d13cf3d81eb3391892917ca497da15bd09f538 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2124712 Auto-Submit: David Jean <djean@chromium.org> Reviewed-by:
Hiroshi Ichikawa <ichikawa@chromium.org> Commit-Queue: David Jean <djean@chromium.org> Cr-Commit-Position: refs/heads/master@{#754833}
-
Hans Wennborg authored
Ran `./tools/clang/scripts/upload_revision.py 4e0d9925d6a3561449bdd8def27fd3f3f1b3fb9f`. Bug: 1065826 Cq-Include-Trybots: chromium/try:mac_chromium_asan_rel_ng Cq-Include-Trybots: chromium/try:linux_chromium_cfi_rel_ng Cq-Include-Trybots: chromium/try:linux_chromium_msan_rel_ng Cq-Include-Trybots: chromium/try:linux_chromium_chromeos_msan_rel_ng Cq-Include-Trybots: chromium/try:linux-chromeos-dbg,win-asan Cq-Include-Trybots: chromium/try:chromeos-amd64-generic-cfi-thin-lto-rel Cq-Include-Trybots: chromium/try:linux_chromium_compile_dbg_32_ng Cq-Include-Trybots: chromium/try:win7-rel,win-angle-deqp-rel-32 Cq-Include-Trybots: chromium/try:win-angle-deqp-rel-64 Cq-Include-Trybots: chromium/try:dawn-win10-x86-deps-rel Cq-Include-Trybots: chrome/try:iphone-device,ipad-device Cq-Include-Trybots: chrome/try:linux-chromeos-chrome Change-Id: Iab8aaf31aa0c281213e425039f13bab9a9ed06d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2125464 Commit-Queue: Hans Wennborg <hans@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Hans Wennborg <hans@chromium.org> Cr-Commit-Position: refs/heads/master@{#754832}
-
Wei-Yin Chen (陳威尹) authored
This should be no-op in prod code. Bug: None Change-Id: I8bd24b4c6d20f9ae461d3e4b4c8e860fd71dee2e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117531Reviewed-by:
Mei Liang <meiliang@chromium.org> Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#754831}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/25a88e75eaed..63a4edd314aa git log 25a88e75eaed..63a4edd314aa --date=short --first-parent --format='%ad %ae %s' 2020-03-31 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/angle2 1b2dd6f92a6b..05fa1a99914a (10 commits) 2020-03-31 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 1c29477aa06e..5cf1e9a31c90 (6 commits) Created with: gclient setdep -r src/third_party/skia@63a4edd314aa 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 michaelludwig@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 Bug: None Tbr: michaelludwig@google.com Change-Id: Ibcb2fbd01d5bf750fc8d43a7e952e23e96701cd8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129387Reviewed-by:
-
Fergus Dall authored
Currently we consider this an error condition, but since we are now able to distinguish this from other failure sources we can retry instead. This CL arbitrarily sets the retry frequency to 5 seconds, which is hopefully fast enough to not be annoying. Bug: 1065817 Change-Id: I532efeb2f4f7ef1aea4be248e1396061edb65d71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2126967 Commit-Queue: Fergus Dall <sidereal@google.com> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#754829}
-
Dominic Mazzoni authored
Eventually all COM interfaces should be on AXPlatformNodeWin and we won't need BrowserAccessibilityComWin at all. This change also provides an implementation of setCurrentValue, which was previously unimplemented, including tests. Bug: none Relnotes: N/A Change-Id: I045625ce7abe695c21e93f244f6c92515f612c44 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119203 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#754828}
-
Hirokazu Honda authored
Bug: 1033799 Test: ip_test on kukui Test: appr.tc on kukui Change-Id: I475f526f8d39535cfe0be696e657b27b82cc135b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2102303 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/1dcb54c744c7..54265a4c5ab7 Created with: gclient setdep -r src-internal@54265a4c5ab7 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 Bug: chromium:1032601,chromium:1043347,chromium:1053811,chromium:1065917 Tbr: jbudorick@google.com Change-Id: I8a39a6b6edb33993b731d18b7dcda20c158caab6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2128883Reviewed-by:
-
Hirokazu Honda authored
The output buffer of ImageProcessor is created by minigbm. The minigbm's buffer alignment can be different from the encoder's one. It is necessary to execute S_FMT with the buffer allocated by minigbm. Bug: 1033799 Test: appr.tc on kukui Test: vea_test on kukui Change-Id: I45659393c84a1bb629afa895349d9c438251810f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2104572 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
-
Wei-Yin Chen (陳威尹) authored
This CL is a mechanical cleanup for instrumented tests under org/chromium/chrome/features/start_surface, suggested by Android Studio. Bug: None Change-Id: I3b6f24e4e2b510d3e6d7ba8d6e648c9480cfc418 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2123643 Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Reviewed-by:
Ganggui Tang <gogerald@chromium.org> Cr-Commit-Position: refs/heads/master@{#754824}
-
Robert Ogden authored
These browsertests have dependancies which are flaky on Windows and some other platforms, and should have been disabled there like the rest of the tests are. Bug: 1023485 Change-Id: I9e2bcd4bf215cc3a07e242e6f038ed33778d50cd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2129089 Commit-Queue: Robert Ogden <robertogden@chromium.org> Auto-Submit: Robert Ogden <robertogden@chromium.org> Reviewed-by:
-
