GitLab

The tests had a hidden dependency on the order that other tests in the
suite would run. This change fixes the hidden dependency by setting the
application context in ContextUtils.

      -f *CastWebContentsSurfaceHelperTest*
      without patch (fails)
      and with patch (passes)

Bug: 834034
Test: Run cast_shell_junit_tests with
Change-Id: I3fe2d4111a6959a58352411ff653cebf2403a437
Reviewed-on: https://chromium-review.googlesource.com/1015978Reviewed-by: Nate Fischer <ntfschr@chromium.org>
Reviewed-by: John Budorick <jbudorick@chromium.org>
Reviewed-by: Luke Halliwell <halliwell@chromium.org>
Commit-Queue: Luke Halliwell <halliwell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551557}

Bug: 834098
Change-Id: Id40def9037ae2f9a7afc10be8b2ddcc4a9153775
Reviewed-on: https://chromium-review.googlesource.com/1016022Reviewed-by: Tommy Steimel <steimel@chromium.org>
Commit-Queue: apacible <apacible@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551556}

Currently |reduction_| in DynamicsCompressorHandler is accessed by
two different threads without any protection. Adding
NoBarrierStore/Load will suppress the warning, and also
indicates that we have a benign data race in |reduction_|.

See also: https://codereview.chromium.org/1461743002/

Bug: 755393
Change-Id: I3946772b55fdd6b6e7c806058306d0f81a855c69
Reviewed-on: https://chromium-review.googlesource.com/927713
Commit-Queue: Hongchan Choi <hongchan@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551555}

base::TimeTicks is default-constructed at a timestamp of 0. If a
keyboard event has a timestamp near 0, TypeAhead will incorrectly return
that there is an active session.

To fix that, track the last typed time as an Optional and use that to
distinguish if a session is actually active.

Change-Id: Id4dfab9cc498b8fee2f51785894bfc1ec04a6502
Reviewed-on: https://chromium-review.googlesource.com/1015972Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551554}

BUG=b:62406437
TEST=Open launcher in caroline-arcnext

Change-Id: I01e8c31ea01578ad86129ea2f96de0b2503c5715
Reviewed-on: https://chromium-review.googlesource.com/1014882Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: Yury Khmel <khmel@chromium.org>
Commit-Queue: Shuhei Takahashi <nya@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551553}

https://chromium.googlesource.com/chromiumos/chromite.git/+log/c46e7967a50f..6e6b9f035719

$ git log c46e7967a..6e6b9f035 --date=short --no-merges --format='%ad %ae %s'
2018-04-12 mikewu chromeos_config: Use latest toolchain in lakitu_next

Created with:
  roll-dep src/third_party/chromite


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

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

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


TBR=chrome-os-gardeners@chromium.org

Change-Id: I0896758fe7d81e3a14fdbe216c68765888128371
Reviewed-on: https://chromium-review.googlesource.com/1015222
Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#551552}

Aura synthesizes a mouse event at certain times, including when the
window comes back from being minimized. Currently it uses the last known
mouse position, which works in most cases. However, the window doesn't
get mouse events when minimized, so the last known position isn't
updated. When the window comes back, if the user's mouse is not over the
window it can cause buttons to get stuck in the hovered state until the
user moves the mouse back into the window.

To fix this, this patch uses the real mouse position.

Bug: 828591
Change-Id: I604d6ddf8f260fe32bd0e4a607f19f1793cf5c5e
Reviewed-on: https://chromium-review.googlesource.com/1006396
Commit-Queue: Bret Sepulveda <bsep@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551551}

In its last action before being turned off the VC++ compiler said:
    warning.cpp(2): warning C4554: '|': check operator precedence for
    possible error; use parentheses to clarify precedence

The code is not wrong, but it is confusing. By default we should use
logical operators for logical expressions, and bitwise operators for
bitwise expressions.

See also this request for a similar warning in clang:
    https://bugs.llvm.org/show_bug.cgi?id=37155

Bug: 825568
Change-Id: I339744f004258137d72b23bfe047901dd40be4ad
Reviewed-on: https://chromium-review.googlesource.com/1015780Reviewed-by: Timothy Loh <timloh@chromium.org>
Commit-Queue: Bruce Dawson <brucedawson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551550}

Also add test to try to replicate the crash, although the best it seems
to do is flakily DCHECKing. At least this CL seems to fix the DCHECK.

