GitLab

Change-Id: If2ff3976bae6216853ba9a70a189ba0932f16c6b
Reviewed-on: https://chromium-review.googlesource.com/c/1257245
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596992}

This creates a blinkpy/preusbmit directory, so that we run also run
audit_non_blink_usage_test.py

Change-Id: I7ce7832a90bc4a02e3f886fad216a4eb82188064
Reviewed-on: https://chromium-review.googlesource.com/c/1261103
Commit-Queue: Fergal Daly <fergal@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596991}

https://chromium.googlesource.com/catapult.git/+log/3f7d74f7cd02..a7a24422d086


git log 3f7d74f7cd02..a7a24422d086 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 chiniforooshan@chromium.org Telemetry: fix thread_other_cpu_time_per_frame_tbmv2


Created with:
  gclient setdep -r src/third_party/catapult@a7a24422d086

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

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=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel

BUG=chromium:891546
TBR=sullivan@chromium.org

Change-Id: I4b92e40272ff65e082998a2ca6379ae661e43d1a
Reviewed-on: https://chromium-review.googlesource.com/c/1263858Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596990}

Now we have the QuickView <webview> infra to detect the 'loaded' state
of thew <webview> content, add a <file-safe-media> image test and also
check the <webview> renders its content on a transparent black body.

Bug: 891150
Change-Id: I8e6aac85ce09e0fcbe0cbba660630d1427e2962b
Reviewed-on: https://chromium-review.googlesource.com/c/1263678Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596989}

This queues a report through the Reporting API whenever
requestPictureInPicture is called in a frame in which the
'picture-in-picture' feature is not allowed according to feature policy.

Bug: 867471
Change-Id: I1cede10385f6a5110fd0ddd5bf124ff5d3737dd9
Reviewed-on: https://chromium-review.googlesource.com/c/1262432
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Reviewed-by: François Beaufort <beaufort.francois@gmail.com>
Reviewed-by: Mounir Lamouri <mlamouri@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596988}

BUG=chromium:854481

Change-Id: I3f6fe2429d9bc762b3f6552811c3d58326b18bac
Reviewed-on: https://chromium-review.googlesource.com/c/1263641Reviewed-by: Sam McNally <sammc@chromium.org>
Commit-Queue: Sergei Datsenko <dats@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596987}

SwipeRefreshHandler is used for the active WebContents to
handle swipe-to-refresh gesture. It's now managed as
TabWebContentsUserData - created on the first
initialization of WebContents, and destroyed at the end of
the life of the Tab it belongs to.

Accompanied changes:

 - TabGestureStateListener should also have the same lifetime
   i.e. created on the first WebContents of a Tab becomes active,
   then have its associated WebContents init/destroyed as active
   one changes over time, and finally destroyed together with Tab.
   Previously it was created repeatedly for new WebContents. This
   CL changes it.
 - All TabWebContentsUserData (so far) need to call
   |cleanupWebContents| upon |destroy|. This CL factors the common
   pattern out to their base class.

Bug: 877878
Change-Id: I6f6264866338e763be7737716b93dab86ea1414b
Reviewed-on: https://chromium-review.googlesource.com/c/1253525
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596986}

KeyboardUI::GetKeyboardWindow lazily creates a keyboard window and
loads web contents. Because the KeyboardController lifecycle is quite
complicated, it's not clear from reading the code whether a call to
GetKeyboardWindow creates a window or not.

In fact, only one particular call to GetKeyboardWindow creates the
keyboard window. All the other calls are guaranteed to occur after
that call. Thus, we keep that one call to GetKeyboardWindow to create
the keyboard window and replace the other calls with KeyboardController
::GetKeyboardWindow, which, by contrast, has no side effects.

Bug: 845780
Change-Id: I56e86932e82ced8b25a6f9eb034c60034e3a3548
Reviewed-on: https://chromium-review.googlesource.com/c/1235353
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596985}

