- 25 Mar, 2020 40 commits
-
-
Anastasiia Nikolaienko authored
Bug: 1043108 Change-Id: Icdcd8d35a5fa1b0452d29ab666e3983ad46a006a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2100939Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Anastasiia Nikolaienko <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#753145}
-
tby authored
We've got a lot of dead code in search_result_ranker* from now-disabled experiments. I'm going to go through and clean it up, starting with the category and query-based mixed-types models. Bug: 1028447 Change-Id: I93f5d050df33d72ee4a1ce6362db87d8ee8ad1e1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117915Reviewed-by:
Jia Meng <jiameng@chromium.org> Reviewed-by:
Rachel Wong <wrong@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#753144}
-
Alex Danilo authored
Sets the end margin for the clear icon in the search box to 6px to align the end of the search box wrapper with the touch target. Increases the width calculation for the search wrapper to allow for the changed margin end change and keep the search wrapper width at 288px. Increases the width of the cr-input for the same reason as above. Bug: 1064514 Change-Id: Ic1e0ba7f31adef6f9b2230455ac8edd3bd5e54d9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120111Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#753143}
-
Ken Rockot authored
These were disabled across the board due to TSAN failures. They were also previously disabled specifically in low-end Android test environments. The low-end test bot that originally produced failures is no more. The TSAN failures also appear to be gone according to local testing. This attempts to turn the tests back on so we can see what happens. Bug: 626299,769903 Change-Id: Id75acf5948e45b72c714af31af74507987f2078f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119110 Auto-Submit: Ken Rockot <rockot@google.com> Commit-Queue: Guido Urdaneta <guidou@chromium.org> Reviewed-by:
Guido Urdaneta <guidou@chromium.org> Cr-Commit-Position: refs/heads/master@{#753142}
-
tby authored
Currently we have presubmits that check the structured metrics XML file is well-formatted (doesn't have duplicated metric names etc), but these only run when you `git cl upload`. It would be better if an invalid structured.xml file caused a compilation failure, because it will lead to confusing bugs when testing locally. This CL expands the compile-time validations to cover the cases currently checked only via presubmit. Bug: 1016655 Change-Id: Iec71eb7db629e566c10c7c997e509a5824373fb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2120070Reviewed-by:
Thanh Nguyen <thanhdng@chromium.org> Commit-Queue: Tony Yeoman <tby@chromium.org> Cr-Commit-Position: refs/heads/master@{#753141}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/d588fbe8cd72..a65af2905412 Created with: gclient setdep -r src-internal@a65af2905412 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:1033098 Tbr: jbudorick@google.com Change-Id: I62e48f21ead9a3a7ba894892bb7efcf030b656a5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119915Reviewed-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@{#753140}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/81c8f4b8027e..8ec90f4c93f6 git log 81c8f4b8027e..8ec90f4c93f6 --date=short --first-parent --format='%ad %ae %s' 2020-03-25 skia-autoroll@skia-public.iam.gserviceaccount.com Roll third_party/externals/swiftshader 6c11cf2ad9ca..540bdf92531d (2 commits) Created with: gclient setdep -r src/third_party/skia@8ec90f4c93f6 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 robertphillips@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: robertphillips@google.com Change-Id: I03864795c66ce29a32c0edf23a5412ef9571ac76 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119916Reviewed-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@{#753139}
-
Bugs Nash authored
The Media App (currently behind a flag) requires FileHandling to be enabled to work. This patch removes the need for users to explicitly enable the FileHandling flag as well when enabling the MediaApp flag Bug: 1030935 Change-Id: I088466129a68b455e4b05793a98cbf2849a6cd42 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108242Reviewed-by:
Alex Danilo <adanilo@chromium.org> Reviewed-by:
Trent Apted <tapted@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Commit-Queue: Bugs Nash <bugsnash@chromium.org> Cr-Commit-Position: refs/heads/master@{#753138}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 1f151e19. With Chromium commits locally applied on WPT: bb6643e9 "Simplify image-loading-subpixel-clip.html." 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: I2b6c0e69a544d73fddc100e7663f3f7ca4f6dcdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119552Reviewed-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@{#753137}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/d0800d29473f..107e4e989e94 git log d0800d29473f..107e4e989e94 --date=short --first-parent --format='%ad %ae %s' 2020-03-25 ynovikov@chromium.org Lift SwANGLE dEQP-GLES3.functional.fbo.blit.conversion.* suppression Created with: gclient setdep -r src/third_party/angle@107e4e989e94 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 cnorthrop@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: cnorthrop@google.com Change-Id: I7ee95b29b7cb444f9f2336a3be50897dbeebb5f9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119912Reviewed-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@{#753136}
-
Brian Sheedy authored
Re-enables the BrowserMinidumpTest tests on Linux and the CrOS amd64 emulator, as the flakiness that caused them to be disabled should be properly attributed to the correct test now that pre/post test minidump checking has been enabled. Also fixes the test failing locally under certain GN arg configurations that caused the crash signature to change a bit. Bug: 1056235 Change-Id: I74c7407e8ded26652cf8fa833c5e11bd2dca9f4a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2118576 Auto-Submit: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Wenbin Zhang <wenbinzhang@google.com> Reviewed-by:
Wenbin Zhang <wenbinzhang@google.com> Cr-Commit-Position: refs/heads/master@{#753135}
-
Yuzu Saijo authored
This CL filters out RenderFrameHostImplBrowserTest.CheckIsCurrentBefore AndAfterUnload, which is failing deterministically. Bug: 1062923 Change-Id: Id016e1f79c8eb63fb8b977862f38d18d2d08d2b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2109513Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Yuzu Saijo <yuzus@chromium.org> Cr-Commit-Position: refs/heads/master@{#753134}
-
Luciano Pacheco authored
Bug: 1064504 Change-Id: I1233f791280d3b7aa56b7ee6713ebb2886fa6048 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119348 Commit-Queue: Noel Gordon <noel@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Alex Danilo <adanilo@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#753133}
-
nancy authored
Move LaunchManager's two functions to launch utils: GetLaunchFilesFromCommandLine CreateNewTabBrowser Follow up: 1. Move ExtensionAppLaunchManager's function to extensions, and remove ExtensionAppLaunchManager 2. Modify web_app::WebAppLaunchManager to remove the dependency on LaunchManager. 3. Remove LaunchManager. BUG=1061843 Change-Id: I29e333a4ef686ab76edee846cf35228ac55e5074 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117930Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#753132}
-
Maggie Cai authored
This CL adds the functionality to delete all entries with the given app id from the preferred app list. This CL also migrates the unit tests from the original preferred app tests. BUG=853604 Change-Id: I475c0c2776c7c2723b7cfdcdfb4a9dccd2ddcbce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117141Reviewed-by:
Nancy Wang <nancylingwang@chromium.org> Commit-Queue: Maggie Cai <mxcai@chromium.org> Cr-Commit-Position: refs/heads/master@{#753131}
-
Luciano Pacheco authored
Reduce the size of the input inside the <cr-input> to avoid long text to run over the "clear button". Bug: 1061311 Change-Id: I281d2388eeadb5001ac35b1b6dca14c301f437fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119340Reviewed-by:
Alex Danilo <adanilo@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#753130}
-
Peter Kasting authored
StartMonitoring() starts the thread asynchronously, which it must do since it happens on the UI thread and cannot block. However, the test checks that the tid() is set immediately, which is only valid once the thread starts up. This race leads to test flakiness. We could busy-loop waiting for the thread to start, or use a condition variable, or do various other fixes to ensure that we don't access the tid() until it's really there; but since the test is basically trying to ensure that a second call to StartMonitoring() is a no-op, this just checks that the thread* is non-null and unchanged between the two calls. This is sufficient and should work consistently. Bug: none Change-Id: I672e229ce07d84d04a7cd0aa2eb498a9d903d5cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117053 Auto-Submit: Peter Kasting <pkasting@chromium.org> Commit-Queue: Cheng-Yu Lee <cylee@chromium.org> Reviewed-by:
Cheng-Yu Lee <cylee@chromium.org> Cr-Commit-Position: refs/heads/master@{#753129}
-
Jeff Yoon authored
Forward a --release flag to indicate a release run for iOS test runners when is_debug=false. Bug: 1056887 Change-Id: Ic9941745084376b85938b1b98e1afb01cc254384 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105070 Commit-Queue: Jeff Yoon <jeffyoon@chromium.org> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#753128}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/cdbe45e832f2..8c53b6f52b29 git log cdbe45e832f2..8c53b6f52b29 --date=short --first-parent --format='%ad %ae %s' 2020-03-25 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. Created with: gclient setdep -r src/third_party/devtools-frontend/src@8c53b6f52b29 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: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: I6bde32ac1f22533a17032d09153a372c99b0d164 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119911Reviewed-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@{#753127}
-
Moe Ahmadi authored
NTP realbox matches support different icons for different match types: "loupe" for search suggestions, "clock" for historical search suggestions, "page" for navigation suggestions, "drive icons" for Google Drive, etc. Currently only the "loupe" and "clock" are supported by hardcoding in CSS. This CL changes that by sending the appropriate icon information along with the matches form C++ to JS allowing for a 1:1 mapping between the icons in the NTP realbox and the Omnibox. A followup CL will complete this mapping and add tests to avoid regression in the future. NTP realbox input has a "loupe" icon by default. When a match with a different icon or favicon is selected, the realbox input also gets that icon/favicon. This CL refactors the way these icons/favicons are displayed for the realbox input as well as the matches: Icons are rendered via webkit-mask-image and background-color and favicons are rendered via background-image set on the element's style property in JS. Also instead of requesting favicons for *every* navigation match which results in display the default "page" favicon for matches without one, favicons are now requested in C++ and sent to JS only if one is available; taking advantage of high res SVG icon for matches without a favicon as well as Google Drive matches, calculator matches, etc. Bug: 1039357,1061221 Change-Id: I90e17aa573ba7ef211e3f1423fa7de511cb7e053 Tbr: dcheng@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106655 Commit-Queue: Moe Ahmadi <mahmadi@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Tommy Li <tommycli@chromium.org> Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#753126}
-
CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-betty-pi-arc-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-eve-compile-chrome CQ_INCLUDE_TRYBOTS=luci.chrome.try:chromeos-kevin-compile-chrome BUG=762641 TBR=chrome-os-gardeners@google.com Change-Id: I8cff5fc0261595f5c33f34eb578e9000b921dcb0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119874Reviewed-by:
ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Commit-Queue: ChromeOS bot <3su6n15k.default@developer.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#753125}
-
Nate Fischer authored
This switches Dev UI from Activities to Fragments. The immediate user benefit is smoother transitions when switching between tools, but this opens up for opportunities for other navigation methods (ex. bottom nav bar or nav drawer). Bug: 1017532 Test: Tested system_webview_apk on L+, monochrome on N+, Trichrome on Q Test: Click flag UI notification, verify it opens the flags UI fragment Change-Id: I32eb7a9b2ada6395bad21488e6bd15c802131b7e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108919 Commit-Queue: Nate Fischer <ntfschr@chromium.org> Reviewed-by:
Hazem Ashmawy <hazems@chromium.org> Cr-Commit-Position: refs/heads/master@{#753124}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/7fdb9a49a600..d588fbe8cd72 Created with: gclient setdep -r src-internal@d588fbe8cd72 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:1053589 Tbr: jbudorick@google.com Change-Id: I2e1eb8b074ca79a9d30762063fb6f442232007f4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119812Reviewed-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@{#753123}
-
Minoru Chikamune authored
Add a mode which doesn't reset the mojo connection when the associated ExecutionContext is detached. If the original code doesn't have `ContextDestroyed()`, then HeapMojoWrapperMode::kWithoutContextObserver should be used. Bug: 1058076 Change-Id: I6ea02f1b663b968d70e7b3b475ce40f183592f57 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2114020Reviewed-by:
Keishi Hattori <keishi@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Kouhei Ueno <kouhei@chromium.org> Commit-Queue: Minoru Chikamune <chikamune@chromium.org> Cr-Commit-Position: refs/heads/master@{#753122}
-
Demetrios Papadopoulos authored
Specifically porting: - sync_encryption_options.js - people_page_test.js (split CrOS tests to people_page_test_cros.js) - people_page_sync_controls_test.js - people_page_sync_page_test.js The remaining people_page_sync_page_interactive_test.js will be ported in a separate CL. Bug: 1026426 Change-Id: I8c3446ad645a492e12cc0d6f7df5cf2d0463f3df Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2113708 Auto-Submit: dpapad <dpapad@chromium.org> Commit-Queue: Rebekah Potter <rbpotter@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#753121}
-
zhaoyangli authored
xcodebuild_runner can detect a list of test names in EG2 and Release build target since this change. - Introduced a |release| arg to indicate if this is running on a release build and used the arg in |SimulatorParallelTestRunner|. - Changed functions in shard_util to return a list of test names. - Fixed minor bugs in shard_util finding app path logics. - Updated affected tests. This is tested locally that a correct list of test names can be detected for {Debug, Release} builds * {EG1, EG2} tests. Bug: 1054651 Change-Id: I0d6fbe794c81df9d128e6bc2ca77ceede549ec87 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2079612 Commit-Queue: Zhaoyang Li <zhaoyangli@chromium.org> Reviewed-by:
Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#753120}
-
Takashi Sakamoto authored
Bug: 1056839 Change-Id: I0aa5ac3a20fc5a5cd1a8584e05c07baf59d2fe08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106980 Commit-Queue: Takashi Sakamoto <tasak@google.com> Reviewed-by:
Bartek Nowierski <bartekn@chromium.org> Reviewed-by:
Erik Chen <erikchen@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#753119}
-
Khushal authored
SurfaceControl relies on transaction acks from the framework to manage pipelining/backpressure for frames sent by display compositor to SurfaceFlinger. These are not getting dispatched when Chrome is rendering on an external samsung dex display due to a platform bug which stalls the rendering pipeline. This change adds a workaround to disable SurfaceControl while the Chrome activity is associated with a samsung dex display. R=boliu@chromium.org Bug: 1042581 Change-Id: I38c067f6f7f48cf04c192cc59efe95a4c21fa4ea Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2099602 Commit-Queue: Khushal <khushalsagar@chromium.org> Reviewed-by:
Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#753118}
-
Chromium WPT Sync authored
Using wpt-import in Chromium ccb5f29c. With Chromium commits locally applied on WPT: bb6643e9 "Simplify image-loading-subpixel-clip.html." 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 Directory owners for changes in this CL: drott@chromium.org, kojii@chromium.org: external/wpt/css/css-fonts NOAUTOREVERT=true TBR=robertma No-Export: true Change-Id: If7bb23043d037fa4e57429cbd8c979041d39083d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119551Reviewed-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@{#753117}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/9a378addafad..81c8f4b8027e git log 9a378addafad..81c8f4b8027e --date=short --first-parent --format='%ad %ae %s' 2020-03-25 brianosman@google.com Revert "Switch sRGB color filter to use GrColorSpaceXformEffect" 2020-03-25 brianosman@google.com Switch sRGB color filter to use GrColorSpaceXformEffect Created with: gclient setdep -r src/third_party/skia@81c8f4b8027e 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 robertphillips@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: robertphillips@google.com Change-Id: I982cf30d910a65409681f48ccb0829f507e8a4f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119195Reviewed-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@{#753116}
-
nancy authored
Launch_service will be integrated to AppService, and launch_service will be removed. So modify related components to use app_service/launch_utils. BUG=1061843 Change-Id: If767a4e91e50cdbc0ec0e5805b88f979a0acfcab Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2115856Reviewed-by:
Scott Violet <sky@chromium.org> Reviewed-by:
Istiaque Ahmed <lazyboy@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#753115}
-
Martin Kreichgauer authored
The kU2fCryptotoken{Register,Sign} WebFeature counters record when a page makes a request to the CryptoToken component extension via the U2F API in order to register or sign with a security key credential. Change-Id: I2ea19f46da9028da000e0e5564a7ade242d9d3ff Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2118851 Auto-Submit: Martin Kreichgauer <martinkr@google.com> Reviewed-by:
Steven Holte <holte@chromium.org> Commit-Queue: Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#753114}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e3a49aa40576..7c5efb2a7e2b git log e3a49aa40576..7c5efb2a7e2b --date=short --first-parent --format='%ad %ae %s' 2020-03-25 tiborg@chromium.org Add format string option to git cl archive 2020-03-25 ehmaldonado@chromium.org Revert "git-cl: Remove unused and duplicate functions." Created with: gclient setdep -r src/third_party/depot_tools@7c5efb2a7e2b 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 Bug: None Tbr: agable@chromium.org Change-Id: Ie6a5e5ee15f382f4013ed7dcc7db787b096b7f71 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119197Reviewed-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@{#753113}
-
John Chen authored
Change-Id: I16aaa1acdbb1cade2bb2f1d579dee77d4c9b7e75 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2118393Reviewed-by:
Caleb Rouleau <crouleau@chromium.org> Commit-Queue: John Chen <johnchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#753112}
-
David Tseng authored
TBR=dtseng@chromium.org Fixed: 1033649 Change-Id: I713ef317499a7ae027ddf63594ee5e8948ede8a1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119265Reviewed-by:
David Tseng <dtseng@chromium.org> Commit-Queue: David Tseng <dtseng@chromium.org> Cr-Commit-Position: refs/heads/master@{#753111}
-
chrome-release-bot authored
# This is an automated release commit. # Do not revert without consulting chrome-pmo@google.com. NOAUTOREVERT=true TBR=govind@chromium.org Change-Id: Ia61ce947694855b305fa92b2070abbe67f3c1eb6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119152Reviewed-by:
Chrome Release Bot (LUCI) <chrome-official-brancher@chops-service-accounts.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#753110}
-
Peter Kasting authored
This also gets rid of event_sink() entirely, since callers can easily get it from the root window. This is in preparation for making AshTestHelper a subclass of AuraTestHelper, which will require these to be virtual. I did not rename the transitive closure of caller getters to ensure everything everywhere was renamed -- that would have required hundreds and hundreds of changes and didn't seem worth it. Bug: none Change-Id: Ief901ed0ffd733a553bfccb445c96e920be53499 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117348 Commit-Queue: Peter Kasting <pkasting@chromium.org> Auto-Submit: Peter Kasting <pkasting@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#753109}
-
Luciano Pacheco authored
Change FilesTooltip to hide the tooltip when the window is resized. Fixes issue 989456, where the tooltip could be misplaced after the window resizing. This patch is due to Giovanni Panaro and his work to resolve this issue, see CL:1743090. Test: browser_tests --gtest_filter="*filesTooltipHidesOnWindowResize*" Bug: 989456 Change-Id: I1a78b047e2c18ff79fa54e6e2ad58d4a1e804315 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119290 Commit-Queue: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Noel Gordon <noel@chromium.org> Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Cr-Commit-Position: refs/heads/master@{#753108}
-
Simon Que authored
Previously reverted: http://crrev.com/c/2098851 Relanding because of changes to iOS build that may have fixed the previously observed build failures. We no longer see intermittent failures in the Chromium WebRTC FYI builders that were migrated: https://ci.chromium.org/p/chromium/g/chromium.webrtc.fyi See crbug.com/1060666 In addition, some builders have been removed from this CL since it was previously landed. Those builders have been migrated by jeffyoon@ to mb_config in http://crrev.com/c/2071293. They will be migrated to Goma RBE in a separate CL: - chromium.fyi/ios-webkit-tot - chromium.fyi/ios13-beta-simulator - chromium.fyi/ios13-sdk-device ============================ These are the remaining iOS builders in Chromium. They are expected to have under 150 QPS at peak: https://docs.google.com/spreadsheets/d/1-XKh-CqSoUcXEqCETVywjmIo1C18P0qEBDCY50ozBb0/edit#gid=555424423 Schedule of migration: https://docs.google.com/spreadsheets/d/1TZhpasm6iceYm7zqkyEtZ1wn9ZcpREfH4LV4cusICJA/edit#gid=0 R=agable@chromium.org, smut@google.com Bug: 950413 Change-Id: Ia8447eae2409eb616c4ca4b9dcb921baf35fa468 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2117552 Commit-Queue: Simon Que <sque@chromium.org> Reviewed-by:
Takuto Ikuta <tikuta@chromium.org> Reviewed-by:
Aaron Gable <agable@chromium.org> Reviewed-by:
smut <smut@google.com> Cr-Commit-Position: refs/heads/master@{#753107}
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/e39d055d085a..d0800d29473f git log e39d055d085a..d0800d29473f --date=short --first-parent --format='%ad %ae %s' 2020-03-24 ianelliott@google.com Vulkan: Fix crashes/failures in array-of-struct uniforms 2020-03-24 jmadill@chromium.org Speculative fix for assertion failure with samplers. Created with: gclient setdep -r src/third_party/angle@d0800d29473f 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 cnorthrop@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: cnorthrop@google.com Change-Id: I99f7ef8a4627091c1a3d162469627451d736de9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2119193Reviewed-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@{#753106}
-