Bug: 829859
Change-Id: I44f416d7c3edf68e2faf15c2eba7d582db947a0a
Reviewed-on: https://chromium-review.googlesource.com/1014400
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551549}

This hides the internal URL used by WKBasedNavigationManager to force
create an entry in WKBackForwardList when reloading with a different
user agent type (see crrev.com/c/919175).

Also fixed a bug in NavigationManagerTest which wasn't testing the
redirect case when GetParam() == TEST_WK_BASED_NAVIGATION_MANAGER.

Bug: 833958
Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I095178302bfc4d209c2d058aab736777cc8b2de4
Reviewed-on: https://chromium-review.googlesource.com/1016002Reviewed-by: Eugene But <eugenebut@chromium.org>
Commit-Queue: Danyao Wang <danyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551548}

https://chromium.googlesource.com/catapult.git/+log/ca7ee7a2c7fe..a903f96d720c

$ git log ca7ee7a2c..a903f96d7 --date=short --no-merges --format='%ad %ae %s'
2018-04-17 simonhatch Dashboard - InstallHooks() in testing_common setup.

Created with:
  roll-dep src/third_party/catapult


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

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

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


TBR=sullivan@chromium.org

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

Bug: 833008
Change-Id: Ie97bd09ba7aa65bf56532790ee0200ae72fd5383
Reviewed-on: https://chromium-review.googlesource.com/1015970Reviewed-by: Yafei Duan <romax@chromium.org>
Commit-Queue: Carlos Knippschild <carlosk@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551546}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/73d9591f..059b1287

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

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

Change-Id: I11d179623a886cd80cf4ae9de11c29816cb8b430
Reviewed-on: https://chromium-review.googlesource.com/1012944Reviewed-by: v8 autoroll <v8-autoroll@chromium.org>
Commit-Queue: v8 autoroll <v8-autoroll@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551545}

In some cases, AXNodeData node id refs (e.g. nextOnLine), refer to nodes that don't exist.

In the js wrapper objects, we did not ensure objects were valid in

AutomationRootNodeImpl: :get().
Change-Id: I7cf5d988ee7dafa6b0595078002524255a072953
Reviewed-on: https://chromium-review.googlesource.com/1015975Reviewed-by: Dominic Mazzoni <dmazzoni@chromium.org>
Commit-Queue: David Tseng <dtseng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551544}

Bug: 790367
Change-Id: I53baf6424746078bbc0ef23c23aa85e6026ebcc9
Reviewed-on: https://chromium-review.googlesource.com/1015226Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Shenghua Zhang <shenghuazhang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551543}

If you deleted or moved a directory containing a PRESUBMIT.py file,
then the presubmit checks would crash trying to run tests in a
directory that no longer existed.

R=agable@chromium.org
BUG=834077

Change-Id: I8deb59c2d00b80f6b5215597076bba34ff1f0872
Reviewed-on: https://chromium-review.googlesource.com/1015976Reviewed-by: Aaron Gable <agable@chromium.org>
Commit-Queue: Dirk Pranke <dpranke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551542}

Change-Id: I1aacf702cc4b2247179c97ab393f141090cd8940
Reviewed-on: https://chromium-review.googlesource.com/1009484
Commit-Queue: Cathy Li <chili@chromium.org>
Reviewed-by: Peter Williamson <petewil@chromium.org>
Reviewed-by: Matt Menke <mmenke@chromium.org>
Reviewed-by: Colin Blundell <blundell@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551541}

We introduce notification_helper_launches_chrome_unittest, which is evolution
of notification_helper_process_unittest. In addition to testing the launch of
notification_helper by the OS which is notification_helper_process_unittest
all about, the new unit test also tests if chrome.exe can be successfully
launched by notification_helper.exe via the NotificationActivator::Activate
function.

This new unit test is compiled into unit_tests.exe under chrome rather than
notification_helper_unittests.exe. This is because unit_tests.exe has data
dependency on chrome.exe which is required by this new unit test, and it's
undesired to make notification_helper_unittests.exe have data dependency on
chrome.exe.

Bug: 734095
Change-Id: I07ac90a84d7f885a359867f79ce613a194e0bb64
Reviewed-on: https://chromium-review.googlesource.com/1002132
Commit-Queue: Xi Cheng <chengx@chromium.org>
Reviewed-by: Greg Thompson <grt@chromium.org>
Reviewed-by: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551540}

This patch makes |SelectionModifier::Modify()| to do nothing for empty selection
to avoid calling |NextWordPosition()| with null-position.

