GitLab

NOTRY=true  # doesn't affect test on CQ

Bug: 825180
Change-Id: I40c8a928e85326aa7e939b7ceb25d1c1c1ead5f1
Reviewed-on: https://chromium-review.googlesource.com/992353
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Reviewed-by: Emily Hanley <eyaich@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547681}

Follow-up to r546408.

Bug: 801583
Change-Id: I76d52b43a103e13054dc9286c9566da40e366661
Reviewed-on: https://chromium-review.googlesource.com/983085
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547680}

https://skia.googlesource.com/skia.git/+log/c0528e2418e0..fd88fe4d57c7

$ git log c0528e241..fd88fe4d5 --date=short --no-merges --format='%ad %ae %s'
2018-04-02 bsalomon Speculative fix for crbug.com/823920

Created with:
  roll-dep src/third_party/skia
BUG=chromium:823920


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;luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel
TBR=reed@chromium.org

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

WebAuthN api should not be exposed in WebView.

BUG=828168
TBR=changwan@chromium.org,perezju@chromium.org
NOTRY=true

Change-Id: Ibe54d6785183bc0aaa557bf3de850e02b58b1f0c
Reviewed-on: https://chromium-review.googlesource.com/992272Reviewed-by: Tim Volodine <timvolodine@chromium.org>
Commit-Queue: Tim Volodine <timvolodine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547678}

Make font shorthand initial/inherit parsing less strict, the
current solution means the special logic in ConsumeFontFamily
is not taken into account. Since all font longhands will reject
css wide keywords and ConsumeFontFamily has special rules for them,
we can just remove the while loop.

Bug: 824611

Change-Id: I9bae2c1497e61bb0d93973bb1bfaac386d0c6d0b
Reviewed-on: https://chromium-review.googlesource.com/982557Reviewed-by: Timothy Loh <timloh@chromium.org>
Commit-Queue: Rob Buis <rob.buis@samsung.com>
Cr-Commit-Position: refs/heads/master@{#547677}

BUG=511300
R=pastarmovj@chromium.org

Change-Id: Ib58a03e9e761d993e05f214d248f53f966a4b577
Reviewed-on: https://chromium-review.googlesource.com/992036Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Greg Thompson <grt@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547676}

Ran `tools/clang/scripts/upload_revision.py 328716`.

BUG=826683

Change-Id: I1a5d85b42cdc4b98919345d442e1ddad328840b4
Reviewed-on: https://chromium-review.googlesource.com/987232Reviewed-by: Hans Wennborg <hans@chromium.org>
Commit-Queue: Nico Weber <thakis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547675}

TBR=schenney@chromium.org

Bug: 824930
Change-Id: I123f2d2c7e95e5bbb0dd90abeab57246d312bc54
Reviewed-on: https://chromium-review.googlesource.com/992037
Commit-Queue: Fredrik Söderquist <fs@opera.com>
Reviewed-by: Fredrik Söderquist <fs@opera.com>
Cr-Commit-Position: refs/heads/master@{#547674}

This creates a new directory "Source/core/execution_context" for
ExecutionContext, SecurityContext and RemoteSecurityContext.

The ownership was not so clear from the previous directory
(core/dom), and this should clarify the ownership for these
classes.

This is a part of moving files that are not strictly relevant
to DOM files to appropriate owner directory (crbug/738794).

Bug: 738794
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;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: I44444edb2f8fe682ac718ea7d5bb6b82382e0727
Reviewed-on: https://chromium-review.googlesource.com/989693
Commit-Queue: Takayoshi Kochi <kochi@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Hayato Ito <hayato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547673}

This test was converted to testharness in CL[1], but
two of cases started to fail only on Windows in CL:458377.
Since Linux searches baselines for linux -> win -> generic,
win expectations affects linux expectaions as well.

This patch fixes non-Windows expectations by adding linux
pass expectations.

[1] https://codereview.chromium.org/2475393002

Change-Id: I3ba07b9ad190b86b73f5ac0c41e9debf1d5e7eda
Reviewed-on: https://chromium-review.googlesource.com/989517Reviewed-by: Emil A Eklund <eae@chromium.org>
Reviewed-by: Dominik Röttsches <drott@chromium.org>
Commit-Queue: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547672}

This enables code caching for <script> tags whose source code is embedded
in the HTML. External scripts store their cached code on metadata
associated with the external script resource; now, inline scripts store
their cached code on metadata associated with the HTML document's
RawResource.

To allow multiple inline scripts per HTML page, the RawResource metadata
can hold multiple code caches. These are keyed by the SHA-256 hash of the
script's source, which is a large enough hash that we don't have to worry
about collisions, and thus don't have to store the script's source in the
cache.

