Publish DEPS for 74.0.3729.61

• DEPS[diff]

Incrementing VERSION to 74.0.3729.61

[email protected]

Change-Id: I649078b2bd70d34e0727721d4993a0e20c7dbe89
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549832
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#645}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Measure Overview Animation as a whole.

This introduces FpsCounter which can measure multiple animtions
instead of individual transform animations.

This also splits the overview animation smoothness into
Clamshell/SingleClamshell/Tablet/SplitView because they
perform quite different animations.

No logical change but contains a few refacotring:
* notify OverviewGrid the cancelation of starting animation so
  that we can measure this scenario.
* Split the animation type computation and applying them so that
  we can specify the type of overview animation in fps counter.

Bug: 923159
Test: covered by unittests
Change-Id: I4cec30f1a327d349316786a0031251fd6420285d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1526713
Commit-Queue: Mitsuru Oshima <[email protected]>
Reviewed-by: Xiaoqian Dai <[email protected]>
Reviewed-by: Steven Holte <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643072}(cherry picked from commit bdb60f94e9177e11ba3f0fac1ffea14d31b62444)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549986
Reviewed-by: Mitsuru Oshima <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#644}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/metrics/histogram_macros.cc[diff]
• ash/metrics/histogram_macros.h[diff]
• ash/public/cpp/BUILD.gn[diff]
• ash/public/cpp/fps_counter.cc[Added - diff]
• ash/public/cpp/fps_counter.h[Added - diff]
• ash/wm/overview/overview_grid.cc[diff]
• ash/wm/overview/overview_grid.h[diff]
• ash/wm/overview/overview_session.cc[diff]
• ash/wm/overview/overview_session.h[diff]
• ash/wm/overview/overview_session_unittest.cc[diff]
• ash/wm/overview/scoped_overview_animation_settings.cc[diff]
• ash/wm/splitview/split_view_controller.cc[diff]
• ash/wm/splitview/split_view_controller.h[diff]
• ash/wm/splitview/split_view_controller_unittest.cc[diff]
• tools/metrics/histograms/histograms.xml[diff]

Do not check if reporting has started.

The overview animation can continue from the previous one,
so it shouldn't check if it's started.

Bug: 923159
Test: none
Change-Id: Iec70a1ffff8a070073fe887c3a594c9577a695d7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1526745
Reviewed-by: Alex Newcomer <[email protected]>
Commit-Queue: Mitsuru Oshima <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#641739}(cherry picked from commit eff4bc6dd37a296e2a9b1ea44d21b1797178e46f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1550003
Reviewed-by: Mitsuru Oshima <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#643}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/app_list/presenter/app_list_presenter_impl.cc[diff]

Record WindowSelector.Smoothness for each clamshell and tablet mode

This CL deprecates the existing WindowSelector.Smoothness UMA stats
and introduces .Clamshell/.Tablet stats for each mode.

Rename WindowSelector -> Overview to match the class name and feature name.

Bug: 923159
Test: covered by unittests
Change-Id: I074d35b0749a25cab1f2bf85f874fe8ca10424b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1431336
Commit-Queue: Mitsuru Oshima <[email protected]>
Reviewed-by: Brian White <[email protected]>
Reviewed-by: Xiaoqian Dai <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#640451}(cherry picked from commit cae84ec07de66308322a3818ebfb56449383e3df)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549319
Reviewed-by: Mitsuru Oshima <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#642}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/BUILD.gn[diff]
• ash/metrics/histogram_macros.cc[Added - diff]
• ash/metrics/histogram_macros.h[Added - diff]
• ash/metrics/histogram_macros_unittest.cc[Added - diff]
• ash/wm/overview/scoped_overview_animation_settings.cc[diff]
• tools/metrics/histograms/histograms.xml[diff]

Record Apps.StateTransition.Smoothness for clamshell/tablet mode separately

Introduced new EnterOvervie/ExitOverview suffixes to record animations
triggered by overview, as it applies quite different animations.

Bug: 923159
Change-Id: Ia0066e96f8d59bbadd40f7ce3f419d13d41f38fd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1513295
Reviewed-by: Robert Kaplow <[email protected]>
Reviewed-by: Alex Newcomer <[email protected]>
Commit-Queue: Mitsuru Oshima <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#640221}(cherry picked from commit 64c01bae1c499364a69f20e19a38f23382603045)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549984
Reviewed-by: Mitsuru Oshima <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#641}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/app_list/presenter/app_list_presenter_impl.cc[diff]
• ash/app_list/presenter/app_list_presenter_impl.h[diff]
• ash/app_list/views/app_list_view.cc[diff]
• ash/app_list/views/app_list_view.h[diff]
• tools/metrics/histograms/histograms.xml[diff]

[DevTools] Fix NodeMain to not use flatten protocol

We cannot route flatten protocol to node targets, so we should
keep wrapping node subtragets with Target.sendMessageToTarget.

[email protected]

(cherry picked from commit bce2ece30c5c3644e22cae84f472166542a51309)

Bug: 941608
Change-Id: I0b04379e143c32b61c3ede6047d6c967cd96af06
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538484
Reviewed-by: Pavel Feldman <[email protected]>
Commit-Queue: Dmitry Gozman <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644616}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549544
Reviewed-by: Dmitry Gozman <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#640}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/devtools/front_end/node_main/NodeMain.js[diff]
• third_party/blink/renderer/devtools/front_end/protocol/InspectorBackend.js[diff]
• third_party/blink/renderer/devtools/front_end/sdk/Target.js[diff]
• third_party/blink/renderer/devtools/front_end/sdk/TargetManager.js[diff]

Work around BeginFrame/Input ordering jank

Currently, we send BeginFrames to the browser/renderer in an arbitrary
order. During a gesture, the browser will then send input to the
renderer.

In cases where the browser gets the BeginFrame first, it can
occasionally send input to the renderer before the renderer gets its
BeginFrame. When this happens sporadically it can lead to jank.

This CL ensures that we send BeginFrame to root frame sinks last,
which greatly reduces the chances of the above scenario ocurring, and
removes all observed jank.

Longer term this code will be removed and we'll replace this with
a more robust system of tagging input. However this CL is designed
to have no downside (the order of BeginFrames is currently arbitrary,
so chosing a different one should be safe) and be merge safe.

Bug: 947717
Change-Id: I85b11aedabb4fa8888e7ddc11ae1b9a839db1e45
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546427
Reviewed-by: Khushal <[email protected]>
Commit-Queue: Eric Karl <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646105}(cherry picked from commit 436ce9dcce6cab80f5832c546718ae30ad88594c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549893
Reviewed-by: Eric Karl <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#639}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/viz/common/frame_sinks/begin_frame_source.cc[diff]
• components/viz/common/frame_sinks/begin_frame_source.h[diff]
• components/viz/common/frame_sinks/begin_frame_source_unittest.cc[diff]
• components/viz/service/frame_sinks/compositor_frame_sink_support.cc[diff]
• components/viz/service/frame_sinks/compositor_frame_sink_support.h[diff]
• components/viz/test/begin_frame_source_test.h[diff]

libvpx: Cherry pick changes related to highbd subtract func.

BUG=945084

Change-Id: Icc89cb273095e3976e52c81bd132cdfbc41fbbdb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548066
Reviewed-by: James Zern <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#638}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• DEPS[diff]
• third_party/libvpx/README.chromium[diff]
• third_party/libvpx/source/config/vpx_version.h[diff]

[M74] [NTP] Fix :not selectors for fakebox and dark mode

The body:not() selectors were improperly defined, which caused the
fakebox to stay light colored when dark mode was enabled. Combining the
two fixes this issue.

Screencast: https://drive.google.com/open?id=1Y3THliuO6W7GzMsa5_iFRAexexvFVdmQ

Bug: 944624
Change-Id: Ib136562bb957613a05bdb54101bd63bdcde3a369
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542145
Reviewed-by: Kyle Milka <[email protected]>
Reviewed-by: Ramya Nagarajan <[email protected]>
Commit-Queue: Kristi Park <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645062}(cherry picked from commit 32ec2e52d4c1452e133c318dfb5f1dc5eb70ca0c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549888
Reviewed-by: Kristi Park <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#637}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/resources/local_ntp/local_ntp.css[diff]

Do not regenerate local-surface-id on DSF change in the client

When the device-scale-factor changes, the change is currently handled
in the window server. The server re-generate the local surface id and
notifies to the client through OnWindowBoundsChanged. When this
mojo call arrives to the client, it's propagated to
TopLevelAllocator::OnDeviceScaleFactorChange, and it re-generate again
the local surface id, and sends back to the window server through
SetWindowBounds(). This causes some troubles.

This means that TopLevelAllocator does not have to generate the local
surface id on DSF change since it's already handled in the server.
This CL skips this regeneration.

Bug: 942647
Test: the new test case in aura_unittests
Change-Id: I86f9d8952082f1b2e7bb9ab74e38f556783447f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540069
Reviewed-by: Scott Violet <[email protected]>
Commit-Queue: Jun Mukai <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644613}(cherry picked from commit adec8ecac6c72db3156b4515295df75a18fb03f0)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549885
Reviewed-by: Jun Mukai <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#636}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/aura/mus/mus_lsi_allocator.cc[diff]
• ui/aura/mus/mus_lsi_allocator.h[diff]
• ui/aura/mus/window_tree_client_unittest.cc[diff]

Network Service: Update URLLoader::OpenFilesForUpload to open limited number of files at a time

This CL updates URLLoader::OpenFilesForUpload to open limited number of
files at a time.

Bug: 939606, 944218
Change-Id: I69f262e122fbdd9216393656d75645cd5b60f450
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1520317
Reviewed-by: Ken Rockot <[email protected]>
Reviewed-by: Wez <[email protected]>
Reviewed-by: Robbie McElrath <[email protected]>
Commit-Queue: Jun Cai <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644509}(cherry picked from commit 647227b51c36eb4fa620dd47104f7a32f76f6fb7)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547959
Reviewed-by: Jun Cai <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#635}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• services/network/url_loader.cc[diff]
• services/network/url_loader.h[diff]

Incrementing VERSION to 74.0.3729.60

[email protected]

Change-Id: I127bf54de693adf672da1e91b8727113c4372aec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549881
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#634}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Revert "[css-flexbox] Correctly calculate min-height with justify-content"

This reverts commit 06c2e9c9e1adc70061462a219e5961cae30fcf9d.

Reason for revert: I'm told this breaks Android

Original change's description:
> [css-flexbox] Correctly calculate min-height with justify-content
> 
> Spacing added by justification is not meaninfully part of the
> intrinsic block size of an item; remove it.
> 
> This is especially problematic with percentage sizes, because
> with height: 100% and justify-content: flex-end we would
> position any flex items at the end and calculate a minimum
> size based on the 100%, even though we ought to ignore percentages.
> 
> R=​​[email protected], [email protected]
> 
> Bug: 945214
> Change-Id: If4e271df5e550807632d30e5dd1c2b3068d45313
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546002
> Commit-Queue: Emil A Eklund <[email protected]>
> Reviewed-by: David Grogan <[email protected]>
> Reviewed-by: Emil A Eklund <[email protected]>
> Auto-Submit: Christian Biesinger <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#646178}(cherry picked from commit ae2cd00736e2f3c07a9ca10aef4da0e5e4ca36df)
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549204
> Reviewed-by: Christian Biesinger <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#628}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

[email protected]

Change-Id: Ic7b292d80c6551199d3c60c065a71c87e012c597
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 945214
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549691
Reviewed-by: Christian Biesinger <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#633}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/layout/flexible_box_algorithm.cc[diff]
• third_party/blink/renderer/core/layout/flexible_box_algorithm.h[diff]
• third_party/blink/renderer/core/layout/layout_flexible_box.cc[diff]
• third_party/blink/renderer/core/layout/ng/ng_flex_layout_algorithm.cc[diff]
• third_party/blink/web_tests/TestExpectations[diff]
• third_party/blink/web_tests/external/wpt/css/css-flexbox/flex-minimum-height-flex-items-014.html[Deleted - diff]

Don't use the SystemRequestContext in ChromeOS CRD.

SystemRequestContext doesn't work when the network service is enabled.
This CL makes ChromotingHostContext create its own URLRequestContext
instead of using the SystemRequestContext, as it does on all other
platforms.

Bug: 944324
Change-Id: Icfeab1ae42978377c43df15c0b81359f773454ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546758
Reviewed-by: Jamie Walch <[email protected]>
Reviewed-by: John Abd-El-Malek <[email protected]>
Reviewed-by: Bartosz Fabianowski <[email protected]>
Commit-Queue: John Abd-El-Malek <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646920}(cherry picked from commit 3a2a6915b37936708ad9d3104072e28956d40cb8)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549569
Reviewed-by: Robbie McElrath <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#632}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/chromeos/policy/remote_commands/crd_host_delegate.cc[diff]
• chrome/browser/extensions/api/messaging/native_message_host_chromeos.cc[diff]
• content/browser/browser_main_loop.cc[diff]
• remoting/host/chromoting_host_context.cc[diff]
• remoting/host/chromoting_host_context.h[diff]
• remoting/host/it2me/it2me_native_messaging_host_chromeos.cc[diff]
• remoting/host/it2me/it2me_native_messaging_host_chromeos.h[diff]

Incrementing VERSION to 74.0.3729.59

[email protected]

Change-Id: Ia7b2b297371f6c82ee6eab78470924291106466e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549614
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#631}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

[m74] CORB allowlist additions - 2019-04-01.

[email protected]

(cherry picked from commit aacf15f70f4bda558d35d82fba19c3c063ba3e89)

Bug: 937081
Change-Id: Ic41418ef66e76375171ba3e36c660cada83ee0bd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546858
Commit-Queue: Łukasz Anforowicz <[email protected]>
Commit-Queue: Karan Bhatia <[email protected]>
Auto-Submit: Łukasz Anforowicz <[email protected]>
Reviewed-by: Karan Bhatia <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646573}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549232
Reviewed-by: Łukasz Anforowicz <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#630}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• extensions/browser/url_loader_factory_manager.cc[diff]

Merge to M74: Move the getUserMedia gating of mDNS protection to FilteringNetworkManager.

This is a reland of 6130949eaaf724cec6c90b2b54515b9e3dfe81e1

The following changes are added compared to the previous commit.
1. FilteringNetworkManager copies the networks from the underlying
   IpcNetworkManager upon each network change, and acts as both the
   default local address provider and the mDNS responder provider for
   each network.
2. FilteringNetworkManager copies the default local addresses from
   IpcNetworkManager.

The above changes are to ensure
1. different FilteringNetworkManagers sharing the same underlying
   IpcNetworkManager do not interfere with each other by mutating
   networks they do not own;
2. copied networks have their default local address provider and mDNS
   responder provider correctly set to their owning
   FilteringNetworkManager.

Original change's description:
> Move the getUserMedia gating of mDNS protection to FilteringNetworkManager.
>
> Our previous implementation would destroy the connection to the mDNS
> responder service host when the media capture permission is granted.
> As a result, any mDNS candidate created before the grant would become
> unresolvable. This issue is particularly evident when we have multiple
> peer connections of the same origin, where a newly created peer
> connection would temporarily toggle the permission to blocked before
> restoring the granted status if the user did give the permission. This
> could cause a peer connection created earlier and in the process of ICE
> candidate gathering to perform unnecessary mDNS obfuscation. After this
> change, the connection to the mDNS responder service host keeps alive,
> but just becomes unavailable for registration of new names when we have
> permissions granted.
>
> Bug: 930339, 937992
> Change-Id: I8ccdf0e6b578bc9e327337c6fbd102cdc542d15c
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1522687
> Commit-Queue: Qingsi Wang <[email protected]>
> Reviewed-by: Sergey Ulanov <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#641046}

[email protected]

(cherry picked from commit afd6f046afbad55e53cef13631a27418a0bc78d1)

Bug: 930339, 937992
Change-Id: I5a5a8d357a4a491bfc97e1a9922670aee7373840
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529659
Commit-Queue: Qingsi Wang <[email protected]>
Reviewed-by: Sergey Ulanov <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646207}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549209
Reviewed-by: Qingsi Wang <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#629}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/renderer/media/webrtc/peer_connection_dependency_factory.cc[diff]
• content/renderer/media/webrtc/peer_connection_dependency_factory.h[diff]
• content/renderer/p2p/filtering_network_manager.cc[diff]
• content/renderer/p2p/filtering_network_manager.h[diff]
• content/renderer/p2p/filtering_network_manager_unittest.cc[diff]
• content/renderer/p2p/ipc_network_manager.cc[diff]
• content/renderer/p2p/ipc_network_manager.h[diff]
• content/renderer/p2p/ipc_network_manager_unittest.cc[diff]

[css-flexbox] Correctly calculate min-height with justify-content

Spacing added by justification is not meaninfully part of the
intrinsic block size of an item; remove it.

This is especially problematic with percentage sizes, because
with height: 100% and justify-content: flex-end we would
position any flex items at the end and calculate a minimum
size based on the 100%, even though we ought to ignore percentages.

R=​[email protected], [email protected]

Bug: 945214
Change-Id: If4e271df5e550807632d30e5dd1c2b3068d45313
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546002
Commit-Queue: Emil A Eklund <[email protected]>
Reviewed-by: David Grogan <[email protected]>
Reviewed-by: Emil A Eklund <[email protected]>
Auto-Submit: Christian Biesinger <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646178}(cherry picked from commit ae2cd00736e2f3c07a9ca10aef4da0e5e4ca36df)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549204
Reviewed-by: Christian Biesinger <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#628}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/layout/flexible_box_algorithm.cc[diff]
• third_party/blink/renderer/core/layout/flexible_box_algorithm.h[diff]
• third_party/blink/renderer/core/layout/layout_flexible_box.cc[diff]
• third_party/blink/renderer/core/layout/ng/ng_flex_layout_algorithm.cc[diff]
• third_party/blink/web_tests/TestExpectations[diff]
• third_party/blink/web_tests/external/wpt/css/css-flexbox/flex-minimum-height-flex-items-014.html[Added - diff]

status-bubble: delays destruction for 10 seconds (M-74 merge)

Perf data indicates deleting the bubble can be expensive. This is because
the deletion of a top-level widget triggers synchronous operations with viz.
During startup there may be lots of things happening in viz, so, delay deletion
for 10 seconds in hopes of things having settled down.

BUG=943268
TEST=covered by tests

Change-Id: Id1d56997553b01f9cf64a3866c013f20d0b75c86
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532583
Reviewed-by: Evan Stade <[email protected]>
Commit-Queue: Scott Violet <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642715}(cherry picked from commit c660475187a1e1354d54dc2633a901591a4a6905)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549489
Reviewed-by: Michael Wasserman <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#627}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/views/status_bubble_views.cc[diff]
• chrome/browser/ui/views/status_bubble_views.h[diff]
• chrome/browser/ui/views/status_bubble_views_browsertest.cc[diff]

Webview should create an InProgressDownloadManager without DB as no history is saved

Currently InProgressDownloadManager is created for each DownloadManager.
And BrowserContext's file path is used to store history downloads.
However, for webview, there is no need to store history information.
So just use an empty dir should work.

BUG=946183,947921

