- 19 Oct, 2020 40 commits
-
-
Luciano Pacheco authored
When the CSS is in-lined in the JS as part of generating the JS module the "\" escaping doesn't work. To workaround this problem we replace the escaping unicode code with the actual UTF-8 char. Note that the code "\00a0" is actually a non-break space and it was replaced with a normal blank space, however the style used doesn't break and this is an old style used in pre-GM2. Note 2: The problem only affects styles in the HTML for Polymer elements, but for consistency, I'm replacing in all files. Bug: 1133186, 1130413 Change-Id: I69fc315a6f4e8b96564b2250b2f8510fed55e307 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2470037 Auto-Submit: Luciano Pacheco <lucmult@chromium.org> Reviewed-by:
Alex Danilo <adanilo@chromium.org> Commit-Queue: Alex Danilo <adanilo@chromium.org> Cr-Commit-Position: refs/heads/master@{#818385}
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/87dd38763359..91f276e903b3 2020-10-19 jleconte@google.com Fix pop up statement when triaging with an existing bug. 2020-10-19 jleconte@google.com Move Monorail projects list from Python to Javascript. 2020-10-19 dberris@chromium.org pinpoint: Handle potentially missing timestamps 2020-10-19 dberris@chromium.org pinpoint: Fix handling of Monorail issue response If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC wenbinzhang@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:chromeos-kevin-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:1133649,chromium:1134978 Tbr: wenbinzhang@google.com Change-Id: I9da365cf89b5ad387d845b088b4cd51c5a9c0418 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483772Reviewed-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@{#818384}
-
Omer Katz authored
Using just the suffix results in applying the clang plugin to Oilpan source code if the chromium checkout resides (transitively) under a directory named "test". In such cases, chromium fails to build. Bug: 1137252 Change-Id: I6a0e7fb5e175c9f33fe7c79ca7f5790b69eb34ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2480569Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#818383}
-
Keishi Hattori authored
Detects invalid freelist_head by seeing if the super page matches. Bug: 1137799 Change-Id: If6f2fb0736a426a14f4f098e0f75eaa6b920fe5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2476876Reviewed-by:
Benoit L <lizeb@chromium.org> Commit-Queue: Keishi Hattori <keishi@chromium.org> Cr-Commit-Position: refs/heads/master@{#818382}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/1400dda5de60..260029435baa 2020-10-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll Chromium from 17855e1b to cb9deed9 (555 revisions) 2020-10-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll ANGLE from b4fb7cc9ec8f to 2be35682cd67 (25 revisions) 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 egdaniel@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: None Tbr: egdaniel@google.com Test: Test: Test: Free Fire MECTest: Test: GLES31/functional_android_extension_pack_extensions_ext_texture_srgb_decodeTest: Test: GLES31/functional_debug_negative_coverage_*_srgb_decode_*Test: Test: GLES31/functional_state_query_sampler_*_srgb_decode_*Test: Test: GLES31/functional_state_query_texture_*_srgb_decode_*Test: Test: PUBG MECTest: Test: angle_end2end_testsTest: Test: angle_end2end_tests --gtest_filter=SRGBTextureTest.*Vulkan*Test: Test: dEQP.GLES31/functional_srgb_texture_decode_skip_decode_* Change-Id: Ia38ca4128f1de8bb0679a3d8d7174b59ac3051ae Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484304Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/64d02136b235..c7e96aec1132 2020-10-19 annaagoha@gmail.com fix uneven outline and padding issue 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/+doc/master/autoroll/README.md Bug: chromium:1137347 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ib3952714f1a331146ed997de5bf1ed5a30a0ba5c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483890Reviewed-by:
-
Melissa Zhang authored
This reverts commit f524b7f0. Reason for revert: I will add a simple fix. Original change's description: > Revert "Change ButtonPressed overrides to callbacks: c/b/ui/views/sharesheet/" > > This reverts commit 1ac9a1da. > > Reason for revert: Breaking UI > > Original change's description: > > Change ButtonPressed overrides to callbacks: c/b/ui/views/sharesheet/ > > > > This allows converting |targets_| from a member to a parameter that's > > passed around. I also did some other cleanup while there. > > > > Bug: 772945 > > Change-Id: Iaa7f70594ba109d1598da3566856e90c8b09629d > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2481246 > > Commit-Queue: Melissa Zhang <melzhang@chromium.org> > > Reviewed-by: Melissa Zhang <melzhang@chromium.org> > > Auto-Submit: Peter Kasting <pkasting@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#818324} > > TBR=pkasting@chromium.org,melzhang@chromium.org > > Change-Id: Ief930a27f44caef219bf782a7534ef8bc76d83ef > No-Presubmit: true > No-Tree-Checks: true > No-Try: true > Bug: 772945 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483745 > Reviewed-by: Melissa Zhang <melzhang@chromium.org> > Commit-Queue: Melissa Zhang <melzhang@chromium.org> > Cr-Commit-Position: refs/heads/master@{#818357} TBR=pkasting@chromium.org,melzhang@chromium.org # Not skipping CQ checks because this is a reland. Bug: 772945 Change-Id: I00d7acd3d7c812dfde8b584c2e53b6b2e1c237a7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483987Reviewed-by:
Melissa Zhang <melzhang@chromium.org> Commit-Queue: Melissa Zhang <melzhang@chromium.org> Cr-Commit-Position: refs/heads/master@{#818379}
-
Daniel Ng authored
This object allows us to queue up events to be run once we observe that the borealis vm has started (or until a specified timeout). Tests: Built and ran tests. Bug: b/162562622 Change-Id: Ieaee96af72bc7fc41b90a48111ef8998af6cdf97 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2473781Reviewed-by:
Nic Hollingum <hollingum@google.com> Commit-Queue: Daniel Ng <danielng@google.com> Cr-Commit-Position: refs/heads/master@{#818378}
-
Tomasz Tylenda authored
We use base::RepeatingClosure and base::Bindepeating for attempt_user_exit_callback_. In other locations we convert Bind to BindOnce, because callers require OnceCallback. BUG=764795 TESTED=build Change-Id: I35172b00c4ccb37383b2adb55d44e8569b129dbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483547Reviewed-by:
Hidehiko Abe <hidehiko@chromium.org> Commit-Queue: Tomasz Tylenda <ttylenda@chromium.org> Cr-Commit-Position: refs/heads/master@{#818377}
-
David Jean authored
Add enum histogram for ios multi-window configuration recording. Each minute one bucket from the enum will be increased by 1. Code is in: https://chromium-review.googlesource.com/c/chromium/src/+/2445415 Bug: 1134568 Change-Id: I418120cc33c8f866b1dea4c2800493160179fcb5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2445491Reviewed-by:
Ilya Sherman <isherman@chromium.org> Reviewed-by:
Mark Cogan <marq@chromium.org> Commit-Queue: David Jean <djean@chromium.org> Cr-Commit-Position: refs/heads/master@{#818376}
-
Jiewei Qian authored
This is a cleanup after BMO is launched. Bug: 1065748 Change-Id: Ib5af0334ca9f9a9f11419fc774d0cd1adfc4307f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2467040 Commit-Queue: Jiewei Qian <qjw@chromium.org> Reviewed-by:
Giovanni Ortuño Urquidi <ortuno@chromium.org> Cr-Commit-Position: refs/heads/master@{#818375}
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/b74a39b2dc30..99f4c7607c8c 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 aboxhall@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: None Tbr: aboxhall@google.com Change-Id: I2114e33a156d7c88738f71d1fa41bcc0cd88246d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483888Reviewed-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@{#818374}
-
sandromaggi authored
This CL adds reporting for which web-action failed. This is useful for actions that execute in multiple steps, e.g. |ClickAction|: * WaitForDocumentToBecomeInteractive * ScrollIntoView * ClickOrTap Bug: b/169924567 Change-Id: I644178737fc21039767670ffcb5853d794ab754b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2475115 Commit-Queue: Sandro Maggi <sandromaggi@google.com> Reviewed-by:
Stephane Zermatten <szermatt@chromium.org> Cr-Commit-Position: refs/heads/master@{#818373}
-
Jimmy Gong authored
- Crash was due to attempting to remove an previously removed notification. - The fix is to verify that the notification_id is a valid id to remove. Bug: 1139561, 1106937 Test: chromeos_components_unittests, ash_unittests Change-Id: Ia606df1dfec13beba9ccc14c2618ec4f6fda7129 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2482428Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Jimmy Gong <jimmyxgong@chromium.org> Cr-Commit-Position: refs/heads/master@{#818372}
-
Kent Tamura authored
- Introduce helper functions to share common code - Use TextControlElement instead of HTMLInputElement This will help testing LayoutTextControlMultiLine. - Fix camelCase variable names This CL has no behavior changes. Bug: 1040826 Change-Id: I4287125d4531151573d839363d182ee2d8d1409e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483741Reviewed-by:
Koji Ishii <kojii@chromium.org> Commit-Queue: Kent Tamura <tkent@chromium.org> Cr-Commit-Position: refs/heads/master@{#818371}
-
Koji Ishii authored
This patch enables the Minikin hyphenation engine on all platforms except Mac. Mac keeps using the engine in the OS. Intent to ship: https://groups.google.com/a/chromium.org/g/blink-dev/c/sIRXYKA4jGE/m/44YfDTNkAwAJ Test expectation changes are: * Linux/Win support `hyphens: auto` as a valid CSS. * Unskip tests that require `hyphens: auto` on Linux/Win. * Some Mac-only failures now fail on all platforms. * Triaged 1 new failure on Linux and 2 on Windows. The causes are existing ones, but results vary by platform as tests rely on fonts and dictionaries. I'll review existing issues. Note, |web_tests| and |content_shell| use the dictionaries in the repository (r818216 crrev.com/c/2478122), while in Chromium, the [component updater] downloads the dictionaries (r816137 crrev.com/c/2437314) and passes to the engine when they become ready (r817902 crrev.com/c/2462893). [component updater]: https://chromium.googlesource.com/chromium/src/+/master/components/component_updater/README.md Bug: 652964 Change-Id: I82e3df40ea59256aa61de24bce4be39870e85adb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2315564Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Yoshifumi Inoue <yosin@chromium.org> Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#818370}
-
Kent Tamura authored
The only callsite of CommitPendingEndOverhang() is NGLineBreaker::HandleText(), and it is called for NGInlineItem::kControl for a TAB character as well as NGInlineItem::kText. So, CommitPendingEndOverhang() should take care of NGInlineItem::kControl. This CL updates CommitPendingEndOverhang() so that a ruby annotation text doesn't hang over a TAB character. This CL also adds a stream printer for blink::NGLineInfo. Bug: 1138943 Change-Id: I478e93086f449b8e4cda4c8828d34f4111d80ea6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483739 Auto-Submit: Kent Tamura <tkent@chromium.org> Commit-Queue: Koji Ishii <kojii@chromium.org> Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/958dc6267f2d..8ecad5261a1f 2020-10-19 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: I27ba5cd0ef6c0a82951ee3c6c827dbd410c604ee Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483532Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/bd239eff..93ed90e6 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: I4ef4af28db36b9f97e0a9bda805ad312f0b934da Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484282Reviewed-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@{#818367}
-
Hiroshige Hayashizaki authored
This CL migrates most of import maps resolution tests by observing the resolution results by intercepting module script requests by a service worker. This CL also introduces `useInternalMethods` flag to clarify the tests still requiring internal methods. Bug: 1026809 Change-Id: I16c2a87bb67b530dc97b1631f6968b2d3bafdac6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2457526 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/master@{#818366}
-
chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/2f56cfe7fc9e..1105ef875a93 2020-10-19 pdfium-autoroll@skia-public.iam.gserviceaccount.com Roll Catapult from 0de987471899 to 89eeef5150cb (76 revisions) If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pdfium-autoroll Please CC pdfium-deps-rolls@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/+doc/master/autoroll/README.md Bug: None Tbr: pdfium-deps-rolls@chromium.org Change-Id: Ie72b91ed5d6924a01e99fd12dfffcd7aa3b3c918 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483886Reviewed-by:
-
Alan Cutter authored
This CL updates AppBannerManager methods: - ShouldDeferToRelatedApplication => ShouldDeferToRelatedNonWebApp - IsSupportedAppPlatform => IsSupportedNonWebAppPlatform - IsRelatedAppInstalled => IsRelatedNonWebAppInstalled to be more clear that they're checking for related non-web apps. This distinction is important now that we have the chromeos_play platform which is used to list web apps on the Play store. Change-Id: I0014b4977078ee46a97b11777b442506b5b2d566 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483712 Commit-Queue: Alan Cutter <alancutter@chromium.org> Reviewed-by:
Dominick Ng <dominickn@chromium.org> Auto-Submit: Alan Cutter <alancutter@chromium.org> Cr-Commit-Position: refs/heads/master@{#818364}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/a5e8f6b40233..1400dda5de60 2020-10-19 skia-autoroll@skia-public.iam.gserviceaccount.com Roll SwiftShader from 04bd4d8e4ece to df17a76102df (2 revisions) 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 egdaniel@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: None Tbr: egdaniel@google.com Change-Id: I994be689013cf27e166a8ca605546651df3cb81d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483529Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/catapult.git/+log/89eeef5150cb..87dd38763359 2020-10-19 dberris@chromium.org chromeperf: Remove Mac testing from presubmit If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/catapult-autoroll Please CC wenbinzhang@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:chromeos-kevin-rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel Bug: chromium:1132884 Tbr: wenbinzhang@google.com Change-Id: I8edece296ed84952c990c0023305050704df2dc7 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483527Reviewed-by:
-
Jérémie Boulic authored
Tests crrev.com/c/2423448. Bug: chromium:912236 Test: browser_tests --gtest_filter=FileManagerJsTest.FileTasks Change-Id: Iefca4742d6fb154c5c1a392d0da14af796a792a0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2467162Reviewed-by:
Luciano Pacheco <lucmult@chromium.org> Commit-Queue: Jeremie Boulic <jboulic@chromium.org> Cr-Commit-Position: refs/heads/master@{#818361}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/a9da93c46a90..64d02136b235 2020-10-19 devtools-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com Update DevTools DEPS. 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/+doc/master/autoroll/README.md Bug: None Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ib348125d88c8c2a951dba18b491af12b0bad1625 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484126Reviewed-by:
-
Shik Chen authored
This CL updates the version for env and parser, and removes some unnecessary globals in the config. Bug: 1139102 Test: Presubmit checks Change-Id: I9b916b0a3a7fa32a744b84ef78bcd333efa85e04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483736 Auto-Submit: Shik Chen <shik@chromium.org> Commit-Queue: Inker Kuo <inker@chromium.org> Reviewed-by:
Inker Kuo <inker@chromium.org> Cr-Commit-Position: refs/heads/master@{#818359}
-
Christopher Lam authored
Bug: 1110666 Change-Id: I5968ba3db606fb07e15bf8972556a80642e8c8db Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483715Reviewed-by:
Jiewei Qian <qjw@chromium.org> Commit-Queue: calamity <calamity@chromium.org> Cr-Commit-Position: refs/heads/master@{#818358}
-
Melissa Zhang authored
This reverts commit 1ac9a1da. Reason for revert: Breaking UI Original change's description: > Change ButtonPressed overrides to callbacks: c/b/ui/views/sharesheet/ > > This allows converting |targets_| from a member to a parameter that's > passed around. I also did some other cleanup while there. > > Bug: 772945 > Change-Id: Iaa7f70594ba109d1598da3566856e90c8b09629d > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2481246 > Commit-Queue: Melissa Zhang <melzhang@chromium.org> > Reviewed-by: Melissa Zhang <melzhang@chromium.org> > Auto-Submit: Peter Kasting <pkasting@chromium.org> > Cr-Commit-Position: refs/heads/master@{#818324} TBR=pkasting@chromium.org,melzhang@chromium.org Change-Id: Ief930a27f44caef219bf782a7534ef8bc76d83ef No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 772945 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483745Reviewed-by:
-
Ramin Halavati authored
A function is added to create Guest test browsers and existing use cases are updated. Bug: 1125474, 1061637 Change-Id: I56c8a11d409f5b0fb2a6baf25447f4e7c535b644 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2474697Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Cr-Commit-Position: refs/heads/master@{#818356}
-
Xianzhu Wang authored
This reverts commit bc2f9ba0. Reason for revert: The test is still flaky. Original change's description: > Reenable scroll-animations/element-based-offset tests > > The underlying source of the test flakes was likely addressed in > recent changes to the handling of play and pause for scroll-linked > animations. > > https://chromium-review.googlesource.com/c/chromium/src/+/2386339 > > This patch also contains a cleanup to remove the unnecessarily > convoluted way of hiding the scrollbar for the clamped offset test. > > Tested with 1000+ retries of the two tests in a virtual threaded > environment. > > Bug: 1080609 > Change-Id: If9097b0a2017a9c87a359b61ba93f35bae56b512 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2475935 > Reviewed-by: Xida Chen <xidachen@chromium.org> > Commit-Queue: Kevin Ellis <kevers@chromium.org> > Cr-Commit-Position: refs/heads/master@{#817568} TBR=kevers@chromium.org,xidachen@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1080609 Change-Id: Ic31eede5561182d78e7bf805cd1f112edfd5379c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483585Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#818355}
-
Alice Boxhall authored
TBR=joonbug@chromium.org Bug: 1139523 Change-Id: I03307649b4d5eb833650d578313e3a37645fe947 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484024Reviewed-by:
Alice Boxhall <aboxhall@chromium.org> Commit-Queue: Alice Boxhall <aboxhall@chromium.org> Cr-Commit-Position: refs/heads/master@{#818354}
-
Adam Rice authored
The WenSocket web_tests connect-error-by-no-websocket-server-closeonclose.html connect-error-by-no-websocket-server-closeonerror.html connect-error-by-no-websocket-server-noclose.html have been failing on Windows 10 1909. In this version of Windows the delay on connecting to a closed localhost port has increased from 1 second to 3 seconds. Adjust the timeout in the tests to match. BUG=1132194 Change-Id: I11b69ea3ed15daa9ab9dff2ce22672c706867da2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483717 Commit-Queue: Yoichi Osato <yoichio@chromium.org> Auto-Submit: Adam Rice <ricea@chromium.org> Reviewed-by:
Yoichi Osato <yoichio@chromium.org> Cr-Commit-Position: refs/heads/master@{#818353}
-
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/fuchsia-sdk-chromium-autoroll Please CC cr-fuchsia+bot@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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-x64-cast Tbr: cr-fuchsia+bot@chromium.org Change-Id: Iaf5562a64bf812cf289f37288538b1e78e91961b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483884Reviewed-by:
-
Noel Gordon authored
Instead of "fake_test_name", collect coverage for the given test_name, which would be the browser test fixture name in practice. Tbr: benreich Bug: 1113941 Change-Id: Ib6b4a30cc09a37def86d3f0267a2596cda370602 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483731Reviewed-by:
Noel Gordon <noel@chromium.org> Commit-Queue: Noel Gordon <noel@chromium.org> Cr-Commit-Position: refs/heads/master@{#818351}
-
Nicholas Hollingum authored
This infrastructure is being built, but for now we can proceed without it. Until it is completed, users may have to retry actions while the vm is actually still starting up. Bug: b/162562622 Change-Id: I093647b931b8d33ba598054d29aea3b69e8aa670 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2472679 Commit-Queue: Nic Hollingum <hollingum@google.com> Reviewed-by:
Daniel Ng <danielng@google.com> Cr-Commit-Position: refs/heads/master@{#818350}
-
Noel Gordon authored
Add a using directive: a C++ shorthand to describe the contents of the of agent hosts to devtools listener map. Tbr: benreich Bug: 1113941 Change-Id: Iea4b34b056654315d8e0e1382f584326b50122a8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2484027Reviewed-by:
-
Shik Chen authored
Bug: 1139102 Test: Pass ESLint with https://crrev.com/c/2478884 Change-Id: I2a443b1e13ab72bb24616acb10cb191564ff71b4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2480102 Commit-Queue: Shik Chen <shik@chromium.org> Commit-Queue: Inker Kuo <inker@chromium.org> Reviewed-by:
-
Wei Lee authored
This CL fixes a bug which will happen when renaming a folder name which is displayed with its i18n name. The underlying folder name will be unexpectedly renamed to its i18n name even when there is no actual modification for rename input. Bug: 1139205 Test: Manually Change-Id: I6059b7158bc01a457ca7888572c40ffdf0fa6413 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2477056Reviewed-by:
-
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/4394032b4d4b..b74a39b2dc30 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 aboxhall@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: None Tbr: aboxhall@google.com Change-Id: I2390f5ced4027d1e85ed9c589bff89d835dae731 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2483883Reviewed-by:
-
