GitLab

privacy.websites.protectedContentEnabled is enabled per-platform, and
the platform list includes "cros" and "cros touch". But the
determination of the platforms in binding.js includes "chromeos" and
"chromeos touch". As a result, the protectedContentEnabled feature
is never actually available on a CrOS platform (despite being documented
as such).

It appears that, since it's addition in
https://crrev.com/cc3a5c8ca42579c0f9f710c18f51c566b4c75e86, this has
been unavailable on ChromeOS: See
https://chromium.googlesource.com/chromium/src/+/cc3a5c8ca42579c0f9f710c18f51c566b4c75e86/chrome/renderer/resources/extensions/schema_generated_bindings.js#318
and
https://chromium.googlesource.com/chromium/src/+/cc3a5c8ca42579c0f9f710c18f51c566b4c75e86/chrome/common/extensions/api/privacy.json#88
where the patch was added.

Update the platforms listed to be those that are used in the features
system.

Also update the browser test to correctly check the
privacy.websites.protectedContentEnabled feature on all platforms, as
well as expanding it to test setting preferences.

BUG=746064

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

None of the implementations do anything, so no point in keeping it
around.

BUG=none
TEST=none

Change-Id: Ic892b01b1f4896105982e2646731d65d53499c54
Reviewed-on: https://chromium-review.googlesource.com/576662
Commit-Queue: Scott Violet <sky@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487930}

First CL! :)

Bug: 
Change-Id: I171a77c0d0d99316cc9d2ce3ac7aa4fdcc83fa9d
Reviewed-on: https://chromium-review.googlesource.com/575703Reviewed-by: Rick Byers <rbyers@chromium.org>
Commit-Queue: Robert Ma <robertma@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487929}

Commit 5ab428852e35dc177a8c37a2df9dc9ccf768c65a adds Contents.json for
all .imageset in the material-design-icons that are required for asset
catalog compilation.

Add //third_party/material_design_icons:ic_touch_app as a dependency
of //remoting/ios/app/resources:assets as it is now fixed.

Bug: 734054
Change-Id: Icc4e484ba1ed22c1af06a4c074b3d0cf0ba00bf9
Reviewed-on: https://chromium-review.googlesource.com/576037Reviewed-by: Yuwei Huang <yuweih@chromium.org>
Commit-Queue: Scott Nichols <nicholss@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487928}

Bug: None
Change-Id: If5029a3b39078bfad7e7d4e8ea21f0ac2acfa8e1
Reviewed-on: https://chromium-review.googlesource.com/576077Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Mike Dougherty <michaeldo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487927}

Reland https://chromium-review.googlesource.com/c/574029 this time with MSAN
disabled.

This reverts commit efe60ec8.

Bug: 742958
Change-Id: I6fb6d84ce3686aac033a24a2b3dc790d81d8e44e
Reviewed-on: https://chromium-review.googlesource.com/577852
Commit-Queue: Hector Dearman <hjd@chromium.org>
Commit-Queue: Primiano Tucci <primiano@chromium.org>
Reviewed-by: Primiano Tucci <primiano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487926}

Change-Id: I00ae6625688bb740ecf0235c98e25f41296b359f
Reviewed-on: https://chromium-review.googlesource.com/553298Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Morten Stenshorne <mstensho@opera.com>
Cr-Commit-Position: refs/heads/master@{#487925}

any notification click.

The issue was that we opened settings on any OnNotificationClicked() callback.
However, this callback is invoked for *any* notification click, even when
notifications from another app/feature are clicked. To fix this issue, we now
explicitly check to make sure that the ID of the clicked notification matches
one of the IDs we care about.

This CL was originally:
https://chromium-review.googlesource.com/c/571856/

But was reverted as part of:
https://chromium-review.googlesource.com/c/576013/

My original CL is the initial upload on this CL, and my changes to fix the
breakage are in the subsequent patch version.

Bug: 672263, 738225
Change-Id: Ieae494d786f5fd3b50535d3d34f4a17ce30b0152
Reviewed-on: https://chromium-review.googlesource.com/577762Reviewed-by: Steven Bennetts <stevenjb@chromium.org>
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487924}