Change-Id: Ifb06065a909fc36d3ca749fd96820e36984eca9a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548141
Commit-Queue: Min Qin <[email protected]>
Reviewed-by: Tao Bai <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646508}(cherry picked from commit ef25e912581a9674dab388838c67a9cb78e8e6ac)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549203
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#626}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• android_webview/browser/aw_browser_context.cc[diff]
• android_webview/browser/aw_browser_context.h[diff]
• components/download/internal/common/in_progress_download_manager.cc[diff]
• components/download/public/common/in_progress_download_manager.h[diff]

Fix some issues in displaying downloaded file names

This CL fixes a couple issues when displaying downloaded file names:
1. If no download is in in-progress DB, we currently skip the display name
   retrieval.This Causes an issue that downloads from the history DB won't
   have the correct name.
2. If a download is removed from files app, we should also remove them
   during loading. Since the removed files won't have a display name, we
   can use that as a hint to remove the download from in-progress or
   history DB.
3. Also fixes an issue that OnAllInProgressDownloadsLoaded() was called too
   early before all the content URIs from history DB can get their display names.

BUG=946183

(cherry picked from commit 881dac60c50b749155cdf0f10c61b9d1f10c767f)

Change-Id: I65e262039c66a8e439af8e8d083ff4ff9543225d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539128
Reviewed-by: Xing Liu <[email protected]>
Commit-Queue: Min Qin <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645880}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549580
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#625}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/download/internal/common/in_progress_download_manager.cc[diff]
• content/browser/download/download_manager_impl.cc[diff]

[Chromeshine] Record UsageStats events in a histogram

Add a histogram and enum that tracks important UsageStats-related events,
e.g. token tracking, consent, suspensions.

Bug: 943618
Change-Id: I81eec695e1b5d49b5d8196e88034d6a43124ec5d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529626
Commit-Queue: Patrick Noland <[email protected]>
Reviewed-by: Brian White <[email protected]>
Reviewed-by: Filip Gorski <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645528}(cherry picked from commit 172d187b408e314627e7ea9e3300bf5b1acd3091)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549286
Reviewed-by: Patrick Noland <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#624}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/EventTracker.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/SuspensionTracker.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/TokenTracker.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/UsageStatsMetricsEvent.java[Added - diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/UsageStatsMetricsReporter.java[Added - diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/UsageStatsService.java[diff]
• chrome/android/java_sources.gni[diff]
• chrome/android/junit/src/org/chromium/chrome/browser/usage_stats/EventTrackerTest.java[diff]
• tools/metrics/histograms/enums.xml[diff]
• tools/metrics/histograms/histograms.xml[diff]

M74: Fix the top toolbar size in the PDF Viewer.

https://crrev.com/620265 added an annotations toolbar underneath the
main toolbar. When the annotations toolbar is not shown, it is invisible
but still takes up space. So it blocks the ability to interact with the
PDF content underneath.

Fix this by hiding the annotations toolbar using the display: none
attribute instead of the visibility: hidden, so the toolbar takes up no
space.

BUG=946389
[email protected], [email protected]

(cherry picked from commit 90c04983feb28d4ad8ebdec5343869e87a5a7225)

Change-Id: Id7b1f657138f54b099caf533a1cb8f8c3c6a81b7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545214
Reviewed-by: Demetrios Papadopoulos <[email protected]>
Commit-Queue: Lei Zhang <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645910}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549197
Reviewed-by: Lei Zhang <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#623}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/resources/pdf/elements/viewer-pdf-toolbar/viewer-pdf-toolbar.html[diff]
• chrome/browser/resources/pdf/elements/viewer-pdf-toolbar/viewer-pdf-toolbar.js[diff]

Fix some book-keeping bugs in MessagePumpKqueue.

Some logic bugs could result in the events_ buffer growing exceptionally
large:

1. In StopWatchingFileDescriptor() and StopWatchingMachPort(), if
   removing the kevents from the kqueue fails, the event_count_ should
   still be decreased because the events are invalid.
2. In ProcessEvents() for EV_ONESHOT FDs, the event_count_ should be
   decremented after it is activated, since the event is removed.
3. If a controller is re-used with WatchFileDescriptor(), event_count_ is
   unconditionally incremented, even if it meant no truly new events were
   added to the kqueue.

(cherry picked from commit 532e430957e651c90d9494ac103a28bedf1d4b04)

Bug: 940540, 932175
Change-Id: Ia2e547ea81b93332edbe20b85b1e80939a0104a4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1516718
Reviewed-by: Mark Mentovai <[email protected]>
Commit-Queue: Robert Sesek <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#639910}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549485
Reviewed-by: Robert Sesek <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#622}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• base/message_loop/message_pump_kqueue.cc[diff]

Fix AppCacheSubresourceURLFactory::CreateLoaderAndStart() 'crashes'

Temporary workaround to avoid crash reporting when
AppCacheSubresourceURLFactory::CreateLoaderAndStart() is called after
the RPHI associated with |appcache_host_->process_id()| has already
been destroyed. This is similar to the behavior allowed before
http://crrev.com/c/1409732 and is intended to be a minimal fix that can
be merged to M73.

Bug: 910287
Change-Id: Ic14013095c2deeadbef81f800bf6d23ce95bdca1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1530003
Commit-Queue: Aaron Colwell <[email protected]>
Commit-Queue: Marijn Kruisselbrink <[email protected]>
Auto-Submit: Aaron Colwell <[email protected]>
Reviewed-by: Marijn Kruisselbrink <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642641}(cherry picked from commit 4cef7af42b793503d6aab595e9e235db45478595)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548681
Reviewed-by: Aaron Colwell <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#621}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/browser/appcache/appcache_subresource_url_factory.cc[diff]

Added flag for download auto resumptions in native

This CL :
1 - Disables the download auto-resumptions in native feature
2 - Adds a flag for testing purposes

Bug: 948299
Change-Id: I4a04eb17e61e40e9a4d471e1bfaccd77c0029270
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547763
Reviewed-by: Min Qin <[email protected]>
Commit-Queue: Shakti Sahu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646590}(cherry picked from commit 07b0527159389ccc303d38d99fcf2f42e4b148ae)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549480
Reviewed-by: Shakti Sahu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#620}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/about_flags.cc[diff]
• chrome/browser/flag-metadata.json[diff]
• chrome/browser/flag_descriptions.cc[diff]
• chrome/browser/flag_descriptions.h[diff]
• components/download/public/common/download_features.cc[diff]
• tools/metrics/histograms/enums.xml[diff]

[Chromeshine] Add feature toggle for UsageStats

This feature is enabled by default, which means it's mostly a kill
switch at this point. This was done because
1) It will be easier for dogfooders
2) There's already an explicit opt-in UI
3) The feature is already gated to Q only

With that said, we could still use this to do a phased rollout if we
wanted to.

Bug: 946200
Change-Id: I786327efb19dd34c77fff3b666fbfaf21cd0e4f8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540291
Commit-Queue: Patrick Noland <[email protected]>
Reviewed-by: Theresa <[email protected]>
Reviewed-by: Filip Gorski <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645602}(cherry picked from commit 2ac4bea670701cbf5937af71c1ad812f88bdfcf1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549285
Reviewed-by: Patrick Noland <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#619}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeFeatureList.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/usage_stats/UsageStatsService.java[diff]
• chrome/browser/android/chrome_feature_list.cc[diff]
• chrome/browser/android/chrome_feature_list.h[diff]

[merge to 74] shelf: Send key up press as well on back button click.

It originally send key down on tap down, and key up on tap, but this
was changed to just key down on tap, because some gestures send the tap
event and the browser listens to just the key up.

But ARC apps need the entire sequence, so this sends both on click.

Original bug: https://bugs.chromium.org/p/chromium/issues/detail?id=943154

[email protected]

(cherry picked from commit b29b632278fb50dd05453105533064ea189050ae)

Test: manual
Bug: b/129248267
Change-Id: I5e5d16f4dc6a675d1a96936ad938e5201ad74aa1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542224
Commit-Queue: Sammie Quon <[email protected]>
Reviewed-by: Scott Violet <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646441}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549284
Reviewed-by: Sammie Quon <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#618}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/public/cpp/caption_buttons/frame_back_button.cc[diff]
• ash/shelf/back_button.cc[diff]
• ash/shelf/back_button.h[diff]
• ash/shelf/back_button_unittest.cc[diff]

[Merge to M74]Don't snap the window if activation changed on window drag.

[email protected]

Do not snap the window if window activation changes because of dragging
a previous snapped window in split view, regardless it is tab drag or
window drag. When single process mash is enabled, race condition may
happens to make the dragged window be snapped because of activation
change before window drag ended. This will lead the split view to enter
a wrong state and then backdrop will be updated to wrong bounds.

(cherry picked from commit d32c9f0f96ebfe052c2c94e911cb92117f22e1fa)

Bug: 941237
Change-Id: I6c44ecc3dbf5bc03b914603520b4e41dd15d94f0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1526804
Reviewed-by: Xiaoqian Dai <[email protected]>
Commit-Queue: Min Chen <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642581}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548064
Reviewed-by: Min Chen <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#617}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/wm/splitview/split_view_controller.cc[diff]
• ash/wm/splitview/split_view_controller_unittest.cc[diff]

media: Vaapi VDA: Enable kReduced/kSuperReduced mode on all intel device if ALLOCATE mode

It is fact that "split |decoder_|s GetRequiredNumOfPictures()", crrev.com/c/1387391,
caused a regression. We tried to resolve the regressions by limiting it on newer
platforms, crrev.com/c/1409353. However, peformance downgrades more which is
side-effected by enabling MojoVideoDecoder crrev.com/c/1405284. This CL backs
the performance status to the time when crrev.com/c/1387391 is landed, which
were better than the present performance.

Test results of video.PlaybackPerfVP92160P30FPS on panther:
ToT: 44-45
ToT + this CL: 27-29

Bug: 912295, 920510, 925709
Test: tast run video.PlaybackPerfVP92160P30FPS on panther
Change-Id: I6bdaa152cf96113ad2edbccc4944ab98708fa178
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1513996
Auto-Submit: Hirokazu Honda <[email protected]>
Reviewed-by: Miguel Casas <[email protected]>
Commit-Queue: Hirokazu Honda <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#640209}(cherry picked from commit e63c9c30bd5ea200a478ff396b764f41af8850dc)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545807
Reviewed-by: Hirokazu Honda <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#616}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/gpu/vaapi/vaapi_video_decode_accelerator.cc[diff]
• media/gpu/vaapi/vaapi_video_decode_accelerator.h[diff]

Merge M74 "SPM: Fix VK not working after launching xls results"

> - Update DesktopFocusRules so that only toplevel window is allowed
>   to be activated in minimized state. Otherwise, a child window could
>   be activated and cause DesktopNativeWidgetAura not getting
>   activation change which in turn causes more problems down the road.
> - Fix DesktopNativeWidgetAura so that RestoreFocusedView happens
>   for a minimized widget
>
> Bug: 942787
> Change-Id: I0e3d35f342fca16cc78e451d6bf7fe32ac2dc337
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536554
> Commit-Queue: Xiyuan Xia <[email protected]>
> Reviewed-by: Scott Violet <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#644565}

(cherry picked from commit 900b50bc8b4e386004b74c55f36ad95e3e0d2b63)

Change-Id: I80edf6b213787cca57068ada1021fdb86156a0d4
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1549279
Reviewed-by: Xiyuan Xia <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#615}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/views/mus/desktop_window_tree_host_mus_unittest.cc[diff]
• ui/views/widget/desktop_aura/desktop_focus_rules.cc[diff]
• ui/views/widget/desktop_aura/desktop_native_widget_aura.cc[diff]
• ui/views/widget/desktop_aura/desktop_native_widget_aura.h[diff]

[Duet] Add flag to fix both top and bottom browser controls

Add a flag to make the top and bottom controls not scroll off-screen.

Bug: 946805
Change-Id: Ibb5ba66de6aaabbe933f5455946828ebd3ae0d24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542676
Commit-Queue: Pedro Amaral <[email protected]>
Reviewed-by: Matthew Jones <[email protected]>
Reviewed-by: Theresa <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646005}(cherry picked from commit bcdceff2816fa1d07b0bf53d0e4656162a2edf67)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548780
Reviewed-by: Pedro Amaral <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#614}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeFeatureList.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tab/TabStateBrowserControlsVisibilityDelegate.java[diff]
• chrome/browser/android/chrome_feature_list.cc[diff]
• chrome/browser/android/chrome_feature_list.h[diff]

Reenable undo snack bar for bottom sheet

This adds plumbing so that ChromeTabbedActivity serves the right
SnackbarManager based on bottom sheet state. I believe support for
this used to work but was not needed, so was removed.

BUG=941301

Change-Id: If88241d813852a3d9520c977ceec697f66921ed6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545496
Reviewed-by: Matthew Jones <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645836}(cherry picked from commit d87e7d103903e374a77ceabef09310470147718e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548864
Reviewed-by: Yusuf Ozuysal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#613}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/SnackbarManager.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarController.java[diff]

Incrementing VERSION to 74.0.3729.58

[email protected]

Change-Id: I16d288f8720ac2a6a73c720a89be06c886faeba6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548740
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#612}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

[m74] ui/gl: Throttle surface control transactions based on acks.

Allow only 1 unacked transaction to be in flight. This ensures that the
next transaction is submitted only after the previous one was latched
and prevents transactions from getting merged, which effectively drops
frames by the framework.

[email protected]
[email protected], [email protected]

(cherry picked from commit e993337f66984df1b10c69c4318602e03b0e6c54)

Change-Id: I350d9944dea681a54aa576bf382e73ee3be70390
Bug: 946849
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546774
Commit-Queue: Khushal <[email protected]>
Reviewed-by: Eric Karl <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646108}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548739
Reviewed-by: Khushal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#611}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/gl/android/android_surface_control_compat.cc[diff]
• ui/gl/android/android_surface_control_compat.h[diff]
• ui/gl/gl_surface_egl_surface_control.cc[diff]
• ui/gl/gl_surface_egl_surface_control.h[diff]

Add ref-counting to LocalFrameView::ukm_aggregator_

M-74 Merge

Because PageWidgetDelegate::Animate can run arbitrary script, it's possible
that a LocalFrameView could call Dispose() -- and delete its
LocalFrameUkmAggregator -- while there is a UKM measurement in progress. By
making LocalFrameUkmAggregator ref-counted, instances of
ScopedUkmHierarchicalTimer can keep it alive while a measurement is in
progress.

BUG=943500
R=​[email protected]

Change-Id: Icc1346db664583a4e5686c2f15a9f261305fb5cf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539182
Reviewed-by: Stephen Chenney <[email protected]>
Commit-Queue: Stefan Zager <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644789}(cherry picked from commit f6825859d4014b6807f01c4b8fc3de81cd6facaa)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546682
Cr-Commit-Position: refs/branch-heads/3729@{#610}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/frame/local_frame_ukm_aggregator.cc[diff]
• third_party/blink/renderer/core/frame/local_frame_ukm_aggregator.h[diff]
• third_party/blink/renderer/core/frame/local_frame_ukm_aggregator_test.cc[diff]
• third_party/blink/renderer/core/frame/local_frame_view.cc[diff]
• third_party/blink/renderer/core/frame/local_frame_view.h[diff]

Use the right string and check for open in new tab

"Open in new tab in group" should show up in context menu on links
long press as well as NTP. And also the check for this should be
through FeatureUtilities.

BUG=945879

Change-Id: I2f20698c1f81729611e75944608637819fa1a6f1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545706
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645907}(cherry picked from commit 27977bb69bdf93c0542d662659d9cdc6bc02e328)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548585
Cr-Commit-Position: refs/branch-heads/3729@{#609}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/contextmenu/ChromeContextMenuItem.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/native_page/ContextMenuManager.java[diff]

Hook up TabModel.closeMultipleTabs() in GTS

Bug: 937679
Change-Id: I42a6f0ffa95a6c081074bd5d3f120cc0ce0f6245
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540102
Commit-Queue: Wei-Yin Chen (陳威尹) <[email protected]>
Reviewed-by: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645915}(cherry picked from commit 70feba5092a09afeb0077dc4548c24efb9adb256)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546526
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#608}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/GridTabSwitcherCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridSheetCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGroupUiCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListMediator.java[diff]
• chrome/android/features/tab_ui/junit/src/org/chromium/chrome/browser/tasks/tab_management/TabListMediatorUnitTest.java[diff]

Add TabModel.closeMultipleTabs()

TabModel.closeMultipleTabs() will be used to close all tabs in a tab group
atomically.

Reference:
https://chromium-review.googlesource.com/c/experimental/chromium/src/+/1356307

Bug: 937679
Change-Id: I8e9067bff35313afb6be6ff1d232b30ac94d7f13
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1525134
Reviewed-by: Yusuf Ozuysal <[email protected]>
Commit-Queue: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644135}(cherry picked from commit 8a3bd2812bfa0bf77488a98ca4739f499ca486c1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548612
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#607}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ActivityTabTaskDescriptionHelper.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/undo/UndoBarController.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/EmptyTabModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/EmptyTabModelObserver.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/IncognitoTabModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/SingleTabModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelFilter.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelImpl.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelObserver.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelObserverJniBridge.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tabmodel/document/DocumentTabModelImpl.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/TabCountProvider.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/widget/emptybackground/EmptyBackgroundViewWrapper.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/TabPersistentStoreTest.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/tabmodel/document/MockDocumentTabModel.java[diff]
• chrome/test/android/javatests/src/org/chromium/chrome/test/util/ChromeTabUtils.java[diff]

Incrementing VERSION to 74.0.3729.57

[email protected]

Change-Id: Ieb21348db3ecebb3add1e90f15f4cff5b69d7f47
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548635
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#606}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Ignore mhtml iframe navigations outside of archive

Clicking on a link inside iframe of mhtml archive tries to navigate the iframe.
If the target url is not captured in the archive, we used to ignore the
navigation (using cancelation error). After the recent refactoring (r626179),
we instead load the error page because the error is now "failure".

This patch restores back the cancelation error.

Adding test for this is very hard:
- unit test does not trigger the error page load (which is a part of
  RenderFrameImpl logic);
- browser test cannot trigger the navigation, because javascript is disabled
  for mhtml pages.
Manually verified that link clicks are ignored.

[email protected]

(cherry picked from commit f3224298cd2176794f4bf71e358289f4d9b65780)

Bug: 945394
Change-Id: Ia92d1d771fe987d9278f14981ac4b9dea8bbce08
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544792
Reviewed-by: Charlie Reis <[email protected]>
Reviewed-by: Nate Chapin <[email protected]>
Commit-Queue: Dmitry Gozman <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645867}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548563
Reviewed-by: Dmitry Gozman <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#605}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/loader/document_loader.cc[diff]

M74: Avoid renderer kills for OOPIFs in MHTML files, for now.

MHTML files should load subframes from the archive in the main
frame's process, but there appear to be some cases where Chrome is
creating out-of-process iframes inside MHTML pages.  Temporarily
allow this to avoid the renderer kills, until we can track down
how it is occurring.

BUG=931895, 948246

