- 18 Mar, 2020 40 commits
-
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/b988e0398277..83e138ca962c git log b988e0398277..83e138ca962c --date=short --first-parent --format='%ad %ae %s' 2020-03-18 enga@chromium.org D3D12: Fix usage of CopyResource Created with: gclient setdep -r src/third_party/dawn@83e138ca962c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/dawn-chromium-autoroll Please CC cwallez@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:dawn-linux-x64-deps-rel;luci.chromium.try:dawn-mac-x64-deps-rel;luci.chromium.try:dawn-win10-x64-deps-rel;luci.chromium.try:dawn-win10-x86-deps-rel Bug: None Tbr: cwallez@google.com Change-Id: I1eedfe78678bcde97eebb2036b3d022e7f3d3d1a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108108Reviewed-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@{#751369}
-
Ian Kilpatrick authored
Instead of separate MinPreferredLogicalWidth/MaxPreferredLogicalWidth this introduces PreferredLogicalWidths to return the pair. There should be no other behaviour changes. Bug: 845235 Change-Id: I66b3cd1eb8eb5df41966e6ae06944651ac2baa51 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097124 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Morten Stenshorne <mstensho@chromium.org> Cr-Commit-Position: refs/heads/master@{#751368}
-
Yann Dago authored
When validating the url of some tabs, they were not necessarily loaded. We now wait for them to be loaded before checks. Bug: 958893 Change-Id: I17f83ec19c790224fc479632df35769da72b7a5a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107060 Commit-Queue: Yann Dago <ydago@chromium.org> Reviewed-by:
Greg Thompson <grt@chromium.org> Cr-Commit-Position: refs/heads/master@{#751367}
-
Peng Huang authored
VulkanImage::CreateWithExternalMemory() is for creating a VkImage with memory which can be exported and used by foreign API. This method also uses vkGetPhysicalDeviceImageFormatProperties2() to query supported handle types instead of hardcoding in the VulkanImplementation. Bug: 848385 Change-Id: Ic8d93798b72e5c7a157c1fa924bcf3f3d15c275c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107704 Commit-Queue: Peng Huang <penghuang@chromium.org> Reviewed-by:
Vasiliy Telezhnikov <vasilyt@chromium.org> Cr-Commit-Position: refs/heads/master@{#751366}
-
Zhenyao Mo authored
Otherwise, all tests except the first one display an error message. BUG=1026447 TEST=viz_perftests R=backer@chromium.org Change-Id: I4e71979592debbc21e3d8e234318f1f4788b4de4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108226Reviewed-by:
Jonathan Backer <backer@chromium.org> Commit-Queue: Zhenyao Mo <zmo@chromium.org> Cr-Commit-Position: refs/heads/master@{#751365}
-
Sam Sebree authored
This change uses the -internal-light-dark-color css property to set correct dark mode styling on the button native controls. Bug: 929098 Change-Id: I15e68cda2a07bb89601c43ef1dc8ce696c75b1ce Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097318Reviewed-by:
Kent Tamura <tkent@chromium.org> Reviewed-by:
Ionel Popescu <iopopesc@microsoft.com> Reviewed-by:
Rune Lillesveen <futhark@chromium.org> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Mason Freed <masonfreed@chromium.org> Commit-Queue: Sam Sebree <sasebree@microsoft.com> Cr-Commit-Position: refs/heads/master@{#751364}
-
Tanmoy Mollik authored
Add a test to check the signIn method in SigninManagerTest Bug: 1059964 Change-Id: I953d4344fc50b892b793d6045aa7bf77d246141c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2095128Reviewed-by:
Alice Wang <aliceywang@chromium.org> Commit-Queue: Tanmoy Mollik <triploblastic@chromium.org> Cr-Commit-Position: refs/heads/master@{#751363}
-
Elly Fong-Jones authored
Change-Id: Ic5d667ad032cc8c6aca9043ccdc5cb892c0bdff7 Tbr: groby@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2101314Reviewed-by:
Rachel Blum <groby@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#751362}
-
chromium-autoroll authored
https://swiftshader.googlesource.com/SwiftShader.git/+log/2aa80df14528..dc0f1319307e git log 2aa80df14528..dc0f1319307e --date=short --first-parent --format='%ad %ae %s' 2020-03-18 amaiorano@google.com Subzero: fix partially initialized ManagedStatic assert Created with: gclient setdep -r src/third_party/swiftshader@dc0f1319307e If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/swiftshader-chromium-autoroll Please CC swiftshader-team+autoroll@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_chromium_msan_rel_ng;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:151653536 Tbr: swiftshader-team+autoroll@google.com Change-Id: I8b8ce125ca885c253c4e612d4ae3b63a6f46db99 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108105Reviewed-by:
-
Dan H authored
This reverts commit 57098dce. Reason for revert: Causing test failures: https://bugs.chromium.org/p/chromium/issues/detail?id=1062658 Original change's description: > Better coordinate swipe to home gesture and drag handle nudge > > Manages the drag handle nudge during window drag from shelf to home or > overview (which may end up in a gesture the nudge is describing). > > If the contextual nudge is shown when the window drag starts, the nudge > will not be hidden until the gesture completes (end it will get hidden > when the gesture completes, even if the user does not end up going > home). > > If the contextual nudge is scheduled to be shown when the gesture > starts, the show request will be canceled. > > Adds logic that prevents drag handle nudge from showing up, unless it's > a result of an explicit user action - tapping the drag handle. > > BUG=1058617 > > Change-Id: I96fa2748308143ec42295dbfb472dc564da5bd03 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107135 > Commit-Queue: Toni Baržić <tbarzic@chromium.org> > Reviewed-by: Manu Cornet <manucornet@chromium.org> > Cr-Commit-Position: refs/heads/master@{#751200} TBR=tbarzic@chromium.org,manucornet@chromium.org,yulunwu@chromium.org Change-Id: I361894e79bf7bb027f690f5a6310874341d51ecf No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 1058617 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108525Reviewed-by:
Dan H <harringtond@chromium.org> Commit-Queue: Dan H <harringtond@chromium.org> Cr-Commit-Position: refs/heads/master@{#751360}
-
Justin Miron authored
Current Popup_Page UKM is recorded when the web contents is destroyed, this can lead to recording pop-up settings for source ids different than the pop-up's opener source id. This changelist also adds browsertests for logging of the UKM and makes the ukm for Popup_Page singular as it should be. BUG=1044601 Change-Id: I241a520b61bda691ce61ab4fdbac875f3397ac08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2087975 Commit-Queue: Justin Miron <justinmiron@google.com> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#751359}
-
Ian Kilpatrick authored
This patch changes ComputeIntrinsicLogicalWidths in two ways: 1) Makes it return by value from changing the type from/to: void ComputeIntrinsicLogicalWidths(LayoutUnit&, LayoutUnit&); MinMaxSizes ComputeIntrinsicLogicalWidths(); 2) Makes ComputeIntrinsicLogicalWidths return the border-box sizes instead of the content-box size. Bug: 845235 Change-Id: If2c2da92fdf7d1173548504dc4785afeb1a2dada Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2088219 Commit-Queue: Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by: -
chromium-internal-autoroll authored
https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/64472dfcc5d1..d390847cf65f Created with: gclient setdep -r src-internal@d390847cf65f 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:1024586,chromium:1024588,chromium:1045456,chromium:1055606,chromium:1058597,chromium:1062473 Tbr: jbudorick@google.com Change-Id: I29cd9a786d7a356beb5aef33ee17d05233bf5951 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108104Reviewed-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@{#751357}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/33e164438ff9..8561f1e80744 git log 33e164438ff9..8561f1e80744 --date=short --first-parent --format='%ad %ae %s' 2020-03-18 lalitm@google.com tp: ingest args from json events Created with: gclient setdep -r src/third_party/perfetto@8561f1e80744 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-chromium-autoroll Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: chromium:132559404 Tbr: perfetto-bugs@google.com Change-Id: I6559dcd5bae7f28737a1aade2463f8b079d4c484 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108102Reviewed-by:
-
Francois Beaufort authored
On KitKat, vector drawables are not supported in chromium permissions prompt. This CL fixes this issue by replacing the unique vector drawable for NFC infobar icon to multiples images. Bug: 1061916 Change-Id: Id96e7e1b496f2849b7d3bfdbb1e81db78e9c793f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106164Reviewed-by:
Reilly Grant <reillyg@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Reviewed-by:
Andrew Grieve <agrieve@chromium.org> Commit-Queue: François Beaufort <beaufort.francois@gmail.com> Cr-Commit-Position: refs/heads/master@{#751355}
-
Patrick Monette authored
Change-Id: Ibf60d1dc438745f10853f5a02e0a92778768b91b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106323Reviewed-by:
Sigurður Ásgeirsson <siggi@chromium.org> Commit-Queue: Patrick Monette <pmonette@chromium.org> Cr-Commit-Position: refs/heads/master@{#751354}
-
Omer Katz authored
Tracing to capacity means we don't need to check size_ during tracing anymore. Bug: 986235 Change-Id: Idae046fcbb6a1705c1522d56479d560091415ea4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107983Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Michael Lippautz <mlippautz@chromium.org> Commit-Queue: Omer Katz <omerkatz@chromium.org> Cr-Commit-Position: refs/heads/master@{#751353}
-
Anna Malova authored
We rely on preferred-color-scheme is set to no-preferences if UA darkening is turned on, so we do not accidentally darken twice. In prefer-web-theme-over-UA-darkening mode (apply web theme if available, otherwise apply UA darkening) we need to ensure that preferred-color-scheme is correctly set for each loaded webpage. Change-Id: Id3246269f5fb9336ff5a924a95e4768420aefe8c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105376 Commit-Queue: Anna Malova <amalova@chromium.org> Reviewed-by:
Tobias Sargeant <tobiasjs@chromium.org> Cr-Commit-Position: refs/heads/master@{#751352}
-
manukh authored
Enabled by default since m79 (crrev.com/c/1782674). Bug: 856335 Change-Id: I2842b63a13545725cf97aaa0cd156e8a87ea0e56 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2065330Reviewed-by:
Tommy Li <tommycli@chromium.org> Commit-Queue: manuk hovanesian <manukh@chromium.org> Cr-Commit-Position: refs/heads/master@{#751351}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/e5c9f6d10689..4c07605f8d55 git log e5c9f6d10689..4c07605f8d55 --date=short --first-parent --format='%ad %ae %s' 2020-03-18 hbos@webrtc.org [Adaptation] VideoStreamAdapter unit tests added. Created with: gclient setdep -r src/third_party/webrtc@4c07605f8d55 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/+/master/autoroll/README.md Bug: None Tbr: webrtc-chromium-sheriffs-robots@google.com Change-Id: I9ab3e2a1e341f689f8414a07fdc9e0d5f5fc90c5 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108101Reviewed-by:
-
ckitagawa authored
In rare cases it was possible a renderer vanished taking the PaintPreviewDocumentData with it before the PaintPreviewClient was destroyed. This resulted in the invocation of an empty callback causing a crash. This CL audits all use of document_data to ensure it is safe. Bug: 1062434 Change-Id: Ie5e800755902f66d5c6b3a46f956fb522bde97f0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106383Reviewed-by:
Mehran Mahmoudi <mahmoudi@chromium.org> Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#751349}
-
Mustaq Ahmed authored
This CL fixes/refactors existing tests to prepare for new WPTs for UAv2. - Fix a test name suffix. - Fix github issue number. - Make postMessage format consistent. - Clarify a child-frame name. - Replace parent refs to "top" to allow deep iframes. Change-Id: I9da97ee827d6556116eab9d592e44665dc71d4c1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106538Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#751348}
-
Francois Doray authored
MessagePump::Delegate::DoWork/DoDelayedWork are being removed in favor of DoSomeWork. This CL updates MessagePumpFuchsia to use DoSomeWork to unblock that migration. Bug: 885371 Change-Id: Ibe7ca894ac866e3287f301f70505195a2b5d6d10 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2068827Reviewed-by:
Wez <wez@chromium.org> Reviewed-by:
Gabriel Charette <gab@chromium.org> Commit-Queue: François Doray <fdoray@chromium.org> Cr-Commit-Position: refs/heads/master@{#751347}
-
Delan Azabani authored
This CL migrates the remaining comparisons involving an ASCII literal from DeprecatedEqualIgnoringCase to EqualIgnoringASCIICase (except the comparisons being reviewed under <https://crrev.com/c/2106983>). These changes are potentially author-facing, but I’ve optimistically made them without any additional tests based on Rick Byers’ reasoning on blink-dev [1]. Please let me know if that’s not sufficient here. The engine changes are a curated subset of the following replacement under Sublime Text 2 (PCRE): Find: (?x-i) (?<escape>\\['"?\\abfnrtv]){0} (?<literal>"(?:(?=[ -~])[^"]|(?&escape))*"){0} (?<any>(?:[^(),]|(\((?:[^()]*|(?-1))\)))*+){0} DeprecatedEqualIgnoringCase (\s*\(\s*+(?: (?&literal)\s*,\s*+(?&any) |(?&any)\s*,\s*+(?&literal) )\s*\)) Where: *.cc,-/path/to/src/out/*,-/path/to/src/.clangd/* Replace: EqualIgnoringASCIICase$5 [1] https://groups.google.com/a/chromium.org/d/msg/blink-dev/sFOpNuQ91UU/3u1HxbnQCQAJ Change-Id: If6e1963da8128e3044df38f1924d98089e7d215a Bug: 627682 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108441 Commit-Queue: Delan Azabani <dazabani@igalia.com> Reviewed-by:
Yoav Weiss <yoavweiss@chromium.org> Reviewed-by:
Yutaka Hirano <yhirano@chromium.org> Cr-Commit-Position: refs/heads/master@{#751346}
-
Yann Dago authored
Bug: 958893 Change-Id: I6d546ba5c5d88bbb1f525f5296e7f44c30c76b2a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2093538 Commit-Queue: Yann Dago <ydago@chromium.org> Reviewed-by:
-
Manu Cornet authored
Change-Id: I4b791af99d81f23f1fb257653f01513cd1f9b82d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108088Reviewed-by:
Matthew Mourgos <mmourgos@chromium.org> Commit-Queue: Manu Cornet <manucornet@chromium.org> Cr-Commit-Position: refs/heads/master@{#751344}
-
Mohamed Amir Yosef authored
This CL reverts most of [1] since a const override of GetPasswordFeatureManager() is actually required and will be used in a follow CL. [1] https://chromium-review.googlesource.com/c/chromium/src/+/2030866 Bug: 1002000 Change-Id: I8a3978b3ab0312c50d2c3667ed90711c19f4cf9b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2105379 Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org> Reviewed-by:
Vasilii Sukhanov <vasilii@chromium.org> Cr-Commit-Position: refs/heads/master@{#751343}
-
Alex Ilin authored
SigninViewController::ShowReauthPopup() will start showing a popup window with a reauth page if "SigninReauthPrompt" feature is enabled. This popup observes all navigations and reports success once a special reauth endpoint is successfully loaded. This endpoint is currently available only on test Gaia. The behavior of ShowReauthPopup() without the feature flag also has changed. Now Chrome will display a fake reauth dialog by default. ShowReauthPopup() is still in development and shouldn't be used without a flag anyways. There is no need to guard it behind another feature flag. The screen recording of the resulting flow is available at https://screencast.googleplex.com/cast/NTM5Njc2MjgxNjY3NTg0MHw0MTg2MGQ0OS02Yg Bug: 1045515 Change-Id: I8a685b593dadec3ec059c77a6a4eccb946f4c98c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107840 Commit-Queue: Alex Ilin <alexilin@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#751342}
-
Tom Anderson authored
BUG=None R=sky Change-Id: I46c44a85089ed1f2dd226c1e3df00991fad95864 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2097463 Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Auto-Submit: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#751341}
-
Omer Katz authored
Vector::Fill uses std::fill, which I believe is not safe for concurrent marking. Currently there are no usages of Vector::Fill for on-heap vectors. Disabling Vector::Fill for on-heap vectors will give a compile time error if anyone tries to use it. I think we can avoid introducing an atomic version of Vector::Fill and revisit it if and when such a usage is wanted. Bug: 986235 Change-Id: I7ac1cb95be3456d86e75f39e0aaf27120e388a84 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106057 Commit-Queue: Omer Katz <omerkatz@chromium.org> Reviewed-by:
-
Elly Fong-Jones authored
It's not used or referenced anywhere. Bug: None Change-Id: Ie3a7716023cc79a22f22980d9c90f9b0609a31be Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108621 Auto-Submit: Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Leonard Grey <lgrey@chromium.org> Reviewed-by:
Leonard Grey <lgrey@chromium.org> Cr-Commit-Position: refs/heads/master@{#751339}
-
Rainhard Findling authored
* Prepares the safety check UI to receive a pluralized string to show in the 'Change password[s]' button, depending on if one or multiple passwords are compromised. * Adds the C++ side string construction w/ tests. Bug: 1015841 Change-Id: I4cf1b2197536f2cd7fbc606f8f167976cfc74f6b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107549Reviewed-by:
Esmael Elmoslimany <aee@chromium.org> Commit-Queue: Rainhard Findling <rainhard@chromium.org> Cr-Commit-Position: refs/heads/master@{#751338}
-
Elly Fong-Jones authored
Bug: 27059 Change-Id: I135c1bbd498cf4d3769718f91ddd486d2cd4c9af Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106564 Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Reviewed-by:
Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#751337}
-
Johannes Kron authored
Bug: 1061376 Change-Id: I4f3e460dc971f76cb8ebb4f37b7bedb0a1cfc9f8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106581 Auto-Submit: Johannes Kron <kron@google.com> Reviewed-by:
-
Domenic Denicola authored
Now that https://crbug.com/1060740 is fixed, we can add a test that parameters are ignored in the allowed=() inner list. Also this adds an additional failure case test that I came up with while working on https://chromium-review.googlesource.com/c/chromium/src/+/2106417. Bug: 1042036 Change-Id: I89ee5423aaafc9a48646e44a605f5bd326256306 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106556Reviewed-by:
Daniel Vogelheim <vogelheim@chromium.org> Commit-Queue: Domenic Denicola <domenic@chromium.org> Cr-Commit-Position: refs/heads/master@{#751335}
-
Domenic Denicola authored
Fixed: 1060184 Change-Id: I8e16621050b2a37d8126b7edc3d9c10d2442758b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2106560Reviewed-by:
Kinuko Yasuda <kinuko@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
-
Nicolás Peña Moreno authored
In this CL, we replace base::Bind() with BindOnce/BindRepeating. BindOnce is preferred, so here is the rationale for the BindRepeating cases: * In SearchBouncer, the base::Bind is used as a parameter of AddInterface(), which requires a repeating callback. It also makes sense to allow for multiple receivers. * In TranslateAgentBrowsertest, it is used as a parameter to SetBinderForTesting, which also requires a repeating callback. Both Repeating cases are associated to methods in Blink which are already marked as receiving repeating callbacks. Bug: 1007641 Change-Id: I03c1b25669a63e58ce5b384f301d2c5b1399e0bc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107716Reviewed-by:
-
v8-ci-autoroll-builder authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/47d5b94b..56a118c8 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: I1036be718b67a798d7aaccb9987cba4202687807 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108640Reviewed-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@{#751332}
-
Jan Wilken Dörrie authored
The current migration to Polymer 3 mistakenly used `clang-format-on` instead of `clang-format on` after turning `clang-format` off via `clang-format off`. This resulted in clang-format effectively being disabled for affected files, this change fixes it. Steps: - sed 's/clang-format-on/clang-format on/g' - git cl format --js --full TBR=dpapad Bug: 1026426 Change-Id: I4364d325e3e7df1add0f83d93e8b18453f6014e6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2107591Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#751331}
-
chromium-autoroll authored
https://chromium.googlesource.com/devtools/devtools-frontend.git/+log/02d03f4ef2db..6d51bf00ea3c git log 02d03f4ef2db..6d51bf00ea3c --date=short --first-parent --format='%ad %ae %s' 2020-03-18 tvanderlippe@chromium.org Factor out Runtime to root/Runtime.js 2020-03-18 aerotwist@chromium.org [global] Remaps self.Bindings.debuggerWorkspaceBinding Created with: gclient setdep -r src/third_party/devtools-frontend/src@6d51bf00ea3c If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/devtools-frontend-chromium Please CC devtools-waterfall-sheriff-onduty@grotations.appspotmail.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md Bug: chromium:1011811,chromium:1058320 Tbr: devtools-waterfall-sheriff-onduty@grotations.appspotmail.com Change-Id: Ic721c20d78b05e596f55d4036258c474456fe812 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2108590Reviewed-by:
-
