- 09 Jul, 2020 26 commits
-
-
Lei Zhang authored
Just call base::bits::Align() and AlignDown() directly. Change-Id: If62719a55caa78b76b3624fb53c5eef8ef9574b8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2277741Reviewed-by:
Khushal <khushalsagar@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#786581}
-
Jinsuk Kim authored
This CL moves the logic in ChromeFullscreenManager that controls viewport to CompositorViewHolder so that it will be the source of viewport information. ContentView sizing/offset update is still initiated by ChromeFullscreenManager but they are decoupled(i.e. no more ViewSizeDelegate), and now the flow is handled through CVH listening to BrowserControlsStateProvider.onControlOffsetChanged. Bug: 966272 Change-Id: I111b47d4fd93f4d1866dc0cab7012c9339278788 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2278742 Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#786580}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/eb7703565a86..6669b0126705 2020-07-08 bungeman@google.com Always check AutoFTAccess::face() for nullptr. 2020-07-08 johnstiles@google.com Update onMakeColorTypeAndColorSpace to consolidate color processors. 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 mtklein@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/+doc/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: mtklein@google.com Change-Id: Ie6104601fa4491d2693585d4162182d202cabe64 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289029Reviewed-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@{#786579}
-
Klaus Weidner authored
This check was added by https://chromium-review.googlesource.com/c/chromium/src/+/2286052 and is failing for WebXR presentation when using immersive-ar on Android. Change-Id: Id47b7fb33ea9b89d5a782decde104c4ef5b4eccf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288824 Commit-Queue: Klaus Weidner <klausw@chromium.org> Commit-Queue: Peng Huang <penghuang@chromium.org> Auto-Submit: Klaus Weidner <klausw@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#786578}
-
Sophie Chang authored
Bug: 1099371 Change-Id: I3fe8f12e0ad827b180efec3963bb59999c2c2b75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288632 Commit-Queue: Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Michael Crouse <mcrouse@chromium.org> Cr-Commit-Position: refs/heads/master@{#786577}
-
Ken MacKay authored
This avoids stereo image shifts due to WSOLA. Bug: internal b/151393870 Change-Id: I44999ef9c9173fbd56c609307475ce691978f182 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288095Reviewed-by:
Yuchen Liu <yucliu@chromium.org> Commit-Queue: Kenneth MacKay <kmackay@chromium.org> Cr-Commit-Position: refs/heads/master@{#786576}
-
Mark Mentovai authored
This gets all Crashpad code building on macOS for arm64, including tests. Not all tests pass yet, and there aren’t any guarantees that anything else works on arm64 yet. This is mostly a lot of CPU context shuffling. Includes: Update Crashpad to 030c58c2959aa839d247e117ac7b28a71e7dd98c d2c22d6b4b50 [fuchsia] Always initialize 17b08b5aab64 Copies the latest setup_ios_gn.py from Chromium 245fee1b6350 Removes Chromium-specific args from setup_ios_gn.py b2061004cec9 Update mini_chromium to 4e8ff0c3a29f 7cf66d83e65e Restore util tests to crashpad_tests used by Chromium 828095ac32f3 Clear the gn sources_assignment_filter when building in Chromium b9c828ea838d android: Fix “gn analyze” 809939c9d110 mac: Crashpad for macOS on arm64, phase 1: build it 0c3f50c8e14a mac: Crashpad for macOS on arm64, phase 2: build tests 030c58c2959a mac: Add missing #include "build/build_config.h" for ARCH_CPU_* Bug: crashpad:345, chromium:1098899 Change-Id: Ie1459d936babc9567c2fe4956f9e2cb81c84f695 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288635 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#786575}
-
Shervin Oloumi authored
Renames the AnonymizerTool to RedactionTool for better representation of what the tool does and better compliance with the definition of anonymization. Replaces all the references to anonymization with redaction, including occurrences in the comments. Bug: 1097367 Test: Passes all 'components_unittests' and 'RedactionToolTest' Change-Id: I49279d0a7ac1ee68421d7334bd2c3fad729020d8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2255203 Auto-Submit: Shervin Oloumi <enlightened@chromium.org> Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Darren Shen <shend@chromium.org> Reviewed-by:
Sergey Poromov <poromov@chromium.org> Reviewed-by:
Ian Barkley-Yeung <iby@chromium.org> Reviewed-by:
Thiemo Nagel <tnagel@chromium.org> Cr-Commit-Position: refs/heads/master@{#786574}
-
Natalie Chouinard authored
This reverts commit def0fa3e. Reason for revert: WebBundleSubresourceLoaderFactoryTest.MetadataParseError failing on android-marshmallow-arm64-rel https://ci.chromium.org/p/chromium/builders/ci/android-marshmallow-arm64-rel Original change's description: > Add URLLoaderFactory for loading resources from a WebBundle > > In follow-up CLs, this factory will be created by LinkWebBundle, and > will be used to load subresources from Web Bundles. > > Bug: 1082020 > Change-Id: I231e8c85f740e3588c2ba4c518378cd26a36a79d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2275986 > Reviewed-by: Hayato Ito <hayato@chromium.org> > Reviewed-by: Tsuyoshi Horo <horo@chromium.org> > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org> > Commit-Queue: Kunihiko Sakamoto <ksakamoto@chromium.org> > Cr-Commit-Position: refs/heads/master@{#786147} TBR=horo@chromium.org,kinuko@chromium.org,hayato@chromium.org,ksakamoto@chromium.org Change-Id: Idbdb87829da52eff4f87306efaa5b136406a79ee No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1082020 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288767Reviewed-by:
Natalie Chouinard <chouinard@chromium.org> Commit-Queue: Natalie Chouinard <chouinard@chromium.org> Cr-Commit-Position: refs/heads/master@{#786573}
-
Ken MacKay authored
Bug: internal b/160107822 Change-Id: I49ac040c261324bf8c29272c7c7007d507e9f6d4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289075Reviewed-by:
-
Kenichi Ishibashi authored
This CL introduces a mojo interface that works as a client of ServiceWorkerResourceReader. Call sites of the reader can pass a remote that implements the interface to receive OnComplete() notification. This kind of notification is required to replace old resource reader in ServiceWorkerInstalledScriptReader. Bug: 1055677 Change-Id: I1038298942d892bcc69a9dd9edebae73a63d29c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2279304 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@{#786571}
-
Ian Barkley-Yeung authored
Investigation is over, clean up code used for investigation. This reverts https://chromium-review.googlesource.com/c/chromium/src/+/2236469, https://chromium-review.googlesource.com/c/chromium/src/+/2229301, and parts of https://chromium-review.googlesource.com/c/chromium/src/+/2166470. Fixed: chromium:1074115 Change-Id: Ib918c78bcedc3b0adbed8b489afbd76bc6f0223f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284095 Commit-Queue: Ian Barkley-Yeung <iby@chromium.org> Reviewed-by:
Brian White <bcwhite@chromium.org> Reviewed-by:
James Cook <jamescook@chromium.org> Reviewed-by:
Ken Rockot <rockot@google.com> Cr-Commit-Position: refs/heads/master@{#786570}
-
Wei-Yin Chen (陳威尹) authored
This is a follow-up for the review comments in: https://crrev.com/c/2266028/5/tools/metrics/actions/actions.xml#22734 Bug: None Change-Id: I8dfe5b810ab0bfd0570e45d9f8168c46d763b690 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284371Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Wei-Yin Chen (陳威尹) <wychen@chromium.org> Cr-Commit-Position: refs/heads/master@{#786569}
-
James Zern authored
This is reland of https://crrev.com/c/2274961 after updating media/filters/gav1_video_decoder.cc to support the addition of kLibgav1StatusNothingToDequeue. https://chromium.googlesource.com/codecs/libgav1.git/+log/bf190c43e5c7..ba8dd2919fca $ git log bf190c43e..ba8dd2919 --date=short --no-merges --format='%ad %ae %s' 2020-06-25 jzern detect fuzzing w/FUZZING_BUILD_MODE_UNSAFE_FOR_PRODUCTION 2020-06-24 jzern fix kDefaultRestorationTypeCdf size 2020-06-23 vigneshv film_grain: Avoid some implicit bool->int conversions 2020-06-23 vigneshv trivial: Isolate some declarations 2020-06-23 vigneshv tile: Move some functions out of Tile class 2020-06-23 vigneshv film_grain: Remove multiplication by 1 2020-06-23 jzern GetMaskBlendFunc,cosmetics: simplify lookup index 2020-06-23 vigneshv decoder_impl: Move some functions out of DecoderImpl 2020-06-23 vigneshv decoder_impl: Remove AllocateCurrentFrame 2020-06-23 vigneshv deblock, trivial: Add a const declaration (...) 2020-05-14 jzern dsp/CdefFilter,cosmetics: make src pointer uint16_t 2020-05-14 jzern ApplyCdefForOneUnit: fix implicit int -> bool conv 2020-05-14 vigneshv trivial: Rename output_frames_ to output_frame_queue_ 2020-05-13 vigneshv Add support for outputting all layers 2020-05-13 linfengz trivial: reduce block buffer size 2020-05-13 linfengz dsp,trivial: add comments in sfg sse4 2020-05-13 jzern fuzzer_temp_file.h: fix unused function warning 2020-05-13 slavarnway arm: Improve CdefFilter_NEON() 2020-05-12 linfengz make PrepareLoopRestorationBlock() a static member of PostFilter 2020-05-12 vigneshv deblock: Change the plane ordering Created with: roll-dep src/third_party/libgav1/src Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2274961Reviewed-by:
Hirokazu Honda <hiroh@chromium.org> Commit-Queue: James Zern <jzern@google.com> Cr-Original-Commit-Position: refs/heads/master@{#784134} Bug: b/160581754 Change-Id: I722117e7cf502b1f8dc93853bbfdec51f3c1bc55 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284229Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#786568}
-
Joel Hockey authored
The .dialog-header.files-ng which this component sits directly below already has a bottom border and we don't want a double line. Bug: 1099557 Change-Id: I808deb34484b80f984ec31b60424ede0b4ecc2f1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2289171Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Joel Hockey <joelhockey@chromium.org> Cr-Commit-Position: refs/heads/master@{#786567}
-
Steve Kobes authored
This finishes conversion of base::Bind and friends in net/reporting, net/server, and net/websockets. Bug: 1007815 Change-Id: I8c8e060198cc9058c865b36e7f8529e49c1f65a6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288106Reviewed-by:
David Benjamin <davidben@chromium.org> Commit-Queue: Steve Kobes <skobes@chromium.org> Cr-Commit-Position: refs/heads/master@{#786566}
-
Mason Freed authored
This is one CL of many, moving tests out of the virtual/controls-refresh directory, into fast/forms. This CL moves: virtual/controls-refresh/calendar-picker -> fast/forms/calendar-picker virtual/controls-refresh/date -> fast/forms/date virtual/controls-refresh/datetimelocal-picker -> fast/forms/datetimelocal-picker virtual/controls-refresh/month-picker -> fast/forms/month-picker virtual/controls-refresh/time-picker -> fast/forms/time-picker virtual/controls-refresh/week-picker -> fast/forms/week-picker This is a pure move of the above folder(s). A few tests share names in the old/new location, but in all cases, the version being moved is updated/new and should overwrite the old version. See also crbug.com/1102647 - there is additional cleanup that will be needed after this move is complete. But to keep the review process simple, this is just a pure move. Bug: 1034611 Change-Id: Id05c8358152bf88f87e26d3283e2780aab58d109 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2283941 Commit-Queue: Mason Freed <masonfreed@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Reviewed-by:
Stephen Chenney <schenney@chromium.org> Auto-Submit: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#786565}
-
George Steel authored
Make interpolation type always run CSSValues through the converter for this property. Update remaining WPT tests to use order-invariant comparison (as most tests for this property already use). spec https://drafts.csswg.org/css-fonts-4/#font-variation-settings-def If the same axis name appears more than once, the value associated with the last appearance supersedes any previous value for that axis. This deduplication is observable by accessing the computed value of this property. resolution https://github.com/w3c/csswg-drafts/issues/1959 RESOLVED: For both font-variation-settings and font-feature-settings, the computed value is a map (and thus specified dupes are removed) Bug: 1005355 Change-Id: I2f6f0f76fdcd69cfece283625cf42f517686dc8a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2253248 Commit-Queue: George Steel <gtsteel@chromium.org> Reviewed-by:
Dominik Röttsches <drott@chromium.org> Cr-Commit-Position: refs/heads/master@{#786564}
-
rbpotter authored
Hook up the +/- and fit to page/width buttons in the new PDF toolbar. Note: This does not update or make use of the new zoom input field. Bug: 1100428 Change-Id: I33a98f6ea2d065aae5a1557ae3a6c6edad619be0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2277333 Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#786563}
-
Mason Freed authored
This is a reland of 72158dea The only change made here is to add the new test to LeakExpectations, pointing to crbug.com/1103082. Local testing shows that this leak is triggered by the new test, both before and after this patch. And since the patch fixes a UAF security bug, I'd like to land it with the test, and then work on the leak. Fixed: 1102408 Bug: 1103082 Original change's description: > Fix UAF in SelectType > > This fixes the UAF detected by ClusterFuzz in [1], caused by [2]. > The test case added here is a minimized version of the clusterfuzz > case, and I verified that it crashes (ASAN UAF) before this patch > and no longer crashes after. > > [1] https://clusterfuzz.com/testcase-detail/6224868955193344 > [2] https://chromium-review.googlesource.com/c/chromium/src/+/1912682 > > Fixed: 1102408 > Change-Id: Ieb6a9582ff5b9676596048920bbcff881fdc2eb2 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2283901 > Commit-Queue: Mason Freed <masonfreed@chromium.org> > Auto-Submit: Mason Freed <masonfreed@chromium.org> > Reviewed-by: Kent Tamura <tkent@chromium.org> > Cr-Commit-Position: refs/heads/master@{#785970} Change-Id: I471cb4abc98a7627803de4e434e0453cb729c15f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288372 Auto-Submit: Mason Freed <masonfreed@chromium.org> Reviewed-by:
Kent Tamura <tkent@chromium.org> Commit-Queue: Mason Freed <masonfreed@chromium.org> Cr-Commit-Position: refs/heads/master@{#786562}
-
Xinghui Lu authored
Add a flag named "SafeBrowsingSecuritySectionUIAndroid". When this flag is enabled: * Remove the Safe Browsing, data breach and Scout settings from SyncAndServicesSettings. They will be moved under the new "Security" section. before: http://screen/sqZdYYgeTui after: http://screen/WomGjbCwT14 * Add a "Security" section under the "Privacy" section. before: http://screen/Knonxg95erZ after: http://screen/EsXPyntvkpV * Rename the "Privacy" section to "Privacy and security" section. before: http://screen/U75bT4sDFmJ after: http://screen/DuqDjmmR1x0 More context can be found in go/esb-clank-dd. Bug: 1097310 Change-Id: I7644167ff36741a8a39feaa269704904ae6c73ef Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2275343Reviewed-by:
Boris Sazonov <bsazonov@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
-
Rakib M. Hasan authored
Bug: 1102480 Change-Id: I885168c2b792d2c50353246e7b387c66427b06f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2278440 Commit-Queue: Rakib Hasan <rmhasan@google.com> Reviewed-by:
Robert Ma <robertma@chromium.org> Cr-Commit-Position: refs/heads/master@{#786560}
-
Andrew authored
BUG:1103025 TEST:Validate the Recovery App can send feedback using feedbackPrivate. Change-Id: I856737cdaacc0d65f7a42f9766ad7b4d3debf28f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2287951 Commit-Queue: Andrew Lassalle <andrewlassalle@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#786559}
-
Bailey Berro authored
This change introduces static factory functions to construct DisplayAlignmentIndicator's in order to make clear the distinction between constructing an indicator with a pill and without an pill. Bug: 1070122 Change-Id: I79978091f045df13412cf7eae3308431e9a6120d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284052 Commit-Queue: Bailey Berro <baileyberro@chromium.org> Reviewed-by:
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-win-chromium 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/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I00b2a602d5e93024ece2a6a558e8968520e6d995 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288833Reviewed-by:
-
Marijn Kruisselbrink authored
Rather than sorting all extensions, keep the website provided extensions first and append the mime-type derived extensions to that list. This should give the website more control over what extension gets appended to a file name if the user doesn't include an extension themselves. Additionally pass 1 as file_type_index, to select the first type by default. Most file dialog implementations did this anyway when passing 0, but it seems better to be explicit about what type to select by default. Bug: 1103133 Change-Id: I531f13e72bdaa4de8d7b3270da21a577ebe56e82 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288429 Commit-Queue: Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#786556}
-
- 08 Jul, 2020 14 commits
-
-
Dale Curtis authored
NaCl always starts from the sandboxed zygote, so we don't need these added. Additionally they trigger a warning message since the NaCl zygote shouldn't be unsandboxed. Fixed: 1085552 Change-Id: Ic41e89dda9d06d533cf88d45228a66bd646ed1d1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288821 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Derek Schuff <dschuff@chromium.org> Reviewed-by:
Derek Schuff <dschuff@chromium.org> Auto-Submit: Dale Curtis <dalecurtis@chromium.org> Cr-Commit-Position: refs/heads/master@{#786555}
-
Songtao Xia authored
The frontend fix CL: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2288419 Bug: crbug.com/1103380 Change-Id: I5d5d037a32f81fe49aa0d6f1f0b3de8260530210 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288108Reviewed-by:
John Emau <John.Emau@microsoft.com> Commit-Queue: Songtao Xia <soxia@microsoft.com> Cr-Commit-Position: refs/heads/master@{#786554}
-
Renjie Tang authored
CloseStream() and SendRstStream() both only partially cover stream closure code paths. So they are not recording all the streams' states upon their closure. QuicSession::OnStreamClosed() is called however when the stream is closed. It makes more sense to do the logging here. Bug: 1101516 Change-Id: I8bfb7b927eef862497587907998f772e290d76fe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288316Reviewed-by:
David Schinazi <dschinazi@chromium.org> Commit-Queue: Renjie Tang <renjietang@chromium.org> Cr-Commit-Position: refs/heads/master@{#786553}
-
Saurabh Nijhara authored
MinimumChromeVersionEnforced is a device policy and only applicable to ChromeOS. Bug: 1048607 Change-Id: I3dcf88c018b2b16aceb82086996b121f60a1b28b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2279882 Commit-Queue: Saurabh Nijhara <snijhara@google.com> Reviewed-by:
Roman Sorokin [CET] <rsorokin@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#786552}
-
Erik Staab authored
Bug: 1103364 Change-Id: Ic27edc10f6b27c4739d9d2cf027794d919811318 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288311Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/a3f6dc2fadb4..900c7e87ce70 2020-07-08 nguyenmh@google.com Serialize d/s buffers + use color buffers' impl format for readPixels 2020-07-08 jonahr@google.com Skip MultithreadingTest.MultiCreateContext on Ozone 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 syoussefi@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/+doc/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;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: chromium:1103009 Tbr: syoussefi@google.com Change-Id: I09e8985100edb6814e1c1f25d4c7cda02fbfa88b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288659Reviewed-by:
-
Lukasz Anforowicz authored
String literals cannot be used to initialize *non-const* fields - such usage would trigger the following warning: ISO C++11 does not allow conversion from string literal to 'char *' [-Wwritable-strings]. Because of the above, this CL ensures makes a change to include in the rewrite the fields below: char* char_ptr; wchar_t* wide_char_ptr; while continuing to exclude the fields below: const char* const_char_ptr; const wchar_t* const_wide_char_ptr; Bug: 1069567 Change-Id: Id660806c9e947220644b1ad10b083eeebaae36e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2283771 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Cr-Commit-Position: refs/heads/master@{#786549}
-
Stephen Roe authored
Build times for the arm64 CQ 50% bot experiment over the last week are similar to the existing x64 CQ trybot: builder fuchsia_arm64 fuchsia_x64 builds 549 1114 build_minutes_p50 27.18 29.33 build_minutes_p70 47.80 47.18 build_minutes_p90 58.42 57.23 Stability is good at 50% https://ci.chromium.org/p/chromium/builders/try/fuchsia_arm64 Capacity is also good at 50%. The average number of arm64 bots running at one time at 50% CQ varies from 24 to 36 by day out of 240 bots. Typical spikes are 50 to 60 bots active when a CL build is in progress or 150 to 240 bots active when multiple CL builds overlap. Bot usage percentages on a weekday are p50 27 bots p90 76 bots p99 131 bots out of 240 bots There is more pending time info at https://docs.google.com/spreadsheets/d/1yzHLqp7oNFXlKkRg3whoZ2QhAwC4DYN9pJz8PaXwRYA There is more build time info at https://docs.google.com/spreadsheets/d/1A1B6zgLw2VnBXD5IQXrPZGr5NavJrhjhj8JKeunSvno There is more capacity and bot usage info at https://docs.google.com/spreadsheets/d/15oFJzlDLbfLLnndaTSH2Vsi14QgfiWpIm1dbgn742mU Bug: 1042511 Change-Id: I4fbb69b3c33f3b6ae172f64fb471f88748f591d3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288709Reviewed-by:
Stephen Martinis <martiniss@chromium.org> Commit-Queue: Stephen Roe <steveroe@google.com> Cr-Commit-Position: refs/heads/master@{#786548}
-
Mason Freed authored
This is one CL of many, moving tests out of the virtual/controls-refresh directory, into fast/forms. This CL moves: virtual/controls-refresh/focus-rect -> fast/forms/focus-rect This is a pure move of the above folder(s). A few tests share names in the old/new location, but in all cases, the version being moved is updated/new and should overwrite the old version. See also crbug.com/1102647 - there is additional cleanup that will be needed after this move is complete. But to keep the review process simple, this is just a pure move. Bug: 1034611 Change-Id: I99e246142240ba872d8b3a0bec0815769634aaf6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2284020 Auto-Submit: Mason Freed <masonfreed@chromium.org> Commit-Queue: Stephen Chenney <schenney@chromium.org> Reviewed-by:
-
dpapad authored
Delegate more logic from pdf-viewer into viewer-zoom-toolbar. Bug: None Change-Id: I4a80f895ce26825dc491cd7eb4291afdc4b5292f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2285360Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Commit-Queue: dpapad <dpapad@chromium.org> Cr-Commit-Position: refs/heads/master@{#786546}
-
Devlin Cronin authored
The current implementation of tabs.update() will perform updates as it goes, potentially firing an error for invalid update properties after making other changes. Update this so that tabs.update() will only update a window once all parameters are validated. This allows the extension to know that if it threw an error, none of the updates were performed. This also hopefully de-flakes the ExtensionTabsTest.InvalidUpdateWindowState API test. Bug: 1081549 Change-Id: Ice1af2c6af599c6529278f249a6a9b54ad7f078b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2261160Reviewed-by:
David Bertoni <dbertoni@chromium.org> Commit-Queue: Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#786545}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/c5e8e8bf91ef..c3a81f99020d 2020-07-08 mmortensen@google.com scripts/cros_sdk.py: Use cros_build_lib.Die rather than rethrow exception. 2020-07-07 ahassani@chromium.org dev_server_wrapper.py: Strictly specify the static directory 2020-07-07 saklein@chromium.org autotest_util: Check for files before creating archive. 2020-07-07 chrome-bot@chromium.org Update config settings by config-updater. 2020-07-07 bpastene@chromium.org deploy_chrome: Add an arg to deploy test binaries. 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:chromeos-betty-pi-arc-chrome;luci.chromium.try:chromeos-kevin-rel Tbr: chrome-os-gardeners@google.com Change-Id: I0c31ee015fc0772acd551884683dd101ba2ea654 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2287779Reviewed-by:
-
Alex Gough authored
31 bit refcounts are pushing up against larger renderer memory limits so should be guarded for overflow. Bug: 1101547 Change-Id: Iaf613e642be43f2dbddf25bed573e50a0eedf7f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288490Reviewed-by:
Bill Budge <bbudge@chromium.org> Commit-Queue: Alex Gough <ajgo@chromium.org> Cr-Commit-Position: refs/heads/master@{#786543}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/973c321add1b..d60500e69944 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 jam@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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: chromium:1090060 Tbr: jam@google.com Change-Id: I4432c33347fafb81cd1e0df5bdeae6f1d37f66c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2288576Reviewed-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@{#786542}
-