Change-Id: I88acedca05f7bd45b2e8c803665549f2eb2d0c22
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548040
Reviewed-by: Alex Moshchuk <[email protected]>
Commit-Queue: Charlie Reis <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646530}(cherry picked from commit a3af9a79b09234841e76fe1b46b84ff354d9b05c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548595
Reviewed-by: Charlie Reis <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#604}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/browser/frame_host/render_frame_host_impl.cc[diff]

Fix DRP retries on net error

When net error is received in OnComplete, and the response has already started,
this leads to a crash. So this CL limits handling net error only when the
response has not been received.

Bug: 936999
Change-Id: If308f062ab3c2f8f75d96387f7126d2491bb1ab6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548539
Reviewed-by: rajendrant <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#603}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/data_reduction_proxy/content/common/data_reduction_proxy_url_loader_throttle.cc[diff]
• components/data_reduction_proxy/content/common/data_reduction_proxy_url_loader_throttle.h[diff]

Cache GTS flag and disable adaptive toolbar when GTS is enabled

This CL caches the GTS flag so it can be used before native is
initialized. It also disables the apative toolbar when the grid tab
switcher is enabled. Also we don't enable adaptive toolbar if the bottom
toolbar is not enabled.

[email protected]

(cherry picked from commit b442e66e683bd3a8a2ba688fb6903cf59e23bc94)

Bug: 944653
Change-Id: I11ca478caf581d1c8903c13fda41266b5371c7a5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546319
Reviewed-by: Yusuf Ozuysal <[email protected]>
Commit-Queue: Pedro Amaral <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646034}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548327
Reviewed-by: Pedro Amaral <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#602}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/preferences/ChromePreferenceManager.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/AppThemeColorProvider.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/TabSwitcherModeTTPhone.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java[diff]

[Bluetooth] Bus blocked LTKs to BlueZ.

Bug: 942089
Change-Id: If20865f189c98aaac521ee9a2bb29c34ce464303
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1533471
Reviewed-by: Sonny Sasaka <[email protected]>
Reviewed-by: Giovanni Ortuño Urquidi <[email protected]>
Commit-Queue: Ryan Hansberry <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643591}(cherry picked from commit 90a778357c2fd72c7f3c6148ab4077218c166451)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548553
Reviewed-by: Ryan Hansberry <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#601}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• device/bluetooth/bluez/bluetooth_adapter_bluez.cc[diff]
• device/bluetooth/bluez/bluetooth_adapter_bluez.h[diff]
• device/bluetooth/bluez/bluetooth_bluez_unittest.cc[diff]
• device/bluetooth/dbus/bluetooth_adapter_client.cc[diff]
• device/bluetooth/dbus/bluetooth_adapter_client.h[diff]
• device/bluetooth/dbus/fake_bluetooth_adapter_client.cc[diff]
• device/bluetooth/dbus/fake_bluetooth_adapter_client.h[diff]
• services/device/bluetooth/bluetooth_system_unittest.cc[diff]

[Bluetooth] Fetch and parse blocked LTKs.

Subsequent CL will bus the parsed array of byte arrays into BlueZ.

TEST: Included unittests, as well as manually testing with a
real Chromebook and local server which supplied the config seed.

Bug: 942089
Change-Id: I1d6288be99da9a11b83da38485923224a80d06cd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1531031
Commit-Queue: Ryan Hansberry <[email protected]>
Reviewed-by: Giovanni Ortuño Urquidi <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643258}(cherry picked from commit 27c9c5873b3d67e8d6d87c10601d39e5809931f1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548391
Reviewed-by: Ryan Hansberry <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#600}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chromeos/constants/chromeos_features.cc[diff]
• chromeos/constants/chromeos_features.h[diff]
• device/BUILD.gn[diff]
• device/bluetooth/chromeos/bluetooth_utils.cc[diff]
• device/bluetooth/chromeos/bluetooth_utils.h[diff]
• device/bluetooth/chromeos/bluetooth_utils_unittest.cc[Added - diff]

[m74] CORB allowlist additions - 2019-03-29.

[email protected]

(cherry picked from commit 81ebfff6fa42efb4260972146ad3ac69b91f5dac)

Bug: 937081
Change-Id: Ic7ba6a76adc589243ea6b12d973dd3cff6fb3d55
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544332
Commit-Queue: Łukasz Anforowicz <[email protected]>
Commit-Queue: Devlin <[email protected]>
Auto-Submit: Łukasz Anforowicz <[email protected]>
Reviewed-by: Devlin <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645999}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548386
Reviewed-by: Łukasz Anforowicz <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#599}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• extensions/browser/url_loader_factory_manager.cc[diff]

[m74] CORB allowlist additions - 2019-03-28.

[email protected]

(cherry picked from commit 725be848c724ec8a95ce8fa5bbddc4aa78dd85c4)

Bug: 937081
Change-Id: I7a48030d184b42701f17671debe3f04a5dcf335e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542432
Commit-Queue: Łukasz Anforowicz <[email protected]>
Commit-Queue: Devlin <[email protected]>
Auto-Submit: Łukasz Anforowicz <[email protected]>
Reviewed-by: Devlin <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645432}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548385
Reviewed-by: Łukasz Anforowicz <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#598}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• extensions/browser/url_loader_factory_manager.cc[diff]

Updating XTBs based on .GRDs from branch 3729

[email protected]

Change-Id: If0a468213e623205ad4badde00e1315c10b23e3d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548251
Reviewed-by: Mustafa Emre Acer <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#597}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/components/strings/ash_components_strings_fil.xtb[diff]
• ash/strings/ash_strings_am.xtb[diff]
• ash/strings/ash_strings_ar.xtb[diff]
• ash/strings/ash_strings_bg.xtb[diff]
• ash/strings/ash_strings_bn.xtb[diff]
• ash/strings/ash_strings_ca.xtb[diff]
• ash/strings/ash_strings_cs.xtb[diff]
• ash/strings/ash_strings_da.xtb[diff]
• ash/strings/ash_strings_de.xtb[diff]
• ash/strings/ash_strings_el.xtb[diff]
• ash/strings/ash_strings_en-GB.xtb[diff]
• ash/strings/ash_strings_es-419.xtb[diff]
• ash/strings/ash_strings_es.xtb[diff]
• ash/strings/ash_strings_et.xtb[diff]
• ash/strings/ash_strings_fa.xtb[diff]
• ash/strings/ash_strings_fi.xtb[diff]
• ash/strings/ash_strings_fil.xtb[diff]
• ash/strings/ash_strings_fr.xtb[diff]
• ash/strings/ash_strings_gu.xtb[diff]
• ash/strings/ash_strings_hi.xtb[diff]
• ash/strings/ash_strings_hr.xtb[diff]
• ash/strings/ash_strings_hu.xtb[diff]
• ash/strings/ash_strings_id.xtb[diff]
• ash/strings/ash_strings_it.xtb[diff]
• ash/strings/ash_strings_iw.xtb[diff]
• ash/strings/ash_strings_ja.xtb[diff]
• ash/strings/ash_strings_kn.xtb[diff]
• ash/strings/ash_strings_ko.xtb[diff]
• ash/strings/ash_strings_lt.xtb[diff]
• ash/strings/ash_strings_lv.xtb[diff]
• ash/strings/ash_strings_ml.xtb[diff]
• ash/strings/ash_strings_mr.xtb[diff]
• ash/strings/ash_strings_ms.xtb[diff]
• ash/strings/ash_strings_nl.xtb[diff]
• ash/strings/ash_strings_no.xtb[diff]
• ash/strings/ash_strings_pl.xtb[diff]
• ash/strings/ash_strings_pt-BR.xtb[diff]
• ash/strings/ash_strings_pt-PT.xtb[diff]
• ash/strings/ash_strings_ro.xtb[diff]
• ash/strings/ash_strings_ru.xtb[diff]
• ash/strings/ash_strings_sk.xtb[diff]
• ash/strings/ash_strings_sl.xtb[diff]
• ash/strings/ash_strings_sr.xtb[diff]
• ash/strings/ash_strings_sv.xtb[diff]
• ash/strings/ash_strings_sw.xtb[diff]
• ash/strings/ash_strings_ta.xtb[diff]
• ash/strings/ash_strings_te.xtb[diff]
• ash/strings/ash_strings_th.xtb[diff]
• ash/strings/ash_strings_tr.xtb[diff]
• ash/strings/ash_strings_uk.xtb[diff]
• ash/strings/ash_strings_vi.xtb[diff]
• ash/strings/ash_strings_zh-CN.xtb[diff]
• ash/strings/ash_strings_zh-TW.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_es-419.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_te.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_bg.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_fil.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_hr.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sk.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sw.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_vi.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_zh-TW.xtb[diff]
• chrome/app/resources/chromium_strings_fil.xtb[diff]
• chrome/app/resources/chromium_strings_hi.xtb[diff]
• chrome/app/resources/generated_resources_am.xtb[diff]
• chrome/app/resources/generated_resources_bg.xtb[diff]
• chrome/app/resources/generated_resources_fil.xtb[diff]
• chrome/app/resources/generated_resources_fr.xtb[diff]
• chrome/app/resources/generated_resources_hr.xtb[diff]
• chrome/app/resources/generated_resources_ko.xtb[diff]
• chrome/app/resources/generated_resources_ms.xtb[diff]
• chrome/app/resources/generated_resources_sk.xtb[diff]
• chrome/app/resources/generated_resources_te.xtb[diff]
• chrome/app/resources/generated_resources_tr.xtb[diff]
• chrome/app/resources/generated_resources_vi.xtb[diff]
• chrome/app/resources/generated_resources_zh-TW.xtb[diff]
• chrome/app/resources/google_chrome_strings_fil.xtb[diff]
• chrome/browser/resources/chromeos/camera/src/strings/camera_strings_fil.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_fil.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_ru.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_sk.xtb[diff]
• chromeos/strings/chromeos_strings_fil.xtb[diff]
• components/embedder_support/android/java/strings/translations/web_contents_delegate_android_strings_fil.xtb[diff]
• components/policy/resources/policy_templates_fil.xtb[diff]
• components/policy/resources/policy_templates_ko.xtb[diff]
• components/policy/resources/policy_templates_ru.xtb[diff]
• components/policy/resources/policy_templates_sk.xtb[diff]
• components/policy/resources/policy_templates_tr.xtb[diff]
• components/policy/resources/policy_templates_zh-TW.xtb[diff]
• components/strings/components_strings_am.xtb[diff]
• components/strings/components_strings_bg.xtb[diff]
• components/strings/components_strings_es-419.xtb[diff]
• components/strings/components_strings_fil.xtb[diff]
• components/strings/components_strings_hr.xtb[diff]
• components/strings/components_strings_te.xtb[diff]
• components/strings/components_strings_vi.xtb[diff]
• components/strings/components_strings_zh-TW.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_fil.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_bg.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_fil.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_zh-TW.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_bg.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_fil.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_hr.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_sw.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_vi.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_zh-TW.xtb[diff]
• remoting/resources/remoting_strings_fil.xtb[diff]
• ui/accessibility/extensions/strings/accessibility_extensions_strings_fil.xtb[diff]
• ui/strings/translations/ui_strings_bg.xtb[diff]
• ui/strings/translations/ui_strings_fil.xtb[diff]
• ui/strings/translations/ui_strings_vi.xtb[diff]

Added temp UMA logging for renderer hangs.

Adding temporary UMA statistics to determine what the
browser was waiting for when it determined that the
renderer was hung.

This is similar to logging added in r406039 and removed in
r455764, but the hang monitor is now different enough that
the UMA values cannot be reused.

[email protected]

(cherry picked from commit ab20718c657194abb5e1a53257267e13c8b91714)

Bug: 947559, 938647
Change-Id: Ia4d44e09ec653f23fcc9d8ea0fcde05a887d21da
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546250
Commit-Queue: Chris Mumford <[email protected]>
Reviewed-by: John Abd-El-Malek <[email protected]>
Reviewed-by: Steven Holte <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645979}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548360
Reviewed-by: Chris Mumford <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#596}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/metrics/chrome_stability_metrics_provider.cc[diff]
• components/metrics/stability_metrics_helper.cc[diff]
• components/metrics/stability_metrics_helper.h[diff]
• content/browser/frame_host/navigation_handle_impl.cc[diff]
• content/browser/renderer_host/render_widget_host_impl.cc[diff]
• content/browser/renderer_host/render_widget_host_impl.h[diff]
• tools/metrics/histograms/enums.xml[diff]
• tools/metrics/histograms/histograms.xml[diff]

Clear DialogPlate when Assistant is no longer visible.

We need to clear the dialog plate when Assistant UI is no longer visible
so that text entered in the textfield does not bleed across sessions.

This is the same logic we perform for standalone UI:
https://cs.chromium.org/chromium/src/ash/assistant/ui/dialog_plate/dialog_plate.cc?dr=CSs&g=0&l=223-232

(cherry picked from commit 265a99b8e54215242033795117e0456e141a622d)

Bug: 941125
Change-Id: I26e4060559e4ea2408b79488effbda7f4fb32cc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546715
Reviewed-by: Xiyuan Xia <[email protected]>
Reviewed-by: Xiaohui Chen <[email protected]>
Commit-Queue: David Black <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646043}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546585
Cr-Commit-Position: refs/branch-heads/3729@{#595}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/app_list/views/assistant/dialog_plate.cc[diff]
• ash/app_list/views/assistant/dialog_plate.h[diff]

speculative fix for status-bubble crash

Crash data seems to indicate after the call to SetBubbleWidth(). AFAICT
the functions that are called after SetBubbleWidth() can be moved *before*
the call to SetBubbleWidth(), so moving them in hopes of fixing the crash.

BUG=943564
TEST=none

Change-Id: I271fdc0e90a89f8d3ce4f45a848e4dcbeee0c1dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540308
Reviewed-by: Evan Stade <[email protected]>
Commit-Queue: Scott Violet <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644903}(cherry picked from commit 81a2702f243f6bea7c626890109376571de913c8)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546587
Cr-Commit-Position: refs/branch-heads/3729@{#594}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/views/status_bubble_views.cc[diff]

AwDebug: write a phony dump if crashpad isn't enabled

Bug: 948187
Change-Id: Ied073362bbed4a7d7504db1d076827e641dc3657
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542957
Reviewed-by: Richard Coles <[email protected]>
Commit-Queue: Joshua Peraza <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645408}(cherry picked from commit fdc73015b5b10cb5f34a70e5d20dfc5d52aaba5f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548373
Reviewed-by: Krishna Govind <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#593}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• android_webview/browser/aw_debug.cc[diff]
• android_webview/common/crash_reporter/aw_crash_reporter_client.cc[diff]
• android_webview/common/crash_reporter/aw_crash_reporter_client.h[diff]

[android] When ConnectivityManager returns null ProxyInfo, clear proxy

Prior to using ConnectivityManager, we treated a null proxy info object
as meaning that we should look up the proxy settings from the system
properties. However ConnectivityManager is authoritative, so we should
treat null in that case as indicating that proxy info should be cleared.

Bug: 851995
Bug: 908085
Change-Id: Ia285a01fa0683dbc39c852774a051e42cf05fe01
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541247
Reviewed-by: Paul Jensen <[email protected]>
Commit-Queue: Tobias Sargeant <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645799}(cherry picked from commit 7475f3376456d336c9c05276d573b327a4012269)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547835
Reviewed-by: Tobias Sargeant <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#592}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• net/android/java/src/org/chromium/net/ProxyChangeListener.java[diff]
• net/proxy_resolution/proxy_config_service_android.cc[diff]
• net/proxy_resolution/proxy_config_service_android.h[diff]
• net/proxy_resolution/proxy_config_service_android_unittest.cc[diff]

[android] Use ConnectivityManager to fetch proxy information.

On Android M+ we can use ConnectivityManager to fetch proxy information
rather than relying on system properties or intent extras.

In order to keep WebView applications that use reflection to trigger the
proxy change broadcast working, we must continue to register the old
ProxyReceiver object. To ensure that this is not triggered by real
system broadcasts, we register it with an empty intent filter.

Bug: 851995
Bug: 908085
Change-Id: Ibfa6c35693301868f8a63ea924649e4c2bc8a328
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1526268
Reviewed-by: Richard Coles <[email protected]>
Reviewed-by: Paul Jensen <[email protected]>
Commit-Queue: Tobias Sargeant <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645223}(cherry picked from commit 281f390d31056c76c472347180698a77cd3556ef)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547973
Reviewed-by: Tobias Sargeant <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#591}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• net/android/BUILD.gn[diff]
• net/android/java/src/org/chromium/net/ProxyBroadcastReceiver.java[Added - diff]
• net/android/java/src/org/chromium/net/ProxyChangeListener.java[diff]
• net/android/javatests/src/org/chromium/net/ProxyChangeListenerTest.java[Added - diff]

[M74] Use dav1d by default

CHERRYPICK
Not able to do a clean cherrypick. These files required manual merge:
  media/media_options.gni
  third_party/libaom/options.gni

This CL turns on dav1d by default. It also disables libaom to save on
binary size.

