GitLab

Allows to throw a stackoverflow exception in ScriptForbiddenScope.

Bug: 813493
Change-Id: I75ebd44533b559369032fc750140bb22cb0c272a
Reviewed-on: https://chromium-review.googlesource.com/980013Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545765}

Introducing the GCPhase enum. In the future, this will be used to simplify the GCState into just handling the next scheduled GC. Right now this isn't connected to anything yet.

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

Bug: 775415
Change-Id: I334dda306d5844784fd13f9e046cbe7b6722e92b
Reviewed-on: https://chromium-review.googlesource.com/973166Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Commit-Queue: Elad Alon <eladalon@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545763}

Makes HeadlessRenderTest use CompositorController to render pages and
adds a way to capture and compare screenshots against golden files.

Also adds a custom font rendering test.

Because of the dependency on BeginFrameControl, these tests can now
only run on linux.

Bug: 787197
Change-Id: If0feaffb6d47cde624851c37db33e598b313668f
TBR: thakis@chromium.org
Reviewed-on: https://chromium-review.googlesource.com/978217
Commit-Queue: Eric Seckler <eseckler@chromium.org>
Reviewed-by: Alex Clarke <alexclarke@chromium.org>
Reviewed-by: Sami Kyöstilä <skyostil@chromium.org>
Reviewed-by: Brian Salomon <bsalomon@chromium.org>
Reviewed-by: enne <enne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545762}

Speculative mac jumbo fix after this CL:
https://chromium-review.googlesource.com/c/chromium/src/+/978860/

TBR=palmer@chromium.org

Bug: 766882
Change-Id: I673edfd250987cd654e45c13b749f800438e2313
Reviewed-on: https://chromium-review.googlesource.com/979806
Commit-Queue: Mostyn Bramley-Moore <mostynb@vewd.com>
Reviewed-by: Daniel Bratell <bratell@opera.com>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545761}

Bug: n/a
Change-Id: Iee9d21bdeaa207d445f16f0bb72ceb2d657e354a
Reviewed-on: https://chromium-review.googlesource.com/979741Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545760}

Rename HonorEditingBoundaryAtOr{Before,After}() to Adjust{Back,Forward}PositionToAvoidCrossingEditingBoundaries()

This patch renames |HonorEditingBoundaryAtOr{Before,After}()| to
|Adjust{Back,Forward}PositionToAvoidCrossingEditingBoundaries()| to follow
|SelectionAfjuster::AdjustSelecitonToAvoidCrossingEditingBoundaries()| for
improving readability.


Bug: 824687
Change-Id: I156c09e884ea8ff805e920cdef928e473f7cb420
Reviewed-on: https://chromium-review.googlesource.com/977343Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545759}

This patch drops "Web" from WebFrameScheduler and other classes having
the same name for consistency with other Web classes.

This patch touches a lot of files, but the changes in those files are
basically simple renames. No behavior change is expected.

LocalFrame::FrameScheduler() was renamed to GetFrameScheduler() because
of the name conflict.

Bug: 825094
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I1c90ac0020c57382bbcb636ba58b9a0d6f1c849b
Reviewed-on: https://chromium-review.googlesource.com/977246
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Alexander Timin <altimin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545758}

This CL applies the Touchable Chrome color spec for the location bar and
omnibox drop down.

The selected keyword icon was not updated to the Touchable Chrome
version in a previous CL; that icon is also updated here along with its
color.

BUG=801583

Change-Id: Ia8a6ab56ef02852d407a0dd1501f2d10422f9455
Reviewed-on: https://chromium-review.googlesource.com/972702
Commit-Queue: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Trent Apted <tapted@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545757}

The script is called run-swarmed.py, not run_swarmed.py.

TBR=thakis

Bug: 495204
Change-Id: I43953019fe4884fac9c7491e87845ef7476091fa
Reviewed-on: https://chromium-review.googlesource.com/980053Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Hans Wennborg <hans@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545756}

content/common/cache_storage/cache_storage_messages.h has been converted
to Mojo thus should be removed on crrev.com/c/612287

Bug: 612287
Change-Id: I88db5d0798891d218c5f1bfdd312a758a45152ff
Reviewed-on: https://chromium-review.googlesource.com/907769
Commit-Queue: Luciano Pacheco (SYD) <lucmult@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Chris Mumford <cmumford@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545755}