Provides a test FeatureEngagementTracker that makes all non-relevant conditions
true so you can test per-feature specific conditions/configurations.

Bug: 734132
Change-Id: I83e6f0be55870ae1eb812d0688d0cb416672ab3f
Reviewed-on: https://chromium-review.googlesource.com/570509
Commit-Queue: Catherine Chung <catherinechung@google.com>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Reviewed-by: Robert Liao <robliao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487923}

This is m61 version of last attempt: https://codereview.chromium.org/2906803002/

renamed
* maximize_mode_xxx files to tablet_mode_xxx
* Class name MaximizeModeXxx to TabletModeXxx
* variable names maximize_mode(xxx) to tablet_mode(xxx)
* comments "maximize mode" to "tablet mode"

BUG=726476
R=skuhne@chromium.org, sky@chromium.org, reveman@chromium.org
TBR=reveman@chromium.org
TEST=None

Rename MaximizeMode to TabletMode

This is m61 version of last attempt: https://codereview.chromium.org/2906803002/

renamed
* maximize_mode_xxx files to tablet_mode_xxx
* Class name MaximizeModeXxx to TabletModeXxx
* variable names maximize_mode(xxx) to tablet_mode(xxx)
* comments "maximize mode" to "tablet mode"

BUG=726476
R=skuhne@chromium.org, sky@chromium.org, reveman@chromium.org
TBR=reveman@chromium.org
TEST=None

Change-Id: I138134efe0b8af559d04c7682ee13b93e63cbbc6
Reviewed-on: https://chromium-review.googlesource.com/572945
Commit-Queue: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Scott Violet <sky@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487922}

BUG=

Change-Id: Idf2062ce75eccd76ee73fdded76628ba8819875f
Reviewed-on: https://chromium-review.googlesource.com/576054Reviewed-by: Julian Pastarmov <pastarmovj@chromium.org>
Commit-Queue: Georges Khalil <georgesak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487921}

This reverts commit af3f414f.

Reason for revert: Already found a lifecycle issue when called from DidMeaningfulLayout(). Hence reverting the CL before M61 branch.

Original change's description:
> Tentatively change TextIterator to CHECK clean layout, instead of DCHECK
> 
> This patch changes the DCHECK of clean layout in TextIterator to CHECK,
> so that we can catch remaining lifecycle issues.
> 
> Bug: 739060
> Change-Id: I0ee0a152b438d3d8e1f97de0acd2844b190ef703
> Reviewed-on: https://chromium-review.googlesource.com/575597
> Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#487454}

TBR=yosin@chromium.org,xiaochengh@chromium.org

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

Bug: 739060
Change-Id: I569f5cf9898e49fdf24de28635db370d0c8047f9
Reviewed-on: https://chromium-review.googlesource.com/577649Reviewed-by: Xiaocheng Hu <xiaochengh@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487920}

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/1195fa0a2590..7d6336b80e30

$ git log 1195fa0a2..7d6336b80 --date=short --no-merges --format='%ad %ae %s'
2017-07-19 loloangela Fixed errors related to bad-continuation

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: Ib308a2fca06144f1e93c12b402a7f525bc89f32a
Reviewed-on: https://chromium-review.googlesource.com/576486
Reviewed-by: <catapult-deps-roller@chromium.org>
Commit-Queue: <catapult-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487919}

Android lint has been failing due to API level mismatch between lint and
platform-tools (26 vs 25). This is causing new lint errors to be
introduced. Ignore the API mismatch for now and re-enable lint so it at
least catches all other issues.

API warning will be re-enabled after platform-tools is rolled.

Bug: 739746,746409
Change-Id: I339560498544462732300c5030381a0c67f25f1b
Reviewed-on: https://chromium-review.googlesource.com/577593Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Michael Thiessen <mthiesse@chromium.org>
Commit-Queue: Peter Wen <wnwen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487918}

This replaces the handwritten ConditionalFeaturesFor{Core,Modules}.cpp files with a script which reads IDL files looking for the appropriate extended attributes (just OriginTrialEnabled currently) and generates the required code to install those features at runtime.

BUG=615060

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