Bug: 924370
Change-Id: I4d331a3b66fe6031930f5f56880866b9a3c92df8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539901
Commit-Queue: Chrome Cunningham <[email protected]>
Auto-Submit: Thomas Guilbert <[email protected]>
Reviewed-by: Tom Finegan <[email protected]>
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/heads/master@{#645600}
(cherry picked from commit ede4345734371ac35b4e50cea7ef3332dabd54d5)

TBR: [email protected]
Change-Id: I4d331a3b66fe6031930f5f56880866b9a3c92df8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548047
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#590}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/base/media_switches.cc[diff]
• media/base/media_switches.h[diff]
• media/cdm/library_cdm/clear_key_cdm/cdm_video_decoder.cc[diff]
• media/media_options.gni[diff]
• media/renderers/default_decoder_factory.cc[diff]
• media/test/pipeline_integration_fuzzertest.cc[diff]
• media/test/pipeline_integration_test_base.cc[diff]
• third_party/libaom/options.gni[diff]

[M74] Add ENABLE_LIBAOM_DECODER buildflag

CHERRYPICK
Not able to do a clean cherrypick. These files required manual merge:
  media/media_options.gni
  third_party/libaom/options.gni

Currently, ENABLE_AV1_DECODER is both used to signal that we have
enabled AV1 decoding, and to guard libaom build files. This makes it
impossible to turn off libaom without also turning off AV1 decoding.

This CL adds a new ENABLE_LIBAOM_DECODER flag, and changes the way we
compute ENABLE_AV1_DECODER. This allows for independent compilation
flags for dav1d and libaom, without affecting general AV1 code (such
as codec parsing, test cases, etc).

Bug: 924370
Change-Id: I0cfeb4e0033d494bf29956a305726c3a47fbb166
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540280
Commit-Queue: Thomas Guilbert <[email protected]>
Reviewed-by: Johann Koenig <[email protected]>
Reviewed-by: Vasilii Sukhanov <[email protected]>
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/heads/master@{#645082}
(cherry picked from commit 642e97d190356d64e2e8df3ad500ea78a05abb1a)

TBR: [email protected]
TBR: [email protected]
TBR: [email protected]
Change-Id: I0cfeb4e0033d494bf29956a305726c3a47fbb166
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547756
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#589}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/BUILD.gn[diff]
• chrome/browser/DEPS[diff]
• chrome/browser/media/encrypted_media_browsertest.cc[diff]
• content/browser/media/encrypted_media_browsertest.cc[diff]
• content/browser/media/media_canplaytype_browsertest.cc[diff]
• content/test/BUILD.gn[diff]
• media/BUILD.gn[diff]
• media/base/BUILD.gn[diff]
• media/base/mime_util_internal.cc[diff]
• media/base/supported_types.cc[diff]
• media/base/video_codecs.h[diff]
• media/cdm/library_cdm/clear_key_cdm/BUILD.gn[diff]
• media/cdm/library_cdm/clear_key_cdm/cdm_video_decoder.cc[diff]
• media/filters/BUILD.gn[diff]
• media/filters/stream_parser_factory.cc[diff]
• media/formats/BUILD.gn[diff]
• media/formats/mp4/box_definitions.cc[diff]
• media/formats/mp4/fourccs.h[diff]
• media/formats/webm/webm_video_client.cc[diff]
• media/media_options.gni[diff]
• media/renderers/BUILD.gn[diff]
• media/renderers/default_decoder_factory.cc[diff]
• media/test/BUILD.gn[diff]
• media/test/pipeline_integration_fuzzertest.cc[diff]
• media/test/pipeline_integration_test.cc[diff]
• media/test/pipeline_integration_test_base.cc[diff]
• third_party/libaom/BUILD.gn[diff]
• third_party/libaom/options.gni[diff]

Reduce bypass duration on 407 auth failures

With network-servicification, 407 failures bypass the proxy for a random duration of
1-5 minutes, which leads to drop in data saver usage. This CL reduces that bypass
duration. This CL also reduces initial delay for fetching the client config.

Bug: 945419
Change-Id: Ifc297a3ee7b681e76ce752612a6b05feba642ce8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539210
Reviewed-by: Tarun Bansal <[email protected]>
Commit-Queue: rajendrant <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644642}(cherry picked from commit 43fd4036df2f25dc0e44225a4c18fc931e0e0665)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548235
Reviewed-by: rajendrant <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#588}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/data_reduction_proxy/data_reduction_proxy_browsertest.cc[diff]
• components/data_reduction_proxy/core/browser/data_reduction_proxy_config_service_client.cc[diff]
• components/data_reduction_proxy/core/common/data_reduction_proxy_headers.cc[diff]

[m74] gpu: Handle paint op serialization failures due to insufficient buffer.

We currently fail serializing an op if it requires more than 512k bytes.
Increase that up to the max transfer buffer size with incremental
attempts.

[email protected]
[email protected]

(cherry picked from commit 6cabe8030922906e2d030c6ac6cc2d1af8bd66d3)

Bug: 924042
Change-Id: I83b4905343eff31af387e2ee6a2d70618cb399f9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1533054
Reviewed-by: Robert Kaplow <[email protected]>
Reviewed-by: enne <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Commit-Queue: Khushal <[email protected]>
Auto-Submit: Khushal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644417}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548039
Reviewed-by: Khushal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#587}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• cc/layers/heads_up_display_layer_impl.cc[diff]
• cc/paint/oop_pixeltest.cc[diff]
• cc/raster/gpu_raster_buffer_provider.cc[diff]
• cc/raster/raster_source.h[diff]
• gpu/command_buffer/client/mock_transfer_buffer.cc[diff]
• gpu/command_buffer/client/mock_transfer_buffer.h[diff]
• gpu/command_buffer/client/raster_implementation.cc[diff]
• gpu/command_buffer/client/raster_implementation.h[diff]
• gpu/command_buffer/client/raster_implementation_gles.cc[diff]
• gpu/command_buffer/client/raster_implementation_gles.h[diff]
• gpu/command_buffer/client/raster_interface.h[diff]
• gpu/command_buffer/client/transfer_buffer.cc[diff]
• gpu/command_buffer/client/transfer_buffer.h[diff]
• tools/metrics/histograms/histograms.xml[diff]

[Feed] Fix Nonsynthetic NotSynthetic typo

Feed should report NotSynthetic instead of Nonsynthetic.

Bug: 943833
Change-Id: I851d7e2c7f604812466ff1df61e1cb2d4bf518e6
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542961
Reviewed-by: Sky Malice <[email protected]>
Commit-Queue: Sky Malice <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645327}(cherry picked from commit 2f7662d9fbdd0718e2ced483dba22e10234d39d3)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545226
Reviewed-by: Gang Wu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#586}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/feed/core/feed_logging_metrics.cc[diff]

[SEC] Adding Search Engine Choice handling code

This patch:
* Creates a class SearchEngineChoiceNotification.
* Creates preferences to track whether the user needs to be prompted or
  whether they have been prompted already.
* Shows a snackbar, which gets the user to appropriate settings page.
* Adds metrics regarding the events and updated search engine.
* Adds tests.

[email protected]

(cherry picked from commit c8d317771e9f99ee731d0ae7b1106ac16db3fe64)

Bug: 945376
Change-Id: Ib1cdcb9a8cc8e4c6bcc72fea514902e1d2207f2b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542673
Commit-Queue: Filip Gorski <[email protected]>
Reviewed-by: Ted Choc <[email protected]>
Reviewed-by: Mark Pearson <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#646113}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1548031
Reviewed-by: Filip Gorski <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#585}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeFeatureList.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/preferences/SearchEngineAdapter.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/search_engines/SearchEngineChoiceNotification.java[Added - diff]
• chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java[diff]
• chrome/android/java/strings/android_chrome_strings.grd[diff]
• chrome/android/java/strings/android_chrome_strings_grd/IDS_SEARCH_ENGINE_CHOICE_PROMPT.png.sha1[Added - diff]
• chrome/android/java_sources.gni[diff]
• chrome/android/junit/src/org/chromium/chrome/browser/search_engines/SearchEngineChoiceNotificationTest.java[Added - diff]
• chrome/browser/android/chrome_feature_list.cc[diff]
• chrome/browser/android/chrome_feature_list.h[diff]
• tools/metrics/histograms/enums.xml[diff]
• tools/metrics/histograms/histograms.xml[diff]

Use existing storage format for download loaded from history DB.

If a download is loaded from history DB, then we should use its existing
storage format.
For new download, we will check whether download collection applies.

BUG=946788

Change-Id: Ia3fef65187b52c29bcca4d4177f6d4f97c9cca82
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542659
Reviewed-by: David Trainor <[email protected]>
Commit-Queue: Min Qin <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645438}(cherry picked from commit a1b62597b199c671739c57f5c3f292c514151805)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545223
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#584}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/download/internal/common/download_item_impl.cc[diff]

Records UMA for the sync XHR duration

This is used to diagnose the renderer hang issue when network service is running
OOP.

BUG=934317

Change-Id: Ifb3ea14eae748ba9ee6c50a3b70bc8e7ec22a596
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541595
Commit-Queue: Min Qin <[email protected]>
Reviewed-by: John Abd-El-Malek <[email protected]>
Reviewed-by: Ilya Sherman <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645135}(cherry picked from commit 1af07b303ccc5563b7960785c10fc7eeb3c82ec3)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546064
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#583}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/renderer/loader/web_url_loader_impl.cc[diff]
• tools/metrics/histograms/histograms.xml[diff]

Use the Lite Mode strings for Data Saver Promo InfoBar, if enabled

Bug: 947122
Change-Id: Ic78377e46b609fd5c8159da5ce27f29c82786c0f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544155
Reviewed-by: Tarun Bansal <[email protected]>
Reviewed-by: Theresa <[email protected]>
Commit-Queue: Robert Ogden <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645560}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546881
Reviewed-by: Robert Ogden <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#582}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/infobar/DataReductionPromoInfoBar.java[diff]
• chrome/android/java/strings/android_chrome_strings.grd[diff]

[omnibox] Set accessibility text on revert

Before, the omnibox accessibility text was set when
OnTemporaryTextMaybeChanged was called, which was not called when the
omnibox was reverted. This was fine, as screen readers were only
triggered when traversing the omnibox with the arrow/page up/down keys,
and not when reverting the omnibox. crrev.com/c/1419257 changed
traversing the selection to the first suggestion to be consistent with
reverting the omnibox. Consequently, the accessibility text was no
longer set when arrowing/paging up to the first suggestion, and the
incorrect text (corresponding to the previously selected match) was
read.

This CL sets the accessibility text on revert as well.

Bug: 943105
Change-Id: I22df51b3a17a34d5a7c35f494a365391d1c67236
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1534138
Reviewed-by: Orin Jaworski <[email protected]>
Reviewed-by: Kevin Bailey <[email protected]>
Reviewed-by: Justin Donnelly <[email protected]>
Commit-Queue: manuk hovanesian <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644345}(cherry picked from commit 4b29136eff0dee370e1c04255edf6233b257783a)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542499
Reviewed-by: manuk hovanesian <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#581}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/views/omnibox/omnibox_view_views.cc[diff]
• chrome/browser/ui/views/omnibox/omnibox_view_views.h[diff]
• components/omnibox/browser/omnibox_edit_model.cc[diff]
• components/omnibox/browser/omnibox_view.h[diff]
• components/omnibox/browser/test_omnibox_view.h[diff]
• ios/chrome/browser/ui/omnibox/omnibox_view_ios.h[diff]

Update text styling of Lite mode tile in the overflow menu

Bug: 947208
Change-Id: Icf26cf2abb588cb4f39337679ff1a243cf9d4e8d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545177
Reviewed-by: Theresa <[email protected]>
Commit-Queue: Robert Ogden <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645583}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547931
Reviewed-by: Robert Ogden <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#580}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/layout/data_reduction_main_menu_item.xml[diff]
• chrome/android/java/src/org/chromium/chrome/browser/preferences/datareduction/DataReductionMainMenuItem.java[diff]

Align Lite Mode illustration to the center of the image

Right now, the body of the car is a bit too far over to the right
because of how the SVG was cropped. This CL adds a bit of whitespace
to the right of the image so that the body of the car will be centered
on the device.

Bug: 947241
Change-Id: I58dbba6a2c986737b008943fe78fec7d260ee70b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546149
Reviewed-by: Theresa <[email protected]>
Commit-Queue: Robert Ogden <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645819}(cherry picked from commit e0e0b61c9e6eb785bf2eb86f3d01ebfa4d004b8f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547345
Reviewed-by: Robert Ogden <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#579}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/drawable/data_reduction_illustration.xml[diff]

Update Lite Mode FRE to use secondary grey text color

Note: This is intentionally done to only the Lite Mode FRE layout, not
Data Saver.

Bug: 947229
Change-Id: Id1c1e5b0f8e41f796fa56d047fa4f8ac47393639
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545171
Reviewed-by: Theresa <[email protected]>
Commit-Queue: Robert Ogden <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645581}(cherry picked from commit cfb6a482648fee3ca8061c23fe8ca42f801e9cd2)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546882
Reviewed-by: Robert Ogden <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#578}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/layout/fre_data_reduction_proxy_lite_mode.xml[diff]

Update Data Saver and Lite Mode empty savings graphic

This is the graphic displayed when there is too little data savings to
show the graph.

Updating it to be the new speedometer icon. It is intentionally a bit
smaller than the old one.

Bug: 947216
Change-Id: I12340a9588bf8ce292ed18daf71ea4b743776301
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546150
Reviewed-by: Theresa <[email protected]>
Commit-Queue: Robert Ogden <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645863}(cherry picked from commit 2b6e347d965738f5ce4bb24a160e6cefcd59d02a)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546866
Reviewed-by: Robert Ogden <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#577}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/drawable/data_reduction_big.xml[diff]

[M74 Merge][Managed Session] Change full warning login string

Bug: 947423
Change-Id: Iae78599ca08a92d38f8ea8fd22afed3602f82f2c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545915
Reviewed-by: Kush Sinha <[email protected]>
Commit-Queue: Ivan Šandrk <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645766}(cherry picked from commit 7964e930a54c98f152340719312e3a878e6125e5)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547792
Reviewed-by: Ivan Šandrk <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#576}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/ash_strings.grd[diff]
• ash/ash_strings_grd/IDS_ASH_LOGIN_MANAGED_SESSION_MONITORING_FULL_WARNING.png.sha1[diff]

[Feed] Import feed@fd6f3e737554d10c7293b4be0a0066559565ece0

Changes for 4 issues in the Feed library (third_party/feed) are merged
to M74 in this CL:
* crbug.com/942081 - More button after dismissing all articles results in an infinite spinner
* crbug.com/942587 - Infinite spinner on expand
* crbug.com/947209 - onSpinnerDesroyedWithoutCompleting() is being called too often
* crbug.com/939970 - Flickering Zero-State

Bug: 942081, 942587, 947209, 939970
Change-Id: I2eb0bd6ca740c963deae5e9a9b56639308f328ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547298
Reviewed-by: Filip Gorski <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#575}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• DEPS[diff]
• third_party/feed/README.chromium[diff]

Fix fragment anchors for SVG documents

In https://crrev.com/c/1395893 I did a refactoring of fragment anchors
that should have been a no-op in terms of visible behavior. This turned
out not to be the case.

Prior to that CL, LocalFrameView::ProcessUrlFragmentHelper would early
out in the case that we're in an SVG document and have no anchor_node:

if (frame_->GetDocument()->IsSVGDocument()) {
    if (SVGSVGElement* svg =
            ToSVGSVGElementOrNull(frame_->GetDocument()->documentElement())) {
      svg->SetupInitialView(name, anchor_node);
      if (!anchor_node)
        return false;
    }

This block of code was moved into FragmentAnchor::TryCreate (now in
ElementFragmentAnchor); however, we lost the
|if(!anchor_node) return false| block. This means we'll return the
document itself as the anchor when we |Implement the rule that "" and
"top" both mean top of page as in other browsers|. For a document
that's transformed this can cause unintended scrolling.

The fix here is to simply bring back the early return in the case of
an SVG document. We use target rather than anchor_node since the code
about implementing "top of page" has been moved up above this section.

Bug: 945609
Change-Id: I4149f5baf7c796cfb1fcb33a042dea291fa64861
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544638
Commit-Queue: Stefan Zager <[email protected]>
Reviewed-by: Stefan Zager <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645677}(cherry picked from commit 2cfaa0c66a29eae2c5ee5e0f3188a02f3a1a5c05)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547340
Reviewed-by: David Bokan <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#574}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/page/scrolling/element_fragment_anchor.cc[diff]
• third_party/blink/renderer/core/page/scrolling/element_fragment_anchor_test.cc[diff]
• third_party/blink/renderer/core/svg/graphics/svg_image.h[diff]

Incrementing VERSION to 74.0.3729.56

[email protected]

Change-Id: If8f371ff5e9de8fe7ad2bc15bd66d3a441e7eee8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547202
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#573}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Restore FeatureList switch for the cache-aware WebFonts loading

This CL is almost equivalent to revert commit
ee469355e81792a154346f10a39d05eefdb9d6b9.

But, the feature is also disabled by default on Android.

Original change's description:
> Remove FeatureList switch for the cache-aware WebFonts loading
>
> Now the feature has been enabled by default, and this change
> removes the kill-switch.
>
> Bug: 570205
> Change-Id: I9408d9c8f60edb562a817736816f376c5739ae61
> Reviewed-on: https://chromium-review.googlesource.com/c/1466542
> Reviewed-by: Kunihiko Sakamoto <[email protected]>
> Reviewed-by: Kinuko Yasuda <[email protected]>
> Reviewed-by: Yutaka Hirano <[email protected]>
> Commit-Queue: Takashi Toyoshima <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#632125}

Bug: 570205, 942440
Change-Id: I107906a7dd3b1629e362d3e5e8e4bad4daab1121
TBR: [email protected]
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1534806
Commit-Queue: Takashi Toyoshima <[email protected]>
Reviewed-by: Yutaka Hirano <[email protected]>
Reviewed-by: Takashi Toyoshima <[email protected]>
Reviewed-by: Kunihiko Sakamoto <[email protected]>
Auto-Submit: Takashi Toyoshima <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645191}(cherry picked from commit a91506df72a5833354c3201f07d800a1ac0dd8a6)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546801
Cr-Commit-Position: refs/branch-heads/3729@{#572}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/common/features.cc[diff]
• third_party/blink/public/common/features.h[diff]
• third_party/blink/renderer/core/css/css_font_face_src_value.cc[diff]
• third_party/blink/renderer/core/loader/resource/font_resource.h[diff]
• third_party/blink/renderer/core/loader/resource/font_resource_test.cc[diff]

Incrementing VERSION to 74.0.3729.55

[email protected]

Change-Id: I3bbba340201d8e215a3b61f03e53c86ef12111f7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547195
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#571}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Incrementing VERSION to 74.0.3729.54

[email protected]

Change-Id: I2bdfa62abc6dec7d227d06d6b77c7c0d004dfd02
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1547192
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#570}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Allow finch config on the expiration duration for download

BUG=938978

Change-Id: I45f887c26d692c32e56cec713d95d922138cee75
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539827
Commit-Queue: Min Qin <[email protected]>
Reviewed-by: Xing Liu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645086}(cherry picked from commit 8411c8ad6f692af9de9796586dd9854e19953d3d)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546935
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#569}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/download/internal/common/android/download_collection_bridge.cc[diff]
• components/download/internal/common/android/download_collection_bridge.h[diff]
• components/download/internal/common/android/java/src/org/chromium/components/download/DownloadCollectionBridge.java[diff]
• components/download/public/common/download_features.cc[diff]
• components/download/public/common/download_features.h[diff]

Refresh the expiration date of the unpublished download on resumption

For resumed download, we should extend the expiration date.
Or otherwise, large downloads may not complete on slow networks

BUG=938978

Change-Id: Ib57c5915a4caa1305e7b9ba6ec7c2abc1f30f3e9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538453
Reviewed-by: Xing Liu <[email protected]>
Reviewed-by: David Trainor <[email protected]>
Commit-Queue: Min Qin <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644566}(cherry picked from commit 840b730a7260a24e902fcd95fb116ba3fd10e7e1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546934
Reviewed-by: Min Qin <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#568}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/download/internal/common/android/java/src/org/chromium/components/download/DownloadCollectionBridge.java[diff]

Incrementing VERSION to 74.0.3729.53

[email protected]

Change-Id: I0cd99ffe34286890642b82d8e4175ff969ed7bc0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544807
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#567}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Incrementing VERSION to 74.0.3729.52

[email protected]

Change-Id: I2aadd19a9a3d51106f4024d1e2bdc55daeb08e27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546775
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#566}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Incrementing VERSION to 74.0.3729.51

[email protected]

Change-Id: I5cb53fd76457d18883e61de0b9505747304fd1d3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546854
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#565}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Expand/ceil default object size in SVGImagePainter

Ceil the default object size passed to LayoutImageResource::GetImage, to
increase the chances of the (container) size round-tripping through
Image::Size() (which we use as the source rect).
SVGImageForContainer::Size uses RoundedIntSize(), so depending on the
fractional part of the default object size it could end up either
smaller or larger.

Bug: 944739
Change-Id: Ifc62ddfad760ade87135b356ce6f6e07ed8e5dc5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538421
Reviewed-by: Stephen Chenney <[email protected]>
Commit-Queue: Fredrik Söderquist <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644238}(cherry picked from commit 6b4d840cc8026fa55141de1e5a6b3f12300a1cbf)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545926
Reviewed-by: Fredrik Söderquist <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#564}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/core/paint/svg_image_painter.cc[diff]
• third_party/blink/web_tests/external/wpt/svg/embedded/image-embedding-svg-viewref-with-viewbox.svg[Added - diff]