This reverts commit 35cc2b25 and
removes "completed_event_->Signal()" from SyncLoadContext::OnReceivedRedirect().
https://chromium-review.googlesource.com/c/chromium/src/+/657787/23/content/renderer/loader/sync_load_context.cc#81

"completed_event_->Signal()" was called twice when SyncLoadContext receives
a cross origin redirect response and caused crashes (crbug.com/797374).

Bug: 706331, 797374
Change-Id: I0562752b4be305a83ac4a0997c50687189d7f6cb

Original change's description:
> Revert "Use DocumentThreadableLoader for sync loading from worker thread."
>
> This reverts commit 86859669.
>
> Reason for revert: threading related crashes in sync XHR code
>
> Original change's description:
> > Use DocumentThreadableLoader for sync loading from worker thread.
> >
> > This change makes the sync loading on worker thread off the main thread.
> >
> > I will create another CL to:
> >  - Rename DocumentThreadableLoader.
> >  - Remove WorkerThreadableLoader.
> >
> >
> > Bug: 706331
> > Change-Id: Ia625c667eb1367d44976478dd0272317b52e60ea
> > Reviewed-on: https://chromium-review.googlesource.com/657787
> > Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
> > Reviewed-by: Kentaro Hara <haraken@chromium.org>
> > Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#525696}
>
> TBR=horo@chromium.org,kinuko@chromium.org,haraken@chromium.org
>
> # Not skipping CQ checks because original CL landed > 1 day ago.
>
> Bug: 706331, 797374
> Change-Id: Ibd2be711ed693084c45d45fda7c51a2d3170361a
> Reviewed-on: https://chromium-review.googlesource.com/843145
> Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
> Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#526111}

Change-Id: I0562752b4be305a83ac4a0997c50687189d7f6cb
Reviewed-on: https://chromium-review.googlesource.com/974989
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545754}

It shouldn't be possible to have an infinite number of these bugs, so
I'll just keep fixing them and hope the Clusterfuzz reports come to an
end :).

Bug: 825679
Change-Id: I899c87b7849cdf7cefc50976b247bce4307b89d2
Reviewed-on: https://chromium-review.googlesource.com/979801Reviewed-by: Olga Sharonova <olka@chromium.org>
Commit-Queue: Max Morin <maxmorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545753}

Bug: 813477
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Idaaf4fbc6d0217b5e0e817f8953095ae5d45e707
Reviewed-on: https://chromium-review.googlesource.com/979744Reviewed-by: Naoki Fukino <fukino@chromium.org>
Commit-Queue: Noel Gordon <noel@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545752}

This is required to support building Microsoft's WinUWP store
application version of WebRTC.

vs_toolchain.py
- needed to perform environment variable expansion for "Program
Files(x86)" to correctly identify Visual Studio installation location

config/BUILD.gn
- remove delayimp.lib, kernel32.lib and ole32.lib from store
applications (instead requires dloadhelper.lib/WindowsApp.lib
must be used)

BUILDCONFIG.gn
- Do not use clang when compiling Windows UWP targets;
- Added declare_args for is_target_winuwp rather and stripped multi
defined variations of the host_os/current_os == "winrt_10", "winrt_81",
"winrt_81_phone" that heavily polluted the platform / target selections
(as the current targeting methodology is incorrect anyway). The
host_os/current_os is always be "win" and only the target should be
Windows UWP / store applications based on the target_os == "winuwp"
rather than all the flavors of UWP.
- Added filter for _winuwp source files (separate from just windows)
- Added default configs for desktop vs store applications to correctly set
the defines according to the desktop vs store targets

config/win/BUILD.gn
- The Windows UWP versioning assumes to be Windows 10 / store
app now although a updated GN allows for targeting older Windows UWP
versions/SDKs/device families. This allows the definitions for the
various application support versioning and application families required
for UWP to be set.
- The linker calls vsvarsall.bat to be executed via
toolchain/win/setup_toolchain.py in order to correctly identify the
correct linker library path information for Windows store SDK targets.
The hard coded and assumed library paths are fixed in all cases to be
discovered from the tooling for forward future platform support in all
cases.
- Added ARM linkage definitions for the Windows ARM CPU required for
properly targeting all three CPUs (x86, x64, arm) for universal store
binaries.
- Added the proper family C++/C defines required to target the various
Windows store application types currently offered for Windows UWP store
applications.

