GitLab

Bug: chromium:804275
Change-Id: Id76cce71074ddde14b7e5179d1e536d0658154a1
Reviewed-on: https://chromium-review.googlesource.com/951785
Commit-Queue: Harald Alvestrand <hta@chromium.org>
Reviewed-by: Henrik Boström <hbos@chromium.org>
Reviewed-by: Steven Holte <holte@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541761}

A video tag should not highlight the play button
and slider thumb when preload is disabled and
no source is present

BUG=665149

Change-Id: I2ee71d95f7c1cf289c2e7892b30845c77bc1bd1f
Reviewed-on: https://chromium-review.googlesource.com/905643
Commit-Queue: Mounir Lamouri <mlamouri@chromium.org>
Reviewed-by: Robert Ma <robertma@chromium.org>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541760}

Cf. https://docs.google.com/document/d/1Zati5ZohwjUM0vz5qj6sWg5r-_I0iisUoSoAMNdd7C8/edit#heading=h.xo57d5v7zhrp.

This CL just updates docs. Movement of existing services in //components
will follow.

Change-Id: I9fceb075f66eadfd9e2ceb6ae55d927affbdf8a0
Reviewed-on: https://chromium-review.googlesource.com/948851Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541759}

This is a follow up CL to
https://chromium-review.googlesource.com/c/chromium/src/+/951245

That renames couple of other occurrences of SharedModelTypeProcessor

Change-Id: I55657cc2b0e98c8a0b7dacf2c4be97ee770f51ef
Reviewed-on: https://chromium-review.googlesource.com/953542Reviewed-by: Jan Krcal <jkrcal@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Commit-Queue: Mohamed Amir Yosef <mamir@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541758}

Following bot results are included.
4108 4111 4112 4117 4119 4120 4122 4126 4129

2 lines were removed and 6 lines were deflaked by consecutive
results since 4065.

TBR=eae@chromium.org
NOTRY=true

Bug: 591099
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I899bb826d5a55529cc73e698880e7cbb13461cc1
Reviewed-on: https://chromium-review.googlesource.com/952112
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541757}

When we observe high memory usage in renderer process, |binding_| could
be already bound when StartMonitoringIfNeeded() is called. Check if it
is already bound in StartMonitoringIfNeeded().

Bug: 817873
Change-Id: I4c75b48471a7de839fd2a159edf5f87db89c8c1d
Reviewed-on: https://chromium-review.googlesource.com/954892Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541756}

A script is added to check network traffic annotation tests. Common
parts of this script and the former annotation checker are separated
as annotation_tools.py.

Bug: 690323
Change-Id: If712cca4359a98549318fafe80d64b651765d800
Reviewed-on: https://chromium-review.googlesource.com/951255
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Martin Šrámek <msramek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541755}

https://skia.googlesource.com/skia.git/+log/df04ce29490a..fa303398d503

$ git log df04ce294..fa303398d --date=short --no-merges --format='%ad %ae %s'
2018-03-08 angle-skia-autoroll Roll skia/third_party/externals/angle2/ 2202680c6..ed8b49195 (5 commits)
2018-03-07 csmartdalton ccpr: Simplify triangle corners

Created with:
  roll-dep src/third_party/skia


The AutoRoll server is located here: https://autoroll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=fmalita@chromium.org

Change-Id: Ia6de38042177eb3dce9ec853f19fd02db0ba9c67
Reviewed-on: https://chromium-review.googlesource.com/954990Reviewed-by: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: skia-chromium-autoroll <skia-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#541754}

interface mixin is a new IDL feature, and has a new grammar definition.
This CL updates IDL parser to accept it.
https://heycam.github.io/webidl/#idl-interface-mixins

Bug: 781257
Change-Id: Ibb771007c9147cf868995c80db8afc4ca43b9a74
Reviewed-on: https://chromium-review.googlesource.com/954763
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541753}

Various unused or redundant APIs are removed trivially.

Bug: 681921
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I6e5b85b69a002b1037e1ee4f778f29af31af1525
Reviewed-on: https://chromium-review.googlesource.com/952922Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541752}

Network traffic annotation is added to two requests in:
components/invalidation/impl/p2p_invalidator.cc
components/invalidation/impl/push_client_channel.cc

Bug: 656607
Change-Id: I34b9dbc22b8d4035270f66225d2424c624bb6217
Reviewed-on: https://chromium-review.googlesource.com/926008
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541751}

Since M60, vibrate always requires user gesture.

Remove the feature flag and M59's deprecation message.

BUG=704650

Change-Id: Ifd1bda521e320fb7c0e8b3378117553e18cad833
Reviewed-on: https://chromium-review.googlesource.com/892581Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541750}