[Tango->FCM] Make lazyness work for the new invalidations

Currently messages are not persisted for the FCM invalidations, because
the sender id for which invaliations are marked lazy mismatches the id
received in the incoming messages. In addition, messages are replayed
before app handler is knows, so they aren't passed to it.

This patch implements the fix.

[email protected]

(cherry picked from commit 5bcb52875e96c60b17963b0898db971993f2434d)

Bug: 882887, 944484
Change-Id: Id660c986707b3cb9875ed18ea25e9bc8bd3fbfec
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1533875
Reviewed-by: Mikel Astiz <[email protected]>
Reviewed-by: Peter Beverloo <[email protected]>
Commit-Queue: Tatiana Gornak <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644773}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546291
Reviewed-by: Tatiana Gornak <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#563}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/gcm_driver/android/java/src/org/chromium/components/gcm_driver/GCMDriver.java[diff]
• components/gcm_driver/android/java/src/org/chromium/components/gcm_driver/LazySubscriptionsManager.java[diff]
• components/gcm_driver/gcm_driver_android.cc[diff]
• components/gcm_driver/gcm_driver_android.h[diff]
• components/invalidation/impl/fcm_invalidation_listener.cc[diff]
• components/invalidation/impl/fcm_invalidation_listener_unittest.cc[diff]
• components/invalidation/impl/fcm_invalidation_service.cc[diff]
• components/invalidation/impl/fcm_invalidator_unittest.cc[diff]
• components/invalidation/impl/fcm_network_handler.cc[diff]
• components/invalidation/impl/fcm_network_handler.h[diff]
• components/invalidation/impl/fcm_sync_network_channel.h[diff]

[Chromoting] Fix mouse coord bug with multimon.

When the default display is not leftmost, then the coordinates for the
mouse events are not calculated correctly. This fixes that.

Bug: 946001

Change-Id: I1d0124c2c5bbb828466ee306e60bb0aedaa2f5d5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536769
Commit-Queue: Gary Kacmarcik <[email protected]>
Reviewed-by: Joe Downing <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644492}(cherry picked from commit 6b6e445607ebe76f0c1ee037ba53b85bebff3927)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546612
Reviewed-by: Gary Kacmarcik <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#562}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• remoting/client/chromoting_client.cc[diff]
• remoting/client/plugin/chromoting_instance.cc[diff]
• remoting/host/BUILD.gn[diff]
• remoting/host/client_session.cc[diff]
• remoting/host/desktop_display_info.cc[diff]
• remoting/host/desktop_display_info.h[diff]
• remoting/host/desktop_display_info_unittest.cc[Added - diff]
• remoting/protocol/mouse_input_filter.cc[diff]
• remoting/protocol/mouse_input_filter.h[diff]
• remoting/protocol/mouse_input_filter_unittest.cc[diff]

Incrementing VERSION to 74.0.3729.50

[email protected]

Change-Id: I90f6dadb5316888e01659c30bf75f73152d5dbeb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546501
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#561}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

[M74 Reland] MediaCapabilities: Set new features *AFTER* finalizing previous record.

Reland: fix the unit test break.

This order ensures stats are saved with the correct stream features.

Bug: 945191, 947382
Change-Id: Ie4abb5dd109ce85fe9ef8145caff56f1e44fd21a
TBR: [email protected]
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546609
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#560}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/learning/common/BUILD.gn[diff]
• media/mojo/services/BUILD.gn[diff]
• media/mojo/services/test_helpers.cc[Added - diff]
• media/mojo/services/test_helpers.h[Added - diff]
• media/mojo/services/video_decode_perf_history_unittest.cc[diff]
• media/mojo/services/video_decode_stats_recorder.cc[diff]
• media/mojo/services/video_decode_stats_recorder_unittest.cc[Added - diff]

Reland "[SEC] Adding snackbar theming"

This is a reland of b4beb4446fc6a9377651f82e31a6655fc854efec

Original change's description:
> [SEC] Adding snackbar theming
> 
> * Allows components using a Snackbar to choose a Theme.
> * Introduces a Google theme for the snackbar.
> 
> [email protected]
> 
> (cherry picked from commit 124ae9d44ef358fbec0b4e879b423719bea693ce)
> 
> Bug: 945376
> Change-Id: Ia4fb4d580d4c2edbd65950a1211f97ec4805ef8d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1535130
> Reviewed-by: Becky Zhou <[email protected]>
> Reviewed-by: Ted Choc <[email protected]>
> Commit-Queue: Filip Gorski <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#644517}
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545519
> Reviewed-by: Filip Gorski <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#533}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

Bug: 945376
Change-Id: I71337132bdc72f982243ff4e185053445ff0207e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546419
Reviewed-by: Filip Gorski <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#559}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/SnackbarView.java[diff]
• ui/android/java/res/values-v17/styles.xml[diff]

[M74]Add back pre-select screen sharing screen for desktopCapture API

In M73 we stop preselect screen view in desktop media picker in the cl
https://chromium-review.googlesource.com/c/chromium/src/+/1407669.
And it causes great usability issues when M73 is in Stable channel:
- some users are confused by the missing pre-selection and don't know
  how to start screen sharing.
- On some Macbooks, it triggers a possible Cocoa defect that after
  selecting the target screen, the Share button will be enabled only a
  short period and disabled again. So the user can't start sharing.

As the discussion, we decide to add the pre-selection back into M74 to
ease the problem and keep to inverstigate solution in M75 to inform
user to choose before sharing.

Also fixed an issue introduced in cl
https://chromium-review.googlesource.com/c/chromium/src/+/1454129

PS: the Cocoa issue is fixed by moving picker implementation from
Cocoa to Chrome Views in M75. But we decided not to merge it into
M74 for the purpose to verify this fix only for a decision if we're
going to merget it back to M73.

Bug: 947328
Change-Id: I9deab5f17922c0ee145c81519f8c26d8f6c8b48f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545095
Reviewed-by: Avi Drissman <[email protected]>
Reviewed-by: Emircan Uysaler <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#558}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/extensions/api/desktop_capture/desktop_capture_base.cc[diff]
• chrome/browser/media/webrtc/desktop_media_picker.h[diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h[diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm[diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_list_view.cc[diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.cc[diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.h[diff]

[Merge M74] app_list: Prevent sending updates over destroyed mojo channel

This observes profile deletion prevents communication over
AppListController. ChromeAppListModelUpdater has longer live scope than
AppListClientImpl and using pointers owned by AppListClientImpl is not
safe on deletion

TEST=Simulate crash condition and make sure app is not loaded.
BUG=936665

Change-Id: Ib07861ac8b98a39af7d538659883ddbfe48f7607
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529633
Reviewed-by: Xiyuan Xia <[email protected]>
Commit-Queue: Yury Khmel <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642106}(cherry picked from commit fc09013b02eb912985afcfb3119ca839d700a9f1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546368
Reviewed-by: Yury Khmel <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#557}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/app_list/app_list_client_impl.cc[diff]
• chrome/browser/ui/app_list/chrome_app_list_model_updater.cc[diff]

Swipers gonna swipe (or reenable swiping for Grid)

1) Routes swipe down through a delegate OverviewModeController passed
during construction in LayoutManagerChrome. This keeps default behavior
as is but swipes down to the Grid when Grid or TabGroups is enabled.

2) Gets back swipe to side for both grid and tab groups. This was
disabled while looking for a solution to 1) above. Now the only caveat
is swipe to side scrolls between tabgroups if there are tabgroups which
is WAI.

3) Disables both side swipe and swipe down from the bottom toolbar for
the tab groups case where tabstrip needs the swiping gesture as it
technically can become long enough that it needs to scroll

BUG=941676

Change-Id: I706a3aa778ab08c5e46fbbcaaec40eb7f69c56fe
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1537250
Reviewed-by: David Trainor <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644630}(cherry picked from commit 27a27b01b12c00322a1b9c53808145d45170c65c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546355
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#556}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChrome.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChromePhone.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerChromeTablet.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/bottom/BottomControlsCoordinator.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/ToolbarControlContainer.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/compositor/layouts/LayoutManagerTest.java[diff]

Polish UI for Tab Grid

This is the CL for following UI polishment:

* Fade out title edge instead of ellipsizing it.
* Change the drawable for close tab in tab grid.
* Adjust padding in recyclerview.
* Adjust padding for incognito toggle indicator.
(TODO: Need a good way to adjust the width of indicator.)

Bug: 937537
Change-Id: Ic56d884a4fb83fa2390c628152cc8d5c94977689
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538809
Commit-Queue: Yue Zhang <[email protected]>
Reviewed-by: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644121}(cherry picked from commit 6441aafe21fecc776904ac6c2ea9f498d3b95623)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546317
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#555}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/layout/tab_grid_card_item.xml[diff]
• chrome/android/java/res/layout/tab_list_recycler_view_layout.xml[diff]
• chrome/android/java/res/layout/tab_switcher_toolbar.xml[diff]

Add swipe to dismiss to grid

Adds a way to close tab using side swipe by attaching an ItemTouchHelper
and also adding a new TabProperties key.

BUG=941676

Change-Id: I6216b430f60aff0cde4b4775f55d78ebb267bbe7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536806
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644117}(cherry picked from commit 62e165a5a53307caaaf9b002de0ca11e8c2d9bc1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546316
Cr-Commit-Position: refs/branch-heads/3729@{#554}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/SilenceLintErrors.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListMediator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java[diff]
• chrome/android/java/res/values/dimens.xml[diff]

Fix tabStrip not show during cold start

Set the visibilty of tab strip after cold start.

Change-Id: I660fb92ef5d4db0da7ed6a439f831591eaf11c5a
Bug: 945040
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538539
Reviewed-by: Yusuf Ozuysal <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643947}(cherry picked from commit 62ea490d67eaefcb723d80c650e8adfc9740f4de)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546258
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#553}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGroupUiMediator.java[diff]

Updating XTBs based on .GRDs from branch 3729

[email protected]

Change-Id: I8e85da1f9d2f4fcce87f6f20e0237337f837f7ee
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546362
Reviewed-by: Mustafa Emre Acer <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#552}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• android_webview/ui/translations/aw_strings_sk.xtb[diff]
• ash/components/strings/ash_components_strings_fa.xtb[diff]
• ash/components/strings/ash_components_strings_id.xtb[diff]
• ash/components/strings/ash_components_strings_iw.xtb[diff]
• ash/components/strings/ash_components_strings_ja.xtb[diff]
• ash/components/strings/ash_components_strings_sr.xtb[diff]
• ash/components/strings/ash_components_strings_uk.xtb[diff]
• ash/strings/ash_strings_da.xtb[diff]
• ash/strings/ash_strings_de.xtb[diff]
• ash/strings/ash_strings_es-419.xtb[diff]
• ash/strings/ash_strings_es.xtb[diff]
• ash/strings/ash_strings_hi.xtb[diff]
• ash/strings/ash_strings_iw.xtb[diff]
• ash/strings/ash_strings_ja.xtb[diff]
• ash/strings/ash_strings_lv.xtb[diff]
• ash/strings/ash_strings_mr.xtb[diff]
• ash/strings/ash_strings_nl.xtb[diff]
• ash/strings/ash_strings_ru.xtb[diff]
• ash/strings/ash_strings_sk.xtb[diff]
• ash/strings/ash_strings_ta.xtb[diff]
• ash/strings/ash_strings_te.xtb[diff]
• ash/strings/ash_strings_tr.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_bn.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_ca.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_cs.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_de.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_es-419.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_fa.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_fil.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_fr.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_gu.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_hi.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_iw.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_kn.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_ml.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_mr.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_no.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_ru.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_sl.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_ta.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_te.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_tr.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_uk.xtb[diff]
• chrome/android/features/autofill_assistant/java/strings/translations/android_chrome_autofill_assistant_strings_zh-TW.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_ar.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_da.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_de.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_el.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_en-GB.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_es-419.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_es.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_fa.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_fr.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_hi.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_hu.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_id.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_it.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_iw.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_ja.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_ko.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_lt.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_nl.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_no.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_pl.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_pt-PT.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_ro.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_ru.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sk.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sr.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sv.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_sw.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_th.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_tr.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_uk.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_vi.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_zh-CN.xtb[diff]
• chrome/android/java/strings/translations/android_chrome_strings_zh-TW.xtb[diff]
• chrome/android/webapk/strings/translations/android_webapk_strings_it.xtb[diff]
• chrome/app/resources/chromium_strings_ar.xtb[diff]
• chrome/app/resources/chromium_strings_en-GB.xtb[diff]
• chrome/app/resources/chromium_strings_es-419.xtb[diff]
• chrome/app/resources/chromium_strings_fil.xtb[diff]
• chrome/app/resources/chromium_strings_fr.xtb[diff]
• chrome/app/resources/chromium_strings_hi.xtb[diff]
• chrome/app/resources/chromium_strings_id.xtb[diff]
• chrome/app/resources/chromium_strings_it.xtb[diff]
• chrome/app/resources/chromium_strings_mr.xtb[diff]
• chrome/app/resources/chromium_strings_no.xtb[diff]
• chrome/app/resources/chromium_strings_vi.xtb[diff]
• chrome/app/resources/generated_resources_ar.xtb[diff]
• chrome/app/resources/generated_resources_bn.xtb[diff]
• chrome/app/resources/generated_resources_ca.xtb[diff]
• chrome/app/resources/generated_resources_cs.xtb[diff]
• chrome/app/resources/generated_resources_da.xtb[diff]
• chrome/app/resources/generated_resources_de.xtb[diff]
• chrome/app/resources/generated_resources_el.xtb[diff]
• chrome/app/resources/generated_resources_en-GB.xtb[diff]
• chrome/app/resources/generated_resources_es-419.xtb[diff]
• chrome/app/resources/generated_resources_es.xtb[diff]
• chrome/app/resources/generated_resources_fa.xtb[diff]
• chrome/app/resources/generated_resources_fi.xtb[diff]
• chrome/app/resources/generated_resources_fil.xtb[diff]
• chrome/app/resources/generated_resources_fr.xtb[diff]
• chrome/app/resources/generated_resources_gu.xtb[diff]
• chrome/app/resources/generated_resources_hi.xtb[diff]
• chrome/app/resources/generated_resources_hu.xtb[diff]
• chrome/app/resources/generated_resources_id.xtb[diff]
• chrome/app/resources/generated_resources_it.xtb[diff]
• chrome/app/resources/generated_resources_iw.xtb[diff]
• chrome/app/resources/generated_resources_ja.xtb[diff]
• chrome/app/resources/generated_resources_kn.xtb[diff]
• chrome/app/resources/generated_resources_ko.xtb[diff]
• chrome/app/resources/generated_resources_lt.xtb[diff]
• chrome/app/resources/generated_resources_lv.xtb[diff]
• chrome/app/resources/generated_resources_ml.xtb[diff]
• chrome/app/resources/generated_resources_mr.xtb[diff]
• chrome/app/resources/generated_resources_nl.xtb[diff]
• chrome/app/resources/generated_resources_no.xtb[diff]
• chrome/app/resources/generated_resources_pl.xtb[diff]
• chrome/app/resources/generated_resources_pt-BR.xtb[diff]
• chrome/app/resources/generated_resources_pt-PT.xtb[diff]
• chrome/app/resources/generated_resources_ro.xtb[diff]
• chrome/app/resources/generated_resources_ru.xtb[diff]
• chrome/app/resources/generated_resources_sk.xtb[diff]
• chrome/app/resources/generated_resources_sl.xtb[diff]
• chrome/app/resources/generated_resources_sr.xtb[diff]
• chrome/app/resources/generated_resources_sv.xtb[diff]
• chrome/app/resources/generated_resources_sw.xtb[diff]
• chrome/app/resources/generated_resources_ta.xtb[diff]
• chrome/app/resources/generated_resources_te.xtb[diff]
• chrome/app/resources/generated_resources_th.xtb[diff]
• chrome/app/resources/generated_resources_tr.xtb[diff]
• chrome/app/resources/generated_resources_uk.xtb[diff]
• chrome/app/resources/generated_resources_vi.xtb[diff]
• chrome/app/resources/generated_resources_zh-CN.xtb[diff]
• chrome/app/resources/generated_resources_zh-TW.xtb[diff]
• chrome/app/resources/google_chrome_strings_ar.xtb[diff]
• chrome/app/resources/google_chrome_strings_da.xtb[diff]
• chrome/app/resources/google_chrome_strings_en-GB.xtb[diff]
• chrome/app/resources/google_chrome_strings_es-419.xtb[diff]
• chrome/app/resources/google_chrome_strings_es.xtb[diff]
• chrome/app/resources/google_chrome_strings_fil.xtb[diff]
• chrome/app/resources/google_chrome_strings_fr.xtb[diff]
• chrome/app/resources/google_chrome_strings_hi.xtb[diff]
• chrome/app/resources/google_chrome_strings_id.xtb[diff]
• chrome/app/resources/google_chrome_strings_it.xtb[diff]
• chrome/app/resources/google_chrome_strings_iw.xtb[diff]
• chrome/app/resources/google_chrome_strings_mr.xtb[diff]
• chrome/app/resources/google_chrome_strings_no.xtb[diff]
• chrome/app/resources/google_chrome_strings_ro.xtb[diff]
• chrome/app/resources/google_chrome_strings_sk.xtb[diff]
• chrome/app/resources/google_chrome_strings_vi.xtb[diff]
• chrome/browser/resources/chromeos/camera/src/strings/camera_strings_ar.xtb[diff]
• chrome/browser/resources/chromeos/camera/src/strings/camera_strings_iw.xtb[diff]
• chrome/browser/resources/chromeos/camera/src/strings/camera_strings_sr.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_ar.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_es.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_hi.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_hu.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_it.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_iw.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_uk.xtb[diff]
• chrome/browser/resources/chromeos/chromevox/strings/chromevox_strings_zh-CN.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_ar.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_es-419.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_fr.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_hi.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_it.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_iw.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_ja.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_lv.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_pt-BR.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_sl.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_th.xtb[diff]
• chrome/credential_provider/gaiacp/strings/gaia_resources_vi.xtb[diff]
• components/policy/resources/policy_templates_ar.xtb[diff]
• components/policy/resources/policy_templates_cs.xtb[diff]
• components/policy/resources/policy_templates_da.xtb[diff]
• components/policy/resources/policy_templates_de.xtb[diff]
• components/policy/resources/policy_templates_el.xtb[diff]
• components/policy/resources/policy_templates_en-GB.xtb[diff]
• components/policy/resources/policy_templates_es-419.xtb[diff]
• components/policy/resources/policy_templates_es.xtb[diff]
• components/policy/resources/policy_templates_et.xtb[diff]
• components/policy/resources/policy_templates_fr.xtb[diff]
• components/policy/resources/policy_templates_hi.xtb[diff]
• components/policy/resources/policy_templates_hr.xtb[diff]
• components/policy/resources/policy_templates_hu.xtb[diff]
• components/policy/resources/policy_templates_id.xtb[diff]
• components/policy/resources/policy_templates_it.xtb[diff]
• components/policy/resources/policy_templates_iw.xtb[diff]
• components/policy/resources/policy_templates_ja.xtb[diff]
• components/policy/resources/policy_templates_ko.xtb[diff]
• components/policy/resources/policy_templates_lv.xtb[diff]
• components/policy/resources/policy_templates_nl.xtb[diff]
• components/policy/resources/policy_templates_pl.xtb[diff]
• components/policy/resources/policy_templates_pt-BR.xtb[diff]
• components/policy/resources/policy_templates_pt-PT.xtb[diff]
• components/policy/resources/policy_templates_ro.xtb[diff]
• components/policy/resources/policy_templates_ru.xtb[diff]
• components/policy/resources/policy_templates_sk.xtb[diff]
• components/policy/resources/policy_templates_sl.xtb[diff]
• components/policy/resources/policy_templates_sr.xtb[diff]
• components/policy/resources/policy_templates_sw.xtb[diff]
• components/policy/resources/policy_templates_th.xtb[diff]
• components/policy/resources/policy_templates_vi.xtb[diff]
• components/strings/components_chromium_strings_da.xtb[diff]
• components/strings/components_google_chrome_strings_da.xtb[diff]
• components/strings/components_strings_ar.xtb[diff]
• components/strings/components_strings_bn.xtb[diff]
• components/strings/components_strings_ca.xtb[diff]
• components/strings/components_strings_cs.xtb[diff]
• components/strings/components_strings_da.xtb[diff]
• components/strings/components_strings_de.xtb[diff]
• components/strings/components_strings_el.xtb[diff]
• components/strings/components_strings_en-GB.xtb[diff]
• components/strings/components_strings_es-419.xtb[diff]
• components/strings/components_strings_es.xtb[diff]
• components/strings/components_strings_fa.xtb[diff]
• components/strings/components_strings_fil.xtb[diff]
• components/strings/components_strings_fr.xtb[diff]
• components/strings/components_strings_gu.xtb[diff]
• components/strings/components_strings_hi.xtb[diff]
• components/strings/components_strings_hu.xtb[diff]
• components/strings/components_strings_id.xtb[diff]
• components/strings/components_strings_it.xtb[diff]
• components/strings/components_strings_iw.xtb[diff]
• components/strings/components_strings_ja.xtb[diff]
• components/strings/components_strings_kn.xtb[diff]
• components/strings/components_strings_ko.xtb[diff]
• components/strings/components_strings_ml.xtb[diff]
• components/strings/components_strings_mr.xtb[diff]
• components/strings/components_strings_nl.xtb[diff]
• components/strings/components_strings_no.xtb[diff]
• components/strings/components_strings_pl.xtb[diff]
• components/strings/components_strings_pt-PT.xtb[diff]
• components/strings/components_strings_ro.xtb[diff]
• components/strings/components_strings_ru.xtb[diff]
• components/strings/components_strings_sk.xtb[diff]
• components/strings/components_strings_sl.xtb[diff]
• components/strings/components_strings_sr.xtb[diff]
• components/strings/components_strings_sv.xtb[diff]
• components/strings/components_strings_ta.xtb[diff]
• components/strings/components_strings_te.xtb[diff]
• components/strings/components_strings_th.xtb[diff]
• components/strings/components_strings_tr.xtb[diff]
• components/strings/components_strings_uk.xtb[diff]
• components/strings/components_strings_zh-CN.xtb[diff]
• components/strings/components_strings_zh-TW.xtb[diff]
• content/app/strings/translations/content_strings_ar.xtb[diff]
• content/app/strings/translations/content_strings_hi.xtb[diff]
• content/app/strings/translations/content_strings_it.xtb[diff]
• content/app/strings/translations/content_strings_iw.xtb[diff]
• content/app/strings/translations/content_strings_ja.xtb[diff]
• content/app/strings/translations/content_strings_ro.xtb[diff]
• content/app/strings/translations/content_strings_ru.xtb[diff]
• content/public/android/java/strings/translations/android_content_strings_da.xtb[diff]
• content/public/android/java/strings/translations/android_content_strings_iw.xtb[diff]
• content/public/android/java/strings/translations/android_content_strings_ja.xtb[diff]
• content/public/android/java/strings/translations/android_content_strings_ro.xtb[diff]
• content/public/android/java/strings/translations/android_content_strings_ru.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_ar.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_en-GB.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_id.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_it.xtb[diff]
• ios/chrome/app/strings/resources/ios_chromium_strings_nl.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_ar.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_da.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_de.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_en-GB.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_es.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_hi.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_hu.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_id.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_it.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_iw.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_nl.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_ro.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_sk.xtb[diff]
• ios/chrome/app/strings/resources/ios_google_chrome_strings_zh-CN.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_ar.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_da.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_el.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_en-GB.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_fr.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_hi.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_hu.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_id.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_it.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_iw.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_ja.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_nl.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_pl.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_pt-PT.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_ro.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_ru.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_sk.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_th.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_tr.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_uk.xtb[diff]
• ios/chrome/app/strings/resources/ios_strings_zh-CN.xtb[diff]
• remoting/resources/remoting_strings_id.xtb[diff]
• remoting/resources/remoting_strings_it.xtb[diff]
• remoting/resources/remoting_strings_iw.xtb[diff]
• remoting/resources/remoting_strings_ro.xtb[diff]
• ui/accessibility/extensions/strings/accessibility_extensions_strings_ar.xtb[diff]
• ui/accessibility/extensions/strings/accessibility_extensions_strings_it.xtb[diff]
• ui/accessibility/extensions/strings/accessibility_extensions_strings_iw.xtb[diff]
• ui/strings/translations/ui_strings_hi.xtb[diff]
• ui/strings/translations/ui_strings_iw.xtb[diff]

Add UMA histograms for TabGroup

This CL adds the following histograms for TabGroup:
  * TabGroups.UserTabGroupCount
  * TabStrip.TabCountOnPageLoad

Change-Id: Id205e83a6f5d1c0331b709dee837219b3943d78f
Bug: 942779
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1526688
Reviewed-by: Yusuf Ozuysal <[email protected]>
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Reviewed-by: Steven Holte <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643664}(cherry picked from commit aef4f6b67cb2978127488ef11ba5cc8f24bca4e4)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546353
Cr-Commit-Position: refs/branch-heads/3729@{#551}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGroupUiCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGroupUiMediator.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/tasks/tabgroup/TabGroupModelFilter.java[diff]
• tools/metrics/histograms/histograms.xml[diff]

[m74] CORB allowlist additions - 2019-03-26.

[email protected]

(cherry picked from commit 83dbc2de113a36c7b8146bfdb485a11a91c6f420)

Bug: 937081
Change-Id: Icf70ae575e75f32bbd5cae614bf9039679c23694
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539865
Reviewed-by: Devlin <[email protected]>
Commit-Queue: Łukasz Anforowicz <[email protected]>
Auto-Submit: Łukasz Anforowicz <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645332}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546352
Reviewed-by: Łukasz Anforowicz <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#550}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• extensions/browser/url_loader_factory_manager.cc[diff]

