- 19 Sep, 2020 40 commits
-
-
Simeon Anfinrud authored
Some downstream chromecast code intends to use this. Bug: None Test: build with downstream patch Change-Id: I1d8c71ea236160a95d105a1aa82787b1091b0cbe Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419508 Auto-Submit: Simeon Anfinrud <sanfin@chromium.org> Commit-Queue: Andrew Grieve <agrieve@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#808637}
-
Yuke Liao authored
This CL makes the feature_compiler support non-default toolchains by avoiding assuming that the gen directory is always gen/. Bug: 1129223 Change-Id: I2a361cbb1ce4a220fca03c8db6457f2fb03d81d7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419815 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/master@{#808636}
-
Andrey Kosyakov authored
Bug: 1116444 Change-Id: Ic50dfc144f8024870131e7586b9dce2dff591e42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419712Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#808635}
-
Yulun Wu authored
Bug: 1110766 Change-Id: I2da47ccab1bdb00274ab2224c40efd3a137a5f88 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417241Reviewed-by:
Yulun Wu <yulunwu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Yulun Wu <yulunwu@chromium.org> Cr-Commit-Position: refs/heads/master@{#808634}
-
Yuke Liao authored
This CL supports user provided ash-chrome to make it easy to test crosapi tests that depend on changes made in ash-chrome side. Bug: 1129223 Change-Id: I1be43c760222ed25f0423e7ca05c5e6965441ae0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2415220 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Sven Zheng <svenzheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#808633}
-
Min Qin authored
This CL implements the time decay for tile score. The equation is score = original_score * exp(-0.099 * time_passed). BUG=1096224 Change-Id: Id623232a7510f2ca63f854e75846bc6119fdba03 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417497 Commit-Queue: Min Qin <qinmin@chromium.org> Reviewed-by:
Shakti Sahu <shaktisahu@chromium.org> Cr-Commit-Position: refs/heads/master@{#808632}
-
Jonah Ryan-Davis authored
Because OS_LINUX includes OS_CHROMEOS, passthrough was accidentally launched on ChromeOS by default. This was eventually refactored to explicitly include OS_CHROMEOS, but we want to exclude that. Bug: chromium:976283 Bug: chromium:1095700 Change-Id: I61f83056f91a174352cdddabce1d35b70ee3728a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419541Reviewed-by:
Kenneth Russell <kbr@chromium.org> Commit-Queue: Jonah Ryan-Davis <jonahr@google.com> Cr-Commit-Position: refs/heads/master@{#808631}
-
Meilin Wang authored
This CL adds the connection error UI that pops up when the Phone Hub feature is not available due to connection issues. Remaining work includes implementing the button click handler. Demo: https://screenshot.googleplex.com/6VSGgicU284Z4Ts.png Misc: updates the comment for |PhoneHubInterstitalView|. BUG=1106937,1126208 Change-Id: I12c05c2bf47dd017af801874170aba29aa3554c6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417296 Commit-Queue: Meilin Wang <meilinw@chromium.org> Reviewed-by:
Tim Song <tengs@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Cr-Commit-Position: refs/heads/master@{#808630}
-
Sergey Ulanov authored
crrev.com/808041 disabled SkiaRenderer on all low-end devices. This broke graphics on Fuchsia (low-end devices with Vulkan is enabled). That change was Android-specific, so it doesn't make sense on Fuchsia. Moved corresponding code to #ifdef OS_ANDROID, so it doesn't break other platforms. TBR=backer@chromium.org Bug: 1126490, 1130036 Change-Id: I10842de84e64eca16d8e904b3b1139ca7ad5abdf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419555Reviewed-by:
Sergey Ulanov <sergeyu@chromium.org> Commit-Queue: Sergey Ulanov <sergeyu@chromium.org> Cr-Commit-Position: refs/heads/master@{#808629}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/788480dcff70..14dbf82961c3 2020-09-18 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision 4ed1b071..f294a94c (808365:808500) 2020-09-18 titovartem@webrtc.org Move SampleStatsCounter to public API 2020-09-18 sprang@webrtc.org Fix flag parsing for lazy pacer starting 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/+doc/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I74667abab796865f5e828fa54a99f868241ddcc2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419628Reviewed-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@{#808628}
-
chinsenj authored
As a part of making the window cycle list more interactive, mouse behavior has been added. This CL adds to this ongoing effort by making user clicks outside of the list close it. MousePressOutsideOfListCancelsCycling Test: InteractiveWindowCycleControllerTest. Bug: 1067327 Change-Id: I25b3696a96ff5834a68bc503ed805a005b52e4bf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414125Reviewed-by:
Jun Mukai <mukai@chromium.org> Commit-Queue: Jeremy Chinsen <chinsenj@chromium.org> Cr-Commit-Position: refs/heads/master@{#808627}
-
Aaron Colwell authored
Removing is_empty() method and updating callers to use the explicit lock state methods is_invalid/allows_any_site/is_locked_to_site instead. Bug: 1085275 Change-Id: Id08adaca960368de16943710c5a35c9a6fb5a350 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414764 Commit-Queue: Aaron Colwell <acolwell@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#808626}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/e1e4781e5ec2..5a2819e1a971 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 fdegans@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:1077274,chromium:1085419,chromium:1085603,chromium:1099917,chromium:1099982,chromium:1105693,chromium:1109276,chromium:1117068,chromium:1117069,chromium:1117244,chromium:1126301,chromium:1126891,chromium:1127088,chromium:1128632,chromium:1128770,chromium:1128990,chromium:1129342,chromium:1129402,chromium:1129540 Tbr: fdegans@google.com Change-Id: Ia977b3d5dd2347c9f8589db0e469b3f573e159e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419773Reviewed-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@{#808625}
-
Yuke Liao authored
These proto files are generated per toolchain, so this change makes the action work correctly when compiled inside a non-default toolchain. Bug: 1129223 Change-Id: I3d040de0f264f92b63e3aa44c63d9f43578ddb66 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419696Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#808624}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/df123b290f75..244d77022cb2 2020-09-18 recipe-mega-autoroller@chops-service-accounts.iam.gserviceaccount.com Roll recipe dependencies (trivial). 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 ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@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 Bug: None Tbr: ajp@google.com,apolito@google.com,ehmaldonado@google.com,sokcevic@google.com Change-Id: I80b8effb211c490a47f3ab7a92f0954087a5be40 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419627Reviewed-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@{#808623}
-
Yuke Liao authored
libmojo_core.so is generated by the a shared_library rule, which always outputs the .so file inside: ${root_build_dir}/${toolchain_used_to_compile} directory. Bug: 1129223 Change-Id: Ib2937510363ccc4db7df09d000d260d344aed4a3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419149Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#808622}
-
David Maunder authored
In backShouldCloseTab we should not try to close the Tab if the Tab is not initialized. Bug: 1117396 Change-Id: I7ef0b4e50315ddc9560b438c66a7f66161c5e1ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410920Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: David Maunder <davidjm@chromium.org> Cr-Commit-Position: refs/heads/master@{#808621}
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/31d58940..80dc1e80 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,vahl@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com Change-Id: I93b541c612609ea2142019bc9a219c2684e42f21 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419475Reviewed-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@{#808620}
-
David Schinazi authored
This change cleans up the codebase and prepares us for using multiple versions in DefaultSupportedQuicVersions(). Our current plan is to eventually set it to (h3-29,h3-Q050) if our latest experiments in Stable go as planned. R=renjietang@chromium.org Change-Id: Ie1f6b35b42df07f65a8d800616e8f34427282913 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419150 Commit-Queue: David Schinazi <dschinazi@chromium.org> Commit-Queue: Renjie Tang <renjietang@chromium.org> Auto-Submit: David Schinazi <dschinazi@chromium.org> Reviewed-by:
Renjie Tang <renjietang@chromium.org> Cr-Commit-Position: refs/heads/master@{#808619}
-
Moe Ahmadi authored
Frequency component must be multiplied by recency decay in numerator. See: http://shortn/_bW5Qq2zTHM Bug: 1096615 Change-Id: I5138dce685607c44289dcab8b42a22968fc8ea0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419533 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Commit-Queue: Justin Donnelly <jdonnelly@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Auto-Submit: Moe Ahmadi <mahmadi@chromium.org> Cr-Commit-Position: refs/heads/master@{#808618}
-
Daniel Murphy authored
The WebAppProvider can be uninitialized. Dependencies should use the on_registry_ready() signal. This change is a quick & mergeable fix for a related crash, after which the codepaths will be turned async if possible. TBR-ing dominick so this change can get a canary roll, and they can look at it when they get in on Monday. Patch is very low-risk - resolving crash when there used to be a crash. R=pwnall@chromium.org TBR: dominickn@chromium.org Bug: 1129921 Change-Id: I57c6d4f181f28c178df85c60626423a5619f9494 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419537 Commit-Queue: Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Cr-Commit-Position: refs/heads/master@{#808617}
-
Sophie Chang authored
Bug: 1086227 Change-Id: I6e144c0d858e88206b768293ca237d5aa56efc19 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417532 Commit-Queue: Sophie Chang <sophiechang@chromium.org> Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Dominic Farolino <dom@chromium.org> Cr-Commit-Position: refs/heads/master@{#808616}
-
Yuke Liao authored
out_camera_app_dir is constructed by: 'out_camera_app_dir = "$root_out_dir/resources/chromeos/camera"', which already has a toolchain name inside the path if compiled by non-default toolchain. 'out_camera_app_dir' in the grit_flags is supposed to be relative to the target_gen_dir according to grit_rules.gni, therefore, when rebasing, it needs to be rebased against root_out_dir instead of root_build_dir so that the toolchain name bits can be stripped off. Bug: 1129223 Change-Id: Ib2a5bf62f2f5957613398e03e20402c95896801e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419658Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Cr-Commit-Position: refs/heads/master@{#808615}
-
Brian Sheedy authored
Sets --expected-vendor-id for info_collection_tests on the ARM-based Mac DTKs, as we don't currently request a specific GPU for tasks and thus cannot determine it automatically during JSON generation. Bug: 1105656 Change-Id: I0f9a527bb1797dcc8af293b6fed9405bae9732dd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419542 Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Yuly Novikov <ynovikov@chromium.org> Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Cr-Commit-Position: refs/heads/master@{#808614}
-
wutao authored
Changes: 1. When get settings fails, we will use backoff policy until max retries. 2. However, we will request settings immediately if the request is called from Settings UI. Bug: b/152921891 Test: added new tests Change-Id: Ied3f02b7dcb24d6fb95b528bac932987b9ee3ee1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2410299 Commit-Queue: Tao Wu <wutao@chromium.org> Reviewed-by:
Jimmy Gong <jimmyxgong@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#808613}
-
David Maunder authored
Bug: 1060181 Change-Id: I57ab52c6a8113370829a902d53a1544e9a6a0cf9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2382734Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Tommy Nyquist <nyquist@chromium.org> Commit-Queue: David Maunder <davidjm@chromium.org> Cr-Commit-Position: refs/heads/master@{#808612}
-
Yuke Liao authored
This CL supports also_build_ash_chrome arg to allow building ash-chrome in an alternate toolchain. A few other fixes are required to work, but they will be landed in separate CLs because they could affect existing builds, so landing them separately will make it easier to reason and detect potential breakages. Bug: 1129223 Change-Id: I94dc3dc1ca2bc96f029fea3c4f4d6e41d018e2ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2416999 Commit-Queue: Yuke Liao <liaoyuke@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/master@{#808611}
-
Mark Mentovai authored
This enables x86_64 Chrome to run under Rosetta binary translation on arm64 macOS systems. The system page size is no longer considered a compile-time constant on macOS. It may be determined dynamically at run time through the vm_page_size variable. The related vm_page_shift and vm_page_mask variables are also available. On arm64, the system page size is currently 16kB. On x86_64, it’s 4kB. x86 code running translated on currently available arm64 systems (the Apple Developer Transition Kit) is subjected to the arm64 system’s native 16kB page size, meaning that code built with an expectation of 4kB pages, previously correct for native x86_64, will not work correctly in the translated environment. This is significant because the DTK is the only mac-arm64 system available for testing now, and it’s become important to test x86_64 Chrome under Rosetta translation on arm64. To overcome this limitation, the system page size and related constants are, on macOS, made to be determined at run time on the basis of the variables provided for this purpose by the system. In order to make this change without violating Google’s and Chrome’s prohibition against load-time initializers, the variables that had been used to convey page size information need to become functions. On all platforms except for macOS, the functions remain constexpr, are marked to be always inlined, and continue to return the same compile time-determined value they had in the past, so while it appears invasive, this change is in fact physically a no-op on these platforms. On macOS, it is a functional no-op change, except when running x86_64 code on arm64 under Rosetta translation, where it causes correct values to be used for the run-time environment. Adopting a compile-time 16kB page size on mac-x86_64 was considered but rejected due to the potential for memory footprint and performance impact. That approach would result in a behavior change for native x86_64 users, which constitute the vast majority of consumers of the x86_64 build, for the benefit of the comparatively small number of arm64 users running x86_64 Chrome under translation. This is undesirable, particularly for a change that may need to be merged to more stable branches. Apple documentation links: https://download.developer.apple.com/Documentation/Universal_App_Quick_Start_Program_Resources/DTK_Release_notes.pdf “Rosetta 2 Behaviors…Applications that depend on a 4K memory page size will not run. Examples of this may include some web browsers and language runtimes.” Also: https://developer.apple.com/documentation/apple_silicon/addressing_architectural_differences_in_your_macos_code#3616877 https://developer.apple.com/videos/play/wwdc2020/10214/?time=544 (9:04) Bug: 1102194 Code-Word: disaster CQ-Include-Trybots: luci.chromium.try:mac-arm64-rel Change-Id: I143f38f06444f96eb8a82ede34f2a44ea48318fb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2361202 Commit-Queue: Mark Mentovai <mark@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Reviewed-by:
Chris Palmer <palmer@chromium.org> Cr-Commit-Position: refs/heads/master@{#808610}
-
Mike Wittman authored
This configuration solely controls the ThreadProfiler, so rename it to be consistent with the naming of that class and with future code in support of enabling the ThreadProfiler on a per-thread basis. Bug: 1129939 Change-Id: I20d19abe579cabaf7a355182f164898c12861c92 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2418675 Commit-Queue: Mike Wittman <wittman@chromium.org> Reviewed-by:
Etienne Pierre-Doray <etiennep@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#808609}
-
Nico Weber authored
- Mention that we now have a tester bot and link to it - Put actual build instructions to top of the "Building" section - Add some text about copying files to the DTK, and mention quarantine error messages, so that search engines can index them Bug: none Change-Id: I5b5f042280b6077ad89e217e4bf907f4b4c12fde Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2418984 Commit-Queue: Nico Weber <thakis@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Cr-Commit-Position: refs/heads/master@{#808608}
-
Yue Li authored
Bug: b/150034512 Test: Manual Test Change-Id: I87a02b3d0ef65a2e6a26c663fe2ee66d43331423 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419263 Commit-Queue: Yue Li <updowndota@chromium.org> Auto-Submit: Yue Li <updowndota@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#808607}
-
Dirk Pranke authored
A "raw" test type in MB was a test executable that didn't conform to one of the other supported test harness APIs (like the chromium gtest api or the test executable API), and which we would handle by simply running the executable and returning the status code. There was no real need to support this directly; it turns out that every raw test that we current run should've either actually been a gtest-based test or was handled by a generated wrapper (for the ChromeOS Tast-based tests). Fixing this and removing support for "raw" test types allows us to simplify MB a bit but more importantly gets us one step closer to having only one supported API for tests. Bug: 816629 Change-Id: I831efa949614719a8aa5106c5ab5c1c0c48b5059 Cq-Include-Trybots: luci.chromium.try:linux_optional_gpu_tests_rel Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2380439 Commit-Queue: Dirk Pranke <dpranke@google.com> Reviewed-by:
Ben Pastene <bpastene@chromium.org> Cr-Commit-Position: refs/heads/master@{#808606}
-
Peter Boström authored
Most of these methods forward to |model_| and then return *this. Inlining these reduces boilerplate. Bug: 1106422 Change-Id: I19724548e73e117f32c38c88fae5c5beaf3f6741 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419142 Auto-Submit: Peter Boström <pbos@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#808605}
-
Yulun Wu authored
Bug: 1110766 Change-Id: I832431d2c500080143bb2b470553764d7262097e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417653 Commit-Queue: Yulun Wu <yulunwu@chromium.org> Reviewed-by:
Yulun Wu <yulunwu@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Cr-Commit-Position: refs/heads/master@{#808604}
-
Kalvin Lee authored
This change moves the Restrictions struct (public in the declaration of PpdProvider) into the PPD metadata parsing code. This is not the change where we cut away the v2 PpdProvider, though, so we retain the old struct definition as PpdProvider::LegacyRestrictions. In addition, this change modifies the semantics of Restrictions; metadata that can be restricted now always contains Restrictions structs, but the struct's members (max and min milestones) are now optional. Bug: chromium:888189 Test: chromeos_unittests Change-Id: I03a3825ec2150e8db93a2187d11285afcd44b07a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2393615 Commit-Queue: Kalvin Lee <kdlee@chromium.org> Reviewed-by:
Luum Habtemariam <luum@chromium.org> Cr-Commit-Position: refs/heads/master@{#808603}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/be0a9ca63d9d..ca4d074d54ab 2020-09-18 johnstiles@google.com Fix unit test for InlinerWrapsEarlyReturnsWithDoWhileBlock. 2020-09-18 johnstiles@google.com Add unit test for a dead do-while loop. 2020-09-18 bsalomon@google.com Expand SkYUVAInfo::PlanarConfig/SkYUVAPixmapInfo::DataType values. 2020-09-18 csmartdalton@google.com Fix CanvasKit warnings compiling on Mac with emsdk 2.0.0 2020-09-18 johnstiles@google.com Add program-settings flag to disable the inliner. 2020-09-18 mtklein@google.com quad-pump avx2 and up 2020-09-18 robertphillips@google.com Switch the thread-safe proxy cache over to using an arena for its entries 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 scroggo@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 Cq-Do-Not-Cancel-Tryjobs: true Bug: chromium:1108408 Tbr: scroggo@google.com Change-Id: Ie69b9281937728bea0132439962ae326e110610d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419625Reviewed-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@{#808602}
-
Nate Chapin authored
* Use LocalDOMWindow rather than LocalFrame/Document for most things. * Change probe::WindowOpen() to take a KURL rather than a url String, to avoid an unnecessary CompleteURL() call. * Remove an obsolete TODO, which refers to a Frame being remote when at a point where it is known to be local. Change-Id: I0860485758fa1ff12843bf965e03c9816e5f2390 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2414277 Commit-Queue: Nate Chapin <japhet@chromium.org> Auto-Submit: Nate Chapin <japhet@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#808601}
-
Daniel Libby authored
This flag is still WIP, but in the meantime it doesn't need to part of about:flags as we don't expect users to toggle it. R=kschmi@microsoft.com Bug: 1110626 Change-Id: I849bd807b609ad39d384177ba081a1109749c90d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2417637 Commit-Queue: Daniel Libby <dlibby@microsoft.com> Auto-Submit: Daniel Libby <dlibby@microsoft.com> Reviewed-by:
Adam Ettenberger <Adam.Ettenberger@microsoft.com> Cr-Commit-Position: refs/heads/master@{#808600}
-
Brian Sheedy authored
Prepares the GPU pixel tests for the upcoming migration from a dedicated GPU Gold instance to the general Chrome instance by: 1. Including a combined hardware identifier that will allow us to make forwarding rules more precise. 2. Performing a duplicate, FYI version of the comparison in the Chrome instance in addition to the existing comparison in the GPU instance. Bug: 1113308, skia:10723 Change-Id: Ie6e9abb5e8b7edaec11bde63ece81a45d47792b2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2415089 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Yuly Novikov <ynovikov@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#808599}
-
Xiaohan Wang authored
Update the string per design doc (see Bug). Also check ShouldShow() which was missed in the last CL due to rebase error. See screenshot at: https://photos.app.goo.gl/Ho4rr76RK4nnckJe7 Tbr: dcheng@chromium.org Bug: 1126568 Change-Id: I775715eba04f3ff243e3db8b73bba04a42771a33 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2419578 Commit-Queue: Xiaohan Wang <xhwang@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#808598}
-