toolchain/win/BUILD.gn
- The name to support the storage of the environment variables now is
passed into the setup script to allow for easier extension of the
CPUs and target combinations (arm, x64, x86 in the desktop vs store
variations)
- "desktop" vs "store" is now specified the setup for the
correct toolchain targeting
- Sets true/false for is_target_winuwp is dependent on the toolchain
activated (so configurations will be set correctly when the toolchain is
specified for host tool targets required for build tools vs finalized
application targets)
- Cleaned up the Windows RT section to properly support Windows UWP
toolchains

toolchain/win/setup_toolchain.py
- the setup was missing the arm CPU for universal binaries required for
the UWP platform
- The calling of vcvarsall.bat was missing the "store" option for store
applications and all the CPU offered
- Added returning of linker paths by searching the library environments
for well-known library files expected in each of the 3 library paths
required "lib", "um" and "atlmfc"

R=phoglund@google.com

Bug: 812814
Change-Id: If1a6b1b1bc3ed940fc8e2ce726ac016e2491e61d
Reviewed-on: https://chromium-review.googlesource.com/923161
Commit-Queue: Patrik Höglund <phoglund@chromium.org>
Reviewed-by: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545751}

This CL adds support for autoGainControl and noiseSuppression properties
to MediaStreamTrack.getCapabilities() and InputDeviceInfo.getCapabilities().

Bug: 823831, 817769
Change-Id: Ifdc0e560001e000b12c8d04634d3c52de0a34cc7
Reviewed-on: https://chromium-review.googlesource.com/977251Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Chandan Padhi <c.padhi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#545750}

Looks like that scoped FeatureList object doesn't affect
ShellBrowserMainParts::SetupFieldTrials() execution and can be removed.

Change-Id: If6c7b3433492ffdde1b1a44762bfed28f5e95824
Reviewed-on: https://chromium-review.googlesource.com/978216Reviewed-by: Jianpeng Chao <chaopeng@chromium.org>
Reviewed-by: Mike West <mkwst@chromium.org>
Commit-Queue: Andrey Kraynov <iceman@yandex-team.ru>
Cr-Commit-Position: refs/heads/master@{#545749}

This CL allows dynamic import() on DedicatedWorkerGlobalScope and adds WPT
tests. Note that ES Modules on DedicatedWorker is an experimental feature behind
the runtime flag.

Design doc:
https://docs.google.com/document/d/1IMGWAK7Wq37mLehwkbysNRBBnhQBo3z2MbYyMkViEnY/edit#heading=h.637avx8i5qtn

Bug: 680046
Change-Id: I1f8fed5c319aab634f96bcfabeb3c95f5dc7d9a7
Reviewed-on: https://chromium-review.googlesource.com/970127
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545748}

I was mostly looking for components/signin/-related things, but
discovered a bunch more stuff along the way, mostly in
ProfileSyncComponentsFactoryImpl.

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

This CL introduces "InsertCommands.h", move the declarations of some
static functions related to insert commands to the header file. Make
EditorCommand simpler for improving code health.

Bug: 818552
Signed-off-by: Zhuoyu Qian <zhuoyu.qian@samsung.com>
Change-Id: I4fc744dc58da885071760ba691baa03e8015d63c
Reviewed-on: https://chromium-review.googlesource.com/977365Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545746}

https://chromium.googlesource.com/chromiumos/platform/system_api.git/+log/72bfdd343136..a0f6142f5733

$ git log 72bfdd343..a0f6142f5 --date=short --no-merges --format='%ad %ae %s'
2018-03-20 dats Add constants for DriveFS D-Bus service.
2018-03-15 emaxx cryptohome: Constants for challenge-response methods
2018-03-15 ejcaruso cryptohome: remove MigrateToDircryptoEx constant

Created with:
  roll-dep src/third_party/cros_system_api
R=sammc@chromium.org

Bug: 823603
Change-Id: I718b17c6ddb00bdeca9a8fb11a097faaecb67500
Reviewed-on: https://chromium-review.googlesource.com/979740Reviewed-by: Sam McNally <sammc@chromium.org>
Commit-Queue: Sergei Datsenko <dats@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545745}

This converts the type of callback parameters on AsyncFileUtil methods
from const-ref to value, and adds std::move() around them for smaller
number of copies.

TBR=baileyberro@chromium.org

Change-Id: I07ef5fa991bcfb2e0d07ef739b4ecad4c89c39ff
Reviewed-on: https://chromium-review.googlesource.com/936345
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Reviewed-by: Hidehiko Abe <hidehiko@chromium.org>
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545744}