Android download: Fix storage space UI on Android Q.

On Android Q, Environment.DIRECTORY_DOWNLOADS is an invalid directory,
and all disk space queries like getUsableSpace() or getTotalSpace()
will return 0, and other related calls may throw exception. This is
because we didn't create the directory in external storage. Q basically
exposes a bug in our code.

This CL adds a utility function to retrieve the directory and tries to
create it. Also updates DownloadUtils.isInPrimaryStorageDownloadDirectory
to work with content URI file path on Q.


[email protected]

(cherry picked from commit 281e07f71aaaac3362989521c0f89d1c99a755bb)

Bug: 945028
Change-Id: I22cce2e5ac81bd574eee96719648689c5a8cdbf7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538876
Reviewed-by: Min Qin <[email protected]>
Reviewed-by: David Trainor <[email protected]>
Commit-Queue: Xing Liu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644621}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546312
Reviewed-by: Xing Liu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#549}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/download/DownloadDirectoryProvider.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/download/home/storage/StorageSummaryProvider.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/download/ui/SpaceDisplay.java[diff]

Draw gray square for tabs without screenshots

Quick follow up to https://chromium-review.googlesource.com/c/chromium/src/+/1531651
I had missed the condition where we have a tab but don't have a
screenshot. In that case, we should still draw the grey square,
before decrementing the count.

BUG=943876

Change-Id: I9703d18541ab00f876827fd514d8db94a3aec5ab
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536215
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Commit-Queue: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643546}(cherry picked from commit 3e754732817a03058cac6909c5f3313eb1814717)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544879
Cr-Commit-Position: refs/branch-heads/3729@{#548}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/MultiThumbnailCardProvider.java[diff]

Disable Duet when TabGroupsAndroid is on

Bug: 944228
Change-Id: I2ac8191844137ab2266c2457eb5e861e4e588fa9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1533090
Reviewed-by: Pedro Amaral <[email protected]>
Reviewed-by: Yusuf Ozuysal <[email protected]>
Commit-Queue: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643531}(cherry picked from commit 2cc7504aaf8c216ecf51c7c92de3e56cf3d92be1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545704
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#547}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/util/FeatureUtilities.java[diff]

Add "Create group" button for GTS

Bug: 944339
Change-Id: I290bfe5c436128d2269cf349940c5d86b12d43fa
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532248
Commit-Queue: Mei Liang <[email protected]>
Reviewed-by: Yusuf Ozuysal <[email protected]>
Reviewed-by: Wei-Yin Chen (陳威尹) <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643509}(cherry picked from commit 687c506f6a45a4fd8335a32119a5726ac3f8182d)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544236
Cr-Commit-Position: refs/branch-heads/3729@{#546}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/GridTabSwitcherCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/GridTabSwitcherMediator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridSheetCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewBinder.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGridViewHolder.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabGroupUiCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListCoordinator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabListMediator.java[diff]
• chrome/android/features/tab_ui/java/src/org/chromium/chrome/browser/tasks/tab_management/TabProperties.java[diff]
• chrome/android/features/tab_ui/junit/src/org/chromium/chrome/browser/tasks/tab_management/TabListMediatorUnitTest.java[diff]
• chrome/android/java/res/layout/tab_grid_card_item.xml[diff]

[Dark] Update filled button text color

+ Update filled button text color to grey 900 in dark mode

Bug: 934550
Change-Id: Ic10d35b130b6d2e8a47cc033f5c3e886e0dd0926
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1522342
Commit-Queue: Becky Zhou <[email protected]>
Reviewed-by: Theresa <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#641249}(cherry picked from commit 119a270a66c0c28dce74ccad9e24f0feeb878b2e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546083
Reviewed-by: Becky Zhou <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#545}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/android/java/res/values-v17/styles.xml[diff]
• ui/android/java/res/values/color_palette.xml[diff]
• ui/android/java/res/values/colors.xml[diff]
• ui/android/java/res_night/values-night/colors.xml[diff]

[Tango->FCM] Ditch invalidation client.

Invalidation client was introduced during migration. Main reason for
having it was that new architecture was heavily invfluenced by old one.
Now it's clear that InvalidationClient is unnessesary.

 Bug: 801985, 878446

[email protected]

(cherry picked from commit 2bab2c79d672f563a95ff37a4e50bb4da1d36b39)

Change-Id: Ibdd832eb98143c02fd8a342f51ae7e509cfb8707
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1508393
Reviewed-by: Tatiana Gornak <[email protected]>
Reviewed-by: Marc Treib <[email protected]>
Commit-Queue: Tatiana Gornak <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#640244}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1546091
Cr-Commit-Position: refs/branch-heads/3729@{#544}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/invalidation/impl/BUILD.gn[diff]
• components/invalidation/impl/fcm_invalidation_listener.cc[diff]
• components/invalidation/impl/fcm_invalidation_listener.h[diff]
• components/invalidation/impl/fcm_invalidation_listener_unittest.cc[diff]
• components/invalidation/impl/fcm_invalidator.cc[diff]
• components/invalidation/impl/invalidation_client.h[Deleted - diff]
• components/invalidation/impl/invalidation_listener.h[diff]
• components/invalidation/impl/per_user_topic_invalidation_client.cc[Deleted - diff]
• components/invalidation/impl/per_user_topic_invalidation_client.h[Deleted - diff]
• components/invalidation/impl/per_user_topic_invalidation_client_unittest.cc[Deleted - diff]

Incrementing VERSION to 74.0.3729.49

[email protected]

Change-Id: If33fa0b2eed80d5da6e85cfa1768e5fd2749c06a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545699
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#543}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Revert "[SEC] Adding snackbar theming"

This reverts commit b4beb4446fc6a9377651f82e31a6655fc854efec.

Reason for revert: https://crbug.com/947394 - build failed

Original change's description:
> [SEC] Adding snackbar theming
> 
> * Allows components using a Snackbar to choose a Theme.
> * Introduces a Google theme for the snackbar.
> 
> [email protected]
> 
> (cherry picked from commit 124ae9d44ef358fbec0b4e879b423719bea693ce)
> 
> Bug: 945376
> Change-Id: Ia4fb4d580d4c2edbd65950a1211f97ec4805ef8d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1535130
> Reviewed-by: Becky Zhou <[email protected]>
> Reviewed-by: Ted Choc <[email protected]>
> Commit-Queue: Filip Gorski <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#644517}
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545519
> Reviewed-by: Filip Gorski <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#533}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

[email protected],[email protected],[email protected]

Change-Id: Ia751a30677d7a7e6a02b66f8cbb4bd2c92e7e224
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 945376
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544643
Reviewed-by: Krishna Govind <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#542}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/SnackbarView.java[diff]
• ui/android/java/res/values-v17/styles.xml[diff]

Android Photo Picker: Request appropriate permission for Q.

(cherry picked from commit 2fac4893c1e64d9eb2e17cf8c8f31e521cb556a8)

Bug: 905257, 656015
Change-Id: Ieefe575a59e215c84b55920f28137a562c8b22b9
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1528573
Commit-Queue: Finnur Thorarinsson <[email protected]>
Auto-Submit: Finnur Thorarinsson <[email protected]>
Reviewed-by: Theresa <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644248}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545306
Reviewed-by: Finnur Thorarinsson <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#541}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/AndroidManifest.xml[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/FileEnumWorkerTask.java[diff]
• ui/android/java/src/org/chromium/ui/base/SelectFileDialog.java[diff]

android: Do not use /proc/self/exe to get dynamic section.

The crazy linker used to parse /proc/self/exe and /proc/self/maps
in order to find the value of the process-global |r_debug| variable
maintained by the system linker. Unfortunately, this operation is
flaky on Android L (see attached bug for details). In the end this
prevents Crashpad from generating valid stack traces, because it
will not see libchrome.so in the current process when this
happens.

This CL uses getauxval() instead to get the same address, since
this function is available from the C library starting with API
level 18 / Android KitKat.

+ Add new unit-test to check that RDebug::GetAddress() works
  correctly.

BUG=946380
R=​[email protected], [email protected], [email protected]

Change-Id: I90212f94206fda05be515bdbd6fc7f1285f041a8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541117
Reviewed-by: Andrew Grieve <[email protected]>
Commit-Queue: David Turner <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644778}(cherry picked from commit f684eadf6f3a9d7cda20429c3dcfd6f550873ca9)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545060
Reviewed-by: David Turner <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#540}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/android_crazy_linker/BUILD.gn[diff]
• third_party/android_crazy_linker/src/src/crazy_linker_rdebug.cpp[diff]
• third_party/android_crazy_linker/src/src/crazy_linker_rdebug_unittest.cpp[Added - diff]

Photo Picker: Convert filePaths to URIs.

(A reland of the original patch, with just a small fix to the test)
https://chromium-review.googlesource.com/c/chromium/src/+/1524262 (Original)

TBR=twellington

Bug: 905257, 656015
Change-Id: Ibe999a7ad65ba531170f8b4c9f2503f5d66bc9fb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1530986
Reviewed-by: Finnur Thorarinsson <[email protected]>
Commit-Queue: Finnur Thorarinsson <[email protected]>
Auto-Submit: Finnur Thorarinsson <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642103}(cherry picked from commit 43ab8952a0148aa9a82d459bf1ae6664dba389ec)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545396
Cr-Commit-Position: refs/branch-heads/3729@{#539}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/DecoderServiceHost.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/FileEnumWorkerTask.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialog.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/PickerBitmap.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/PickerBitmapViewHolder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/photo_picker/PickerCategoryView.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/photo_picker/PhotoPickerDialogTest.java[diff]
• ui/android/java/src/org/chromium/ui/PhotoPickerListener.java[diff]
• ui/android/java/src/org/chromium/ui/base/SelectFileDialog.java[diff]

app_banner: Fix the crash when the frame is detached when BeforeInstallPromptEvent is created

This CL fixes the crash when getting a task runner in the constructor of
BeforeInstallPromptEvent even though the frame is detached. When a frame
is detached, its document still lives but its scheduler does not, then
getting task runners from such frame would not work.

Bug: 926282
Change-Id: Ib1b218eb920a6b62c0bf58b5b2214b9bfb66ab79
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1535695
Commit-Queue: Hajime Hoshi <[email protected]>
Reviewed-by: Dominick Ng <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643331}(cherry picked from commit d7508e0230611749a423707155339e14fd11ca3f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545446
Reviewed-by: Hajime Hoshi <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#538}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/modules/app_banner/app_banner_controller.cc[diff]

Incrementing VERSION to 74.0.3729.48

[email protected]

Change-Id: I9ab69dbbadbc93f2935e7c9550e9b38380ef4469
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545692
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#537}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Reland "[AudioOutputDevices] Check frame before checking device authorization."

This is a reland of 3d8e2c2aa3646c4a5507eead750b8e1e1d4e29c6

Original change's description:
> [AudioOutputDevices] Check frame before checking device authorization.
> 
> This attempts to address an unusual crash involving WebLocalFrame and
> SetSinkIdResolver.
> This CL also removes a timer to fire an asynchronous task and uses
> a per-frame task runner instead.
> 
> Bug: 941380
> Change-Id: I9acf17ca3fd7d3dd8253b4e278adf583f4205f85
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1524267
> Reviewed-by: Marina Ciocea <[email protected]>
> Commit-Queue: Guido Urdaneta <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#641563}

Bug: 941380
Change-Id: I6e61eb632e22fa81230b2420eed0b1c40c5ba992
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1543632
Reviewed-by: Marina Ciocea <[email protected]>
Commit-Queue: Guido Urdaneta <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645390}(cherry picked from commit 781973a7282d999eed7f663547828a4d4fed3fbc)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545291
Reviewed-by: Guido Urdaneta <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#536}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• third_party/blink/renderer/modules/audio_output_devices/html_media_element_audio_output_device.cc[diff]

Incrementing VERSION to 74.0.3729.47

[email protected]

Change-Id: I53326b433ff789529f2d2e15e72f0c9acacecd64
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545691
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#535}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Revert "[M74] MediaCapabilities: Set new features *AFTER* finalizing previous record."

This reverts commit 1e39a6495f5b5607b18b783a8207ed7c8a3dc981.