Note: |NextWordPosition()| doesn't accept null-position since the patch[1].

[1] http:/crrev.com/c/1004536 Make NextWordPosition() to utilize
TextOffsetMapping

Bug: 832061
Change-Id: I534af079f068f709c9b48a408988cb413a520b86
Reviewed-on: https://chromium-review.googlesource.com/1014841Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551539}

changwan@ has been looking into WebView startup performance; add as an
owner of the relevant benchmark to get notifications about bugs. Also
add component.

Change-Id: I34cac9a71d2c36129d40a0a34b1b2b940282b154
Reviewed-on: https://chromium-review.googlesource.com/1015540Reviewed-by: Changwan Ryu <changwan@chromium.org>
Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Commit-Queue: Richard Coles <torne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551538}

This CL ensures that querying for ARC apps is gracefully aborted if the
owning WebContents is destroyed during the asynchronous process.

ArcNavigationThrottle is converted to be a self-owning object with a
private constructor and destructor. When the static app query methods
are invoked, an instance of the object is instantiated to handle the
querying. By overriding WebContentsDestroyed(), it can cancel all
in-flight callbacks upon WebContents destruction and safely bail out of
querying.

BUG=824598
TBR=sky@chromium.org

Change-Id: I2ba6d391b9babec46b277cbcebe567786215bf5a
Reviewed-on: https://chromium-review.googlesource.com/1003432
Commit-Queue: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Yusuke Sato <yusukes@chromium.org>
Reviewed-by: David Jacobo <djacobo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551537}

This CL renames and refactors DeviceCapabilityManager. Now, this class
makes use of the SoftwareFeature proto message instead of relying on its
own Capability enum. Likewise, now the class utilizes new fields in
ToggleEasyUnlockRequest (e.g., is_exclusive). Finally, this class no
longer has any promo-related functionality, since that is being removed
in favor of the unified setup flow.

Bug: 824568, 752273
Change-Id: I23f5c642a3974c2723f29591d67fe79d0426effb
Reviewed-on: https://chromium-review.googlesource.com/1011543Reviewed-by: Jeremy Klein <jlklein@chromium.org>
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551536}

TBR=fdoray@chromium.org

Bug: 833989
Change-Id: I6cf7e67add2310af3b09c9659209f5dc60cd4c73
Reviewed-on: https://chromium-review.googlesource.com/1015846Reviewed-by: Michael Wasserman <msw@chromium.org>
Reviewed-by: François Doray <fdoray@chromium.org>
Commit-Queue: Michael Wasserman <msw@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551535}

Model variables and logic were stored in TraySessionLengthLimit. This CL
adds SessionLengthLimitModel to SystemTrayModel and separates them.

In the upcoming CL, session length limit notification will be generated
outside TraySessionLengthLimit, as it will be removed with
UnifiedSystemTray launch.

TEST=TraySessionLengthLimitTest
BUG=none

Change-Id: Ic3994f45f0ad48460654512fa8f8e23874503a73
Reviewed-on: https://chromium-review.googlesource.com/1015444Reviewed-by: James Cook <jamescook@chromium.org>
Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Tetsui Ohkubo <tetsui@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551534}

TBR=smut

Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs
Change-Id: I6af654c734a4337083800d6efe2598d5e995579d
Reviewed-on: https://chromium-review.googlesource.com/1015704
Commit-Queue: iOS Autoroller <ios-autoroller@chromium.org>
Reviewed-by: iOS Autoroller <ios-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551533}

Bug: 833586
Change-Id: I87e956cadfa62a79aa549f22fe09211ee7cf25ce
Reviewed-on: https://chromium-review.googlesource.com/1015773Reviewed-by: Erik Luo <luoe@chromium.org>
Commit-Queue: Pavel Feldman <pfeldman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551532}

Because of |NextWordPositionInternal()| doesn't accept null-position, this
patch adds |DCHECK()| to catch such callers for ease of debugging.

Bug: 832061
Change-Id: Id987553c939db8f061e58eca09d7ebcd946482c9
Reviewed-on: https://chromium-review.googlesource.com/1013642Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Yoshifumi Inoue <yosin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551531}

Bug: 826306
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Idb639ff99a917dc59fd04110c918a8d67583db08
Reviewed-on: https://chromium-review.googlesource.com/986287
Commit-Queue: Scott Chen <scottchen@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Hector Carmona <hcarmona@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551530}