BUG=762641
TBR=xdai@google.com

Change-Id: Ibae58c9e8c7ca4f8135de38786c051e42d210d05
Reviewed-on: https://chromium-review.googlesource.com/954844
Commit-Queue: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Reviewed-by: ChromeOS Commit Bot <chromeos-commit-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541749}

When an Android accessibility service sends us a CLICK or FOCUS action
on a specific node in a WebView, we should first focus the WebView itself.
Otherwise clicking on a text field using an accessibility service doesn't
actually move input focus there and pop up the virtual keyboard.

Bug: 819669
Change-Id: I8e4069324e45516fbc1621723885ad8448fdc862
Reviewed-on: https://chromium-review.googlesource.com/953322Reviewed-by: Paul Miller <paulmiller@chromium.org>
Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541748}

Local annotation is added to SpdySession::TryCreatePushStream.

Bug: 656607
Change-Id: I164d9d6df34b9f38e6077c5746a04f2e53a7061d
Reviewed-on: https://chromium-review.googlesource.com/951728
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541747}

This reverts commit 7a87490b.

Reason for revert: Crash. See crbug.com/819916

Original change's description:
> Add a SubresourceLoadInfo struct to content/
> 
> This data type is useful for sending stats from the renderer to the
> browser, in SubresourceResponseStarted methods.  Currently there are six
> fields that are passed around, but soon there may be more in order to
> get PageLoadMetrics working with the Network Service.
> 
> BUG=816684
> 
> Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
> Change-Id: I26a62ebb5aa82f225a70fe54098433c5c3342941
> Reviewed-on: https://chromium-review.googlesource.com/947582
> Commit-Queue: Conley Owens <cco3@chromium.org>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Reviewed-by: Jay Civelli <jcivelli@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#541550}

TBR=dcheng@chromium.org,jcivelli@chromium.org,jam@chromium.org,cco3@chromium.org

Change-Id: I48effa0d5a7407dfbbaf41d5d1708a1491f31639
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 816684,819916
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/954588Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Commit-Queue: Xiaohan Wang <xhwang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541746}

This CL may cause a small binary size increase, roughly proportional
to how long it's been since our last AFDO profile roll. For larger
increases (around or exceeding 100KB), please file a bug against
gbiv@chromium.org. Additional context: https://crbug.com/805539

The AutoRoll server is located here: https://afdo-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


TBR=gbiv@chromium.org

Change-Id: I74ff926d9dd9173fc2e8abbe800700631873629a
Reviewed-on: https://chromium-review.googlesource.com/954984
Commit-Queue: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: afdo-chromium-autoroll <afdo-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#541745}

This has been enabled on Android since Chrome 62 by server-side field
trial configuation, and since Chrome 64 by the client code. As part of
the launch process for issue 733466, we received approval to enable it
on all platforms after Android 62 rolled out widely, which is definitely
true now.

This CL still keeps the feature flag, but it should be removed completely later.

Bug: 733466, 819476
Change-Id: Idcd85b1afb3e7724a1ff6a3b9cd8e1db6ec23788
Reviewed-on: https://chromium-review.googlesource.com/952955Reviewed-by: Justin Donnelly <jdonnelly@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Commit-Queue: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541744}

The ImeTextSpanThickness enum has 3 values:
* kNone: When you don't want to paint the underline
  (this was done with transparent color before).
* kThin: For regular ImeTextSpan.
* kThick: For thick ones.

With the enum the API calls are much clearer than using a boolean.

BUG=554893

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I813716c08d73fc962080952c9729d3167fc12c0e
Reviewed-on: https://chromium-review.googlesource.com/919494Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Reviewed-by: Shu Chen <shuchen@chromium.org>
Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Commit-Queue: Manuel Rego Casasnovas <rego@igalia.com>
Cr-Commit-Position: refs/heads/master@{#541743}

Improve the initial implementation by adding proper pause/resume/seek support.

The AudioDecoderForMixer will notify the AvSync of these actions. The
AvSync will turn on/off its AV monitoring as needed, and will reset its
linear regressions as needed.

BUG=internal 73746352
TEST=On device, pause/play work very well now, without any
noticeable jitter by myself at all. Correction logs indicate that we
will do a hard correction after Seek and usually a soft correction after
resume, but there is no visible jitter to me at all

Change-Id: I21376e41eee15161098578208625974ff0385cf8
Reviewed-on: https://chromium-review.googlesource.com/947720
Commit-Queue: Mina Almasry <almasrymina@chromium.org>
Reviewed-by: Kenneth MacKay <kmackay@chromium.org>
Reviewed-by: Sergey Volk <servolk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541742}