The feature is already existing, and previously we just
by default disabled it. This CL ensures we can config this
feature from Chrome://flags

Bug: 852904
Change-Id: I9c5e37070eed995e03b5bfddcb7fd05c744a9091
Reviewed-on: https://chromium-review.googlesource.com/c/1262929
Commit-Queue: Siyu An <siyua@chromium.org>
Reviewed-by: Jared Saul <jsaul@google.com>
Cr-Commit-Position: refs/heads/master@{#596984}

Using wpt-import in Chromium 71098fe5.
With Chromium commits locally applied on WPT:
92605c16 "[Unified Plan] Remote MediaStreamTracks should be muted by default."
65d72632 "Make DTMF tone change more deterministic"


Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/25850

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

Directory owners for changes in this CL:
rego@igalia.com:
  external/wpt/css/selectors

NOAUTOREVERT=true
TBR=markdittmer

No-Export: true
Change-Id: Ia0cff643cfe95b7fa0aea7460abb60051238234f
Reviewed-on: https://chromium-review.googlesource.com/c/1263591
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596983}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/8e9443f5b7f8..d1d4dc8eb878


git log 8e9443f5b7f8..d1d4dc8eb878 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 recipe-roller@chromium.org Roll recipe dependencies (trivial).


Created with:
  gclient setdep -r src/third_party/depot_tools@d1d4dc8eb878

The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll

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=agable@chromium.org

Change-Id: I6ad82b1795c42dc41426f292ea03023f36a52770
Reviewed-on: https://chromium-review.googlesource.com/c/1263855Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596982}

This reverts commit 8aa00e1b.

Reason for revert: Causes crash on shutdown:

Received signal 11 SEGV_MAPERR 000000000088
#0 0x7f228f7b5a3f base::debug::StackTrace::StackTrace()
#1 0x7f228f7b5521 base::debug::(anonymous namespace)::StackDumpSignalHandler()
#2 0x7f22832c80c0 <unknown>
#3 0x7f228a01add1 std::__1::__tree<>::find<>()
#4 0x7f228a0294dc dbus::ObjectManager::UnregisterInterface()
#5 0x7f228a1978b3 bluez::BluetoothAdapterClientImpl::~BluetoothAdapterClientImpl()
#6 0x7f228a1a05b0 bluez::BluetoothDBusClientBundle::~BluetoothDBusClientBundle()
#7 0x7f228a1c808e bluez::BluezDBusManager::Shutdown()
#8 0x559074b4094e ChromeBrowserMainPartsLinux::PostDestroyThreads()


Original change's description:
> Reland "bluetooth: Add a new BluetoothAdapterClient instance and use it for GetState"
> 
> This is a reland of 837b3c20
> 
> The original patch attempted to init the alternate dbus client on Linux where
> there is no alternate bus connection. The latest patchset fixes this by
> returning early if there is no alternate bus connection.
> 
> TBRing since it's a minor addition to the original patch.
> 
> Original change's description:
> > bluetooth: Add a new BluetoothAdapterClient instance and use it for GetState
> >
> > There are two parts to this change:
> >
> > 1. Create a new BluetoothAdapterClient instance. For this we add
> >    "alternate_bluetooth_adapter_client()" to the bundle of DBus clients.
> >    This new instance uses a separate DBus Connection through
> >    BluezDBusThreadManager. This ensures actions on one client won't
> >    affect the other client.
> >
> > 2. Implement BluetoothSystem::GetState using the new client.
> >
> > Bug: 870192, 882771
> > Change-Id: I9faa92e8234b14dd374a04b4c9e9acbcfd7e6201
> > Reviewed-on: https://chromium-review.googlesource.com/c/1215427
> > Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> > Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> > Reviewed-by: Dan Erat <derat@chromium.org>
> > Reviewed-by: Sam McNally <sammc@chromium.org>
> > Reviewed-by: Reilly Grant <reillyg@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#596520}
> 
> TBR=stevenjb@chromium.org, derat@chromium.org, sammc@chromium.org, reillyg@chromium.org
> 
> Bug: 870192, 882771
> Change-Id: I0e92cc90f095c6049b2b967e46249e8488f71fcb
> Reviewed-on: https://chromium-review.googlesource.com/c/1263217
> Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596975}