Bug: NONE
Change-Id: I4267bed9b5e2d47822dfe023c138417d3b60c643
Reviewed-on: https://chromium-review.googlesource.com/577399
Commit-Queue: Hans Wennborg <hans@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487916}

Bug: 685897
Change-Id: Ib08627d9b1f99baa7f50d97e30b7f6ba271d3495
Reviewed-on: https://chromium-review.googlesource.com/576707
Commit-Queue: Joshua Pawlicki <waffles@chromium.org>
Reviewed-by: Sorin Jianu <sorin@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487915}

Revert of Handle Java assert using a customized function in base (patchset #8 id:390001 of https://codereview.chromium.org/2971063003/ )

Reason for revert:
Broke compile on Cronet bots with error about "can't find referenced class
org.chromium.base.JavaExceptionReporter": https://build.chromium.org/p/chromium.android/builders/Android%20Cronet%20x86%20Builder%20%28dbg%29/builds/61854

Original issue's description:
> Create java_assertion_handler, a Java binary that takes in a JAR file
> and modifies the bytecode of classes.
>
> Replace the Java ASSERT with JavaExceptionReporter.assertFailureHandler
> that throws Exception.
>
> Output is written to a new JAR file.
>
>
> BUG=672945
>
> Review-Url: https://codereview.chromium.org/2971063003
> Cr-Commit-Position: refs/heads/master@{#487903}
> Committed: https://chromium.googlesource.com/chromium/src/+/33133e5767a9c1ca1f7e9defad2d7b3347065589

TBR=yfriedman@chromium.org,agrieve@chromium.org,zpeng@chromium.org,ranj@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=672945

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

Since we are not going to implement the send email feature soon, this CL
hides the send email button in the host instruction page.

This CL replaces the HostSetupFooterView with an empty view.

Bug: 736932
Change-Id: Id3ce3faea6f2352667904eb48d4e811587254d82
Reviewed-on: https://chromium-review.googlesource.com/575302Reviewed-by: Scott Nichols <nicholss@chromium.org>
Commit-Queue: Yuwei Huang <yuweih@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487913}

This CL adds unit test files for Xcode objects in
tools/gn/xcode_object.h, and specifically exercises class
instantiations and tests that it doesn't crash.

Bug: 742258
Change-Id: I234b0e643e718edf09a011c390c70c1d5ed0743f
Reviewed-on: https://chromium-review.googlesource.com/572702Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487912}

The original XCTest files search code was in pretty bad shape, not only
ridiculously long, but also not extensible enough to accommodate
XCUITest.

This CL re-factors it to clean it up and makes it more general.

Bug: 709289
Change-Id: I14c0ad95af8365e3bbbf1db899560d5cac103b7a
Reviewed-on: https://chromium-review.googlesource.com/576990Reviewed-by: Sylvain Defresne <sdefresne@chromium.org>
Commit-Queue: Yuke Liao <liaoyuke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487911}

This CL changes the logic for showing the Lightweight First Run
Experience to show the Lightweight First Run Experience for "Unbound
WebAPKs" only.

"Unbound WebAPKs" are WebAPKs whose package name does not start with
"org.chromium.webapk"

This CL adds AsyncInitializationActivity#shouldPreferLightweightFre().
Sub classes should override the method to request the Lightweight First Run
Experience.

BUG=738133

Change-Id: Icd7796c57d928751c595e6b02467b1054119546a
Reviewed-on: https://chromium-review.googlesource.com/563885Reviewed-by: Ted Choc <tedchoc@chromium.org>
Reviewed-by: Ganggui Tang <gogerald@chromium.org>
Reviewed-by: Xi Han <hanxi@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Yaron Friedman <yfriedman@chromium.org>
Commit-Queue: Peter Kotwicz <pkotwicz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487910}

- Creates a PaymentRequestCache class as KeyedService along with its
  factory to ensure only one instance of it exists per BrowserState.
- PaymentRequestCache maintains a map of web::WebState to a list of
  payments::PaymentRequest instances maintained for that WebState.
- The PaymentRequestCache instance can be accessed in browser tests via
  the instance of BrowserState in order to verify the correct creation
  and state of payments::PaymentRequest objects.