In the original Android and the previous ARC, the bundle notification
was not expanded automatically, because their height may be too long
vertically to expand.

We recentlly re-implementing auto-expander in Chrome-side. This CL
implements that behavior to Chrome auto-expander.

Bug: 810656
Test: manual test
Change-Id: Ie6cc878fff04148402448ddd2aa4f02f6541164b
Reviewed-on: https://chromium-review.googlesource.com/942129
Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
Reviewed-by: Mattias Nissler <mnissler@chromium.org>
Reviewed-by: Eliot Courtney <edcourtney@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541741}

This reverts commit 4eeae7ba and adds
the pyobjc wheel to Chromium's root .vpython. At this point, vpython
is used on all of the bots for the system Python, so it should be
suitably warmed up before getting into any of Chromium's tests.

Bug: 776804, 804174
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I35c602ca6e0368532685ae8afc3139692a6153fa
Reviewed-on: https://chromium-review.googlesource.com/954704Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541740}

Brings Chrome's implementation inline with the latest spec changes.

Bug: 819744
Change-Id: I716bab14bdbcc1c452a4dd447adbe28ffef05f98
Reviewed-on: https://chromium-review.googlesource.com/953259
Commit-Queue: Brandon Jones <bajones@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541739}

pkg_config added -isystem in abspath.

Let me use relative path here for cache utilization on goma backend when building chrome in different directory.

Change-Id: Id48db3036711c07788bb370753acc7a9b217a89d
Reviewed-on: https://chromium-review.googlesource.com/954526Reviewed-by: Thomas Anderson <thomasanderson@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@google.com>
Cr-Commit-Position: refs/heads/master@{#541738}

These were copy-pasted and edited instead of going back to the source
of truth, chromium_gpu_fyi.py.

TBR=nodir@chromium.org
NOTRY=true

Bug: 811799, 811814
Change-Id: Ic3f4e5063058940024adc330f7129f2d6fb24275
Reviewed-on: https://chromium-review.googlesource.com/954717
Commit-Queue: Kenneth Russell <kbr@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541737}

This change fixes a browser crash when the PiP media controls button
is toggled more than once.

BUG: 817984 726621
Change-Id: I53d504b9de44b724a3992c709fa26c0fca9be5bc
Reviewed-on: https://chromium-review.googlesource.com/951909
Commit-Queue: apacible <apacible@chromium.org>
Reviewed-by: CJ DiMeglio <lethalantidote@chromium.org>
Reviewed-by: Elly Fong-Jones <ellyjones@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541736}

This updates //courgette to use base::RepeatingCallback instead of
legacy base::Callback. No intended functional change is included.

This reduces the number of 'base::Bind' in //courgette from 3 to 0 as
tracked at http://goo.gl/LUVhDj

Bug: 714018
Change-Id: I40ec7c5cd7486f6b1110f13e35fd0368a4db9f3c
Reviewed-on: https://chromium-review.googlesource.com/954886Reviewed-by: Samuel Huang <huangs@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541735}

This renames the existing GetMock{,Tick}Clock on TestMockTimeTaskRunner
with DeprecatedGetMock{,Tick}Clock, and adds new GetMock{,Tick}Clock.
The old one returns a new Clock or TickClock object for each call, and
passes its ownership. OTOH, the new one returns a raw pointer to a clock
held by the TestMockTimeTaskRunner instance.

This is needed to update the ownership policy of base::Clock and
base::TickClock from the injectee-owned style to the injector-owned
style.

Bug: 789079
Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel
Change-Id: I8b831eff74e6a8d0f1f67dc1806e3cc24831196f
Reviewed-on: https://chromium-review.googlesource.com/937148
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Charlie Andrews <charliea@chromium.org>
Reviewed-by: Adam Langley <agl@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Xiaohan Wang <xhwang@chromium.org>
Reviewed-by: Yuwei Huang <yuweih@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541734}

Adding the write barrier to collection backings regressed blink_perf.parser by 16%

Bug: 818696
Change-Id: If964e2ccf39beb3010063bf785e97c60ddf007c2
Reviewed-on: https://chromium-review.googlesource.com/953443Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Keishi Hattori <keishi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541733}

Without this code thinks an event has been processed when it may not
have been, which leads to assertions failing. This is only an issue
with the Window Service, which triggers throttling more often.

BUG=817112
TEST=test only change

Change-Id: Ie290dcc827e4be67263f0430c540af271b16ab02
Reviewed-on: https://chromium-review.googlesource.com/954702Reviewed-by: Fady Samuel <fsamuel@chromium.org>
Commit-Queue: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541732}