TBR=derat@chromium.org,stevenjb@chromium.org,reillyg@chromium.org,sammc@chromium.org,ortuno@chromium.org,sonnysasaka@chromium.org

Change-Id: I92bd726f551e48e0b2013d9e58f76e10b90196ec
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 870192, 882771
Reviewed-on: https://chromium-review.googlesource.com/c/1263639Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596981}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/40a16ffeefe6..15082f69653d


git log 40a16ffeefe6..15082f69653d --date=short --no-merges --format='%ad %ae %s'
2018-10-05 zhuohao@chromium.org kalista: add new paladin


Created with:
  gclient setdep -r src/third_party/chromite@15082f69653d

The AutoRoll server is located here: https://autoroll.skia.org/r/chromite-chromium-autoroll

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=chrome-os-gardeners@chromium.org

Change-Id: I7eb20e75a5454e6e191fab94a07c0bdf1718c395
Reviewed-on: https://chromium-review.googlesource.com/c/1263815Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596980}

https://webrtc.googlesource.com/src.git/+log/aa43b7bb2fcd..84583f618399


git log aa43b7bb2fcd..84583f618399 --date=short --no-merges --format='%ad %ae %s'
2018-10-04 benwright@webrtc.org Enable End-to-End Encrypted Audio Payloads.


Created with:
  gclient setdep -r src/third_party/webrtc@84583f618399

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

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=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: I7b51964a2fc00e848f4b89a25ffe6d9ae11e1cf5
Reviewed-on: https://chromium-review.googlesource.com/c/1262993Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596979}

Before this patch, we cleared the delegate for web_content
unconditionally. That means we would remove an already existing delegate
from web_content.

This patch introduces a bool in the controller to track whether the delegate
should be cleared in the destructor.

After this patch, we will no longer stomp on other components setting
a delegate in web_contents.

R=gogerald@chromium.org

Bug: 806868
Change-Id: I92ce1ef2dcf4a48f6709bddee0f76871a58c2df4
Reviewed-on: https://chromium-review.googlesource.com/c/1263118Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Commit-Queue: Ganggui Tang <gogerald@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596978}

Bug: 888668
Change-Id: I7afdab5a71a6650b218e5fc16c930103d3069938
Reviewed-on: https://chromium-review.googlesource.com/c/1243114Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596977}

# This is an automated release commit.
# Do not revert without consulting chrome-pmo@google.com.
NOAUTOREVERT=true
TBR=mmoss@chromium.org

Change-Id: I1b19917ea7a375a81a79868edbf36b62c7cec786
Reviewed-on: https://chromium-review.googlesource.com/c/1263915Reviewed-by: chrome-release-bot@chromium.org <chrome-release-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596976}

This is a reland of 837b3c20

The original patch attempted to init the alternate dbus client on Linux where
there is no alternate bus connection. The latest patchset fixes this by
returning early if there is no alternate bus connection.

TBRing since it's a minor addition to the original patch.

Original change's description:
> bluetooth: Add a new BluetoothAdapterClient instance and use it for GetState
>
> There are two parts to this change:
>
> 1. Create a new BluetoothAdapterClient instance. For this we add
>    "alternate_bluetooth_adapter_client()" to the bundle of DBus clients.
>    This new instance uses a separate DBus Connection through
>    BluezDBusThreadManager. This ensures actions on one client won't
>    affect the other client.
>
> 2. Implement BluetoothSystem::GetState using the new client.
>
> Bug: 870192, 882771
> Change-Id: I9faa92e8234b14dd374a04b4c9e9acbcfd7e6201
> Reviewed-on: https://chromium-review.googlesource.com/c/1215427
> Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
> Reviewed-by: Dan Erat <derat@chromium.org>
> Reviewed-by: Sam McNally <sammc@chromium.org>
> Reviewed-by: Reilly Grant <reillyg@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#596520}