As is said in the bug, we decided not to show the title in the
title bar for ARC apps. However, debugging information (stored
in extra_title_) wants to appear, and the overview mode wants
to show the actual title (app name) instead of the debugging
information.

So this CL changed ClientControlledShellSurface as:
- GetWindowTitle always return extra_title_ and no title_ so
  title won't appear anymore
- when the widget's title is updated, directly specify the
  native window's title which is used by the overview mode

Bug: 831383
Test: manually, exo_unittests
Change-Id: Ifa275bd6f5c72b8ca36cccf7f9c629498c4a670e
Reviewed-on: https://chromium-review.googlesource.com/1012783
Commit-Queue: Jun Mukai <mukai@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: David Reveman <reveman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551529}

LocalCaretRect should use the "flipped blocks" coordinate space for
vertical-rl writing mode. This patch applies that in the NG implementation.

Note: this patch only changes behavior for text-anchored carets. When
caret is anchored to atomic inline, there is no reference as the legacy
behavior is broken, and LocalCaretRect is consumed by legacy.

Bug: 822575
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_layout_tests_layout_ng
Change-Id: I4019f560f8410ef14c7d52312d7fc1de8db6ee28
Reviewed-on: https://chromium-review.googlesource.com/1011296Reviewed-by: Koji Ishii <kojii@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551528}

This patch adds the required aria labels to display zoom slider.
The aria labels added are:
  - aria-valuemin
  - aria-valuemax
  - aria-valuenow
  - aria-valuetext

Bug: 791382, 790723
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ibb1c05d160027cefb91ca270beb5b5c54e3e669b
Component: Settings, display settings, display size slider
Reviewed-on: https://chromium-review.googlesource.com/1015778Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Malay Keshav <malaykeshav@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551527}

This is a reland of 4df8a2dd

Original change's description:
> Add a Mojo-based transportation layer for Perfetto
>
> This adds a base layer that individual logging protocols (like tracing)
> can use to send their data to Perfetto, using Mojo messaging and
> shared memory.
>
> First CL of several, after this comes the Chrome tracing layer: https://chromium-review.googlesource.com/c/chromium/src/+/981270
>
> A preview of the full integration can be seen here: https://chromium-review.googlesource.com/c/chromium/src/+/852808
>
> The design doc for the integration: https://docs.google.com/document/d/1a8lvN3zYWEyAsBjtuRRsy0fWOWdUdbw6bGnKLBscGdw/edit
>
> R=primiano@chromium.org,skyostil@chromium.org
>
> Change-Id: Ie1db5745ebffdcf1e2d74e2f6b12395f447c8552
> Reviewed-on: https://chromium-review.googlesource.com/980578
> Reviewed-by: oysteine <oysteine@google.com>
> Reviewed-by: Daniel Cheng <dcheng@chromium.org>
> Reviewed-by: Primiano Tucci <primiano@chromium.org>
> Commit-Queue: oysteine <oysteine@google.com>
> Cr-Commit-Position: refs/heads/master@{#551019}

Change-Id: Ic47afdb6c2e09924a5f540fa56552ceb6e29e6d7

TBR=dcheng@chromium.org,primiano@chromium.org,oysteine@google.com,skyostil@chromium.org

Change-Id: Ic47afdb6c2e09924a5f540fa56552ceb6e29e6d7
Reviewed-on: https://chromium-review.googlesource.com/1015189Reviewed-by: oysteine <oysteine@chromium.org>
Commit-Queue: oysteine <oysteine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551526}

Previously, storage::FileSystemUsageCache used a custom timer
implementation (TimedTaskHelper). Crash involving this custom timer
implementation have been reported (e.g. http://crash/e2c6ff1ef4863460).
Therefore, we are moving all uses of TimedTaskHelper to the well
maintained base::Timer.

Note: The TaskRunner passed to the TimedTaskHelper in
storage::FileSystemUsageCache was always the one on which the
storage::FileSystemUsageCache was used, so there is no need to
an explicit call to base::Timer::SetTaskRunner in this CL.

Bug: 760326
Change-Id: I4b0727e4f70c2560a2948015dd28236cd760a9f3
Reviewed-on: https://chromium-review.googlesource.com/1012586
Commit-Queue: François Doray <fdoray@chromium.org>
Reviewed-by: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551525}

https://skia.googlesource.com/skia.git/+log/eedecc6046b2..c186e17a6b78