From an offline conversation.

Rationale: reflect the fact that this interface is:
- somewhat specific to navigations
- basically to intercept requests/responses

We plan to define content::URLLoaderRequestInterceptor in
//content/public separately, which only intercepts requests
(and not necessarily specific to navigations)

Note: This does NOT rename subclasses (e.g. FooRequestHandler).

R=jam@chromium.org,jianli@chromium.org

Change-Id: Ie6321d4e9ec8be626a505f242f3aace5d1c45eea
Reviewed-on: https://chromium-review.googlesource.com/977642
Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545743}

Rename the last holdout that is using the old name.

Bug: 821215
Change-Id: I45cafaab27f7ef42eaef748088910d8d28f5260c
Reviewed-on: https://chromium-review.googlesource.com/979450Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545742}

This reverts commit d03adf5a.

Reason for revert:

This has caused some tests to fail flakily and reflect a crash that could happen in the wild. See http://crbug.com/825448

Original change's description:
> RDH:  Ensure there's at most one LoadInfoList in flight.
> 
> ResourceDispatcherHost pushes pending load state of all frames to the UI
> thread once every 250 milliseconds when there's a pending load.  This
> happens even when the UI thread is blocked.  Apparently it's causing an
> OOM when the UI thread is blocked.  Whatever is blocking the UI thread
> is another issue entirely, but we shouldn't use unbounded memory in the
> browser process.  This CL prevents load state updates from being sent
> until the previous set was ACKed by the UI thread.
> 
> Bug: 824869
> TBR: jam@chromium.org
> Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
> Change-Id: I3b68522c5281d477d3a621ac02e5df1f75153d99
> Reviewed-on: https://chromium-review.googlesource.com/978383
> Commit-Queue: Matt Menke <mmenke@chromium.org>
> Reviewed-by: John Abd-El-Malek <jam@chromium.org>
> Reviewed-by: Charlie Harrison <csharrison@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#545616}

TBR=jam@chromium.org,mmenke@chromium.org,csharrison@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 824869
Change-Id: I350dd7b357c32ffc7fb85a2417afaf779552ff77
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Reviewed-on: https://chromium-review.googlesource.com/979674Reviewed-by: Ben Wells <benwells@chromium.org>
Commit-Queue: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545741}

Added ClickBreadcrumbs user action for click in breadcrumb.

Fixed handling of MockFileSystem root to always include trailing slash.
The root dir is referenced in the breadcrumb and causes errors
in tests  with inconsistency of trailing slash.



Bug: 824607
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I5ff72f4d2542c25f87fd495bdb088d0d3315424f
Reviewed-on: https://chromium-review.googlesource.com/977381
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Sasha Morrissey <sashab@chromium.org>
Reviewed-by: Ilya Sherman <isherman@chromium.org>
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545740}

The test fails flakily on Linux, Windows and ChromeOS.
See bug for details.

TBR=foolip@chromium.org, dmazzoni@chromium.org

Bug: 825735
Change-Id: I5953f23c9c87e7b4ab2b0a6b9fd9d604a1bf27c3
Reviewed-on: https://chromium-review.googlesource.com/979799Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545739}

When notification inline settings is opened from the top right gear icon
using by keyboard, it crashed as animation requires the event to be
LocatedEvent but it's not.

TEST=manual
BUG=823646

Change-Id: Ia683de86e042f010ac4bb0ee645fff170547b9fa
Reviewed-on: https://chromium-review.googlesource.com/974922
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545738}

Was trying to fix something in this code for an unrelated bug and came
across a few things that could be improved:
* There were too many stateful booleans making the code unnecessarily
complex.
* The type of event that initialized the drag should be in the
DragDescriptor, because it describes the drag and also has the same
life cycle.
* Flatten complicated if statements into chains of if/else's that only
use simple conditions.
* Use a switch statement for the event type.

Change-Id: I74712df2502960c6a98be0a705a5ced94a61ec75
Reviewed-on: https://chromium-review.googlesource.com/979378Reviewed-by: Yuichiro Hanada <yhanada@chromium.org>
Commit-Queue: Blake O'Hare <blakeo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545737}

Fix MockFileSystem to delete entries rather than nulling them.
As null, the key is still iterable which causes errors.

Make root path consistent to always have trailing slash which
matches prod behaviour.

Set default size/date metadata for MockDirectoryEntry.

Put all test helper methods in 'test' namespace.