TBR=stevenjb@chromium.org, derat@chromium.org, sammc@chromium.org, reillyg@chromium.org

Bug: 870192, 882771
Change-Id: I0e92cc90f095c6049b2b967e46249e8488f71fcb
Reviewed-on: https://chromium-review.googlesource.com/c/1263217Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596975}

While deserializing V8 contexts from snapshot, the process can
fail, unfortunately.
This CL makes the routine to catch such unexpected cases and
disables the feature to avoid crashes.


Bug: 881417
Change-Id: I874a16b63fd6b8ae7a3532bc5c662531a895e3ed
Reviewed-on: https://chromium-review.googlesource.com/c/1258786
Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596974}

Adds a STEP trace with the decoder name being initialized as well
as a STEP trace when DecryptingDemuxerStream is initialized.

BUG=883620
TEST=none

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I958b4f98836e8e4d43ec59931d3cd952c52a09e1
Reviewed-on: https://chromium-review.googlesource.com/c/1263576
Commit-Queue: Dale Curtis <dalecurtis@chromium.org>
Commit-Queue: Dan Sanders <sandersd@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596973}

Hide |filenameEdit_| line edit box when there is no image to be displayed.
This fixes an undesired grey line appearing as the bottom of this edit box.
Changing the "disabled" state doesn't change the style of this box,
"disabled" state is only used for read-only behaviour not style.

R=fukino@chromium.org

Bug: 796503
Change-Id: I6fc9a4eb218231781325ba7a5af992b12610b929
Reviewed-on: https://chromium-review.googlesource.com/c/1212263Reviewed-by: Noel Gordon <noel@chromium.org>
Reviewed-by: Luciano Pacheco <lucmult@chromium.org>
Commit-Queue: Alexander Nohe <nohe@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596972}

BUG=888904

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: I17c89d0ffd9b1dece4bd8779572ff5605e3072fc
Reviewed-on: https://chromium-review.googlesource.com/c/1258784Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Eric Karl <ericrk@chromium.org>
Commit-Queue: Takashi Sakamoto <tasak@google.com>
Cr-Commit-Position: refs/heads/master@{#596971}

According to Intel, the NV12 overlay format should not be used if
DXGI_COLOR_SPACE_YCBCR_STUDIO_G22_LEFT_P709 is not supported. At least one Intel
Gen9 SKU will not support NV12 overlay. This color space info is recorded through
UMA.


BUG=889633

Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: I0406b833e8cfbe0af0a9a408f8e665e3d6e87d77
Reviewed-on: https://chromium-review.googlesource.com/c/1247183Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Sunny Sachanandani <sunnyps@chromium.org>
Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Commit-Queue: Maggie Chen <magchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596970}

https://skia.googlesource.com/skia.git/+log/d945698c5dfa..1b4ad765627f


git log d945698c5dfa..1b4ad765627f --date=short --no-merges --format='%ad %ae %s'
2018-10-05 csmartdalton@google.com ccpr: Unblacklist PowerVR Rogue


Created with:
  gclient setdep -r src/third_party/skia@1b4ad765627f

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

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=luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux-chromeos-compile-dbg;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
TBR=mtklein@chromium.org

Change-Id: Ib0c8aa8e1e9b19ccf2a98e5b71a0e8cdf112980b
Reviewed-on: https://chromium-review.googlesource.com/c/1262992Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596969}