$ git log eedecc604..c186e17a6 --date=short --no-merges --format='%ad %ae %s'
2018-04-17 robertphillips Add handling to ViaDDL for images that are too large to be uploaded in one go
2018-04-17 herb Move node information to SkStrikeCache
2018-04-17 bungeman IWYU for some more test files starting with 'C'.
2018-04-17 reed detect if we made an infinite rect
2018-04-17 borenet [infra] Support recursive configs in builder_name_schema
2018-04-16 egdaniel No long require copy as draw for vulkan wrapped resources.
2018-04-17 caryclark path is rect diag test is required
2018-04-17 mtklein SkStrikeDifferences::operator() -> add()
2018-04-17 reed avoid overflow by using tmp 64bits
2018-04-17 brianosman Move external texture extension logic into SkSL
2018-04-17 benjaminwagner [infra] Add service account override in gen_tasks.go.
2018-04-17 caryclark path is rect bug number twelve
2018-04-17 bungeman Add paint flags to viewer GUI.
2018-04-17 mtklein skcms→3be11ac refactor a bit
2018-04-17 liyuqian Revert "Fix the fat rect bug in the threaded backend"
2018-04-17 reed only recheck fNeedsTiling if we computed fSrcBounds
2018-04-17 brianosman Add SkColorSpace::Make, taking an skcms_ICCProfile
2018-04-16 mtklein Reland "low-hanging bilerp_clamp_8888 wins"
2018-04-17 mtklein skcms→38f22d2 name types in skcms.h
2018-04-17 herb Reorganize strike cache and glyph cache code
2018-04-16 liyuqian Fix the fat rect bug in the threaded backend
2018-04-17 caryclark yet another path is rect exploit
2018-04-17 angle-skia-autoroll Roll third_party/externals/angle2/ 08b9e046c..86de76b69 (1 commit)
2018-04-16 halcanary SkColorSetARGB in constants
2018-04-16 caryclark a more involved path is rect bug
2018-04-16 egdaniel Update our config fallback switch in GrContext.
2018-04-17 borenet Revert "Reland "[infra] Run recipes through Kitchen""
2018-04-16 borenet Reland "[infra] Run recipes through Kitchen"

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


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;luci.chromium.try:win_optional_gpu_tests_rel
TBR=scroggo@chromium.org

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

Extends 8dp corners to ToolbarActionView which was missed in a previous
change. This makes them consistent with other toolbar buttons.

Bug: chromium:822069
Change-Id: Id894664ec698504e3ff997c1b12691821ab1062e
Reviewed-on: https://chromium-review.googlesource.com/1015286
Commit-Queue: Peter Boström <pbos@chromium.org>
Reviewed-by: Bret Sepulveda <bsep@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551523}

Following bot results are included.
4828 4834 4835 4836 4837 4838 4839 4846

2 lines were removed and 0 lines were deflaked by consecutive
results since 4772.

TBR=eae@chromium.org, mstensho@chromium.org
NOTRY=true

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

This CL detects the case that a invisible composited layer may have bogus
subpixel accumulation on it, and ignore the DCHECK in such case.

BUG=816490

Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Iba98cb3a57dfe24f8a69f1f7469bf78482530d7e
Reviewed-on: https://chromium-review.googlesource.com/1012441Reviewed-by: Xianzhu Wang <wangxianzhu@chromium.org>
Commit-Queue: Tien-Ren Chen <trchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551521}

Disabling all tests in CorrectEventFiringTest class.
Tests are flaky and we'll enable these tests again as soon as the bug is fixed.

Change-Id: I5b02e385869165cb2d2e7cfd87c5611fc81eb9fb
Reviewed-on: https://chromium-review.googlesource.com/1015973Reviewed-by: John Chen <johnchen@chromium.org>
Commit-Queue: John Chen <johnchen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551520}

R=brettw@chromium.org, rsesek@chromium.org, sdefresne@chromium.org, sergeyu@chromium.org

Bug: 821738
Change-Id: Ib1dcca4070ad7019c082653217df48adda69ab32
Reviewed-on: https://chromium-review.googlesource.com/962282Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551519}

Bug: none
Change-Id: If04229d0caf53848f6dde4f0f7aabde053954f67
Reviewed-on: https://chromium-review.googlesource.com/1008526Reviewed-by: James Cook <jamescook@chromium.org>
Commit-Queue: Evan Stade <estade@chromium.org>
Cr-Commit-Position: refs/heads/master@{#551518}