BUG=602666

Change-Id: I4aef5be49ab93261bcbebd6f742378bcca59938e
Reviewed-on: https://chromium-review.googlesource.com/577379Reviewed-by: Rouslan Solomakhin <rouslan@chromium.org>
Commit-Queue: mahmadi <mahmadi@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487909}

RuntimeCallStats creates TracedValue objects using the same set of
strings repeatedly. For this case, it makes sense to not create string
copies when adding an entry to a TracedValue. The current TracedValue
class defined in third_party/WebKit only has methods that create
copies of strings. This CL renames the existing methods to match
the base::trace_event::TracedValue method they call, and adds
methods that call the versions that do not make copies.
 

Bug: 724543
Change-Id: Ice4582c250247fc6caf1123de260fa0d1861c4bf
Reviewed-on: https://chromium-review.googlesource.com/550435Reviewed-by: Jeremy Roman <jbroman@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Adithya Srinivasan <adithyas@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487908}

Finish wiring up native extension bindings for service-worker based
contexts. Following the IPC refactoring, the only missing piece is
adding logic for getting the service worker contexts (which aren't in
the normal ScriptContextSet).

This makes ServiceWorkerTest.Events, ServiceWorkerTest.TabsCreate, and
ServiceWorkerAppTest.RegisterAndPostMessage pass with native bindings
enabled.

Bug: 653596
Change-Id: Ic09745caab760084a94d85b79f11add5d94d7a74
Reviewed-on: https://chromium-review.googlesource.com/570992
Commit-Queue: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Istiaque Ahmed <lazyboy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487907}

TBR=zmo@chromium.org
NOTRY=true

BUG=602688

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
Change-Id: I8d33707758586b53d0459d0e5ddc04396e5b5b93
Reviewed-on: https://chromium-review.googlesource.com/578047
Commit-Queue: Geoff Lang <geofflang@chromium.org>
Reviewed-by: Geoff Lang <geofflang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487906}

This is for services like UKM who want to report different field trial names.

Bug:746402
Change-Id: I0ca7cf7abb519583ecb35aa40a1ea672dfb80343

TBR=calamity@chromium.org,oysteine@chromium.org,nparker@chromium.org

Change-Id: I0ca7cf7abb519583ecb35aa40a1ea672dfb80343
Reviewed-on: https://chromium-review.googlesource.com/577634
Commit-Queue: Robert Kaplow <rkaplow@chromium.org>
Reviewed-by: Nathan Parker <nparker@chromium.org>
Reviewed-by: Alexei Svitkine (slow) <asvitkine@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487905}

There was a lot of data being reported as part of
the Event.ScrollUpdate.Touch UKM. For now we decided
to reduce the volume of the data by a simple sampling
scheme until we find a more permanent solution.

BUG=742363

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

and modifies the bytecode of classes.

Replace the Java ASSERT with JavaExceptionReporter.assertFailureHandler
that throws Exception.

Output is written to a new JAR file.

BUG=672945

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

The ChromeOS code in platform2/authpolicy has been sending
AuthPolicy.TimeToRunNetAdsGpo for a while now (see
https://chromium-review.googlesource.com/c/440084/),
but it was never added to histograms.xml.

BUG=chromium:676299
TEST=PCQ

Change-Id: I48ba1d1fed31a11dde3c8994d4e0adbef3c39ecc
Reviewed-on: https://chromium-review.googlesource.com/574912
Commit-Queue: Lutz Justen <ljusten@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487902}

In order to make the RenderFrameHost ready for simulating RenderFrame
originated IPCs, the RenderFrame needs to be initialized.

Bug: 728571
Change-Id: Ia80080a2e304dc2c37582599406b0cde3a22b027
Reviewed-on: https://chromium-review.googlesource.com/575237
Commit-Queue: Arthur Sonzogni <arthursonzogni@chromium.org>
Reviewed-by: Camille Lamy <clamy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487901}

This reverts commit c2fadc96.

Reason for revert: Added static initializer to MonochromePublic.apk.