Reason for revert: Breaks build, bug 947382

Original change's description:
> [M74] MediaCapabilities: Set new features *AFTER* finalizing previous record.
> 
> This order ensures stats are saved with the correct stream features.
> 
> Change-Id: Iacf7cda7960438aae1a8638e3eae17e03b5e4a2d
> Bug: 945191
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536897
> Commit-Queue: Chrome Cunningham <[email protected]>
> Reviewed-by: Frank Liberato <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#643746}(cherry picked from commit d601f2fe894602e132f1725a997691376573d639)
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545193
> Reviewed-by: Chrome Cunningham <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#528}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

[email protected]

Change-Id: I9125569bd87b66930011894e1ff39e88bd25dbf5
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 945191
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545689
Reviewed-by: Krishna Govind <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#534}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/learning/common/BUILD.gn[diff]
• media/mojo/services/BUILD.gn[diff]
• media/mojo/services/test_helpers.cc[Deleted - diff]
• media/mojo/services/test_helpers.h[Deleted - diff]
• media/mojo/services/video_decode_perf_history_unittest.cc[diff]
• media/mojo/services/video_decode_stats_recorder.cc[diff]
• media/mojo/services/video_decode_stats_recorder_unittest.cc[Deleted - diff]

[SEC] Adding snackbar theming

* Allows components using a Snackbar to choose a Theme.
* Introduces a Google theme for the snackbar.

[email protected]

(cherry picked from commit 124ae9d44ef358fbec0b4e879b423719bea693ce)

Bug: 945376
Change-Id: Ia4fb4d580d4c2edbd65950a1211f97ec4805ef8d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1535130
Reviewed-by: Becky Zhou <[email protected]>
Reviewed-by: Ted Choc <[email protected]>
Commit-Queue: Filip Gorski <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644517}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545519
Reviewed-by: Filip Gorski <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#533}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/snackbar/Snackbar.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/snackbar/SnackbarView.java[diff]
• ui/android/java/res/values-v17/styles.xml[diff]

Revert "macviews: switch to Views media picker on Mac"

This reverts commit 65ce7cbe76ed194369c3d22df6dd2d9e2cfa16a4.

Reason for revert: <INSERT REASONING HERE>

Original change's description:
> macviews: switch to Views media picker on Mac
> 
> This change:
> 1) Adds a Mac implementation of DesktopMediaWindowRegistry, which is
>    essentially a stub because Mac windows already have identifying
>    numbers;
> 2) Makes DesktopMediaID::RegisterNativeWindow and ::GetNativeWindowById
>    available on Mac, backed by the new DesktopMediaWindowRegistryMac;
> 3) Builds all the Views DesktopMediaPicker code on Mac, including tests;
> 4) Removes all the Cocoa DesktopMediaPicker code;
> 5) Fixes DesktopMediaPickerViews to not try to use MODAL_TYPE_CHILD on
>    Mac when there is no parent window - this isn't allowed, so it now
>    falls back to MODAL_TYPE_WINDOW;
> 6) Adds a Mac codepath in DesktopMediaPickerViews to ignore the picker
>    window in the media sources list;
> 7) Disables modal animations on Mac in DesktopMediaPickerViewTest.*,
>    since these animations can otherwise make the tests flakily hang
>    during teardown;
> 
> To test the new UI:
> * Install this helper extension: https://chrome.google.com/webstore/detail/screen-capturing/ajhifddimkapgcifgcodmmfdlknahffk
> * Navigate to this test page: https://www.webrtc-experiment.com/screen-sharing/#32685442571862986
> * Click the big blue button :)
> 
> Work still to be done:
> * The TableView used for selecting tabs has some odd appearance glitches
> * The focus ring on windows in the source list is an old-style Views
>   ring
> * Accessibility has not been tested thoroughly
> 
> Bug: 726005
> Change-Id: I0927dd94ede9b9315ffa41ac5ecc83579d456161
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1524521
> Reviewed-by: Avi Drissman <[email protected]>
> Reviewed-by: Weiyong Yao <[email protected]>
> Commit-Queue: Elly Fong-Jones <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#641287}(cherry picked from commit 447f22268ced83b914233365ae1c8ebc7fa8aa73)
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544189
> Cr-Commit-Position: refs/branch-heads/3729@{#510}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

[email protected],[email protected]

Change-Id: I8389e02aeb5b316f65691fbd8b5ce2530769d962
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 726005
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545495
Reviewed-by: Abdul Syed <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#532}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/BUILD.gn[diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.h[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.mm[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.mm[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller_unittest.mm[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h[Added - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.mm[Added - diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.cc[diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views_unittest.cc[diff]
• chrome/test/BUILD.gn[diff]
• content/browser/BUILD.gn[diff]
• content/browser/media/desktop_media_window_registry_mac.mm[Deleted - diff]
• content/public/browser/desktop_media_id.cc[diff]
• content/public/browser/desktop_media_id.h[diff]

Incrementing VERSION to 74.0.3729.46

[email protected]

Change-Id: I13375ad871a092c581a9fb80324cf75f6dc11fa2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545511
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#531}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Custom Margins: Default to 0 for missing values (M74)

Tolerate non-existent custom margins values in C++, by setting the
margin field to 0 if it does not exist. This will prevent the crashes
in the linked bug, and is safer than merging larger changes to the
Print Preview JS that will prevent this from happening.

This is a short term workaround; we will change this back to a stricter
check after landing larger changes in Print Preview JS to prevent bad
custom margin values from being sent.

Bug: 942211
Change-Id: I978fdf65519efc0d6a0d6379c2266e330908fc0d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529314
Reviewed-by: Lei Zhang <[email protected]>
Commit-Queue: Lei Zhang <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642002}(cherry picked from commit 9f1465fc2d48f0b57fcd3bddc9da45946c97fe5d)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545491
Reviewed-by: Rebekah Potter <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#530}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• printing/print_settings_conversion.cc[diff]

[Video Capture Service] Fix for service no longer shutting down on idle

CL https://chromium-review.googlesource.com/c/chromium/src/+/1504230
caused a regression leading to the video capture service no longer
shutting down on idle.

This CL fixes this and fills in the missing test coverage.

Bug: 941240
Change-Id: I1f2309bf6806d8430ba0f035d85122e45899baf2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1520880
Reviewed-by: Robert Sesek <[email protected]>
Reviewed-by: Emircan Uysaler <[email protected]>
Commit-Queue: Christian Fremerey <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643969}(cherry picked from commit 2940d5e8eed78248420369ffc17ed4c61ff4a856)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545509
Reviewed-by: Christian Fremerey <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#529}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• services/video_capture/device_factory_provider_impl.cc[diff]
• services/video_capture/device_factory_provider_impl.h[diff]
• services/video_capture/public/cpp/mock_video_source_provider.cc[diff]
• services/video_capture/public/cpp/mock_video_source_provider.h[diff]
• services/video_capture/public/mojom/video_source_provider.mojom[diff]
• services/video_capture/test/device_factory_provider_connectortest.cc[diff]
• services/video_capture/test/mock_device_shared_access_unittest.cc[diff]
• services/video_capture/video_source_provider_impl.cc[diff]
• services/video_capture/video_source_provider_impl.h[diff]

[M74] MediaCapabilities: Set new features *AFTER* finalizing previous record.

This order ensures stats are saved with the correct stream features.

Change-Id: Iacf7cda7960438aae1a8638e3eae17e03b5e4a2d
Bug: 945191
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536897
Commit-Queue: Chrome Cunningham <[email protected]>
Reviewed-by: Frank Liberato <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643746}(cherry picked from commit d601f2fe894602e132f1725a997691376573d639)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545193
Reviewed-by: Chrome Cunningham <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#528}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/learning/common/BUILD.gn[diff]
• media/mojo/services/BUILD.gn[diff]
• media/mojo/services/test_helpers.cc[Added - diff]
• media/mojo/services/test_helpers.h[Added - diff]
• media/mojo/services/video_decode_perf_history_unittest.cc[diff]
• media/mojo/services/video_decode_stats_recorder.cc[diff]
• media/mojo/services/video_decode_stats_recorder_unittest.cc[Added - diff]

Reland "Use dark theme controlled palette for UrlBar text colors."

This is a reland of 4cbdf1a56760b13481156372d39375ebde71fa88

Original change's description:
> Use dark theme controlled palette for UrlBar text colors.
>
> This change:
> - reduces code complexity and dependency on duplicate resources,
> - fixes a glitch on Android P and Q where text colors would not be
>   correctly reflected by the theme,
> - drops redundant color definitions.
>
> Bug: 944496
>
> Change-Id: I3058edc9c6414ead398bb4e7f3d223fd0a8c77d8
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536541
> Reviewed-by: Yusuf Ozuysal <[email protected]>
> Reviewed-by: Becky Zhou <[email protected]>
> Commit-Queue: Ender <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#644378}
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544164
> Reviewed-by: Matthew Jones <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#518}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

Bug: 944496
Change-Id: I68a9bd9a45b43980eadf5f6b02e61265270cb303
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545253
Reviewed-by: Becky Zhou <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#527}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/values/colors.xml[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizer.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBarViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/BasicSuggestionProcessor.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/SuggestionViewViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/LocationBarModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/CustomTabToolbar.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizerTest.java[diff]

cros: Don't SchedulePaint if it is redundant

Reduce calls to SchedulePaint if nothing has changed.
On average we called SchedulePaint on the background rect of the
launcher 178 times per drag on show. Now we call it <10 times.

misc change: Clean up un-used launcher member variable.

Bug: 946536
Change-Id: I924006c2452d30dd077be2c848478182ec759284
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541858
Reviewed-by: Mitsuru Oshima <[email protected]>
Commit-Queue: Alex Newcomer <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644855}(cherry picked from commit 7e3af2dd5e36db2331f2444b47331e78d754586f)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545252
Reviewed-by: Alex Newcomer <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#526}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/app_list/views/app_list_view.cc[diff]
• ash/app_list/views/app_list_view.h[diff]

Revert "Use dark theme controlled palette for UrlBar text colors."

This reverts commit 4cbdf1a56760b13481156372d39375ebde71fa88.

Reason for revert: Change causes build break: there is an additional dependency on color definition this needs that apparently is not available on m74

Original change's description:
> Use dark theme controlled palette for UrlBar text colors.
> 
> This change:
> - reduces code complexity and dependency on duplicate resources,
> - fixes a glitch on Android P and Q where text colors would not be
>   correctly reflected by the theme,
> - drops redundant color definitions.
> 
> Bug: 944496
> 
> Change-Id: I3058edc9c6414ead398bb4e7f3d223fd0a8c77d8
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536541
> Reviewed-by: Yusuf Ozuysal <[email protected]>
> Reviewed-by: Becky Zhou <[email protected]>
> Commit-Queue: Ender <[email protected]>
> Cr-Original-Commit-Position: refs/heads/master@{#644378}
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544164
> Reviewed-by: Matthew Jones <[email protected]>
> Cr-Commit-Position: refs/branch-heads/3729@{#518}
> Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

[email protected],[email protected],[email protected],[email protected]

Change-Id: I62a3418eac4ceab2bdf0eef329fbdd3ae2488995
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 944496
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545173
Reviewed-by: Matthew Jones <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#525}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/values/colors.xml[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizer.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBarViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/BasicSuggestionProcessor.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/SuggestionViewViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/LocationBarModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/CustomTabToolbar.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizerTest.java[diff]

Avoid autocompleting calculations for answers.

This change ensures that Calculation answer shows query as-is, without further
autocompletion (eg. to avoid changing "pint" to "pinterest").

Bug: 945732
Change-Id: Id0a87a9286fc2568c53c9d83910340cd4247d823
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539068
Reviewed-by: Theresa <[email protected]>
Reviewed-by: Matthew Jones <[email protected]>
Commit-Queue: Ender <[email protected]>
Auto-Submit: Ender <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645048}(cherry picked from commit e1475e4e81d2eb72e44df25779d6ad47225e5fd6)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544814
Cr-Commit-Position: refs/branch-heads/3729@{#524}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerSuggestionProcessor.java[diff]

[m74] ui/android: Add plumbing for keeping a list of supported refresh rates.

Maintain a list of refresh rates supported by the framework in
WindowAndroid. In subsequent patches, this list can be sent to the GPU
process to choose the preferred refresh rate based on the content.

Also add a kill switch to force 60Hz refresh rate, if available on the
device.

[email protected], [email protected]
[email protected]

(cherry picked from commit 019a87ec2d1fffab534347f89b23f6faee71bc0a)

Bug: 938106
Change-Id: Ib2c00af96f21dffe04151d94a201267cacb3071d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1525411
Commit-Queue: Khushal <[email protected]>
Auto-Submit: Khushal <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Reviewed-by: Yaron Friedman <[email protected]>
Reviewed-by: Eric Karl <[email protected]>
Reviewed-by: David Trainor <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642804}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545170
Reviewed-by: Khushal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#523}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/features/vr/java/src/org/chromium/chrome/browser/vr/VrShell.java[diff]
• content/browser/renderer_host/compositor_impl_android.cc[diff]
• content/public/common/content_features.cc[diff]
• content/public/common/content_features.h[diff]
• ui/android/java/src/org/chromium/ui/base/WindowAndroid.java[diff]
• ui/android/java/src/org/chromium/ui/display/DisplayAndroid.java[diff]
• ui/android/java/src/org/chromium/ui/display/PhysicalDisplayAndroid.java[diff]
• ui/android/java/src/org/chromium/ui/display/VirtualDisplayAndroid.java[diff]
• ui/android/window_android.cc[diff]
• ui/android/window_android.h[diff]

Downloads : Fixed share for the new download backend

This CL adds :
1 - Provided implementation for GetShareInfoForItem in the download offline content provider.
2 - Added the missing remapGenericMimeType call before sending out the offline item updates.

Bug: 945229
Change-Id: I0b654e158c03d3ad6cb24452b345547f1086c3a3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542022
Reviewed-by: Xing Liu <[email protected]>
Commit-Queue: Shakti Sahu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645043}(cherry picked from commit 7ea9f165b66ad7e1abc44e59f0f8fcad6562c953)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545249
Reviewed-by: Shakti Sahu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#522}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java[diff]
• chrome/browser/android/download/download_utils.cc[diff]
• chrome/browser/android/download/download_utils.h[diff]
• chrome/browser/download/download_offline_content_provider.cc[diff]
• chrome/browser/download/offline_item_utils.cc[diff]

Keep caBLE enabled for CrOS, as originally intended

caBLE was disabled on Linux in crrev.com/c/1457716, but CrOS also returns
true for IS_LINUX and was accidentally disabled as well.

Bug: 945007
Change-Id: If3ec24299701092d43325702c4764746b9827d27
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536545
Reviewed-by: Dmitry Gozman <[email protected]>
Commit-Queue: Kim Paulhamus <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643958}(cherry picked from commit 6af89cea2bcae0690a2191f305ebf5489f4d22c8)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1545151
Reviewed-by: Kim Paulhamus <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#521}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/browser/webauth/authenticator_impl_unittest.cc[diff]
• content/public/common/content_features.cc[diff]

Fix a crash in JPN IME when using TSF on windows.

The root cause of this issue is that there is a wrong assumption in
current TSF implementations for JPN IMEs. IMEs may start new
composition at offset prior to current composition. We should not
commit current composition if the start of new composition is before
current composition. This change also fix an issue in Confirm/cancel
composition scenario. We should notify text change for the whole buffer
to let IME to terminate composition.

[email protected]

(cherry picked from commit 6f46c031929687404917dcbf13bb21e9da08c31b)

Bug: 944004
Change-Id: Iff8759481bfcb2da622483dabecfb6cc4c50da7f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536637
Commit-Queue: Siye Liu <[email protected]>
Reviewed-by: Yohei Yukawa <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644491}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544910
Reviewed-by: Lan Wei <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#520}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/base/ime/win/tsf_text_store.cc[diff]

[CrOS MultiDevice] Fix multi-device setup UI in OOBE.

OOBE support for multi-device setup broke due to a transition from the
"full" JS bindings to the smaller "lite" JS bindings.

This CL updates the first run (i.e., OOBE) flow to use the correct type
provided by the lite bindings. It also adds a Closure compile BUILD
target to ensure that this does not regress.

[email protected]

(cherry picked from commit 8a8caf5ca8881a97dfcd8cc3ebab4616aeed2def)

Bug: 946743
Change-Id: Iaa2f879bb57bf26deebfc8e29baae0a9bd78c2fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542152
Reviewed-by: Alexander Alekseev <[email protected]>
Commit-Queue: Alexander Alekseev <[email protected]>
Auto-Submit: Kyle Horimoto <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#645454}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544812
Reviewed-by: Kyle Horimoto <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#519}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/resources/chromeos/login/BUILD.gn[diff]
• chrome/browser/resources/chromeos/login/multidevice_setup_first_run.html[diff]
• chrome/browser/resources/chromeos/login/multidevice_setup_first_run.js[diff]

Use dark theme controlled palette for UrlBar text colors.

This change:
- reduces code complexity and dependency on duplicate resources,
- fixes a glitch on Android P and Q where text colors would not be
  correctly reflected by the theme,
- drops redundant color definitions.

Bug: 944496

Change-Id: I3058edc9c6414ead398bb4e7f3d223fd0a8c77d8
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536541
Reviewed-by: Yusuf Ozuysal <[email protected]>
Reviewed-by: Becky Zhou <[email protected]>
Commit-Queue: Ender <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644378}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544164
Reviewed-by: Matthew Jones <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#518}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/res/values/colors.xml[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/LocationBarLayout.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizer.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/UrlBarViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/BasicSuggestionProcessor.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/basic/SuggestionViewViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/LocationBarModel.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/top/CustomTabToolbar.java[diff]
• chrome/android/javatests/src/org/chromium/chrome/browser/omnibox/OmniboxUrlEmphasizerTest.java[diff]

[Sync::USS] Fix for frequent saving of bookmark model file

Before this CL, BookmarkModelTypeProcessor schedules a save upon
changes in ModelTypeState even if they dont't involve changes in
Sync entity metadata.
This results in frequent writes of the bookmarks file which
could be potentially large for some users.

This CL makes sure that the file is saved only upon changes in the
entity metadata.

Bug: 945820
Change-Id: Ib0d581b1b585e5cc4bff1721bf8767abc047316d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538526
Commit-Queue: Mohamed Amir Yosef <[email protected]>
Reviewed-by: Mikel Astiz <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644372}(cherry picked from commit a9f4d8f23e5e2171b74a1cef8433a51efebb55b7)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1543779
Reviewed-by: Mohamed Amir Yosef <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#517}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/sync_bookmarks/bookmark_model_type_processor.cc[diff]
• components/sync_bookmarks/bookmark_model_type_processor_unittest.cc[diff]

Adding bottom toolbar metrics

Added:

MobileBottomToolbarHomeButton
MobileBottomToolbarShareButton
MobileBottomToolbarStackViewButton
MobileBottomToolbarStackViewNewTab

MobileMenuShowFromBottom
MobileMenuShowFromTop

MobileToolbarHomeButton

MobileToolbarStackViewButtonInBrowsingView -

