Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] main from chromium:main #116

Merged
merged 133 commits into from
Dec 15, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
133 commits
Select commit Hold shift + click to select a range
5ddf797
[signin] Extend expiring histograms
Dec 15, 2021
4047ab6
[iOS] Fix TableCellCatalogViewController
Dec 15, 2021
35c5646
[Autofill Assistant] Remove dependency on ChromePreferenceKeys
Dec 15, 2021
4ab8a1a
Roll src-internal from 3772b4e1977a to 56b07585df4f (1 revision)
Dec 15, 2021
48f99d5
[iOS] Allow new overflow menu destinations to wrap to 2 lines
rkgibson2 Dec 15, 2021
4cc0ff9
Remove usage of std::function
Dec 15, 2021
777d701
Roll AFDO from 98.0.4758.7_rc-r1-merged to 98.0.4758.8_rc-r1-merged
Dec 15, 2021
8fe5ad9
Support empty fallback in var().
Dec 15, 2021
636e509
cros: Update Lock screen string in OS Settings
Dec 15, 2021
3b0bb9d
Sheriff: Disable policy.AllowWakeLocks tast test
Dec 15, 2021
41f191c
media: Allow render node, AMD, and libv4l2 in hw vd sandbox.
andrescj-chromium Dec 15, 2021
ee3d460
Roll androidx from 51KZuMRRe... to 22GLME4Ax...
Dec 15, 2021
6f0a7c8
CCA: Handle case when ML service is not available
Dec 15, 2021
5b59d2e
Sheriff: Attempt to fix SBNavigationObserverTest on TSan
Dec 15, 2021
b114256
Allow cross-thread use of RenderableGpuMemoryBufferVideoFramePool
dogben Dec 15, 2021
6a20493
Import wpt@bd81cda45784b2dfb12fd8487f01bc15ea9c0170
Dec 15, 2021
58ff673
[devtools] Migrate Database to use AsyncTaskContext
bmeurer Dec 15, 2021
be1f047
Add more threading tests for RenderableGpuMemoryBufferVideoFramePool
dogben Dec 15, 2021
f10a6bf
bindings: Fix the undefined behavior of conversion to int16_t
yuki3 Dec 15, 2021
949fb8c
support-tool: Add OWNERS file for Support Tool folders
Dec 15, 2021
854350f
[devtools] Migrate DOMTask to use AsyncTaskContext
bmeurer Dec 15, 2021
2d70046
CCA: Migrate snackbar.js to TypeScript
peter50216 Dec 15, 2021
20b6952
CHIPS: Add partitioned cookie support to collected cookies view
sauski-alternative Dec 15, 2021
fbb0600
[iOS] Better separator in the web sign-in dialog
Dec 15, 2021
64b7127
Roll ChromeOS Bigcore AFDO profile from 98-4744.1-1639397847-benchmar…
Dec 15, 2021
d773dfc
[iOS] Increase the context menu lines number
Dec 15, 2021
145de12
crosapi: Remove two deprecated fields from BrowserInitParams
GeorgNeis Dec 15, 2021
537ed4e
Sheriff: Disable flaky test NoAutofillSugggestionForCompanyName
Dec 15, 2021
de8c857
Roll ChromeOS Atom AFDO profile from 98-4744.1-1639393599-benchmark-9…
Dec 15, 2021
1b94b6b
Roll Projector App from FchxTKMvf... to jZ1wDChdu...
Dec 15, 2021
528c7e6
[sync] Leverage bookmark parent GUID for incremental updates
Dec 15, 2021
8da7ce7
Update SyncedTabDelegate interface to use Profile::IsChild terminology.
Dec 15, 2021
7b06ca7
[LayoutNG] Clearer name for RemovePositionedObjects's parameter
Loirooriol Dec 15, 2021
078d9f4
Reland "[Passwords] Fix assertion failure when opening password setti…
Dec 15, 2021
3549191
Revert "[LR] Add fieldtrial config entry"
Dec 15, 2021
895853b
CCA: Migrate animation.js to TypeScript
peter50216 Dec 15, 2021
a884462
Roll Chrome Win32 PGO Profile
Dec 15, 2021
a1f2c6e
Roll abseil_revision 9336be04a2..1065514ef3
Dec 15, 2021
2d63950
Roll Fuchsia Internal SDK from 7.20211214.3.1 to 7.20211215.1.1
Dec 15, 2021
caf396e
Remove checks for USE_LIBINPUT in EventDeviceInfo
MaslankaMarek Dec 15, 2021
1597896
[Autofill] Fixed performance regression in AutofillAgent.
schwering Dec 15, 2021
2bd4bba
renderer/core: Allow concurrent marking of selected types
mlippautz Dec 15, 2021
3550211
Fix multidevice_wifi_sync_item_tests
Dec 15, 2021
bdb3591
Diagnostics: Hide empty sections in the input device list
HarryCutts Dec 15, 2021
381c8cf
Before removing any old layout result, clear fragment item associations.
mstensho Dec 15, 2021
3152fa8
Fixed task runner issue in PerfettoPlatform in services_unittests
Dec 15, 2021
c1e37a3
ExternalVkImageBacking: directly create signaled VkSemaphore
Kangz Dec 15, 2021
5074b61
Avoid further initialization of chrome if migration is required.
Dec 15, 2021
004daf6
[Sheriff] Undo 'standalone' for blocklisted-service-in-filter
Dec 15, 2021
1cf8f62
Remove SyncConsentOptional and OsSyncFeatureEnabled
Dec 15, 2021
3dce8d6
DLP: Get Files source urls from dlp daemon
ayamahmod Dec 15, 2021
6de0398
Roll src-internal from 56b07585df4f to 3b07df95bd2e (6 revisions)
Dec 15, 2021
646b8af
Roll WebRTC from 63b97de330fe to d525e2d9f71f (10 revisions)
Dec 15, 2021
f0ad3e3
Implement KeyedService::Shutdown in AccountAppsAvailability
Dec 15, 2021
faaf5db
Roll Chrome Linux PGO Profile
Dec 15, 2021
eae22cf
Migrate LoggedInUserMixin to EmbeddedPolicyTestServer
rryk Dec 15, 2021
f4b56b9
[CustomTabs] Remove expired histograms
Dec 15, 2021
7b2fc96
Allow empty CSS custom properties values.
Dec 15, 2021
dd7cb63
[Autofill Assistant] Remove TabObscuringHandler from dependencies
Dec 15, 2021
43f81c2
Fix crash with computed nodes
aleventhal Dec 15, 2021
8516986
Roll Chrome Mac PGO Profile
Dec 15, 2021
43511eb
[block-in-inline] Rebaseline MultiColumnRenderingTest.Continuation
kojiishi Dec 15, 2021
34d9502
Reland "Reland "Reland "Mac: enable ThinLTO (without unit splitting)"""
speednoisemovement Dec 15, 2021
970e50b
OOBE: Add browsertests for GuestTosScreen
Dec 15, 2021
95252b0
Reapply visible_rect and natural_size after CopyRGBATextureToVideoFrame
dogben Dec 15, 2021
c710e34
[profiles] Re-enable ProfileBrowserTest.CreateOldProfileAsynchronous
Dec 15, 2021
2e773f8
allow alpha formats in WebRtcVideoFrameAdapter for GMB readbacks
dogben Dec 15, 2021
868b597
Remove unused states from HttpStreamFactory::Job
bashi Dec 15, 2021
b5fce3b
[Sheriff] Disable ProfileBrowserTest.CreateNewProfileAsynchronous on Ash
Dec 15, 2021
ffa8fb0
ExternalVkImageBacking: Shim WritePixelsWithData on top of WithCallback
Kangz Dec 15, 2021
e74a859
[Third-Party Client Hint Delegation] (7) Add ability to extract raw A…
arichiv Dec 15, 2021
5321750
Roll ANGLE from 36fcf80b1f2a to 8419f4fcd1ec (12 revisions)
Dec 15, 2021
577cdcf
Roll DevTools Frontend from 8b004bd56791 to b47975a82916 (7 revisions)
Dec 15, 2021
328adb6
Roll Perfetto from f9abf9948a18 to 22467673a50f (2 revisions)
Dec 15, 2021
de533fc
IWYU improvements for usage of base/no_destructor.h
dsanders11 Dec 15, 2021
2ba63f0
Regression test for certificateProvider memory leak fix
Dec 15, 2021
b1a6ea5
Re-enabling
palak8669 Dec 15, 2021
4450d94
Ensure WaylandPopup can be linked for accessibility properly in Lacros
dtsengchromium Dec 15, 2021
ea34ce4
Rename WebLaunchFilesHelper and eliminate some todos.
Dec 15, 2021
1c146c1
[Passwords] Support other types in ShadowTrafficMetricsRecorder
mohamedamir Dec 15, 2021
5f13593
Roll Chrome Win64 PGO Profile
Dec 15, 2021
12f12bf
Fix memory leak in ProfilePolicyConnectorTest.
Dec 15, 2021
496df95
Remove an unneeded GetMainFrame call.
dtapuska Dec 15, 2021
1295314
Roll Depot Tools from 9e5809e98f33 to 48b35e123dff (1 revision)
Dec 15, 2021
64bb3f9
AngleVulkan: support skia write access
phuang Dec 15, 2021
b12be14
[memories] Avoid copies when collapsing duplicates.
Dec 15, 2021
de0e3fe
[launcher]: Show ripple on continue task with context menu open.
Dec 15, 2021
8662d51
Roll RTS model from K9BP5YhVb... to c47NwE87K...
Dec 15, 2021
16731e8
Roll src-internal from 3b07df95bd2e to b52ee54b2a18 (1 revision)
Dec 15, 2021
8c75381
fix typo
brgoldstein Dec 15, 2021
1470658
Reenable DevTools console layout test
paullewis Dec 15, 2021
fe99486
[iOS] Feature Flag for Share Highlighting V2
ciss1995 Dec 15, 2021
5bf7c51
Updating trunk VERSION from 4768.0 to 4769.0
Dec 15, 2021
2eb939e
oobe: Make gesture animation stoppable
Dec 15, 2021
ebd08ae
Handle kScrollByLine in ResolveScrollGranularityToPixels
mehdi-kazemi Dec 15, 2021
ca2b5db
Avoid UAF in ShellJavascriptDialog on Windows
danakj Dec 15, 2021
5d0b6aa
borealis: Fix signed vs unsigned comparison warning
Dec 15, 2021
3d8e7da
Roll Chrome Win32 PGO Profile
Dec 15, 2021
b21fd24
[Extensions] Add UserIsInDeveloperMode metric for the devtools API.
Dec 15, 2021
7482bd5
Fix memory leak in DisplayOverlayController.
Dec 15, 2021
c580363
[Fast Pair] Log time it takes for a successful GATT connection.
julietlevesque Dec 15, 2021
6cae880
Roll TFLite to Next Green Version
mcrouse-chrome Dec 15, 2021
3f478f2
Roll RTS model from h4OErwvGM... to pOC_fgXF3...
Dec 15, 2021
e746b0d
Revert "[Autofill Assistant] Enable animations + Remove filter"
Dec 15, 2021
0038472
Fix typos in trailing namespace comments
dsanders11 Dec 15, 2021
6b865f1
Import wpt@84ce703733f91ea73315762a42ac183784e555ba
Dec 15, 2021
da0ec20
Cleanup: Remove unnecessary url.is_valid() check in process reuse.
creis Dec 15, 2021
2442486
Roll Perfetto Trace Processor Linux from f9abf9948a18 to 22467673a50f
Dec 15, 2021
0763f4e
Automatic update for log_list.json
Dec 15, 2021
38ad3e8
Load source maps via chrome-extension:// on the backend.
danilsomsikov Dec 15, 2021
de2b9ae
[MPArch] Correct GetMainFrame access for apps/digital_goods
cdesouza-chromium Dec 15, 2021
9546e5a
[sheriff] Disable flaky TabSuggestionMessageCardTest tests
Dec 15, 2021
eedd4fa
Roll Fuchsia SDK from 7.20211215.0.1 to 7.20211215.1.1
Dec 15, 2021
41e571c
[Android] Disable TabObserverRegistrarTest on arm64
Dec 15, 2021
fd5a1a3
variations: Remove test config for PowerScheduler experiments.
betasheet Dec 15, 2021
b425c00
ash: Enable test WindowSizerTest.OpenBrowserUsingShelfItem
Dec 15, 2021
d0a771d
[Android] Disable FullscreenManager test while investigating
colinblundell Dec 15, 2021
ec40a2d
Roll DevTools Frontend from b47975a82916 to d692cac31ffe (6 revisions)
Dec 15, 2021
ef9c865
Build HarfBuzz color implementation
drott Dec 15, 2021
c31ecb7
Add crash directory to no copy list for profile migration.
Dec 15, 2021
22cfae3
Audio service: stop mixing if no listeners and the last track is gone
Dec 15, 2021
7256603
Don't miss AREA elements during block-fragmented pre-paint.
mstensho Dec 15, 2021
2aaf5b4
Roll Chromite from 7f13e5c5ff2d to 1cad40c1e720 (20 revisions)
Dec 15, 2021
d0eca2a
[Autofill] Fixed invalidated WeakPtr<AutofillPopupControllerImpl> deref.
schwering Dec 15, 2021
be497a2
Roll Help App from RcDM8Mq_L... to 27KUeowb-...
Dec 15, 2021
b0878c4
Fix 'selected' voiced on unselected buttons for VoiceOver
spectranaut Dec 15, 2021
950bf24
Fix null-dereference READ in Element::Blur
bebeaudr Dec 15, 2021
a965e95
[Passwords] Fix logging AddCredentialFromSettings.UserAction
Dec 15, 2021
3b14798
Remove NonSFI mode part 11.
Dec 15, 2021
edc6c31
Referrer policy: Try re-enabling mac tests
domfarolino Dec 15, 2021
6f719eb
Bootstrap resolution implementation
bemasc-uproxy Dec 15, 2021
45a47d8
Roll androidx from 22GLME4Ax... to rjqESjGVn...
Dec 15, 2021
26055a3
Use OS11 as host for WebRTC Chromium FYI ios-simulator
Dec 15, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
26 changes: 13 additions & 13 deletions DEPS
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ vars = {
# Three lines of non-changing comments so that
# the commit queue can handle CLs rolling ANGLE
# and whatever else without interference from each other.
'angle_revision': '36fcf80b1f2a99fdaa46d044994dfe96a08d7362',
'angle_revision': '8419f4fcd1ec05ffac0ac51a1ac73f343dab00f4',
# Three lines of non-changing comments so that
# the commit queue can handle CLs rolling SwiftShader
# and whatever else without interference from each other.
Expand Down Expand Up @@ -314,7 +314,7 @@ vars = {
# Three lines of non-changing comments so that
# the commit queue can handle CLs rolling devtools-frontend
# and whatever else without interference from each other.
'devtools_frontend_revision': '8b004bd567919de941703655d5f312465428fc0a',
'devtools_frontend_revision': 'd692cac31ffea822377407e775750d9f6a042eba',
# Three lines of non-changing comments so that
# the commit queue can handle CLs rolling libprotobuf-mutator
# and whatever else without interference from each other.
Expand Down Expand Up @@ -716,7 +716,7 @@ deps = {
'packages': [
{
'package': 'chromium/rts/model/linux-amd64',
'version': 'h4OErwvGMzWmTzk59jLdAUp9o00HkQ6vKZlwtDCstocC',
'version': 'pOC_fgXF3UQ176xy8dq5Ue_0mDiLot4PGslfbyWcVZsC',
},
],
'dep_type': 'cipd',
Expand All @@ -738,7 +738,7 @@ deps = {
'packages': [
{
'package': 'chromium/rts/model/windows-amd64',
'version': 'K9BP5YhVbtP-iXTbJU0dbwRi9gaMI_w2x0OtulD3R_oC',
'version': 'c47NwE87KkjD2MrTSkYBCdjdqtM-53Q3x4D55ICz5HgC',
},
],
'dep_type': 'cipd',
Expand Down Expand Up @@ -799,7 +799,7 @@ deps = {
'packages': [
{
'package': 'chromium/third_party/androidx',
'version': '51KZuMRReilT3sNyF1-vaB_9ciuTsRJxv8kUH1eLbDgC',
'version': 'rjqESjGVnqj61OsILGz2zAx9aZu_63YrdoiwrUlJrHoC',
},
],
'condition': 'checkout_android',
Expand Down Expand Up @@ -1018,7 +1018,7 @@ deps = {
# Tools used when building Chrome for Chrome OS. This affects both the Simple
# Chrome workflow, as well as the chromeos-chrome ebuild.
'src/third_party/chromite': {
'url': Var('chromium_git') + '/chromiumos/chromite.git' + '@' + '7f13e5c5ff2def048f60931a4f09bdcc0c7c3965',
'url': Var('chromium_git') + '/chromiumos/chromite.git' + '@' + '1cad40c1e7201ae3c1273066c335771590bdce2b',
'condition': 'checkout_chromeos',
},

Expand All @@ -1038,7 +1038,7 @@ deps = {
},

'src/third_party/depot_tools':
Var('chromium_git') + '/chromium/tools/depot_tools.git' + '@' + '9e5809e98f33a5f9f9d3b69a5bd826ce8a7a5a81',
Var('chromium_git') + '/chromium/tools/depot_tools.git' + '@' + '48b35e123dff26b1ede11104316cbb7f462db4e8',

'src/third_party/devtools-frontend/src':
Var('chromium_git') + '/devtools/devtools-frontend' + '@' + Var('devtools_frontend_revision'),
Expand Down Expand Up @@ -1421,7 +1421,7 @@ deps = {
},

'src/third_party/perfetto':
Var('android_git') + '/platform/external/perfetto.git' + '@' + 'f9abf9948a180a56a3595ec54ff9f2f5c2c9947c',
Var('android_git') + '/platform/external/perfetto.git' + '@' + '22467673a50f5599e7d8b1f49982faf7c1bc30d9',

'src/third_party/perl': {
'url': Var('chromium_git') + '/chromium/deps/perl.git' + '@' + '6f3e5028eb65d0b4c5fdd792106ac4c84eee1eb3',
Expand Down Expand Up @@ -1582,7 +1582,7 @@ deps = {
},

'src/third_party/tflite/src':
Var('chromium_git') + '/external/github.com/tensorflow/tensorflow.git' + '@' + 'aa79adb41c2f60c81d5d46d994c061abab686bb1',
Var('chromium_git') + '/external/github.com/tensorflow/tensorflow.git' + '@' + 'dd57f5328f37a81197b0dadd052e05c9d9461b16',

'src/third_party/turbine': {
'packages': [
Expand Down Expand Up @@ -1642,7 +1642,7 @@ deps = {
Var('chromium_git') + '/external/github.com/gpuweb/cts.git' + '@' + 'c843f8d63c8c17acfbb7d48e09059a581ba779b9',

'src/third_party/webrtc':
Var('webrtc_git') + '/src.git' + '@' + '63b97de330fe3d4775b9b4df8ad15c7593d58fc0',
Var('webrtc_git') + '/src.git' + '@' + 'd525e2d9f71f02832a8c8d2d63d47b96c16a00fe',

'src/third_party/libgifcodec':
Var('skia_git') + '/libgifcodec' + '@'+ Var('libgifcodec_revision'),
Expand Down Expand Up @@ -1700,7 +1700,7 @@ deps = {
Var('chromium_git') + '/v8/v8.git' + '@' + Var('v8_revision'),

'src-internal': {
'url': 'https://chrome-internal.googlesource.com/chrome/src-internal.git@3772b4e1977a26513f2da475016bd6912a1c9c65',
'url': 'https://chrome-internal.googlesource.com/chrome/src-internal.git@b52ee54b2a18637068d39e2aa0f4d94588076891',
'condition': 'checkout_src_internal',
},

Expand All @@ -1719,7 +1719,7 @@ deps = {
'packages': [
{
'package': 'chromeos_internal/apps/help_app/app',
'version': 'RcDM8Mq_LrCGiuxfQto-gkDVFbqCRy6FE8lzPeHsNEwC',
'version': '27KUeowb-avScJevs9PyckybR1s34lIOg_CMEaolREQC',
},
],
'condition': 'checkout_chromeos and checkout_src_internal',
Expand All @@ -1741,7 +1741,7 @@ deps = {
'packages': [
{
'package': 'chromeos_internal/apps/projector_app/app',
'version': 'FchxTKMvfxSkGTLMU23cSyqvexH52wHP-zPlLYF6QqUC',
'version': 'jZ1wDChducDMF-MWFvXLkLZpk_AWan76x5hDRvEhqLoC',
},
],
'condition': 'checkout_chromeos and checkout_src_internal',
Expand Down
1 change: 0 additions & 1 deletion android_webview/browser/aw_pac_processor.cc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
#include "base/bind.h"
#include "base/logging.h"
#include "base/memory/raw_ptr.h"
#include "base/no_destructor.h"
#include "base/task/post_task.h"
#include "base/task/thread_pool/thread_pool_instance.h"
#include "base/threading/thread_restrictions.h"
Expand Down
1 change: 0 additions & 1 deletion android_webview/browser/gfx/task_queue_webview.cc
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@
#include "base/bind.h"
#include "base/containers/queue.h"
#include "base/logging.h"
#include "base/no_destructor.h"
#include "base/synchronization/condition_variable.h"
#include "base/synchronization/lock.h"
#include "base/thread_annotations.h"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@
#include "base/files/file_path.h"
#include "base/metrics/histogram_functions.h"
#include "base/metrics/persistent_histogram_allocator.h"
#include "base/no_destructor.h"
#include "base/time/time.h"
#include "components/metrics/metrics_pref_names.h"
#include "components/metrics/metrics_service.h"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
#include "base/callback_helpers.h"
#include "base/check.h"
#include "base/command_line.h"
#include "base/no_destructor.h"
#include "base/threading/thread_task_runner_handle.h"
#include "base/time/time.h"
#include "components/component_updater/component_installer.h"
Expand Down
1 change: 1 addition & 0 deletions android_webview/test/shell/src/draw_fn/allocator.cc
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@

#include "android_webview/public/browser/draw_fn.h"
#include "base/logging.h"
#include "base/no_destructor.h"
#include "base/notreached.h"

namespace draw_fn {
Expand Down
53 changes: 53 additions & 0 deletions ash/app_list/views/continue_section_view_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@
#include "testing/gtest/include/gtest/gtest.h"
#include "ui/events/event.h"
#include "ui/events/keycodes/keyboard_codes.h"
#include "ui/views/animation/ink_drop.h"
#include "ui/views/controls/textfield/textfield.h"

namespace ash {
Expand Down Expand Up @@ -726,5 +727,57 @@ TEST_P(ContinueSectionViewTest, ShowContinueSectionWhithMinimumFiles) {
EXPECT_TRUE(GetContinueSectionView()->GetVisible());
}

TEST_P(ContinueSectionViewTest, TaskViewHasRippleWithMenuOpen) {
AddSearchResult("id1", AppListSearchResultType::kFileChip);
AddSearchResult("id2", AppListSearchResultType::kDriveChip);
AddSearchResult("id3", AppListSearchResultType::kDriveChip);

EnsureLauncherShown();
VerifyResultViewsUpdated();

ContinueTaskView* continue_task_view = GetResultViewAt(0);
EXPECT_EQ(continue_task_view->result()->id(), "id1");

GetContinueSectionView()->GetWidget()->LayoutRootViewIfNecessary();
SimulateRightClickOrLongPressAt(
continue_task_view->GetBoundsInScreen().CenterPoint());
EXPECT_TRUE(continue_task_view->IsMenuShowing());

EXPECT_EQ(views::InkDropState::ACTIVATED,
views::InkDrop::Get(continue_task_view)
->GetInkDrop()
->GetTargetInkDropState());
}

TEST_P(ContinueSectionViewTest, TaskViewHidesRippleAfterMenuCloses) {
AddSearchResult("id1", AppListSearchResultType::kFileChip);
AddSearchResult("id2", AppListSearchResultType::kDriveChip);
AddSearchResult("id3", AppListSearchResultType::kDriveChip);

EnsureLauncherShown();
VerifyResultViewsUpdated();

ContinueTaskView* continue_task_view = GetResultViewAt(0);
EXPECT_EQ(continue_task_view->result()->id(), "id1");

GetContinueSectionView()->GetWidget()->LayoutRootViewIfNecessary();
SimulateRightClickOrLongPressAt(
continue_task_view->GetBoundsInScreen().CenterPoint());
EXPECT_TRUE(continue_task_view->IsMenuShowing());

// Click on other task view to hide context menu.
GetContinueSectionView()->GetWidget()->LayoutRootViewIfNecessary();
SimulateRightClickOrLongPressAt(
GetResultViewAt(2)->GetBoundsInScreen().CenterPoint());
EXPECT_FALSE(continue_task_view->IsMenuShowing());

// Wait for the view to update the ink drop.
base::RunLoop().RunUntilIdle();

EXPECT_EQ(views::InkDropState::HIDDEN, views::InkDrop::Get(continue_task_view)
->GetInkDrop()
->GetTargetInkDropState());
}

} // namespace
} // namespace ash
17 changes: 16 additions & 1 deletion ash/app_list/views/continue_task_view.cc
Original file line number Diff line number Diff line change
Expand Up @@ -152,6 +152,8 @@ gfx::Size ContinueTaskView::CalculatePreferredSize() const {
}

void ContinueTaskView::OnButtonPressed(const ui::Event& event) {
views::InkDrop::Get(this)->GetInkDrop()->AnimateToState(
views::InkDropState::ACTION_TRIGGERED);
OpenResult(event.flags());
}

Expand All @@ -173,12 +175,15 @@ void ContinueTaskView::OnMetadataChanged() {

void ContinueTaskView::UpdateResult() {
SetVisible(!!result());
views::InkDrop::Get(this)->GetInkDrop()->AnimateToState(
views::InkDropState::HIDDEN);
CloseContextMenu();

if (!result()) {
SetIcon(gfx::ImageSkia());
title_->SetText(std::u16string());
subtitle_->SetText(std::u16string());
GetViewAccessibility().OverrideName(std::u16string());
CloseContextMenu();
return;
}

Expand All @@ -195,6 +200,9 @@ void ContinueTaskView::OnResultDestroying() {
}

void ContinueTaskView::SetResult(SearchResult* result) {
if (result_ == result)
return;

search_result_observation_.Reset();

result_ = result;
Expand Down Expand Up @@ -222,6 +230,8 @@ void ContinueTaskView::ShowContextMenuForViewImpl(
source->GetWidget(), nullptr /*button_controller*/,
source->GetBoundsInScreen(), views::MenuAnchorPosition::kBubbleTopRight,
source_type);
views::InkDrop::Get(this)->GetInkDrop()->AnimateToState(
views::InkDropState::ACTIVATED);
}

void ContinueTaskView::ExecuteCommand(int command_id, int event_flags) {
Expand Down Expand Up @@ -254,6 +264,11 @@ ui::SimpleMenuModel* ContinueTaskView::BuildMenuModel() {
return context_menu_model_.get();
}

void ContinueTaskView::MenuClosed(ui::SimpleMenuModel* menu) {
views::InkDrop::Get(this)->GetInkDrop()->AnimateToState(
views::InkDropState::HIDDEN);
}

void ContinueTaskView::OpenResult(int event_flags) {
DCHECK(result());
view_delegate_->OpenSearchResult(
Expand Down
1 change: 1 addition & 0 deletions ash/app_list/views/continue_task_view.h
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ class ASH_EXPORT ContinueTaskView : public views::Button,

// SimpleMenuModel::Delegate:
void ExecuteCommand(int command_id, int event_flags) override;
void MenuClosed(ui::SimpleMenuModel* source) override;

private:
void SetIcon(const gfx::ImageSkia& icon);
Expand Down
1 change: 0 additions & 1 deletion ash/clipboard/clipboard_nudge_controller.cc
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
#include "base/logging.h"
#include "base/metrics/histogram_functions.h"
#include "base/metrics/histogram_macros.h"
#include "base/no_destructor.h"
#include "components/prefs/pref_registry_simple.h"
#include "components/prefs/pref_service.h"
#include "components/prefs/scoped_user_pref_update.h"
Expand Down
1 change: 0 additions & 1 deletion ash/components/arc/session/arc_vm_client_adapter.cc
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@
#include "base/logging.h"
#include "base/memory/weak_ptr.h"
#include "base/metrics/histogram_functions.h"
#include "base/no_destructor.h"
#include "base/posix/eintr_wrapper.h"
#include "base/process/launch.h"
#include "base/process/process_metrics.h"
Expand Down
1 change: 1 addition & 0 deletions ash/components/drivefs/fake_drivefs_launcher_client.cc
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
#include <vector>

#include "base/bind.h"
#include "base/no_destructor.h"
#include "base/strings/strcat.h"
#include "base/strings/string_util.h"
#include "base/system/sys_info.h"
Expand Down
12 changes: 0 additions & 12 deletions ash/constants/ash_features.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1154,14 +1154,6 @@ const base::Feature kSmartLockUIRevamp{"SmartLockUIRevamp",
const base::Feature kSyncSettingsCategorization{
"SyncSettingsCategorization", base::FEATURE_DISABLED_BY_DEFAULT};

// Updates the OOBE sync consent screen
//
// NOTE: The feature will be rolled out via a client-side Finch trial, so the
// actual state will vary. TODO(https://crbug.com/1227417): Migrate config in
// chrome/browser/ash/sync/sync_consent_optional_field_trial.cc
const base::Feature kSyncConsentOptional{"SyncConsentOptional",
base::FEATURE_DISABLED_BY_DEFAULT};

// Enables battery indicator for styluses in the palette tray
const base::Feature kStylusBatteryStatus{"StylusBatteryStatus",
base::FEATURE_ENABLED_BY_DEFAULT};
Expand Down Expand Up @@ -1822,10 +1814,6 @@ bool IsSyncSettingsCategorizationEnabled() {
return base::FeatureList::IsEnabled(kSyncSettingsCategorization);
}

bool IsSyncConsentOptionalEnabled() {
return base::FeatureList::IsEnabled(kSyncConsentOptional);
}

bool IsStylusBatteryStatusEnabled() {
return base::FeatureList::IsEnabled(kStylusBatteryStatus);
}
Expand Down
2 changes: 0 additions & 2 deletions ash/constants/ash_features.h
Original file line number Diff line number Diff line change
Expand Up @@ -438,7 +438,6 @@ extern const base::Feature kSmartLockUIRevamp;
COMPONENT_EXPORT(ASH_CONSTANTS)
extern const base::Feature kSnoopingProtection;
COMPONENT_EXPORT(ASH_CONSTANTS) extern const base::Feature kStylusBatteryStatus;
COMPONENT_EXPORT(ASH_CONSTANTS) extern const base::Feature kSyncConsentOptional;
COMPONENT_EXPORT(ASH_CONSTANTS)
extern const base::Feature kSyncSettingsCategorization;
COMPONENT_EXPORT(ASH_CONSTANTS)
Expand Down Expand Up @@ -629,7 +628,6 @@ COMPONENT_EXPORT(ASH_CONSTANTS) bool IsShelfLauncherNudgeEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsShimlessRMAFlowEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsSnoopingProtectionEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsSyncSettingsCategorizationEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsSyncConsentOptionalEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsStylusBatteryStatusEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsSystemChinesePhysicalTypingEnabled();
COMPONENT_EXPORT(ASH_CONSTANTS) bool IsSystemJapanesePhysicalTypingEnabled();
Expand Down
1 change: 0 additions & 1 deletion ash/drag_drop/tab_drag_drop_delegate.cc
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,6 @@
#include "ash/wm/splitview/split_view_drag_indicators.h"
#include "ash/wm/splitview/split_view_utils.h"
#include "ash/wm/tablet_mode/tablet_mode_browser_window_drag_session_windows_hider.h"
#include "base/no_destructor.h"
#include "base/pickle.h"
#include "base/strings/utf_string_conversions.h"
#include "ui/base/clipboard/clipboard_format_type.h"
Expand Down
1 change: 0 additions & 1 deletion ash/drag_drop/tab_drag_drop_delegate_unittest.cc
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
#include "ash/wm/splitview/split_view_controller.h"
#include "ash/wm/tablet_mode/tablet_mode_controller_test_api.h"
#include "base/containers/flat_map.h"
#include "base/no_destructor.h"
#include "base/pickle.h"
#include "base/strings/utf_string_conversions.h"
#include "base/test/gmock_callback_support.h"
Expand Down
1 change: 0 additions & 1 deletion ash/hud_display/ash_tracing_handler.cc
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@
#include "base/files/file.h"
#include "base/files/platform_file.h"
#include "base/logging.h"
#include "base/no_destructor.h"
#include "base/task/task_traits.h"
#include "base/task/thread_pool.h"
#include "base/threading/sequenced_task_runner_handle.h"
Expand Down
7 changes: 7 additions & 0 deletions ash/quick_pair/common/fast_pair/fast_pair_metrics.cc
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,8 @@ const char kRetroactiveEngagementFlowMetric[] =
const char kPairingMethodMetric[] = "Bluetooth.ChromeOS.FastPair.PairingMethod";
const char kRetroactivePairingResultMetric[] =
"Bluetooth.ChromeOS.FastPair.RetroactivePairing.Result";
const char kTotalGattConnectionTimeMetric[] =
"Bluetooth.ChromeOS.FastPair.TotalGattConnectionTime";

} // namespace

Expand Down Expand Up @@ -84,5 +86,10 @@ void RecordRetroactivePairingResult(bool success) {
base::UmaHistogramBoolean(kRetroactivePairingResultMetric, success);
}

void RecordTotalGattConnectionTime(base::TimeDelta total_gatt_connection_time) {
base::UmaHistogramTimes(kTotalGattConnectionTimeMetric,
total_gatt_connection_time);
}

} // namespace quick_pair
} // namespace ash
Loading