Change-Id: I6c00beafaf5d1eea6a7452ed187924ae9f4433fa
Reviewed-on: https://chromium-review.googlesource.com/944666
Commit-Queue: Steven Holte <holte@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Robert Kaplow <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541731}

The inline settings should be triggered by long tap.

BUG=818734
TEST=manual

Change-Id: Ieeb7a0c009b15157bd90c823ece84ed39bf676eb
Reviewed-on: https://chromium-review.googlesource.com/950504Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Megumi Hattori <megumihattori@google.com>
Cr-Commit-Position: refs/heads/master@{#541730}

This allows a bunch of code to be deleted as well.

Need to fix up unit tests which was still using synchronous draw, and
thus having wrong expectations.

Change-Id: Ia5504545f213bf709951eb2213dc40aaca4fc6be
Reviewed-on: https://chromium-review.googlesource.com/949085
Commit-Queue: Bo <boliu@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Tobias Sargeant <tobiasjs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541729}

Makes CallbackInterfaceBase and its subclasses use wrapper-tracing
to retain the underlying v8::Object.  Lets existing clients of
CallbackInterfaceBase, which don't support wrapper-tracing, use
V8PersistentCallbackInterface<T> instead.  The behavior and lifetime
shouldn't change.

Bug: 778580
Change-Id: If5db42a3b8cd5622de0576a920128a0b5ab0861f
Reviewed-on: https://chromium-review.googlesource.com/951163Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541728}

This is a simple clean up CL that has no changes in behaviour.
The bubble anchor code can use the recently added ButtonProvider
instead of a specialised getter on FrameView.

Change-Id: Ia067afe36f07c3ce4b77344c28f6467e4a9a1801
Bug: 811178
Reviewed-on: https://chromium-review.googlesource.com/950642Reviewed-by: calamity <calamity@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Commit-Queue: Alan Cutter <alancutter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541727}

https://webrtc.googlesource.com/src.git/+log/98bb968f7bed..8861da8356ee

$ git log 98bb968f7..8861da835 --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc
BUG=None


The AutoRoll server is located here: https://webrtc-chromium-roll.skia.org

Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_chromium_archive_rel_ng;master.tryserver.chromium.mac:mac_chromium_archive_rel_ng;master.tryserver.chromium.win:win-msvc-dbg
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I457bf7ff2fff46c97fe3c04f333458d23b7473eb
Reviewed-on: https://chromium-review.googlesource.com/954356Reviewed-by: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: webrtc-chromium-autoroll <webrtc-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#541726}

kRtcPeerConnectionId is 2384
2383 is already taken by kRTCConstraintEnableDtlsSrtpFalse

tools/metrics/histograms/enums.xml was already correct
but web_feature.mojom needing correcting.

No release branches affected:
M66 Branched Chromium @ revision: 540276
Bug was introduced: Cr-Commit-Position: refs/heads/master@{#540486}

BUG=775415
TBR=eladalon@chromium.org

Change-Id: I4baccfe7ce21cc24ca704fbc1b49e92ae3365a80
Reviewed-on: https://chromium-review.googlesource.com/954524Reviewed-by: Eric Willigers <ericwilligers@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Eric Willigers <ericwilligers@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541725}

This patch makes PasteMode to be an enum class for better type safety
and improving code health. Using forward declaratuion in other header
files instead of including |PasteMode.h|.
Signed-off-by: Zhuoyu Qian <zhuoyu.qian@samsung.com>
Change-Id: I74ea67e08607779cb823cb379c415263d537303f
Reviewed-on: https://chromium-review.googlesource.com/954423Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541724}

Notify click wont work because it fires after the release event. We need
to observe both press and release events to mimic keyboard press and
release.

Test: none
Bug: none
Change-Id: I314aed2dcc870c8579a48c8abed45b0065737819
Reviewed-on: https://chromium-review.googlesource.com/954048Reviewed-by: Michael Wasserman <msw@chromium.org>
Commit-Queue: Sammie Quon <sammiequon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#541723}

We need to distinguish logout and restart in app push install logger.
Currently there is no easy way to get this information. At this moment
only BootTimesRecorder is explicitly notified. This CL makes this
information public and let log collector use it.

Bug: b/73277923
Test: Manual + unit test
Change-Id: I64eea3b748e4ad089365fe3453e9c5fc0e6c3162
Reviewed-on: https://chromium-review.googlesource.com/949826Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Bartosz Fabianowski <bartfab@chromium.org>
Commit-Queue: Yury Khmel <khmel@google.com>
Cr-Commit-Position: refs/heads/master@{#541722}