If an extension using the webRequest API has access to a.com, and the
user visits b.com, and b.com requests a script from a.com, the request
will be withheld from the extension. However, we shouldn't badge the
extension as wanting to run in this scenario, since invoking the
extension won't grant access to a.com (because it's not the tab's
origin) or b.com (because the extension didn't request access). Thus,
the extension still won't be able to intercept the request. Because
there's no actionable behavior, don't badge.

Add a test for the same.

Bug: 891586
Change-Id: I673bac4cd936f0c659d3c32467c4976daaafe6a7
Reviewed-on: https://chromium-review.googlesource.com/c/1260211
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Karan Bhatia <karandeepb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596968}

https://chrome-internal.googlesource.com/chrome/src-internal.git/+log/7b52a7683757..56f1cb670eca


Created with:
  gclient setdep -r src-internal@56f1cb670eca

The AutoRoll server is located here: https://autoroll-internal.skia.org/r/src-internal-chromium-autoroll

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=mmoss@chromium.org

Change-Id: I75298dc30da5661706edf7a5e7986132e4fe1b8b
Reviewed-on: https://chromium-review.googlesource.com/c/1262991Reviewed-by: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Commit-Queue: chromium-internal-autoroll <chromium-internal-autoroll@skia-corp.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596967}

TtsHandler doesn't remove itself as a VoicesChangedDelegate on
destruction, causing use after frees.

Bug: 885311
Change-Id: Id6ed9cd5e9a1caa16e7f095eeeeb6baae1baa327
Reviewed-on: https://chromium-review.googlesource.com/c/1260604
Commit-Queue: Sam McNally <sammc@chromium.org>
Reviewed-by: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596966}

This queues a report through the Reporting API whenever
requestMediaKeySystemAccess is called in a frame in which the 'encrypted-media'
feature is not allowed according to feature policy.

Bug: 867471
Change-Id: I781cf9d9719bcbaffe0b328c2b6bfa5ca197920a
Reviewed-on: https://chromium-review.googlesource.com/c/1261921Reviewed-by: David Dorwin <ddorwin@chromium.org>
Commit-Queue: Ian Clelland <iclelland@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596965}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/5b1fa949bba4..8e9443f5b7f8


git log 5b1fa949bba4..8e9443f5b7f8 --date=short --no-merges --format='%ad %ae %s'
2018-10-05 recipe-roller@chromium.org Roll recipe dependencies (trivial).


Created with:
  gclient setdep -r src/third_party/depot_tools@8e9443f5b7f8

The AutoRoll server is located here: https://autoroll.skia.org/r/depot-tools-chromium-autoroll

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=agable@chromium.org

Change-Id: I79e3204fc145a5370c84a4f7daf0cbc38821be58
Reviewed-on: https://chromium-review.googlesource.com/c/1262990Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596964}

Change-Id: Ie5b7ffcc5c3e57a0c19895885f8fcdfb843ff59e
Reviewed-on: https://chromium-review.googlesource.com/c/1256061Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Reviewed-by: Tommy Li <tommycli@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596963}

This is a followup to https://crrev.com/593254 and stops changing
property trees via the cc::Layer API when using layer lists. The only
remaining change is through Layer::UpdateScrollOffset.

Bug: 879650
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Id2ff7211b7717f0954fe37d6c9aedb7f9a6f7d14
Reviewed-on: https://chromium-review.googlesource.com/c/1262455Reviewed-by: Chris Harrelson <chrishtr@chromium.org>
Commit-Queue: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596962}

Rename the remaining uses of print_style to histograms_print_style left
from https://crrev.com/596921.

Bug: 714123
Change-Id: Ia5caab102a50bbee2c28573808ad592d48c6cc1b
Tbr: asvitkine@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/c/1263636
Commit-Queue: Alexei Svitkine <asvitkine@chromium.org>
Reviewed-by: Alexei Svitkine <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596961}

This CL fixes the condition of IME feature pod button to be shown.
It should hide the button when a separate IME tray is shown in Shelf.

