GitLab

BUG=636993
TBR=eae@chromium.org, atotic@chromium.org, ikilpatrick@chromium.org

Review-Url: https://codereview.chromium.org/2910393002
Cr-Commit-Position: refs/heads/master@{#475851}

This patch expands |TextIterator::Initialize()| into constructor where the
only call sites to simplify code for improving code health.

This patch is a follow-up of patch[1], which makes call sites of
|Initialize()| to just one.

[1] http://crrev.com/2912053002: Call TextIterator constructor with proper range

BUG=n/a
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2919453002
Cr-Commit-Position: refs/heads/master@{#475850}

The "updatecheck" tag is unnecessary when we want to fetch default
settings. Additionally its presence causes the servers to infer that
Chrome needs to be updated.

BUG=727829
TBR=vasilii@chromium.org

Review-Url: https://codereview.chromium.org/2908073006
Cr-Commit-Position: refs/heads/master@{#475849}

Bug: 712963
Change-Id: I629e4bc40e23b7fad8efe2b0278254b171e2853f
Reviewed-on: https://chromium-review.googlesource.com/518762Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Stuart Langley <slangley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475848}

NV8 characters are invalid in IDN 2008. Four of them are currently
allowed. ( https://goo.gl/LSgs1R )

  [:Idn_2008β=NV8:] & [:Identifier_Statusβ=Allowed:]

Mozilla began to block them earlier this year.

Besides, block 2 quote-like characters that are easy to miss when
put next to other characters.

Bug: chromium:678812
Test: components_unittests --gtest_filter=*IDNToUni*
Change-Id: I994e123335df5446a7a3f41bbb48326319aa1c5f
Reviewed-on: https://chromium-review.googlesource.com/517505
Commit-Queue: Jungshik Shin <jshin@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475847}

This CL also moves the Launch.Modes histogram and its enum into the
Chromium repo.

BUG=717091
R=pkasting@chromium.org

Review-Url: https://codereview.chromium.org/2902963002
Cr-Commit-Position: refs/heads/master@{#475846}

This can be replaced with WebURLRequest::FetchRequestMode which is more
readable and well corresponds to the spec terms.

- Merged the signal indicated by kForcePrelight into fetch_request_mode
- Removed lines setting preflight_policy to the default value
  kConsiderPreflight
- Removed preflight_policy from NotificationImageLoader as it's
  meaningless when the mode is set to NoCORS

Bug: 727596
Change-Id: I5da2b549e4e497f371b21f7acee53920e29b3acc
Reviewed-on: https://chromium-review.googlesource.com/515182Reviewed-by: Mike West <mkwst@chromium.org>
Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Brad Nelson <bradnelson@chromium.org>
Reviewed-by: Reilly Grant <reillyg@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475845}

BUG=n/a
TEST=n/a; no behavioral change

Review-Url: https://codereview.chromium.org/2913953002
Cr-Commit-Position: refs/heads/master@{#475844}

BUG=724932

Review-Url: https://codereview.chromium.org/2895123002
Cr-Commit-Position: refs/heads/master@{#475843}

Mac OS UI fonts renders U+0620 as blank. (radar/32458012)

Bug: chromium:725660
Test: components_unittests --gtest_filter=*IDNToUni*
Change-Id: Ib7d678aae0379e8ac53064663660a7840e5fd3bd
Reviewed-on: https://chromium-review.googlesource.com/517669
Commit-Queue: Jungshik Shin <jshin@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475842}

Android API allows these methods to return null, but converters from
jni_string.h expect jstrings to be non-null with a DCHECK. See
https://developer.android.com/reference/android/hardware/usb/UsbDevice.html
> Returns ... or null if the property could not be read

R=reillyg@chromium.org
TEST=Don't crash in debug on null jstring DCHECK on KeyedService creation, or
any other scenarios that call UsbDeviceAndroid::Create. Was reproducible only
on certain devices, like Motorola Nexus 6.

Review-Url: https://codereview.chromium.org/2914453002
Cr-Commit-Position: refs/heads/master@{#475841}

Currently there's a lot of duplicated code in all the MarkerType-specific
DocumentMarkerController::Add*Marker() methods. This CL factors out the common
logic into an AddMarkerInternal() method, and renames the existing (private)
AddMarker() method to AddMarkerToNode().

This CL also adds some filtering to avoid constructing DocumentMarkers with
start/end offsets that are the same (the current version of the code constructs
these DocumentMarkers, then filters them out before actually adding them), or
when the range returned by TextIterator is not a text node (we don't currently
do this filtering).

BUG=707867

Review-Url: https://codereview.chromium.org/2916463002
Cr-Commit-Position: refs/heads/master@{#475840}

The test RenderThreadImplBrowserTest.InputHandlerManagerDestroyedAfterCompositorThread
is flaky on Windows, with first obvservation being
https://build.chromium.org/p/chromium.win/builders/Win%207%20Tests%20x64%20%281%29/builds/24771

BUG=728034
TBR=skyostil@chromium.org

Change-Id: Icf3a073407eb8eaa13c111987544d887bcac68ca
Reviewed-on: https://chromium-review.googlesource.com/518822Reviewed-by: Henrik Kjellander <kjellander@chromium.org>
Commit-Queue: Henrik Kjellander <kjellander@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475839}

This change removes usages of raw pointers where the underlying data has
been moved from. Currently this code still works fine, but it would
result in uses-after-free with the upcoming change to
Value::DictStorage.

BUG=646113

Change-Id: I99c1498c4c157fbc60ccc240dc243056acb8be13
Reviewed-on: https://chromium-review.googlesource.com/518129
Commit-Queue: jdoerrie <jdoerrie@chromium.org>
Reviewed-by: Brett Wilson <brettw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475838}

This CL adds a base::SharedMemoryHandle getter to gpu::BufferBacking.
The getter will be used by gpu::BufferBacking users when OnMemoryDump is
called to create base::SharedMemory dumps.

This is part of work for exposing SharedMemory usages on tracing
memory-infra dumps.

Design doc: https://docs.google.com/document/d/16Mi5_puxKgQ-9IX7ANfbji0WiqBCK3yrnJ0Qi2wa3w0/edit#heading=h.lofdeze1a2hr

Note: This can conflict with https://codereview.chromium.org/2912723002/

BUG=604726
TEST=gpu_unittests --gtest_filter=Buffer.*
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

Review-Url: https://codereview.chromium.org/2909003002
Cr-Commit-Position: refs/heads/master@{#475837}

Bug: 727991
Change-Id: Ife0e27562727cf08ff40615baf3f024ed621534d
Reviewed-on: https://chromium-review.googlesource.com/518745Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475836}

Disable a flaky test InputRouterImplTest.AckedTouchEventState in content_unittests on Win, Linux and ChromiumOS

TBR=aelias@chromium.org

Bug: 728025
Change-Id: Icb1df3b7e353b4626c9ba263dd3b4c64dcd04276
Reviewed-on: https://chromium-review.googlesource.com/518783
Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475835}

:host and :host-context() should not have implicit type selectors pre-
pended for universal rules with default @namespace. Default @namespace
applies to type and universal selectors inside :host() and
:host-context(), but a default @namespace rule should not affect :host.
That is, :host matches the host element regardless of any @namespace
rules.

R=hayato@chromium.org,kochi@chromium.org
BUG=727603

Change-Id: I78d2919275aa6bdc4fdc5b02a4772d2352819258
Reviewed-on: https://chromium-review.googlesource.com/518019Reviewed-by: Takayoshi Kochi <kochi@chromium.org>
Commit-Queue: Rune Lillesveen <rune@opera.com>
Cr-Commit-Position: refs/heads/master@{#475834}

Currently the bookmark bubble builds its own button row, but that's
unnecessary and causes it to miss out on shared framework code for
layout and button style.

BUG=726187

Review-Url: https://codereview.chromium.org/2908963002
Cr-Commit-Position: refs/heads/master@{#475833}

This adds Feature Policy checks to the mojo permission service and
corresponding tests. Because multiple permissions can be requested at the same
time, part of a request can be answered synchronously (by feature policy) and
the other part asynchronously (by quering the PermissionManager). Thus we need
to track the partially answered request in the PendingRequest class.

Feature policy checks are done here (in content) as they will form part of the
specifications for various features and thus it is best if other embedders of
content also benefit from these checks.

BUG=689802
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Review-Url: https://codereview.chromium.org/2874053003
Cr-Commit-Position: refs/heads/master@{#475832}

In an empty text prompt with ghost text visible, the ghost text would
capture the first character typed.

BUG=none

Review-Url: https://codereview.chromium.org/2907553004
Cr-Commit-Position: refs/heads/master@{#475831}

Currently, the line-break property is a 'storage_only' field,
so it has no generated public getters/setters. This patch changes it to
a 'keyword' field so that its getters/setters can be generated as well.
We also generate the ELineBreak enum and use the generated
CSSValueID <-> ELineBreak mappings.

BUG=628043

Review-Url: https://codereview.chromium.org/2911383002
Cr-Commit-Position: refs/heads/master@{#475830}

910cca2 Fix crash detected by asan test

BUG=726198
TEST=gclient sync, make
TBR=brettw@chromium.org

Change-Id: Ie46fc45719262f11a142d335edd920161366eea4
Reviewed-on: https://chromium-review.googlesource.com/518763
Commit-Queue: Jinsuk Kim <jinsukkim@chromium.org>
Reviewed-by: Jinsuk Kim <jinsukkim@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475829}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/20ee3efa..02777351

Please follow these instructions for assigning/CC'ing issues:
https://github.com/v8/v8/wiki/Triaging%20issues

Please close rolling in case of a roll revert:
https://v8-roll.appspot.com/
This only works with a Google account.

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;master.tryserver.chromium.android:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,vogelheim@chromium.org

Review-Url: https://codereview.chromium.org/2916503002
Cr-Commit-Position: refs/heads/master@{#475828}

A couple of tricky points:
* The requirement to always call CloseStream is added to
  media/audio/audio_output_ipc.h. This requirement isn't new, but it
  wasn't documented before.
* If the RendererAudioOutputStreamFactory used by MojoAudioOutputIPC is
  destroyed prior to receiving authorization (e.g. because of frame
  destruction), we need to signal the AudioOutputDevice anyways to
  avoid waiting forever. We pass a ScopedClosureRunner into the
  authorization callback to make sure we get a callback even if the
  factory is destroyed early.
* Code for intergrating this into the AudioDeviceFactory is in
  https://codereview.chromium.org/2890753003
* Browser tests (using this and the dependent CL) in
  https://codereview.chromium.org/2812883003/#ps380001.

BUG=425368
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

Review-Url: https://codereview.chromium.org/2821203005
Cr-Commit-Position: refs/heads/master@{#475827}

This adds feature policy checks for Geolocation, Midi/sysex, Protected media,
and Mic/Camera to the browser process. These checks are also implemented in
the content layer in the Permission Service
(see https://codereview.chromium.org/2874053003/). The reason to have these
checks implemented in the content/ layer is because they are standardized
checks and we want to ensure that other embedders of content implement the same
behavior.

However there are 2 pitfalls to this:
1) The results returned from the permission service are used to make decisions
in the renderer process and so are not trustworthy. Because of this, most
features that have permission will add additional checks in the browser
process.
2) The permission checks that happen in the browser process most often happen
in the chrome/ layer (through the PermissionManager) so this is also where
feature policy checks need to happen.

In the long term, both chrome/ and content/ should make permission checks
through the same API/mojo service where common permission checks like this one
can live.

BUG=689802
TBR=tommycli@chromium.org,peter@chromium.org

Review-Url: https://codereview.chromium.org/2898663002
Cr-Commit-Position: refs/heads/master@{#475826}

This reverts commit 9a9457b3.

Reason for revert: Relanding this change as the broken test was invalid anyway and has since been removed. Sorry for the churn!

Original change's description:
> Revert "Mojo EDK: Refactor layered message serialization"
> 
> This reverts commit 77c59a87.
> 
> Reason for revert: Broke build. See the comment on the original review.
> 
> Original change's description:
> > Mojo EDK: Refactor layered message serialization
> > 
> > Consolidates and simplifies user message serialization within the EDK.
> > Namely:
> > 
> >   * The ports layer no longer operates under the assumption that all
> >   events exist exclusively in serialized form. This eliminates the
> >   complexity of layered message allocation at the cost of a small amount
> >   of extra copying in some cases.
> > 
> >   * ports::Message is now more aptly named ports::Event, and different
> >   event types are subclasses.
> > 
> >   * Introduces a ports::UserMessage base class for ports embedder
> >   message types. These can be attached to ports::UserMessageEvent
> >   objects.
> > 
> >   * The MessageForTransit and PortsMessage types have been flattened
> >   into a single UserMessageImpl type, a subclass of ports::UserMessage.
> > 
> >   * Nearly all user message serialization and deserialization logic has
> >   been consolidated into UserMessageImpl. A few small pieces remain
> >   isolated in node_controller.cc and will eventually move elsewhere
> >   after additional refactoring.
> > 
> > All of this work is a precursor to supporting lazy serialization.
> > 
> > BUG=725321
> > 
> > Change-Id: Ib9e66cd0d5b88ac0e6dc898f248c958039078023
> > Reviewed-on: https://chromium-review.googlesource.com/516402
> > Commit-Queue: Ken Rockot <rockot@chromium.org>
> > Reviewed-by: Jay Civelli <jcivelli@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#475731}
> 
> TBR=jcivelli@chromium.org,rockot@chromium.org
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> BUG=725321
> 
> Change-Id: I84f950290e6c45dc830f2bc3177af09ae955f432
> Reviewed-on: https://chromium-review.googlesource.com/518742
> Reviewed-by: Takeshi Yoshino <tyoshino@chromium.org>
> Commit-Queue: Takeshi Yoshino <tyoshino@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#475805}

TBR=abarth@chromium.org,jcivelli@chromium.org,aa@chromium.org,rockot@chromium.org,tyoshino@chromium.org,darin@chromium.org,chromium-reviews@chromium.org,findit-for-me@appspot.gserviceaccount.com
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
BUG=725321

Change-Id: I5a2dbe4abf2e1f576bcf3f7f651f3557583d9033
Reviewed-on: https://chromium-review.googlesource.com/518427Reviewed-by: Ken Rockot <rockot@chromium.org>
Commit-Queue: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475825}

We decided DocumentMarkers shouldn't ever be copied. After
https://codereview.chromium.org/2905753002, we no longer need to copy
DocumentMarkers to create TextMatchMarkers, so we can remove the copy
constructor. We can also remove the TextMatchMarker constructor that takes a
DocumentMarker.

We need to remove the copy constructor anyway because we're turning
DocumentMarker into a polymorphic class hierarchy, and copying the base class
will no longer produce correct behavior.

BUG=707867

Review-Url: https://codereview.chromium.org/2902393003
Cr-Commit-Position: refs/heads/master@{#475824}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/32a3f0b8f49b..deb4a2aa8f11

$ git log 32a3f0b8f..deb4a2aa8 --date=short --no-merges --format='%ad %ae %s'
2017-05-30 dtu [pinpoint] Exclude StandardError from the Execution exception block.
2017-05-30 dtu [pinpoint] Request builds using Buildbucket.

Created with:
  roll-dep src/third_party/catapult


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=sullivan@chromium.org

Change-Id: I6c026faa2672fbaec85d13bf85582aad461fdaa8
Reviewed-on: https://chromium-review.googlesource.com/518564
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475823}

Revert of Compute tail time from Biquad coefficients (patchset #7 id:120001 of https://codereview.chromium.org/2862373002/ )

Reason for revert:
The added layout tests are failing on "WebKit Linux Trusty MSAN" bot.
https://build.chromium.org/p/chromium.webkit/builders/WebKit%20Linux%20Trusty%20MSAN/builds/1242

Original issue's description:
> Compute tail time from Biquad coefficients
>
> Instead of using a fixed tail time of 200 ms for all biquad filters,
> we compute the actual tail time from filter coefficients.  Many
> typical filters in applications have a tail time of 10 ms or less.
> This allows biquads and downstream nodes to stop processing much sooner
> than what currently happens, reducing CPU usage and allowing GC to
> collect nodes sooner.
>
> BUG=612215
> TEST=
>
> Review-Url: https://codereview.chromium.org/2862373002
> Cr-Commit-Position: refs/heads/master@{#475632}
> Committed: https://chromium.googlesource.com/chromium/src/+/0be695dcb495cade0ed5c4b70aba95eb3dce49bd

TBR=hongchan@chromium.org,rtoy@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=612215

Review-Url: https://codereview.chromium.org/2912293002
Cr-Commit-Position: refs/heads/master@{#475822}

BUG=none

Review-Url: https://codereview.chromium.org/2867133002
Cr-Commit-Position: refs/heads/master@{#475821}

R=kouhei@chromium.org

Bug: 723841
Change-Id: I1a87c594c90ecec6388122cf3796931b3cc4fa7e
Reviewed-on: https://chromium-review.googlesource.com/518018Reviewed-by: Kouhei Ueno <kouhei@chromium.org>
Commit-Queue: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475820}

This patch makes |TextIterator| constructor to take only proper range to
simplify code for improve code health.

Note: This patch is passed all tests, but there are some cases to pass
improper range to |TextIterator|. To catch such case and avoid infinite loop,
we use |CHECK()| to make blink crash.

BUG=727537
TEST=n/a; no behavior changes

Review-Url: https://codereview.chromium.org/2912053002
Cr-Commit-Position: refs/heads/master@{#475819}

Bug: 
Change-Id: I79db155731d01055cf6121b14b18fbd550d62a6f
Reviewed-on: https://chromium-review.googlesource.com/518664Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Joel Hockey <joelhockey@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475818}

Currently, the ruby-position property is a 'storage_only' field,
so it has no generated public getters/setters. This patch changes it to
a 'keyword' field so that its getters/setters can be generated as well.
We also generate the ERubyPosition enum and use the generated
CSSValueID <-> ERubyPosition mappings.

BUG=628043

Review-Url: https://codereview.chromium.org/2915683002
Cr-Commit-Position: refs/heads/master@{#475817}

Revert of Enable redirection of SequencedWorkerPool to TaskScheduler in TestBrowserThreadBundle. (patchset #3 id:40001 of https://codereview.chromium.org/2769693002/ )

Reason for revert:
Caused ServiceWorkerResourceStorageDiskTest.* tests in content_unittests to crash.

See https://bugs.chromium.org/p/chromium/issues/detail?id=727994

Reverting locally fixed the crashes, so I'm going to revert this CL.

Original issue's description:
> Enable redirection of SequencedWorkerPool to TaskScheduler in TestBrowserThreadBundle.
>
> Motivation:
> - Redirection of SequencedWorkerPool to TaskScheduler is enabled in
>   browser tests and on Canary and Dev. It makes sense to enable it
>   in unit tests too.
> - It is not possible to wait until there are no more blocking pool
>   *and* TaskScheduler tasks (new tasks in the blocking pool during
>   a FlushForTesting() call on TaskScheduler and vice versa). If
>   the blocking pool is redirected to TaskScheduler, calling
>   FlushForTesting() on TaskScheduler guarantees that there are no
>   more blocking pool *and* TaskScheduler tasks.
>
> BUG=667892
>
> Review-Url: https://codereview.chromium.org/2769693002
> Cr-Commit-Position: refs/heads/master@{#475144}
> Committed: https://chromium.googlesource.com/chromium/src/+/e1aa28915a28da266d406202a07112d44d6cc018

TBR=gab@chromium.org,robliao@chromium.org,sky@chromium.org,fdoray@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=667892,727994

Review-Url: https://codereview.chromium.org/2909393003
Cr-Commit-Position: refs/heads/master@{#475816}

This patch makes TextUnderlinePosition an enum class for better type
safety.

BUG=684966
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

Review-Url: https://codereview.chromium.org/2909423002
Cr-Commit-Position: refs/heads/master@{#475815}

This patch renames some StyleRareNonInheritedData members so that they
match the format expected by the generator.

BUG=628043

Review-Url: https://codereview.chromium.org/2911403002
Cr-Commit-Position: refs/heads/master@{#475814}

This is an initial step to merge MainThreadWorklet and ThreadedWorklet into
Worklet. No tests added, no behavior change.

In the current implementation, ThreadedWorklet implements the loader client for
classic script loading. That's blocking the merge effort. This CL moves the
client implementation from ThreadedWorklet to ThreadedWorkletMessagingProxy as
LoaderClient and reduces differences between MainThreadWorklet and
ThreadedWorklet.

DesignDoc: https://docs.google.com/document/d/1cgLcrua7H_7x_o5GlzYrAi2qt-TqTzgtOeixFAugR6g/edit?usp=sharing

Bug: 727194
Change-Id: I9b9adc04ff5a633a2d9afef3ded6a2f57b89e6ba
Reviewed-on: https://chromium-review.googlesource.com/517454
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475813}

https://skia.googlesource.com/skia.git/+log/b365cf590f87..b5e484254331

$ git log b365cf590..b5e484254 --date=short --no-merges --format='%ad %ae %s'
2017-05-30 mtklein clean up now that min_stride == 1
2017-05-30 mtklein call compile in convert_with_pipeline()
2017-05-30 egdaniel Allow GPU blur fast path for circles and rects when using ignore xform
2017-05-31 fmalita Revert "Delete SkGaussianEdgeShader"

Created with:
  roll-dep src/third_party/skia


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

If the roll is causing failures, see:
http://www.chromium.org/developers/tree-sheriffs/sheriff-details-chromium#TOC-Failures-due-to-DEPS-rolls


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;master.tryserver.chromium.android:android_optional_gpu_tests_rel
TBR=herb@chromium.org

Change-Id: I72c40a57d7eb0c2bfd81b069c6ea65565031049c
Reviewed-on: https://chromium-review.googlesource.com/518450Reviewed-by: Skia Deps Roller <skia-deps-roller@chromium.org>
Commit-Queue: Skia Deps Roller <skia-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#475812}