- 11 Aug, 2020 40 commits
-
-
chromium-internal-autoroll authored
Release_Notes:http://go/help_app-x20/relnotes/Nightly/help_app_nightly_202008110800_RC00.html https://chrome-infra-packages.appspot.com/p/chromeos_internal/apps/help_app/app/+/qm8oRNi8gU7naD1qdw0-8dq2k2YWuPswsrYsoz6u_n8C If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://skia-autoroll.corp.goog/r/help-app-chromium-autoroll Please CC help-app@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 Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Bug: b/159394333,b/162906760 Tbr: help-app@grotations.appspotmail.com Change-Id: I1beff4b08702c8c390417f5062aef2aed2802628 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2349320Reviewed-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@{#796879}
-
Muyao Xu authored
A new |error_msg| data member is added to LaunchSessionResponse struct to provide more descriptive error messages. Bug: 687380, b/154273698 Change-Id: If334dbfb0499cddcf3cfe36b3423ebf7005efb0a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2335879 Commit-Queue: Muyao Xu <muyaoxu@google.com> Reviewed-by:
mark a. foltz <mfoltz@chromium.org> Reviewed-by:
Takumi Fujimoto <takumif@chromium.org> Cr-Commit-Position: refs/heads/master@{#796878}
-
Nick Diego Yamane authored
Mechanical change renaming platform window's handlers subdir so making it more consistent with the other sub-directories. R=sky@chromium.org Bug: None Change-Id: I5a355b4ed70a30ae803ce96c6ffee60cbfdd0bbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347401 Auto-Submit: Nick Yamane <nickdiego@igalia.com> Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#796877}
-
Hirokazu Honda authored
media/gpu/video_encode_accelerator_tests: Settle target bitrate by the video resolution and framerate Originally video_encode_accelerator_tests tests a fixed bitrate with any video. This enables to settle a good bitrate for testing by determining it by the resolution and frame rate of a test video. Bug: 1045825 Test: video_encode_accelerator_tests on atlas Change-Id: I129158cf0e877a0d06ce304f201c01969abd3be0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2325671 Commit-Queue: Hirokazu Honda <hiroh@chromium.org> Reviewed-by:
David Staessens <dstaessens@chromium.org> Cr-Commit-Position: refs/heads/master@{#796876}
-
Jeffrey Young authored
BUG=none TEST=browser_tests --gtest_filter="OSSettingsAmbientModePageV3Test.All" Change-Id: Ia097fb823b0e5f828444befe64b50b8e677c71cb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347344Reviewed-by:
Kyle Horimoto <khorimoto@chromium.org> Commit-Queue: Jeffrey Young <cowmoo@chromium.org> Cr-Commit-Position: refs/heads/master@{#796875}
-
Antonio Gomes authored
if (bleh) { (...) } else if (!bleh) { (...) } BUG=None R=sky@chromium.org Change-Id: I7e303732499e1cfc77a23e08186a921612cbff27 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346608 Auto-Submit: Antonio Gomes (GMT-4) <tonikitoo@igalia.com> Commit-Queue: Scott Violet <sky@chromium.org> Reviewed-by: -
Dale Curtis authored
Creates a vpcc box based on the configuration info. Reland now uses a forward declaration for the kCMVideoCodecType_VP9 which is in the 11.0/10.16 SDK. R=avi, sandersd Bug: 1103432, 1105187 Test: Updated unittests. Change-Id: I15158ec3329054981cab4eefc3316283952f2f48 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348330 Commit-Queue: Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Avi Drissman <avi@chromium.org> Cr-Commit-Position: refs/heads/master@{#796873}
-
Seokho Song authored
Bug: 1010217 Change-Id: I2ae48bd21aa792b32d2ddf736a567483295219a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2342237Reviewed-by:
Darwin Huang <huangdarwin@chromium.org> Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Darwin Huang <huangdarwin@chromium.org> Cr-Commit-Position: refs/heads/master@{#796872}
-
Victor Hugo Vianna Silva authored
This CL adds a field trial parameter to the passwords account storage feature. It is used to limit the number of times Chrome offers a user that hasn't gone through the opt-in flow yet to move a password to their account. The methods from crrev.com/c/2340966 are used to query the number of times the user has already refused the move. No behavior is actually changed however, since the refusal counter is not yet incremented (future CLs). TBR=vasilii@chromium.org Bug: 1082152 Change-Id: I248f0964e1b406026016288d5e41fb64b2a6e10d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2345309 Commit-Queue: Victor Vianna <victorvianna@google.com> Reviewed-by:
Victor Vianna <victorvianna@google.com> Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Reviewed-by:
Marc Treib <treib@chromium.org> Cr-Commit-Position: refs/heads/master@{#796871}
-
Jesse Schettler authored
The scanning UI is still under development. Extend the flag's expiration to M90. Bug: 1059779 Change-Id: I19bb927b02dbe916e74425a2f1b2b0b93139d1ac Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346829 Commit-Queue: Jesse Schettler <jschettler@chromium.org> Reviewed-by:
Bailey Berro <baileyberro@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Auto-Submit: Jesse Schettler <jschettler@chromium.org> Cr-Commit-Position: refs/heads/master@{#796870}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/21e5b4554a5f..0376edf14e58 2020-08-11 lalitm@google.com Merge "trace_processor: Make GetColumnsForTable report errors" 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/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:linux-perfetto-rel Bug: None Tbr: perfetto-bugs@google.com Change-Id: I4548f77e6b03dc19db2bc9119ea731b50aa5ac46 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2349317Reviewed-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@{#796869}
-
HyunJun Shin authored
Bug: 1010217 Change-Id: Iccba7f237dee5ea57a0ea77eedd1ac56a58f2fa1 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2344106 Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org> Reviewed-by:
Hiroshige Hayashizaki <hiroshige@chromium.org> Cr-Commit-Position: refs/heads/master@{#796868}
-
Dirk Pranke authored
This reverts commit 06578271. Reason for revert: Getting reports of the version check not working from various users; see crbug.com/1115140 Original change's description: > Rewrote PRESUBMIT.py > > Rewrote PRESUBMIT.py (and associated tests) to new format. Runs > REQUIRE_PRESUBMIT_VERSION at the top so that depot_tools can be > made aware of the change and can time/rdb wrap each of the Check... > functions separately, while also ensuring that they have the new > version of depot_tools. > > Requires the newer version of presubmit_support.py in depot_tools, > which is described in crrev.com/c/2341828 > > Change-Id: I102e8c75834aff1eccdf4a27ffe6f0953b76ee57 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2341362 > Reviewed-by: Dirk Pranke <dpranke@google.com> > Commit-Queue: Saagar Sanghavi <saagarsanghavi@google.com> > Cr-Commit-Position: refs/heads/master@{#796595} TBR=dpranke@google.com,estaab@chromium.org,saagarsanghavi@google.com Change-Id: Ib3295d141e997fc6792c381fea10e8892204bcdb No-Presubmit: true No-Tree-Checks: true No-Try: true Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2349428Reviewed-by:
Dirk Pranke <dpranke@google.com> Commit-Queue: Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/master@{#796867}
-
David Roger authored
The enterprise icon is changed into a badge on top of the gaia avatar. The various texts are also updated to match the mocks in various cases, depending whether the accounts are managed or not. Screenshots: Normal/Normal: https://drive.google.com/file/d/1mE22Ro44l3IxstpFDNEz3oGMqxxfeYa4/view?usp=sharing https://drive.google.com/file/d/1bVrkJ_vC-NTjE5hXh69Gf4FSEqemQkdp/view?usp=sharing Managed/Normal: https://drive.google.com/file/d/1VipZs_BjEtzWtBc_P-pjkR3HdYZB6C-H/view?usp=sharing https://drive.google.com/file/d/1J9Io65diEFqwZq9kZbWnYixKLayKHJZn/view?usp=sharing Normal/Managed: https://drive.google.com/file/d/14R8iadPPOy5zYGdS7GsaCU8hlLQDL3CE/view?usp=sharing Bug: 1076880 Change-Id: Ib66872356ed06d840a956e7d7b18686d29564a12 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2339337 Commit-Queue: David Roger <droger@chromium.org> Reviewed-by:
Monica Basta <msalama@chromium.org> Cr-Commit-Position: refs/heads/master@{#796866}
-
spdonghao authored
Bug: 1112874 Change-Id: Ibba008295747a9ac834309dad09771b7d0ff37c8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2344946Reviewed-by:
Xi Han <hanxi@chromium.org> Reviewed-by:
Wei-Yin Chen (陳威尹) <wychen@chromium.org> Commit-Queue: Hao Dong <spdonghao@chromium.org> Cr-Commit-Position: refs/heads/master@{#796865}
-
adamta authored
Adds enough height to the CS collection view to allow for scroll position to be maintained in Discover feed. Added height is calculated from the cached NTP offset. Bug: 1085419, 1108403 Change-Id: Ia9c2ed7dd7d5d3d8aa3413266404f956ae524cbf Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2317811 Commit-Queue: Adam Trudeau-Arcaro <adamta@google.com> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Sergio Collazos <sczs@chromium.org> Cr-Commit-Position: refs/heads/master@{#796864}
-
chromium-autoroll authored
https://chromium.googlesource.com/chromiumos/chromite.git/+log/a1d5ae72b241..5245bcfaf6a9 2020-08-11 chrome-bot@chromium.org Update config settings by config-updater. 2020-08-11 erikchen@chromium.org Two small updates to deploy_chrome.py for --lacros. 2020-08-11 gbiv@google.com toolchain_util: fix bug 2020-08-10 erikchen@chromium.org Update README to reflect manual testing instructions. 2020-08-10 chrome-bot@chromium.org Update config settings by config-updater. 2020-08-10 denik@google.com chromite: Pass config_lib useflags to cros_setup_toolchains 2020-08-10 zhuohao@chromium.org Revert "kukui-tablet: Adds new builder and signer type" If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/chromite-chromium-autoroll Please CC chrome-os-gardeners@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chrome.try:chromeos-betty-pi-arc-chrome;luci.chromium.try:chromeos-kevin-rel Tbr: chrome-os-gardeners@google.com Change-Id: Ia7724f34613933ad964649fde76af0af0478aac0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347179Reviewed-by:
-
chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/de914d018068..9385eb35f507 2020-08-11 lehoangq@gmail.com Metal: Generate default shaders to separate files. 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 ynovikov@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win-asan;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:linux-swangle-try-x64;luci.chromium.try:win-swangle-try-x86 Bug: None Tbr: ynovikov@google.com Change-Id: If6c9d4328eda0e315d8e84f885fe16188413ef83 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347176Reviewed-by:
-
Jérôme Lebel authored
The goal of those 5 patches is to remove const on the following methods: * IsValidIdentity() * GetIdentityWithEmail() * GetIdentityWithGaiaID() * GetCanonicalizeEmailsForAllIdentities() * HasIdentities() * GetAllIdentities() * GetAllIdentitiesSortedForDisplay() This is required for crrev.com/i/3208484, to not start async identity fetch in ChromeIdentityServiceImpl instance when the cache is not populated. * crrev.com/c/2345305 Add temporary methods * crrev.com/i/3201649 Switch to the temporary methods => crrev.com/c/2345146 Remove const in the methods * crrev.com/i/3201650 Switch back the original methods * crrev.com/c/2344467 Remove temporary methods Bug: 897470 Change-Id: Id1f2c88442b835472e54a5bc74516780460e3b89 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2345146 Commit-Queue: Jérôme Lebel <jlebel@chromium.org> Reviewed-by:
Nohemi Fernandez <fernandex@chromium.org> Cr-Commit-Position: refs/heads/master@{#796861}
-
chromium-autoroll authored
https://webrtc.googlesource.com/src.git/+log/64a60835a190..f867ba8b5d48 2020-08-11 nisse@webrtc.org Delete deprecated variant of RtpVideoStreamReceiver::AddReceiveCodec 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: I7c572c1e9a7646c67a701af00ee7e972f15b6707 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347178Reviewed-by:
-
Morten Stenshorne authored
These fragmentainers are created naturally by the block layout algorithm. CL:1766070 added code to ignore them (just drop them on the floor), but it seems that it's less cumbersome to just keep them. We kept them anyway, if there were unhandled OOFs inside (and manually propagating such OOFs sounds messy). In addition, we end up with empty fragmentainers if the first child of a multicol container is a spanner. Even if we discarded the actual fragmentainer, we'd still keep the break token generated (to resume at column content after the spanner(s)), and this in turn would cause trouble when calucalting the fragmentainer index in NGBlockNode::AddColumnResult() (we'd essentially never add a fragment at index 0, but jump straight to 1). We may need to revisit this again in the future, though. Overflow calculation comes to mind. We'll now create a 1px tall fragmentainer even inside 0px tall multicol containers. But we can either just prevent fragmentainers from contributing to overflow (this is not what other browsers do, though), or actually make such fragmentainers 0px tall (they become 1px because the spec says that fragmentainers may not be shorter than that, but we might get away with it anyway). Add a DCHECK in LayoutBox::AddLayoutResult() to assert that we don't mess up the index (like we used to, which in turn made us keep fragments associated with deleted layout objects). Quite a few unit tests had to be updated because of this. I changed some of them slightly, to give the new empty fragmentainers a nicer width that they'd otherwise get. Bug: 1105758 Change-Id: I44e1b93dea4cd5e0c5f7944798e2336530a6d1a4 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346389 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Koji Ishii <kojii@chromium.org> Cr-Commit-Position: refs/heads/master@{#796859}
-
Yue Ru Sun authored
UKM enums are used by `UKM.Entries.Recorded.ByEntryHash` and `UKM.Entries.Dropped.ByEntryHash` when displaying a breakdown of event counts by name. Some events were incorrectly displayed as "obsolete" in the histogram table here https://screenshot.googleplex.com/CRmtGDOcAwQ.png compared to the the event definition https://source.chromium.org/chromium/chromium/src/+/master:tools/metrics/ukm/ukm.xml;l=4254;drc=ec5f934779d00e3bf817be19297f72ef5bd98e9c This patch fixes the logic to consider an event as obsolete iff - the event itself is marked as obsolete, or - all the metrics under this event are marked as obsolete. Tested locally with chromium/src/tools/metrics/histograms$ ./merge_xml.py --output output.xml ../ukm/ukm.xml enums.xml Bug: b/163387426 Change-Id: I06ae60edb4e025cf693ad37063a45dc3bf6edd31 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347386 Commit-Queue: Yue Ru Sun <yrsun@chromium.org> Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Cr-Commit-Position: refs/heads/master@{#796858}
-
Richard Knoll authored
Attachment ids are transferred via wire_format.proto and need to be serialized/deserialized to int64. Also removes virtual methods from the base Attachment class and adds |text_title| to TextAttachment. Bug: 1085067 Change-Id: I39f0820f6682718c88e6b92947ad5a156c0ab294 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346807Reviewed-by:
Himanshu Jaju <himanshujaju@chromium.org> Commit-Queue: Richard Knoll <knollr@chromium.org> Cr-Commit-Position: refs/heads/master@{#796857}
-
Shu-yu Guo authored
Bug: v8:7883 Change-Id: Ic91546ea953d66d3742e599ad0a8da778b6363b6 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346942Reviewed-by:
Kentaro Hara <haraken@chromium.org> Commit-Queue: Shu-yu Guo <syg@chromium.org> Cr-Commit-Position: refs/heads/master@{#796856}
-
Elly Fong-Jones authored
Bug: 1115106 Change-Id: I82a797d19e6c70e7983221128f769ffb5596bc71 Tbr: lazyboy@chromium.org Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348695Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#796855}
-
Friedrich Horschig authored
This CL ensures that the referrer is considered in the decision to run the password check. When coming from safety check: use last status, don't run check When coming from anywhere else: run the check when possible. The initial status is not relevant for the decision to run. If it's RUNNING, the call is a noop. If it's IDLE or cancelled, it makes sense to run a new check. If it's any error state, rerunning might show the error was fixed since the last attempt. Reaching a new failure won't take long. Bug: 1109691, 1092444 Change-Id: I573a5f9f0a8861e598541fa38faa2d8bd42adba3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348773Reviewed-by:
Ioana Pandele <ioanap@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#796854}
-
sczs authored
- Prevents the same FeedVC from being re-added to the VC hierarchy, and if a new FeedVC is set, removes the old one from the VC hierarchy. - Uses KVO to observe of Feed contentSize changes, so we can reload the Collection view and display the Feed. Without this the Feed wouldn't appear after it has been loaded. Credit to adamta@ crrev.com/c/2342580 - Sets a minimum height for the Feed Cell, this way the loading spinner will always be displayed. Bug: 1085419 Change-Id: I9d9ef7978a667a2eddc62c0b8e1aad5300185cc0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347372 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
-
Anastasiia N authored
- Use Chrome OS Account Manager in identity_test_utils instead of calling UpdateCredentials on token service. Account Manager is created in BuildIdentityManagerForTests. - Change FakeProfileOAuth2TokenService to allow passing custom delegate and use ProfileOAuth2TokenServiceDelegateChromeOS instead of fake delegate on Chrome OS. See details: http://doc/1_JHwE0NioyMVBfebGxuN0EjM91JhyDz1-Hi5PBOPtEw Bug: 1068240 Change-Id: I083e806b37cd967ed0759a2e6add13e07bfa68e2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2232324Reviewed-by:
David Roger <droger@chromium.org> Reviewed-by:
Kush Sinha <sinhak@chromium.org> Commit-Queue: Anastasiia N <anastasiian@chromium.org> Cr-Commit-Position: refs/heads/master@{#796852}
-
Kyle Ju authored
Testing Command: python third_party/blink/tools/wpt_export.py --dry-run --credentials <your credentials> --surface-failures-to-gerrit --verbose A Sample Comment: The exported PR, https://github.com/web-platform-tests/wpt/pull/24650, has failed the following check(s) on GitHub: wpt-firefox-nightly-stability (https://github.com/web-platform-tests/wpt/pull/24650/checks?check_run_id=883668859) ... These failures will block the export. They may represent new or existing problems; please take a look at the output and see if it can be fixed. Unresolved failures will be looked at by the Ecosystem-Infra sheriff after this CL has been landed in Chromium; if you need earlier help please contact ecosystem-infra@chromium.org. Gerrit CL SHA: 0e4d176d93a1a70e03720dbeb8142139e0df5871 Patchset Number: 2 Change-Id: I3376a7913c6677ed219717d2a4ddbc065ce83fbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2336542 Commit-Queue: Kyle Ju <kyleju@chromium.org> Reviewed-by:
Robert Ma <robertma@chromium.org> Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#796851}
-
Dave Tapuska authored
The OneShotTimer needs to be destroyed on the main thread. Destroy it during ClearClient. BUG=1112931 Change-Id: I9fb7946947943f7ff2a5bbb19d7ae1af94335b08 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2347392 Commit-Queue: Dave Tapuska <dtapuska@chromium.org> Reviewed-by:
David Bokan <bokan@chromium.org> Cr-Commit-Position: refs/heads/master@{#796850}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/01e0ccdc9b8b..dc4568899168 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-linux-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I24e32f1e18c446f923ee88a2bc09f0e4fc987693 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348256Reviewed-by:
-
chromium-autoroll authored
If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/fuchsia-aemu-chromium-autoroll Please CC chonggu@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: chonggu@google.com Change-Id: I34b051b9e27d8fda1e6443e65c58c232a44b3fb9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348199Reviewed-by:
-
Niels Möller authored
This flag (not to be confused with EncoderInfo::is_hardware_accelerated) is no longer used by webrtc. Bug: None Change-Id: I5a0524cced0f6f4a45699d2b8caa0ccfcb98336f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346656 Commit-Queue: Gary Kacmarcik <garykac@chromium.org> Reviewed-by:
Gary Kacmarcik <garykac@chromium.org> Cr-Commit-Position: refs/heads/master@{#796847}
-
Kramer Ge authored
WaylandOverlayManager treats any overlay candidate as eligible for hardware overlay. Overlays and primary plane will be forwarded to wayland server and whether they're picked to be displayed as HW overlay is decided by wayland server. This is 4/? CL for overlay forwarding using wl_subsurface. Bug: 1063865 Change-Id: I63e9e3bca152f5202c0f2c35d26d5f9d3663eecb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2321514 Commit-Queue: Kramer Ge <fangzhoug@chromium.org> Reviewed-by:
Maksim Sisov (GMT+3) <msisov@igalia.com> Reviewed-by:
Robert Kroeger <rjkroege@chromium.org> Reviewed-by:
Peng Huang <penghuang@chromium.org> Cr-Commit-Position: refs/heads/master@{#796846}
-
nancylingwang authored
The icon callback may outlive ArcAppShortcutSearchResult, so bind the callback to a weak ptr. BUG=1114754 Change-Id: If261827920d5a4780a80695095937a9f7dd59c5e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348492Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Nancy Wang <nancylingwang@chromium.org> Cr-Commit-Position: refs/heads/master@{#796845}
-
Lukasz Anforowicz authored
The original fix (r719801) for https://crbug.com/1026474 is undesirable, because it may put an about:blank subframe in a separate BrowsingInstance from its parent. Therefore the current CL partially reverts r719801 and implements an alternative fix for preserving the SiteInstance of about:blank after session restore. The alternative fix is to: 1. Prefer |source_site_instance| (in RFHM::DetermineSiteInstanceForURL) over a site-less |current_instance|. 2. Make sure that |source_site_instance| is created by NavigationRequest constructor in session restore cases requiring initiators (and not just in |is_history_navigation_in_new_child_frame| case). The changes above break the following COOP test: NewPopupCOOP_SameOriginPolicyAndCrossOriginIframeSetsNoopener, because COOP explicitly wants to suppress the opener and doesn't want to use *same-BrowsingInstance* |source_site_instance|. The fix for this is to 3. Make sure that |source_site_instance| is not populated if |opener_suppressed| in WebContentsImpl::CreateNewWindow. The change above means that the following DCHECK in the NavigationRequest's constructor fails: DCHECK(!RequiresSourceSiteInstance() || source_site_instance_) The fix for this is to: 4. Stop requiring |source_site_instance| if the opener was suppressed (see the changes in NavigationRequest::RequiresSourceSiteInstance). Summary of tests relevant to this CL: - MultiOriginSessionRestore - new asserts have been added to require that the subframe is in the same BrowsingInstance as the parent. - NewPopupCOOP_SameOriginPolicyAndCrossOriginIframeSetsNoopener - shows the need to avoid using |source_site_instance| when the opener has been suppressed - IncognitoApiTest - shows that there are scenarios where it is okay to use a cross-BrowsingInstance |source_site_instance| (navigations initiated via extension API should switch BrowsingInstance). - SubframeBlankUrlsAfterRestore - highlights the need to check |browser_initiated_| in RequiresSourceSiteInstance. Bug: 1107269 Change-Id: Ibbdcb6c3a5d309a8fe17991b06f380dffcd225d2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2327963 Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org> Reviewed-by:
Charlie Reis <creis@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#796844}
-
Zarah Tesfai authored
Purpose: Move UI Logic of retry() in PRImpl to PaymentUIsManager Change: * onRetry() method in PaymentRequestLifecycleObserver * onRetry() implemented in PaymentUIsManager Bug: 1113337 Change-Id: If4692bf3cd3539d5379b108651205ac2fb2345ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2339705Reviewed-by:
Rouslan Solomakhin <rouslan@chromium.org> Reviewed-by:
Liquan (Max) Gu <maxlg@chromium.org> Commit-Queue: Zarah Tesfai <ztesfai@google.com> Cr-Commit-Position: refs/heads/master@{#796843}
-
arthursonzogni authored
I was suggested the "isolate" argument of ReportCoopAccess could be determined by the DomWindow instance itself: https://chromium-review.googlesource.com/c/chromium/src/+/2339353/4 This patch removes it from the ReportCoopAccess method and updates the callers. Bug: 1090273 Change-Id: I0286f13ea8700c419728f89f706baf71bb0f0642 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2346569 Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Reviewed-by:
Yuki Shiino <yukishiino@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Cr-Commit-Position: refs/heads/master@{#796842}
-
George Steel authored
Change-Id: I47f6de8275c1b17399671eb069794ad2857fdae8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2341827Reviewed-by:
Kevin Ellis <kevers@chromium.org> Commit-Queue: George Steel <gtsteel@chromium.org> Cr-Commit-Position: refs/heads/master@{#796841}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/01e0ccdc9b8b..dc4568899168 If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/perfetto-trace-processor-mac-chromium Please CC perfetto-bugs@google.com on the revert to ensure that a human is aware of the problem. To report a problem with the AutoRoller itself, please file a bug: https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md Tbr: perfetto-bugs@google.com Change-Id: I4111b739c6dda15890f9c1a00a92279ee82de464 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2348203Reviewed-by:
-