TEST=manual
BUG=882824

Change-Id: I4907f3cb01facbcbe7e14cd3329c413bb1adce24
Reviewed-on: https://chromium-review.googlesource.com/c/1255753Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596960}

Bug: None
Change-Id: Idf29937d34e7709957b483a6f25754eb9bf68cdb
Reviewed-on: https://chromium-review.googlesource.com/c/1263558Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Commit-Queue: Istiaque Ahmed <lazyboy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596959}

It appears in these crashes that AppCacheURLLoaderJob::Start was never called. Specifically, we know that it's not called with a null client (since binding_ is empty and has no connection error handler).

Bug: 882538
Change-Id: I68f1f32cab8d8c2150177702aaa33dccc876f0a0
Reviewed-on: https://chromium-review.googlesource.com/c/1261560Reviewed-by: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596958}

To provide a way to switch IME while ARC IMEs show up, we want to show
the IME menu button while the virtual keyboard shows up even if the
option is not enabled.

Bug: 845079, 882373
Test: manual testing on DUT.
Change-Id: Icd5119f3b880e1bf5243d422b9e075c73396d8d0
Reviewed-on: https://chromium-review.googlesource.com/c/1257471
Commit-Queue: Yuichiro Hanada <yhanada@chromium.org>
Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596957}

https://webrtc.googlesource.com/src.git/+log/264079a57aec..aa43b7bb2fcd


git log 264079a57aec..aa43b7bb2fcd --date=short --no-merges --format='%ad %ae %s'
2018-10-04 chromium-webrtc-autoroll@webrtc-ci.iam.gserviceaccount.com Roll chromium_revision c5c13a1e..0af97cea (596716:596847)


Created with:
  gclient setdep -r src/third_party/webrtc@aa43b7bb2fcd

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

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=luci.chromium.try:linux_chromium_archive_rel_ng;luci.chromium.try:mac_chromium_archive_rel_ng

BUG=chromium:None
TBR=webrtc-chromium-sheriffs-robots@google.com

Change-Id: Iac69c67b1742bf683c73600c70fb8ca50db9a171
Reviewed-on: https://chromium-review.googlesource.com/c/1262984Reviewed-by: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Commit-Queue: chromium-autoroll <chromium-autoroll@skia-public.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#596956}

Some unit-tests refers files in LayoutTests/. If
third_party/blink/web_tests exists, we use it instead of
third_party/WebKit/LayoutTests.

Bug: 843412
Change-Id: If40d4f8318ccc4914fec07021f1f102b0b68e41b
Reviewed-on: https://chromium-review.googlesource.com/c/1260742Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596955}

To prevent potentially regressing legacy buffering-by-DTS MSE apps that
may deliberately provide incorrect video keyframeness metadata in their MP4,
this change preserves trust in that metadata when MseBufferByPts is not
enabled. If MseBufferByPts is enabled, this change preserves the recently
added behavior where video frame analysis results' keyframeness (if
implemented and determined) is used instead of the MP4's metadata.

BUG=892365
R=sandersd@chromium.org
TEST=MP4StreamParserTest.*AVC*Mismatches*, also manually confirmed
SPS/PPS injection/lack of injection matches the expected sequence of
Parsed{Non}Keyframe in those tests.

Change-Id: If756e0ee882ec17c0d13d7b400aa43626cedaca9
Reviewed-on: https://chromium-review.googlesource.com/c/1263426
Commit-Queue: Matthew Wolenetz <wolenetz@chromium.org>
Reviewed-by: Dan Sanders <sandersd@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596954}

Add a small 100x100px jpeg test image. No change in behavior so no new
tests, just adding a resource.

Bug: 891150
Change-Id: I3ce460b3e8778b55fdd04fd9fd63891f32b7259f
Reviewed-on: https://chromium-review.googlesource.com/c/1263515
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: Stuart Langley <slangley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#596953}