The multi-script metadata has a simple serialization format optimized for
easy serialize/deserialize:

    marker (uint32)
    num_entries (int)
    key 1 (Key type)
    len data 1 (size_t)
    type data 1
    data for key 1
    ...
    key N (Key type)
    len data N (size_t)
    type data N
    data for key N

Inline scripts receive a CachedMetadataHandler implementation which is a
view onto this multi-script handler.

Bug: chromium:823663
Change-Id: Ib14edfaf460aa8bfc90cfe12b8a6b4b7af050de7
Reviewed-on: https://chromium-review.googlesource.com/893401
Commit-Queue: Leszek Swirski <leszeks@chromium.org>
Reviewed-by: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547671}

Blink should set the request_initiator correctly to use OOR-CORS
for security checks. Blink had several Origin variants, but now
the right SecurityOrigin is set to the blink::ResourceRequest's
RequestorOrigin. If something get broken, we need to modify Blink-site
to set a suitable origin.

Bug: 803766
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ic5e184a58009189f128082c4232526e046e743cc
Reviewed-on: https://chromium-review.googlesource.com/964103
Commit-Queue: Takashi Toyoshima <toyoshim@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547670}

The metadata will store additional developer-provided fields that will
help retain & recover the state of the background fetch.
Create/Get/Update registration database tasks are updated to support
the new metadata proto.

Bug: 826257, 757760
Change-Id: I5b97638d3ab90e482911fb50479e5b8c312b4331
Reviewed-on: https://chromium-review.googlesource.com/984356
Commit-Queue: Rayan Kanso <rayankans@chromium.org>
Reviewed-by: Dan Elphick <delphick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547669}

UpdateStartEnd takes start and end position as parameters,
and then does a DCHECK_LE for start and end.
EphemeralRange constructor also does same DCHECK.
Since UpdateStartEnd is called from many places,
so to catch a bug at the call site, function
UpdateStartEnd is made to take EphemeralRange
instead.

Bug: 826106
Change-Id: I6e1cd23e9aac2846a2f083cd6f4d8110fa3a9085
Reviewed-on: https://chromium-review.googlesource.com/989852Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Commit-Queue: Tanvir Rizvi <tanvir.rizvi@samsung.com>
Cr-Commit-Position: refs/heads/master@{#547668}

Bug: 813068
Change-Id: If8b7489c69b85237d422c6b3d030fdd79ca7017a
Reviewed-on: https://chromium-review.googlesource.com/983955Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547667}

Bug: 822276
Change-Id: I8ed04d4681e2f0e7bb41b869d6b855547b023965
Reviewed-on: https://chromium-review.googlesource.com/986294Reviewed-by: Dan Elphick <delphick@chromium.org>
Commit-Queue: Mugdha Lakhani <nator@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547666}

Add support for the following properties:
- image-rendering
- text-anchor
- text-combine-upright
- text-overflow
- text-rendering
- transform-style
- user-select

Lots of failing tests, usually because a keyword isn't implemented in
Blink yet.

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

This requirement is derived from clause 5.1.3.19.1 in the spec:

  "If options.authenticatorSelection.authenticatorAttachment is present
   and its value is not equal to authenticator’s attachment modality,
   continue",

and the fact that there are no U2F platform devices.

This CL also fixes the error message when the all U2F devices are
filtered out by the resident key and user verification criteria. While
the CL (crrev.com/c/949453) introducing these options correctly
documented the intended behavior in a comment in u2f_sign, it did not
actually change the DOMException itself.

Finally, the CL moves related webauthn tests from
credentialmanager_browsertests to webauth_browsertests.

Bug: 803832
Change-Id: Ie8f124af3783c2aaec85abd3baac050cfbb926b7
Reviewed-on: https://chromium-review.googlesource.com/989413
Commit-Queue: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547664}

https://webrtc.googlesource.com/src.git/+log/5b4f075f9c59..644fde40a9c3

$ git log 5b4f075f9..644fde40a --date=short --no-merges --format='%ad %ae %s'

Created with:
  roll-dep src/third_party/webrtc
BUG=chromium:827917,chromium:828220


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: I51cd91f0fca62013f2f17b6e113f783c90b45f0a
Reviewed-on: https://chromium-review.googlesource.com/991606Reviewed-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@{#547663}

On Mac, audio devices may have an active source.
For example, the built-in audio device might be using the internal
speakers or headphones (same device, different sources).
This CL changes audio device enumerations so that the device label
uses the source instead of the device name if the source is available.

The reason is that the source (together with the transport, added in
crrev.com/c/980951) is a better description of the device.

Some examples:

Before: "Built-in Microphone"
Now: "External Microphone (Built-in)" or "Internal Microphone (Built-in)"

Before: "Built-in Output"
Now: "Internal Speakers (Built-in)" or "Headphones (Built-in)"

Before: "HDMI"
Now: "HP Z30i (HDMI)"