Replace waitUntil with repeatUntil/pending from
integration_tests/test_util.js which provides better error reporting.


Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ie648388a7d6fe6c79951d5b4330234c378ffe2ee
Reviewed-on: https://chromium-review.googlesource.com/965722
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Reviewed-by: Naoki Fukino <fukino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545736}

This patch introduces tests for hanling of "inline-block" and "float" of
|NGInlineNode::CollectInlines()| to avoid regression for improving code health.

This patch is a preparation the patch[1].

[1] http://crrev.com/c/737981 Simplify word granularity handling

Bug: 778507
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: Ifead65d6df83a19dbe2b82b509ef39427b56962c
Reviewed-on: https://chromium-review.googlesource.com/979751
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545735}

See bug for details.

TBR=hubbe@chromium.org,dalecurtis@chromium.org
notry=true

Bug: 754986
Change-Id: I33b05a30137c2c793d1704242dbb28d15c5aefe3
Reviewed-on: https://chromium-review.googlesource.com/979800Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545734}

Bug: 824566
Test: Manual
Change-Id: I58609375c31f9fc86e16ef2e214747ced8f24218
Reviewed-on: https://chromium-review.googlesource.com/974572
Commit-Queue: Yue Li <updowndota@chromium.org>
Reviewed-by: Xiaohui Chen <xiaohuic@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545733}

Accessibility fix for "Open With" dialog that wasn't being read by
Chromevox.

Change DefaultTaskDialog to set tabindex=-1 for dialog items, so they
can be navigated via JS focus() method, but not via TAB key.

Change cr.ui.List to issue an focus to the item when navigated via
keyboard arrow up/down.

Dialog items already have the correct text to be read, see bug for
screenshot.

Bug: 435010
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I2abc1d2d3d6cb6e78a373ccc51c79e34a0b756bd
Reviewed-on: https://chromium-review.googlesource.com/968004Reviewed-by: Naoki Fukino <fukino@chromium.org>
Reviewed-by: Sasha Morrissey <sashab@chromium.org>
Commit-Queue: Luciano Pacheco (SYD) <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545732}

This CL is a follow-up on https://crrev.com/c/956383.

Bug: 817769
Change-Id: I83a87fc0ee5889e9efa2c6e7a86ae8f2c6c70b90
Reviewed-on: https://chromium-review.googlesource.com/978225Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Commit-Queue: Chandan Padhi <c.padhi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#545731}

The media/color-profile-video-seek-filter.html fails flakily on Win7. See bug
for details.

notry=true
TBR=chcunningham@chromium.org

Bug: 825733
Change-Id: I42edc7907e4de80e2201689796de4ff5d10ff487
Reviewed-on: https://chromium-review.googlesource.com/979795Reviewed-by: Dominic Battré <battre@chromium.org>
Commit-Queue: Dominic Battré <battre@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545730}

When converting CSSCustomPropertyDeclaration to CSSUnparsedValues, we
used to assume that CSSCustomPropertyDeclaration will always have a
CSSVariableData, but it turns out that css-wide keywords are stored
as a CSSValueID instead of a CSSVariableData.

This patch handles css-wide keywords on CSSUnparsedValues correctly.

We added a unit test and not a WPT since this is likely a blink-only
implementation detail.

Bug: 824740
Change-Id: Iefbdca4abc736562b81f5c6a3f0332975b842961
Reviewed-on: https://chromium-review.googlesource.com/977141
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545729}

Previously network::WebSocket objects were stored as raw pointers
in a set and deleted manually. Use a set<unique_ptr<WebSocket>,
UniquePtrComparator> instead, making ownership explicit and reducing the
risk of leaking.

Change-Id: I0337551d8fd073341241723538cf9864eb206c87
Reviewed-on: https://chromium-review.googlesource.com/978282Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Adam Rice <ricea@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545728}

TBR=dmazzoni@chromium.org

Bug: 825429
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I677963581b6594c014f05a717330f4b5f827a372
Reviewed-on: https://chromium-review.googlesource.com/979745
Commit-Queue: Darren Shen <shend@chromium.org>
Reviewed-by: Darren Shen <shend@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545727}

Bug: 788251
Change-Id: I328cb3c1c812fab565eafc4009e0210653e236c7
Reviewed-on: https://chromium-review.googlesource.com/958768Reviewed-by: Trent Apted <tapted@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Commit-Queue: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Cr-Commit-Position: refs/heads/master@{#545726}