- 08 Jan, 2021 40 commits
-
-
chromium-autoroll authored
https://chromium.googlesource.com/chromium/src/third_party/freetype2.git/+log/cbc14b10e623..0901a6f74c70 2021-01-08 apodtele@gmail.com Tweak .gitignore. 2021-01-07 apodtele@gmail.com [builds/windows] Fix up DLG build with VC++. 2021-01-07 apodtele@gmail.com [builds] Relocate `FTMODULE_H'. If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/freetype-chromium Please CC bungeman@google.com,drott@google.com,thestig@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_chromium_msan_rel_ng;luci.chromium.try:linux-blink-rel;luci.chromium.try:mac10.12-blink-rel;luci.chromium.try:win10-blink-rel;luci.chromium.try:win7-blink-rel Tbr: bungeman@google.com,drott@google.com,thestig@google.com Change-Id: I43d784af6c7391934f557c53628d0dfda7394b0c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616843Reviewed-by:
chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com> Reviewed-by:
Ben Wagner <bungeman@chromium.org> Commit-Queue: Ben Wagner <bungeman@chromium.org> Cr-Commit-Position: refs/heads/master@{#841494}
-
dpapad authored
Previously PDF Viewer resources were registered as part of component_extension_resources.grd. This CL adds a dedicated and auto-generated grd file to hold the PDF Viewer's resources. This CL also opens up the possibility of using optimize_webui() to improve the startup performance of the PDF Viewer. Bug: 1132403 Change-Id: Ib1ed6ebc9f51dfe0ffd375b73186cfb3b07c6806 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2614504 Commit-Queue: dpapad <dpapad@chromium.org> Auto-Submit: dpapad <dpapad@chromium.org> Reviewed-by:
Karan Bhatia <karandeepb@chromium.org> Reviewed-by:
Rebekah Potter <rbpotter@chromium.org> Cr-Commit-Position: refs/heads/master@{#841493}
-
Evan Stade authored
This allows Chrome to do logging for the IPH feature. Bug: 1164069 Change-Id: I57bd7dc364d1b9d913b1ea316a4df8b1951c3f04 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616489Reviewed-by:
Dominick Ng <dominickn@chromium.org> Commit-Queue: Evan Stade <estade@chromium.org> Cr-Commit-Position: refs/heads/master@{#841492}
-
Nicolas Ouellet-Payeur authored
As pointed out by rhalavati@, the kWaitingForBrowserWindow ref was removed when _any_ ScopedProfileKeepAlive was added. This does not match intended use. This CL only removes kWaitingForBrowserWindow when a kBrowserWindow ref is added. Also, record histograms in ~ProfileManager(), to detect leaked keepalives. Bug: 88586 Change-Id: Iaec7d54109b7268059c8475261fef56ca1594c9f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611050Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Nicolas Ouellet-Payeur <nicolaso@chromium.org> Cr-Commit-Position: refs/heads/master@{#841491}
-
Liviu Tinta authored
There are a few wpt pointerlock tests that fail under the new wpt try bots. The wpt try bots are running wpt run in headless mode. Turns out that, when running wpt tests in headless mode with the wpt runner,the code is reaching this line in WebContentsImpl::RequestToLockMouse [1]. The problem here is that delegate_->RequestToLockMouse is actually the empty implementation from WebContentsDelegate::RequestToLockMouse. This is why the WebContents::GotResponseToLockMouseRequest is never called. The expectation for WebContentsDelegate::RequestToLockMouse is to call WebContents::GotResponseToLockMouseRequest like other code paths. The culprit is HeadlessWebContentsImpl:Delegate, which inherits the empty implementation of WebContentsDelegate::RequestToLockMouse. When running the wpt tests without --headless the delegate is an instance of Browser class. Implement HeadlessWebContentsImpl:Delegate::RequestToLockMouse similar to what we do for content shell. [1] https://www.google.com/url?q=https://source.chromium.org/chromium/chromium/src/%2B/master:content/browser/web_contents/web_contents_impl.cc;drc%3D3fab4813ccc76de821bb74b441ff491afc8a7a6f;l%3D3402&sa=D&source=calendar&ust=1608479687279000&usg=AOvVaw2Q3jZFrFsbSnQd8osPwpOU TEST=external/wpt/pointerevents/pointerlock/pointerevent_getCoalescedEvents_when_pointerlocked.html,external/wpt/pointerevents/pointerlock/pointerevent_coordinates_when_locked.html,external/wpt/pointerevents/pointerlock/pointerevent_getCoalescedEvents_when_pointerlocked.html,external/wpt/pointerevents/pointerlock/pointerevent_movementxy_with_pointerlock.html,external/wpt/pointerevents/pointerlock/pointerevent_pointerlock_after_pointercapture.html,external/wpt/pointerevents/pointerlock/pointerevent_pointermove_in_pointerlock.html,external/wpt/pointerevents/pointerlock/pointerevent_pointermove_on_chorded_mouse_button_when_locked.html Bug: 1127055 Change-Id: Ic1da165fed1dc1902c5e8c0dbb5f7cfc7513ee30 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2594009Reviewed-by:
Mustaq Ahmed <mustaq@chromium.org> Reviewed-by:
Peter Kvitek <kvitekp@chromium.org> Commit-Queue: Liviu Tinta <liviutinta@chromium.org> Cr-Commit-Position: refs/heads/master@{#841490}
-
Fergal Daly authored
This is a reland of 2cda8338 Test is Flaky on Mac, so relanding with Mac disabled. Original change's description: > Reland "[Paint Preview] Post task captured callback" > > This is a reland of c669ac6b > > This is the test only, the fix was landed in https://crrev.com/c/2598087 > > - the previous test did not reproduce the crash, this one has crash a > crash that matches the trace in the wild > https://ci.chromium.org/p/chromium/builders/try/linux-dcheck-off-rel/16863? > It differs from the previous test by > - calling SetNeedsReload (emulating Clank's autorefresh) > - navigating to another frame afterwards, triggering Unload > > - the previous test timed out on some builders. It's possible that > the callback was already called before the test's own RunlLoop > started, which would mean that the loop never exits. This is avoided > by recording whether it has run or not. > > Original change's description: > > [Paint Preview] Post task captured callback > > > > This CL attempts to fix a bug where calling > > WebContents::DecrementCapturerCount in the callback passed to > > PaintPreviewClient can result in reloading a crashed render frame. > > This is problematic and crashes if it occurs inside RenderFrameDeleted > > and PaintPreviewClient::OnFinished is invoked. By posting the callback > > it should be invoked after RenderFrameDeleted cleanup is done resulting > > in DecrementCapturerCount no longer being called inside > > RenderFrameDeleted. > > > > Bug: 1146573 > > Change-Id: Iccb612d89135e058d8fc06de018e7c95cd91657a > > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2595247 > > Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> > > Commit-Queue: Mehran Mahmoudi <mahmoudi@chromium.org> > > Reviewed-by: Mehran Mahmoudi <mahmoudi@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#837649} > > Bug: 1146573, 1160608 > > > Cq-Include-Trybots: luci.chromium.try:linux-dcheck-off-rel > Change-Id: I3ef0a81bb4fa99e2f42c6278ba53931aa934793e > Cq-Do-Not-Cancel-Tryjobs: true > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596697 > Reviewed-by: Calder Kitagawa <ckitagawa@chromium.org> > Commit-Queue: Fergal Daly <fergal@chromium.org> > Auto-Submit: Fergal Daly <fergal@chromium.org> > Cr-Commit-Position: refs/heads/master@{#840497} Bug: 1146573 Bug: 1160608 Change-Id: Ib26af42633d55e49752623930ce63245b6f4482e Cq-Include-Trybots: luci.chromium.try:linux-dcheck-off-rel Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615724 Auto-Submit: Fergal Daly <fergal@chromium.org> Commit-Queue: Calder Kitagawa <ckitagawa@chromium.org> Reviewed-by:
Calder Kitagawa <ckitagawa@chromium.org> Cr-Commit-Position: refs/heads/master@{#841489}
-
Jordan Demeulenaere authored
This CL introduces a new SEND_FEEDBACK chip type that shows the feedback form when it is selected. This new chip type can only be used inside a Prompt atm, but follow-up CLs will expose the feedback chip for more use cases (e.g. client errors). Bug: b/173401004 Change-Id: Idea938ed9f3706ab88be7da2bcf1c34be5a3eb5b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617923Reviewed-by:
Clemens Arbesser <arbesser@google.com> Commit-Queue: Jordan Demeulenaere <jdemeulenaere@chromium.org> Cr-Commit-Position: refs/heads/master@{#841488}
-
Antonio Sartori authored
Initially, this parameter was just an enum, so we were passing it by value. Now it became a struct, so we should pass it by const reference. Bug: 1130587 Change-Id: I6c02e4453259914c01a7d6c218be66cc5af30ad3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616560 Auto-Submit: Antonio Sartori <antoniosartori@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org> Cr-Commit-Position: refs/heads/master@{#841487}
-
dpapad authored
With the addition of the |resource_path| grd attribute, there is no longer a need to manually handle generated paths in ChromeComponentExtensionResourceManager. This is in preparation of auto-generating the grd file for the PDF Viewer. Bug: 1132403 Change-Id: I7d788c5cfaa3da21471a76bf4dd7585a56339ef3 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2614130Reviewed-by:
-
Alexei Svitkine authored
Feature params cannot change at runtime. Retrieve them only once, instead of repeatedly. We're seeing lots of crashes with SIGABRT on shutdown retrieving UKM feature params by these code paths. SIGABRT may suggest that it's just shutdown taking too long and it just happens to land here (because retrieving params takes a bit of time). Eliminate this possibility by caching these values. This should slightly speed up UKM recording, including on shut down. If shut down still takes too long and is the cause of the SIGABRTs we're seeing, the stack traces should hopefully change and the field trial param code won't be blamed anymore (perhaps replaced by some other slow code, which can be fixed separately). Bug: 1131877 Change-Id: I365e7a6fac5d3ef941cd0a26e9b280d03df6b28b Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615820 Commit-Queue: Robert Kaplow <rkaplow@chromium.org> Auto-Submit: Alexei Svitkine <asvitkine@chromium.org> Reviewed-by:
-
Austin Sullivan authored
This finishes renaming in blink, with the exception of web platform tests. Renames "NativeFileSystem" to FileSystemAccess or FileSystem. Generally, NativeFileSystem is renamed to FileSystemAccess. In cases where IDL exposes something as "FileSystem", NativeFileSystem is renamed to "FileSystem". Bug: 1110509 Change-Id: I483c50ae0ffa2de7946c0bd802c5082715113d6e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2601576Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Daniel Murphy <dmurph@chromium.org> Reviewed-by:
Tom Sepez <tsepez@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Marijn Kruisselbrink <mek@chromium.org> Reviewed-by:
Darwin Huang <huangdarwin@chromium.org> Commit-Queue: Austin Sullivan <asully@chromium.org> Cr-Commit-Position: refs/heads/master@{#841484}
-
Ryan Hansberry authored
Change-Id: I2643b6473c8b88fb901cef59b265af336878bfda Fixed: 1160901 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612034 Auto-Submit: Ryan Hansberry <hansberry@chromium.org> Commit-Queue: Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
-
Owen Min authored
The test is timeout on trybots which is likely caused by the RunUntilIdle() function which is put here just in case. Trying to solve the timeout issue by removing the function. Bug: 1137094 Change-Id: I4e48b937901ffa23f2daff6c41050eaa7b9f7f5f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615324Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Commit-Queue: Owen Min <zmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#841482}
-
chromium-autoroll authored
https://skia.googlesource.com/skia.git/+log/cb3bcf88e382..d2f51b18065a 2021-01-08 johnstiles@google.com Fix fuzzer-discovered optimizer crash. 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 johnstiles@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: johnstiles@google.com Change-Id: I85a386f3f3a1998c3d9438949dae0f3939c41c4f Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618068Reviewed-by:
-
Joe Mason authored
This conflicts with the use of 0 bytes as a sentinel for frames that hadn't been measured and should be dropped from the result. So this also makes the bytes optional in WebMemoryBreakdownEntry, so that frames with a byte count of 0 are reported and frames without a byte count are dropped from the result. R=ulan Bug: 1085129 Change-Id: Ie838c72049465e3a3bef9c0a9bb3a11aa648e62c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616818Reviewed-by:
Ulan Degenbaev <ulan@chromium.org> Commit-Queue: Joe Mason <joenotcharles@chromium.org> Cr-Commit-Position: refs/heads/master@{#841480}
-
Kim-Anh Tran authored
This disables a web test until a change in the TimelineUIUtils has landed in Devtools. Bug: chromium:1132260 Change-Id: Ib5acd9c22522f92e2746947ce8bbebc5f3043c1e Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617919Reviewed-by:
Benedikt Meurer <bmeurer@chromium.org> Commit-Queue: Kim-Anh Tran <kimanh@chromium.org> Cr-Commit-Position: refs/heads/master@{#841479}
-
Ramin Halavati authored
During profile signin intercept, if the new account already has a profile, do not offer moving to Guest mode. Bug: 1157764 Change-Id: I2095eeae7c61d5ff8c91082b2f1d52d040d9a055 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611246 Commit-Queue: Ramin Halavati <rhalavati@chromium.org> Reviewed-by:
David Roger <droger@chromium.org> Cr-Commit-Position: refs/heads/master@{#841478}
-
Scott Violet authored
This gives the same logic as that of 'overflow: clip.' BUG=1159155 TEST=overflow-clip-margin-00[456].html Change-Id: I1396fad5fbf4464daee8d021191797ba7d01856c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2596828Reviewed-by:
Ian Kilpatrick <ikilpatrick@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#841477}
-
Xida Chen authored
At this moment, our implementation only keeps the first and the last keyframes for composite background color animation. This is wrong, for example, we can having multiple keyframes like: 0%: { background-color: red } 10%: { background-color: green} 100% { background-color: blue} This CL fixes the issue. The idea is to keep the offset which we stored in CompositorKeyframeValues. In the above case, the offsets would be [0, 0.1, 1]. Once we have that, here is how we do interpolation. Say that the current progress is 0.4, then we know that it falls in to the range of [0.1, 1], which means we need to interpolate from green to blue. We need to adjust the progress (0.4) based on the offsets. Basically scale [0.1, 1] to [0, 1], then the adjusted progress should be (0.4-0.1) / (1-0.1). Layout tests are added to ensure correctness. Bug: 1153671 Change-Id: I6dffc82b5821fdf418f2b94331f567a239079730 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2597844 Commit-Queue: Xida Chen <xidachen@chromium.org> Reviewed-by:Kevin Ellis <kevers@chromium.org> Cr-Commit-Position: refs/heads/master@{#841476}
-
chromium-autoroll authored
https://dawn.googlesource.com/dawn.git/+log/00fcab636a05..a2e02836a260 2021-01-08 cwallez@chromium.org Suppress EntryPointTests.FragAndVertexSameModule on GL 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 enga@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: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: chromium:1163767 Tbr: enga@google.com Change-Id: I1be11624448a3250c8affcabefe574bf28b7e0ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618079Reviewed-by:
-
Victor Hugo Vianna Silva authored
This CL makes sure the `opened` and `notification_dismissed` fields are shown in the "Sync Node Browser" tab of chrome://sync-internals by adding some missing code to proto_visitors.h. Bug: 1164334 Change-Id: I226b37e07709acd8eb164f67c85a22d72cc70e5d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616201Reviewed-by:
Marc Treib <treib@chromium.org> Commit-Queue: Victor Vianna <victorvianna@google.com> Cr-Commit-Position: refs/heads/master@{#841474}
-
Mihai Sardarescu authored
Methods OnPrimaryAccountSet/Cleared() are deprecated, so this CL changes the chrome/browser/password_manager/* to override the method OnPrimaryAccountChanged(). Bug: 1163126 Change-Id: I1357bc6a398089f71ba72363a0b2f24e4a7c7c9c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2611084 Auto-Submit: Mihai Sardarescu <msarda@chromium.org> Reviewed-by:
Tanmoy Mollik <triploblastic@chromium.org> Commit-Queue: Marc Treib <treib@chromium.org> Commit-Queue: Mihai Sardarescu <msarda@chromium.org> Cr-Commit-Position: refs/heads/master@{#841473}
-
Egor Pasko authored
In http://crrev.com/840993 after addressing a comment to rename the inner class in LibraryLoader I forgot to rename the member field accordingly. Doing it here. Bug: 1154224 Change-Id: I5445d303919660e6853eccfc9e947250bf8cf828 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612895 Auto-Submit: Egor Pasko <pasko@chromium.org> Commit-Queue: Benoit L <lizeb@chromium.org> Reviewed-by:
Benoit L <lizeb@chromium.org> Cr-Commit-Position: refs/heads/master@{#841472}
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/070ef598892a..3ce4e8ee1301 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: Ia3b6ebc29b2bba9b818f5663eed762ed8aa3bcdb Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616760Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/3ce4e8ee1301..afc78f5e6ae9 2021-01-08 treehugger-gerrit@google.com Merge "Formalize comment into static_assert." 2021-01-08 skyostil@google.com Merge "sdk: Make the tracing SDK compatible with MSVC" 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: Iea002d008e4f5cf7e35c4efd51b0cc47a6f6ccba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618066Reviewed-by:
-
Alexander Timin authored
Tests are really slow when both tracing and sanitizers are enabled. Increase the RunLoop timeout for tracing tests to avoid test failures in that case. R=eseckler@chromium.org BUG=1157954,1041392 Change-Id: Icc5f077d342587b82cf699ff41122e3c30a7b5e8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2616018 Commit-Queue: Eric Seckler <eseckler@chromium.org> Reviewed-by:
Eric Seckler <eseckler@chromium.org> Cr-Commit-Position: refs/heads/master@{#841469}
-
chromium-autoroll authored
Roll Chrome Win32 PGO profile from chrome-win32-master-1610085457-9d25c44b81df119265a48970c3964c59f218f094.profdata to chrome-win32-master-1610096386-b7d9ff85a7e5a6b9d422dd8ae3e7ff1b6461dd2b.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-win32-chromium Please CC pgo-profile-sheriffs@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:win-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I455e5bca7acce1bd8aca323dc52e897d2d4f541a Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617561Reviewed-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-sdk-chromium-autoroll Please CC chrome-fuchsia-gardener@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.chromium.try:fuchsia-arm64-cast;luci.chromium.try:fuchsia-deterministic-dbg;luci.chromium.try:fuchsia-x64-cast Tbr: chrome-fuchsia-gardener@grotations.appspotmail.com Change-Id: Iac40b0e1f7b3a5cf3fba726c9806a5d96fe02bbc Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618078Reviewed-by:
-
chromium-autoroll authored
Roll Chrome Linux PGO profile from chrome-linux-master-1610085553-90a8ab5c2cfb6eb3765af407df5af94b81fb1c7d.profdata to chrome-linux-master-1610107094-a1b03a814715c7959032a16f0512a58b7ebd318d.profdata If this roll has caused a breakage, revert this CL and stop the roller using the controls here: https://autoroll.skia.org/r/pgo-linux-chromium Please CC pgo-profile-sheriffs@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-chrome Tbr: pgo-profile-sheriffs@google.com Change-Id: I442ceafbc45094fff8adbce5376ad9d0a1ce13b9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2618067Reviewed-by:
-
arthursonzogni authored
Remove a test fixture trying to turn on the flag "EmbedderCSPEnforcement". It doesn't exist anymore. We got: ``` RuntimeEnabledFeature not recognized: EmbedderCSPEnforcement ``` Bug: None Change-Id: I703055949a6665aef5c089af17cf2ae50c3b2bec Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2612855 Auto-Submit: Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Camille Lamy <clamy@chromium.org> Reviewed-by:
Camille Lamy <clamy@chromium.org> Cr-Commit-Position: refs/heads/master@{#841465}
-
Chromium WPT Sync authored
Using wpt-import in Chromium 9ec63731. With Chromium commits locally applied on WPT: 9ec63731 "[selectors] Fix leaks on :focus-visible tests" 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: cbiesinger@chromium.org: external/wpt/css/css-sizing hbos@chromium.org, hta@chromium.org, guidou@chromium.org, philipp.hancke@googlemail.com: external/wpt/webrtc NOAUTOREVERT=true TBR=foolip@google.com No-Export: true Cq-Include-Trybots: luci.chromium.try:linux-wpt-identity-fyi-rel,linux-wpt-input-fyi-rel Change-Id: Id8e5a35932ab1693aee511f32fb8cec31fb2a2fd Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615344Reviewed-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@{#841464}
-
Stephen McGruer authored
Bug: 1163537 Change-Id: Ibf607274909ddc7435f45dc55a005c5d4f14ff15 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2613148Reviewed-by:
Philip Jägenstedt <foolip@chromium.org> Commit-Queue: Stephen McGruer <smcgruer@chromium.org> Cr-Commit-Position: refs/heads/master@{#841463}
-
Nohemi Fernandez authored
Currently, Chrome does not properly dismiss all of its views when the sign-in flow has been interrupted. Add a DCHECK to ensure all flows lead to proper dismissal. Bug: 1115997 Change-Id: I86151297b16c0a791b2e4b4b4671cf4c0dcbfb81 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617921 Commit-Queue: Nohemi Fernandez <fernandex@chromium.org> Reviewed-by:
Jérôme Lebel <jlebel@chromium.org> Cr-Commit-Position: refs/heads/master@{#841462}
-
Robbie Gibson authored
Now, instead of having the BVC manually set the contentInset, it instead alerts the fullscreen controller that the thumb strip has appeared. The fullscreen controller can then use this to correctly set the insets itself, even when new tabs are added. Also, the BVC now keeps track of whether the thumb strip has caused the view to be translated, so it can re-apply this translation when adding a new web state. Fixed: 1155160 Change-Id: I827d9edf058667a535275367605518512ebca7e9 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2599866Reviewed-by:
Mark Cogan <marq@chromium.org> Reviewed-by:
Chris Lu <thegreenfrog@chromium.org> Commit-Queue: Robbie Gibson <rkgibson@google.com> Cr-Commit-Position: refs/heads/master@{#841461}
-
Ewann Pellé authored
This reverts commit 60e867a2. Reason for revert: this CL breaks ios-simulator-noncq bot Original change's description: > [iOS] Updates system colours to use google material colours > > This CL replaces the existing system colours by Google material colours. > > Bug: 1155472 > Change-Id: Ia9c9e8f31d65fe1db70fd29ad94ca370d0aede07 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2595050 > Reviewed-by: Gauthier Ambard <gambard@chromium.org> > Reviewed-by: Javier Flores <javierrobles@chromium.org> > Commit-Queue: Ewann Pellé <ewannpv@chromium.org> > Cr-Commit-Position: refs/heads/master@{#840999} TBR=gambard@chromium.org,javierrobles@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,ewannpv@chromium.org # Not skipping CQ checks because original CL landed > 1 day ago. Bug: 1155472 Change-Id: Id0fc2b406f14f385366ed14d80fd704924c61aa8 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617920Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Friedrich [CET] <fhorschig@chromium.org> Commit-Queue: Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#841460}
-
Gauthier Ambard authored
This CL extracts the JavaScript logic to find the DOM element on which the user is long pressing and creating the ContextMenuParams out of it. It is also renaming HTMLElementFetchRequest in CRWHTMLElementFetchRequest to respect ios/web standard. Bug: 1140387 Change-Id: I62e6e2df9be54f67706ff440354912b00d77337c Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2580073 Commit-Queue: Gauthier Ambard <gambard@chromium.org> Reviewed-by:
Mike Dougherty <michaeldo@chromium.org> Cr-Commit-Position: refs/heads/master@{#841459}
-
Sergei Datsenko authored
This reverts commit ccba8bc4. Reason for revert: Causes Chrome to crash on Chrome OS: crbug.com/1164258 Original change's description: > Consolidate interactions with |CrasAudioHandler|. > > Previously 2 places in the code were interacting with |CrasAudioHandler|: > - |PlatformApiImpl| subscribes to device changes, and passes this > information to |AudioInputHost|. > - |AudioInputHost| posts the new hotword model to |CrasAudioHandler|. > > This made it hard/impossible to split |AudioInputHost| from > |PlatformApiImpl|, which is needed to handle audio input in the > Libassistant mojom service. > > To solve this I moved all interactions with |CrasAudioHandler| inside a > new class |AudioDevices|, of which an instance is owned by > |AudioInputHost|. > > This also allowed me to add a bunch of new unittests. > > deployed on real hardware. > > Bug: b/171748795 > Test: chromeos_unittests --gtest_filter="Assistant*:Audio*" and manually > Change-Id: I8ec9cec6672236f6365345b292efbe828518be69 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2610953 > Commit-Queue: Xiaohui Chen <xiaohuic@chromium.org> > Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> > Auto-Submit: Jeroen Dhollander <jeroendh@chromium.org> > Cr-Commit-Position: refs/heads/master@{#841155} TBR=xiaohuic@chromium.org,meilinw@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,jeroendh@chromium.org Change-Id: I1f11ed9e14063f91e7f734caf0696748b3d3b7de No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b/171748795 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615734 Commit-Queue: Sergei Datsenko <dats@chromium.org> Reviewed-by:
Sergei Datsenko <dats@chromium.org> Cr-Commit-Position: refs/heads/master@{#841458}
-
Sergei Datsenko authored
This reverts commit c307ddb3. Reason for revert: Blocks revert of crrev.com/c/2610953 Original change's description: > Move |AudioInputHost| to |AssistantManagerServiceImpl| > > The main goal is to ensure we can create |AudioInputHost| independently > of |PlatformApiImpl|. > > This CL also makes |AudioInputHost| virtual so it can be faked during > the unittests of |AssistantManagerServiceImpl|. > > Bug: b/171748795 > Test: chromeos_unittests --gtest_filter="Assistant*:Audio*" > Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome > Change-Id: Ic41ff343b28dae801b0c5f4c8c6417267270d8a2 > Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2610047 > Commit-Queue: Jeroen Dhollander <jeroendh@chromium.org> > Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org> > Cr-Commit-Position: refs/heads/master@{#841226} TBR=xiaohuic@chromium.org,meilinw@chromium.org,chromium-scoped@luci-project-accounts.iam.gserviceaccount.com,jeroendh@chromium.org Change-Id: Ibd2ba5ed1a0388bbf80289812f356ba353015b9c No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: b/171748795 Cq-Include-Trybots: luci.chrome.try:linux-chromeos-chrome Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2615735Reviewed-by:
-
chromium-autoroll authored
https://android.googlesource.com/platform/external/perfetto.git/+log/070ef598892a..3ce4e8ee1301 2021-01-07 primiano@google.com Enable enable_perfetto_trace_processor_httpd on Windows 2021-01-07 primiano@google.com Fix debug, fuzzer and bazel build 2021-01-07 primiano@google.com Merge "base: port UnixSocket to Windows" 2021-01-07 treehugger-gerrit@google.com Merge "gn: Also look for Visual Studio Community edition" 2021-01-07 hjd@google.com Merge "NDEBUG builds of traced should work with both DEBUG and NDEBUG clients" 2021-01-07 ssid@google.com Merge "Fix the chrome thread enum to string conversion" 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: chromium:174454879 Tbr: perfetto-bugs@google.com Test: Test: manual on Windows: open https://ui.perfetto.dev Change-Id: I5de830bda75ec754d5275a39a6e9172ee8cc21c2 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617655Reviewed-by:
-
Friedrich Horschig authored
Redundant line caused a consistent lint failure. NoTry: True Bug: 1046784 Change-Id: I74cf7cc3afeb2b01f5c7ca7c91335009078997ba Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2617787Reviewed-by:
Jan Wilken Dörrie <jdoerrie@chromium.org> Commit-Queue: Friedrich [CET] <fhorschig@chromium.org> Cr-Commit-Position: refs/heads/master@{#841455}
-