This change also requires firing the devicechange event if the source for
a device changes, since that results in a change to the enumeration results.
This CL includes the required changes to AudioDeviceListenerMac.

Bug: 811687
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;master.tryserver.chromium.win:win_optional_gpu_tests_rel
Change-Id: Ieb6801fe979cfed528036862471203f5d29e4603
Reviewed-on: https://chromium-review.googlesource.com/984312
Commit-Queue: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547662}

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I03347d01754a2fb7e28ffd73696f7dd03fdbba8b
Reviewed-on: https://chromium-review.googlesource.com/987952Reviewed-by: Menglu Huang <huangml@chromium.org>
Commit-Queue: Eric Noyau <noyau@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547661}

Per http:crrev.com/336357 the variable existed when the old file
manager browser test was split up after growing too large. It is
not used in GalleryBrowserTestBase today: remove scripts_.

Test: browser_tests --gtest_filter=GalleryBrowserTest*
Bug: 804413
Change-Id: Iada83b00c5644d8ea8d9d3a334affe103a59d1db
Reviewed-on: https://chromium-review.googlesource.com/991772
Commit-Queue: Noel Gordon <noel@chromium.org>
Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547660}

This switches tools/cygprofile/run_tests to vpython, which should have
been done as part of http://crrev.com/546845.

After the vpython switch the workaround in http://crrev.com/546845
should not be necessary, so revert this commit as well:
> cygprofile: Fix unittests for cygprofile on CQ.
>
> orderfile_generator_backend_unittests imports
> orderfile_generator_backend, which transitively requires psutil, which
> is not available on the CQ bots.
>
> As psutil is not used in unit tests, handle the import error to
> allow unit tests to run on CQ, and CLs in //tools/cygprofile to pass
> presubmit checks.
>
> Change-Id: Id06bd2b7936509ef44a02d7b6497fa818d439291
> Reviewed-on: https://chromium-review.googlesource.com/985975
> Commit-Queue: Benoit L <lizeb@chromium.org>
> Reviewed-by: Egor Pasko <pasko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#546790}

Bug: None
Change-Id: Ib0737acf652b283c212a61a2bae83a9c373c9fbf
Reviewed-on: https://chromium-review.googlesource.com/988072Reviewed-by: Benoit L <lizeb@chromium.org>
Commit-Queue: Egor Pasko <pasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547659}

The test were replicated from the one in zip_archiver/unpacker_test and
rewritten to be based on JsTest.

Test: browser_tests --gtest_filter=ZipArchiverJsTest.*
Bug: 454910
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I141f310cdab10ca10e6084caac545b8be8c26ec8
Reviewed-on: https://chromium-review.googlesource.com/978224
Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Reviewed-by: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Reviewed-by: Shinya Kawanaka <shinyak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547658}

Add a new test, it makes use of chrome.app.window.create() with a
sandboxed page. It checks no window is returned to the background app.

There were no tests trying to open a new window with a sandboxed
document already.

Bug: 820995
Change-Id: I0e59723ee23b31845545774b3ff3dfede0278af0
Reviewed-on: https://chromium-review.googlesource.com/982627
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547657}

Make it a public method on SpaceSplitString so that it can be used in
unit tests etc.

Also add some unit tests.

Change-Id: I2169259c29bb6e5a5a62c8dc6245ba8af31bc053
Reviewed-on: https://chromium-review.googlesource.com/991693Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Fergal Daly <fergal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547656}

Bug: None
Change-Id: I72665789d93e1eb0b5fe8ac8920cac8a5e0b2615
Reviewed-on: https://chromium-review.googlesource.com/984873Reviewed-by: Theresa <twellington@chromium.org>
Reviewed-by: Friedrich Horschig <fhorschig@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547655}

Fix GCState transition checks for when incrmental marking is enabled

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

In jumbo builds many files are compiled in the same
translation unit and then their anonymous namespaces will also
be one and the same. In blink/inspector two (identical)
UrlWithoutFragment functions end up colliding. This patch
changes it so that one of them has a more narrow scope.

The patch also removes the special casing of these files
from the build system.