Original change's description:
> Enable bounded mode logging on chrome://net-export.
> 
> There is an option to configure the maximum size of the NetLog. The
> default is 100MiB.
> 
> Bug: 679030
> Change-Id: I9260d7ee895d589c4789a350ca4fe1dfc8617b65
> Reviewed-on: https://chromium-review.googlesource.com/571514
> Commit-Queue: Eric Roman <eroman@chromium.org>
> Reviewed-by: Helen Li <xunjieli@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#487602}

TBR=eroman@chromium.org,xunjieli@chromium.org

Change-Id: I7d8fb8daee69e381097b441f3ed42bd24ae252d1
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 679030, 746430
Reviewed-on: https://chromium-review.googlesource.com/577197
Commit-Queue: Eric Stevenson <estevenson@chromium.org>
Reviewed-by: Eric Stevenson <estevenson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487900}

https://pdfium.googlesource.com/pdfium.git/+log/23c93ef6e3b9..4442d453f570

$ git log 23c93ef6e..4442d453f --date=short --no-merges --format='%ad %ae %s'
2017-07-19 janeliulwq Added testing flag --save-attachments that saves embedded attachments
2017-07-18 janeliulwq Removed public/ in include
2017-07-18 janeliulwq Basic APIs and tests for extracting attachments
2017-07-18 npm Remove JBig2_GsidProc class

Created with:
  roll-dep src/third_party/pdfium


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


TBR=dsinclair@chromium.org

Change-Id: Ib054d6b6f2f2f6b3945803e3275752af5ac95939
Reviewed-on: https://chromium-review.googlesource.com/577787
Reviewed-by: <pdfium-deps-roller@chromium.org>
Commit-Queue: <pdfium-deps-roller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487899}

Updates Contextual Search to correctly handle words in alphabets
with unreliable word-break characters. The Android method
Character.isIdeographic does not do what I thought it did!
Instead we now use a regular expression to scan a string
for non latin_basic characters.

Also updates the short/long word suppression to be less
confusing when applied to non-latin languages. Since these
characters in CJKV alphabets are data-dense the shortness rule
should not apply.

BUG=745970, 746097

Change-Id: I65527c614a2ae3a1e74df90a01f00f5e60fea197
Reviewed-on: https://chromium-review.googlesource.com/576909
Commit-Queue: Donn Denman <donnd@chromium.org>
Reviewed-by: Theresa <twellington@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487898}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/5e433d5a..67c78739

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

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

Revert this after M61 branches.

Bug: 737589
Change-Id: Icdd293b970b2577d785a6454d2bb14070457229e
Reviewed-on: https://chromium-review.googlesource.com/577848Reviewed-by: Rohit Rao <rohitrao@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487896}

(remove SK_SUPPORT_LEGACY_HQ_SCALER)

Change-Id: If5430b136027c50d01c9f832907b3cc7a518dea4
Reviewed-on: https://chromium-review.googlesource.com/568363
Commit-Queue: Florin Malita <fmalita@chromium.org>
Reviewed-by: Mike Reed <reed@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487895}

Windows Ash support was removed in 2016, so remove Windows code from
this test. Also remove the ash::test namespace, since we don't use
that anymore.

Bug: 373238
Test: chrome unit_tests
Change-Id: I7dba77222fc36b7b8a5a372e7ed883eb20077332
Reviewed-on: https://chromium-review.googlesource.com/576871Reviewed-by: Pavel Yatsuk <pavely@chromium.org>
Commit-Queue: James Cook <jamescook@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487894}

Adding more unit-test coverage to ChildProcessConnection.

Bug: 702316
Change-Id: I8ffee3636f944db7fa0e36fcc9a825aa1bd1766f
Reviewed-on: https://chromium-review.googlesource.com/569403
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Reviewed-by: Tommy Nyquist <nyquist@chromium.org>
Reviewed-by: Bo Liu <boliu@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487893}

BUG=670879

Change-Id: Ie108aa5a070020782cc304d5fdcbd1d51a7795ad
Reviewed-on: https://chromium-review.googlesource.com/576394Reviewed-by: Benjamin Pastene <bpastene@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Yuly Novikov <ynovikov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487892}