Bug: 936529
Change-Id: I7b5611bec57ccf8b6e85f893f654f99af1e0402b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1521818
Commit-Queue: Pedro Amaral <[email protected]>
Reviewed-by: Theresa <[email protected]>
Reviewed-by: Ilya Sherman <[email protected]>
Reviewed-by: Matthew Jones <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644575}(cherry picked from commit bbe4c217459811fea2713f0d26b29b6d806053d1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542668
Reviewed-by: Pedro Amaral <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#516}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/ChromeActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/ChromeTabbedActivity.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/appmenu/AppMenuPropertiesDelegate.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/toolbar/ToolbarManager.java[diff]
• tools/metrics/actions/actions.xml[diff]

[Feed] Rename polluted histogram and fix histogram size

ContentSuggestions.Feed.TokenCompleted.ContentCount.Synthetic was
supposed to be recorded with a max value of 50 (kMaxContentCount),
but due to a mistake, it's recorded with a max size of 10.
This makes the data mostly useless, as usually values of ~35 are
recorded.

This CL change the max value to 50, and make a new histogram name.

Bug: 943838
Change-Id: I7f84bfcaec7741867f80a6d0692cdbceb5f6f786
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542056
Reviewed-by: Ilya Sherman <[email protected]>
Reviewed-by: Sky Malice <[email protected]>
Commit-Queue: Gang Wu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644934}(cherry picked from commit 9f5cd3abf3cc3a4248ad075b2ed8584d55664588)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544160
Reviewed-by: Gang Wu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#515}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/feed/core/feed_logging_metrics.cc[diff]
• tools/metrics/histograms/histograms.xml[diff]

Fix BroadcastChannelProvider::ConnectToChannel() 'crashes'.

Temporary workaround to avoid crash reporting when
BroadcastChannelProvider::ConnectToChannel() is called after the RPHI
associated with the |process_id| has already been destroyed. This is
similar to the behavior allowed before http://crrev.com/c/1409732 and
is intended to be a minimal fix that can be merged to M73.

Bug: 943027
Change-Id: Ia841a072293ea14f681d696e58e769248bc48271
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1531715
Commit-Queue: Marijn Kruisselbrink <[email protected]>
Auto-Submit: Aaron Colwell <[email protected]>
Reviewed-by: Marijn Kruisselbrink <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642545}(cherry picked from commit 641773a07e160e2f9660d6106a5ddbf915176a6d)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544430
Reviewed-by: Aaron Colwell <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#514}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/browser/broadcast_channel/broadcast_channel_provider.cc[diff]

Android download: Fix missing SD card snackbar on Android Q.

This CL fixed an issue that all downloads on Q with content URI are
recognized as downloads on external SD card.


[email protected]

(cherry picked from commit b1170bcfe88b2ab311543c7bcce195376f3dcbed)

Bug: 945615
Change-Id: I3ddfb3305359e05e8a38ed17a1d413f8ebb8a529
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539181
Reviewed-by: Min Qin <[email protected]>
Commit-Queue: Xing Liu <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644451}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544154
Reviewed-by: Xing Liu <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#513}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/download/DownloadManagerService.java[diff]

[autofill] Pin preview font-family to a system font

(Merge to M74)

Bug: 916838
Change-Id: I4e874105262f2e15a11a7a18a7afd204e5827400
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1423109
Reviewed-by: Fabio Tirelo <[email protected]>
Reviewed-by: Koji Ishii <[email protected]>
Commit-Queue: Roger McFarlane <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#640884}(cherry picked from commit 0bd10e13a008389ec14bbe7cc95f17d82ea151c1)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544309
Reviewed-by: Roger McFarlane <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#512}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/renderer/autofill/form_autofill_browsertest.cc[diff]
• third_party/blink/renderer/core/html/resources/html.css[diff]

Don't regenerate browser frame on native theme changes.

BrowserView was calling BrowserFrame::FrameTypeChanged() whenever the
native theme changed. While Chrome theme changes can indeed cause frame
changes, native theme changes should largely be color-based and should
not require re-creation of the frame and re-composition of the entire
window. This was causing flicker, which was especially pronounced on
older machines and computers running Windows 8.x.

See attached bug for details.

[email protected]

(cherry picked from commit b153488cab7173359d93cc0c64af13125d857619)

Bug: 945138
Change-Id: I002a6535de507866b781643ce9afb81002c6946c
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538984
Commit-Queue: Dana Fried <[email protected]>
Reviewed-by: Peter Kasting <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644544}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544153
Reviewed-by: Dana Fried <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#511}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/browser.cc[diff]
• chrome/browser/ui/browser_window.h[diff]
• chrome/browser/ui/views/frame/browser_view.cc[diff]
• chrome/browser/ui/views/frame/browser_view.h[diff]
• chrome/test/base/test_browser_window.h[diff]

macviews: switch to Views media picker on Mac

This change:
1) Adds a Mac implementation of DesktopMediaWindowRegistry, which is
   essentially a stub because Mac windows already have identifying
   numbers;
2) Makes DesktopMediaID::RegisterNativeWindow and ::GetNativeWindowById
   available on Mac, backed by the new DesktopMediaWindowRegistryMac;
3) Builds all the Views DesktopMediaPicker code on Mac, including tests;
4) Removes all the Cocoa DesktopMediaPicker code;
5) Fixes DesktopMediaPickerViews to not try to use MODAL_TYPE_CHILD on
   Mac when there is no parent window - this isn't allowed, so it now
   falls back to MODAL_TYPE_WINDOW;
6) Adds a Mac codepath in DesktopMediaPickerViews to ignore the picker
   window in the media sources list;
7) Disables modal animations on Mac in DesktopMediaPickerViewTest.*,
   since these animations can otherwise make the tests flakily hang
   during teardown;

To test the new UI:
* Install this helper extension: https://chrome.google.com/webstore/detail/screen-capturing/ajhifddimkapgcifgcodmmfdlknahffk
* Navigate to this test page: https://www.webrtc-experiment.com/screen-sharing/#32685442571862986
* Click the big blue button :)

Work still to be done:
* The TableView used for selecting tabs has some odd appearance glitches
* The focus ring on windows in the source list is an old-style Views
  ring
* Accessibility has not been tested thoroughly

Bug: 726005
Change-Id: I0927dd94ede9b9315ffa41ac5ecc83579d456161
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1524521
Reviewed-by: Avi Drissman <[email protected]>
Reviewed-by: Weiyong Yao <[email protected]>
Commit-Queue: Elly Fong-Jones <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#641287}(cherry picked from commit 447f22268ced83b914233365ae1c8ebc7fa8aa73)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1544189
Cr-Commit-Position: refs/branch-heads/3729@{#510}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/BUILD.gn[diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.h[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_bridge.mm[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.h[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_cocoa.mm[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.h[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller.mm[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_controller_unittest.mm[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.h[Deleted - diff]
• chrome/browser/ui/cocoa/media_picker/desktop_media_picker_item.mm[Deleted - diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views.cc[diff]
• chrome/browser/ui/views/desktop_capture/desktop_media_picker_views_unittest.cc[diff]
• chrome/test/BUILD.gn[diff]
• content/browser/BUILD.gn[diff]
• content/browser/media/desktop_media_window_registry_mac.mm[Added - diff]
• content/public/browser/desktop_media_id.cc[diff]
• content/public/browser/desktop_media_id.h[diff]

Announce answers in natural order.

To catch up with recent discussions about AiS and Accessibility
we are flipping order of *announcements* of AiS elements.

This change also changes the order of presentation of calculation
results and restores the "=" prefix of a calculation result
to make calculations also easier to understand for visually
impaired people.

[email protected]

(cherry picked from commit 46813f47456cb4ccd824a364328d7aeca25d2bbd)

Bug: 941539
Change-Id: Ibb53f24019e188f80e6c60886a13a5ecb0e4badf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532245
Commit-Queue: Ender <[email protected]>
Reviewed-by: Yusuf Ozuysal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643431}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1543869
Reviewed-by: Matthew Jones <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#509}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerSuggestionProcessor.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerSuggestionView.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerSuggestionViewBinder.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerSuggestionViewProperties.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerText.java[diff]
• chrome/android/java/src/org/chromium/chrome/browser/omnibox/suggestions/answer/AnswerTextNewLayout.java[diff]

[ios] Check for nil item in side swipe helper.

Both -updateNavigationEdgeSwipeForWebState and GetBackwardItems can return nil
NavigationItems, so check in the UseNativeSwipe helper for completeness.

Bug: 945989
Change-Id: Ie8ad8a425722e699372013d8c31db039e91ca32b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539121
Auto-Submit: Justin Cohen <[email protected]>
Reviewed-by: Eugene But <[email protected]>
Commit-Queue: Justin Cohen <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644582}(cherry picked from commit 63bcb4d9e0f93cd5d3403db391ec5838e03c1135)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542429
Reviewed-by: Justin Cohen <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#508}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ios/chrome/browser/ui/side_swipe/side_swipe_controller.mm[diff]
• ios/chrome/browser/ui/side_swipe/side_swipe_util.mm[diff]

[Passwords] Implement IOSChromePasswordManager::IsMainFrameSecure

This change implements IOSChromePasswordManager::IsMainFrameSecure by
exposing the WebState on the PasswordManagerClientDelegate and then
using it to call WebStateContentIsSecureHtml().

Bug: 940906
Change-Id: Ia5afca02f141cf601c1b09e7f693f4dc2810daa7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538510
Reviewed-by: Vasilii Sukhanov <[email protected]>
Commit-Queue: Jan Wilken Dörrie <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644373}(cherry picked from commit 0da700bff51a22e37dc5dea1cf76da6e9fa3587e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1543631
Reviewed-by: Jan Wilken Dörrie <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#507}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ios/chrome/browser/passwords/ios_chrome_password_manager_client.h[diff]
• ios/chrome/browser/passwords/ios_chrome_password_manager_client.mm[diff]
• ios/chrome/browser/passwords/password_controller.mm[diff]

[Merge M74][AF] Show datalist labels on two-line layout

Bug: 945225
Change-Id: I15646c1f1ecaff70bb269ab20d37b6773c4fba72
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541657
Reviewed-by: Tommy Martino <[email protected]>
Commit-Queue: Fabio Tirelo <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644831}(cherry picked from commit 2445a275cd65fb6707be2ac54484fdb96c6c33a2)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542423
Reviewed-by: Fabio Tirelo <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#506}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/autofill/autofill_popup_controller_impl.cc[diff]
• chrome/browser/ui/autofill/autofill_popup_controller_unittest.cc[diff]
• components/autofill/core/browser/autofill_external_delegate.cc[diff]
• components/autofill/core/browser/autofill_external_delegate_unittest.cc[diff]

Incrementing VERSION to 74.0.3729.45

[email protected]

Change-Id: I73724ef1467e403f09cbd7788d63a3385d9a8f11
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541674
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#505}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

Incrementing VERSION to 74.0.3729.44

[email protected]

Change-Id: I6e576a0b8161af5da8c6b6a55359e6d7832c1d34
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542694
Reviewed-by: [email protected] <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#504}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/VERSION[diff]

media/android: Don't access GL during ImageReaderTextureOwner cleanup.

We were accidentally creating GL fences when tearing down the
ImageReader. Fix that.

R=​[email protected]

Bug: 942793
Change-Id: Id7588974a741d0e8a48509e8cdb83957e0a659ff
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536553
Auto-Submit: Khushal <[email protected]>
Reviewed-by: Frank Liberato <[email protected]>
Commit-Queue: Khushal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643896}(cherry picked from commit af55c35f35821843bee66cbcf39b4769ce683bcd)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541948
Reviewed-by: Khushal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#503}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• media/gpu/android/image_reader_gl_owner.cc[diff]

Reland "Use NetworkChangeNotifier to initialize the connection type in NetworkChangeManagerClient."

This is a reland of d0310b0c0221e3f44da813443ca11c9869620cb4

Original change's description:
> Use NetworkChangeNotifier to initialize the connection type in NetworkChangeManagerClient.
> 
> Bug: 942782
> Change-Id: Ia2971348153d54cd87a48e8045efc9d31948db8a
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529354
> Reviewed-by: Steven Bennetts <[email protected]>
> Commit-Queue: Robbie McElrath <[email protected]>
> Cr-Commit-Position: refs/heads/master@{#641801}

Bug: 942782
Change-Id: Ie2331fe506b7b8839d9ae998bb3fd338747e0b24
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529664
Reviewed-by: Steven Bennetts <[email protected]>
Reviewed-by: Paul Jensen <[email protected]>
Commit-Queue: Robbie McElrath <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642597}(cherry picked from commit 5db69e59c9c38230e5b2223de0de0e42998a5763)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542662
Reviewed-by: Robbie McElrath <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#502}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/chromeos/network_change_manager_client.cc[diff]
• chrome/browser/chromeos/network_change_manager_client.h[diff]
• chrome/browser/chromeos/network_change_manager_client_unittest.cc[diff]
• net/base/network_change_notifier.cc[diff]

Adds source to Assistant URLs.

Assistant Reminders, Settings, etc. needs to be able to distinguish
between being loaded in the browser and being embedded in Assistant UI.
They need this information to fork behavior and UI treatments, such as
deep links that should only be provided if embedded UI or top bars that
should only be shown if in the browser.

To indicate that we are in an embedded state, we will use a URL
parameter: |source|.

Note that we will default |source| to Assistant, and can support
parameterizing this value later if need be.

Bug: b:128863104
Change-Id: Ia76a53a5c68352313a0015154f5793fad0e8c9cb
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1529918
Commit-Queue: David Black <[email protected]>
Reviewed-by: Xiaohui Chen <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643058}(cherry picked from commit 770a3435cc0067fd11ce66fe6ed946b85be6dc1e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542021
Cr-Commit-Position: refs/branch-heads/3729@{#501}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ash/assistant/util/deep_link_util.cc[diff]
• ash/assistant/util/deep_link_util_unittest.cc[diff]

ui/gl: Fix synchronization bug for release fences with SurfaceControl.

The change in [1] made an incorrect assumption that
GLImage::BindTexImage will be issued each time the associated texture is
bound for a write so issuing a service wait could be deferred until then.
The call is actually made only once at initialization and the state is
cached as Texture::BOUND on the image. As a result we don't wait on the
release fence received from the framework when reusing buffers.

[1]: https://chromium-review.googlesource.com/c/chromium/src/+/1526638

R=​[email protected]

Bug: 945443
Change-Id: I870adfb66f2935ff77c41ea5c714e1458aaf6716
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1538562
Auto-Submit: Khushal <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Reviewed-by: Tommy Nyquist <[email protected]>
Reviewed-by: Frank Liberato <[email protected]>
Commit-Queue: Khushal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644036}(cherry picked from commit 9d0e9a2122eda570db0a35ffcd48afeb0182c692)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542474
Reviewed-by: Khushal <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#500}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• base/android/scoped_hardware_buffer_fence_sync.h[diff]
• media/gpu/android/image_reader_gl_owner.cc[diff]
• ui/gl/gl_image_ahardwarebuffer.cc[diff]
• ui/gl/gl_image_ahardwarebuffer.h[diff]
• ui/gl/gl_surface_egl_surface_control.cc[diff]

Set download shelf, infobar and status bubble colors for GTK themes.

Bug: 944941
Change-Id: I550c11f4ff5177be7ec7c4d23be457b1c05e3b2f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1540070
Commit-Queue: Gayane Petrosyan <[email protected]>
Reviewed-by: Peter Kasting <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644688}(cherry picked from commit 9f743848f23211296a4e3a27f712b39b8e137836)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542033
Reviewed-by: Gayane Petrosyan <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#499}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/ui/libgtkui/gtk_ui.cc[diff]

Fix an issue for some JPN IMEs when using TSF support on Windows.

Some JPN IMEs such as CorvusSKK or SKK日本語FEP behave differently when
using TSF on Windows. The root cause is that uses may commit existing
composition and start new composition in the same edit session with
same composition text.

[email protected]

(cherry picked from commit 3decd3ac1bc0a6240dbe81cd203a0c0c4e6f44b7)

Bug: 944452
Change-Id: Ife8032742afd1cf2be372f015fdc2cb77f73a59a
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532911
Commit-Queue: Siye Liu <[email protected]>
Reviewed-by: Yohei Yukawa <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644592}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1541841
Reviewed-by: Lan Wei <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#498}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• ui/base/ime/win/tsf_text_store.cc[diff]
• ui/base/ime/win/tsf_text_store.h[diff]

Enable SurfaceSynchronization when AndroidSurfaceControl is enabled

Currently we enable VizDisplayCompositor but not SurfaceSynchronization
which is not a supported configaration.

Bug: 943314
Change-Id: I7864d25393f9f975135e701bda07325670aec126
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1536357
Commit-Queue: Saman Sami <[email protected]>
Commit-Queue: Khushal <[email protected]>
Reviewed-by: Khushal <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#643512}(cherry picked from commit d83c1a1364250630ca8e67e4823c902be3bf71a6)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542470
Cr-Commit-Position: refs/branch-heads/3729@{#497}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• components/viz/common/features.cc[diff]

[IndexedDB] Stop requesting locks in the Delete request.

Locks were assumed to be granted synchronously when all previous locks
were destructed. This isn't the case when there is a chain of dependent
locks, as granting a lock posts a task.

Since the delete code doesn't need a transaction to work (yet), remove
the locks here, and refactor this area for the new Scopes integration.

R=​[email protected]

Bug: 945370
Change-Id: Ifce3163852db6b2eae2127ee7062e788a19b9546
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1539211
Reviewed-by: Victor Costan <[email protected]>
Commit-Queue: Victor Costan <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644653}(cherry picked from commit 8008668d66a9b9e45dd38a03f69530e734d9208c)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542035
Reviewed-by: Daniel Murphy <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#496}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• content/browser/indexed_db/indexed_db_database.cc[diff]

Make "on-demand" video surface layer the hardcoded default.

Bug: 924249
Change-Id: Id91994c87910d19884222a7060ecca28d0894bb0
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1427599
Commit-Queue: Mounir Lamouri <[email protected]>
Reviewed-by: Dale Curtis <[email protected]>
Reviewed-by: Bo <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#644568}(cherry picked from commit ea892a6e5498846ba111537205952e4728b0087e)
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542231
Reviewed-by: Mounir Lamouri <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#495}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• android_webview/lib/aw_main_delegate.cc[diff]
• content/renderer/media/media_factory.cc[diff]
• media/base/media_switches.cc[diff]
• media/base/media_switches.h[diff]

Merge M74: Check for null web_contents in OfflinePageUtils::ScheduleDownload

[email protected]

(cherry picked from commit f447fa2ebad87cf57baad062e42cd3c3fce88282)

Bug: 942426
Change-Id: I24f43ac82827633b6390e012b9a583b364c9167b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1532673
Reviewed-by: Peter Williamson <[email protected]>
Commit-Queue: Jian Li <[email protected]>
Cr-Original-Commit-Position: refs/heads/master@{#642672}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1542147
Reviewed-by: Jian Li <[email protected]>
Cr-Commit-Position: refs/branch-heads/3729@{#494}
Cr-Branched-From: d4a8972e30b604f090aeda5dfff68386ae656267-refs/heads/master@{#638880}

• chrome/browser/offline_pages/offline_page_utils.cc[diff]