Bug: 713137
Change-Id: If0b8bfb8a13c1191aa4e1c07d2aa1472ed168f29
Reviewed-on: https://chromium-review.googlesource.com/985978Reviewed-by: Pavel Feldman <pfeldman@chromium.org>
Commit-Queue: Daniel Bratell <bratell@opera.com>
Cr-Commit-Position: refs/heads/master@{#547653}

[Sheriff] Mark svg/W3C-SVG-1.1/text-intro-01-t.svg and svg/W3C-SVG-1.1/text-intro-04-t.svg as failure on Win10

The following 2 tests are failing (Image/text failures) on Win10:

 svg/W3C-SVG-1.1/text-intro-01-t.svg
 svg/W3C-SVG-1.1/text-intro-04-t.svg

TBR=jam@chromium.org

Bug: 828314
Change-Id: I7768c906400f48fbaed0b8e69e4232278e04fbf7
Reviewed-on: https://chromium-review.googlesource.com/991894Reviewed-by: Takashi Sakamoto <tasak@google.com>
Commit-Queue: Takashi Sakamoto <tasak@google.com>
Cr-Commit-Position: refs/heads/master@{#547652}

This patch Introduces |TextOffsetMapping|[1] class to mapping between character
offset in block and DOM position for ease of implementing selection modification
with text segments, such as word/sentence/paragraph.

This patch is a preparation of the patch[2].

[1] https://goo.gl/v2Ax8d Text Offset Mapping
[2] http://crrev.com/c/737981 Simplify word granularity handling

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

FormParser provides parsing PasswordForm structures from HTML data. It
currently is only used on iOS, and hence resides inside //ios. However,
in order to fuzz it, it has to be compiled on Linux and/or Mac.
Therefore the file itself is being moved to the component.

A side effect of this change is:
 * Dropping iOS PlatformTest fixture, because the test does not need it.
 (The fixture was responsible for draining the autorelease pool, but the
 test is pure C++, no Objective C).
 * Renaming to IOSFormParser, to ensure that iOS is still mentioned in
 the file path. It will also help to keep two different form parsers
 once the FormParser for desktop is added into the new target directory.

Bug: 827945
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I114e01af40d5d29f221e15e44f0fdf635ea1c8b4
Reviewed-on: https://chromium-review.googlesource.com/989952Reviewed-by: Vadym Doroshenko <dvadym@chromium.org>
Commit-Queue: Vaclav Brozek <vabr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547650}

Using a set of exlude pattern it is possible to exclude all out/ files
except for files in out/Debug/gen. This removes the need for a symlink.

Change-Id: Ie2b410fa4d4fe12eecbd9c37e5d677b813819b63
Reviewed-on: https://chromium-review.googlesource.com/985977
Commit-Queue: Christian Dullweber <dullweber@chromium.org>
Reviewed-by: Lutz Justen <ljusten@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547649}

String16MojomTraits*.* in platform/mojom/ should be renamed to
string16_mojom_traits*.* for consistency
with //base/strings/string16*.*.

Bug: 578345
Change-Id: Ic84f8e356281a446baaadaf498b31788202d1f97
Reviewed-on: https://chromium-review.googlesource.com/991795Reviewed-by: Hitoshi Yoshida <peria@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547648}

This patch adds a layout test that confirm no warning if
imported document adds style to the main document by appendChild.

Bug: 523952
Change-Id: Idd5a603d3e585378fc57958969b491d866abf997
Reviewed-on: https://chromium-review.googlesource.com/970046Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Yoichi Osato <yoichio@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547647}

This patch changes "transpose-empty.html" to utilize |selection_test()| for ease
of maintenance.

Note: "transpose-empty.html" is failed on layout_ng bot due by pixel test.

Bug: 679977, 636993
Change-Id: Ibeb10cf4f7b6d0f270a30467060b9fba2891999d
Reviewed-on: https://chromium-review.googlesource.com/991496Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547646}

This patch changes "5272440.html" to utilize |selection_test()| for ease of
maintenance.

Following patch will rename "5272440.html" to
"delete_all_contents_in_root_editable.html".

Note: "5272440.html" is failed on layout_ng bot due by pixel test.

Bug: 679977, 636993
Change-Id: I1455f820e8e0baf729fc8a1cca87705db4046674
Reviewed-on: https://chromium-review.googlesource.com/991493Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547645}

This patch changes "type-delete-after-quote{1,-2}.html" to utilize
|selection_test()| and integrate into "type-delete-after-quote.html", since both
tests use similar test input, for ease of maintenance.

Note: "type-delete-after-quote.html" is failed on layout_ng bot due by pixel
test.

Bug: 679977, 636993
Change-Id: I27db14fae0ab951df387e4d58c45f556470209c6
Reviewed-on: https://chromium-review.googlesource.com/991796Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547644}

This is failing because we don't support byte-for-byte update check
to all service worker importScript() resources. Update test expectations
to point an appropriate bug entry and remove the entry from S13nSW
expectation as this failure isn't specific to S13nSW.

Bug: 648295
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: I65dca2a31baa500ae8f1d347af8c77d9e9fd8c29
Reviewed-on: https://chromium-review.googlesource.com/991472Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Commit-Queue: Kenichi Ishibashi <bashi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547643}

Getting rid of another mutable style access.

Bug: 813068
Change-Id: Ied084dcce6c9477ce43763ba42dd4256e87efa25
Reviewed-on: https://chromium-review.googlesource.com/983953Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Rune Lillesveen <futhark@chromium.org>
Cr-Commit-Position: refs/heads/master@{#547642}