GitLab

BUG=703369

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

https: //codereview.chromium.org/2877553003/)
Review-Url: https://codereview.chromium.org/2881603002
Cr-Commit-Position: refs/heads/master@{#471079}

Before this CL, the ConditionValidator::MeetsConditions(...) method only
returned a bool value whether all conditions were met. This made it
harder to track what went wrong in a generic way across all validators,
so this CL changes the return value to be a struct that contains
information about all conditions, and whether they were met.

In addition, to prepare for an upcoming CL that will need to use the
current day to figure out whether conditions are met, that parameter
is added in the current CL.

BUG=706309

Change-Id: I73815d88facfc1fbee9e6a0058790cd4b73dc714
Reviewed-on: https://chromium-review.googlesource.com/502731Reviewed-by: David Trainor <dtrainor@chromium.org>
Commit-Queue: Tommy Nyquist <nyquist@chromium.org>
Cr-Commit-Position: refs/heads/master@{#471078}

BUG=666502
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

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

After ES6 modules loading for PaintWorklet (issue 627945) is introduced, layout
tests for PaintWorklet are no longer able to run on non-http environment because
ES6 modules does not support "file://" url (See https://crbug.com/594639#c86).

This CL moves the tests in inspector/console/ to http/tests/inspector/console/
to enable them to run with module loading.

BUG=719857

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

Since the dep is also conditional, not doing this fails gn check.
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/2876873002
Cr-Commit-Position: refs/heads/master@{#471075}

BUG=522168

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

ScopedTaskEnvironment allows usage of ThreadTaskRunnerHandle and
base/task_scheduler/post_task.h within its scope. It should be
instantiated in everytest that uses either of these APIs
(i.e. no test should instantiate a MessageLoop directly).

Switching from MessageLoopForIO to ScopedTaskEnvironment in
NetTestSuite allows Scoped(Async)TaskScheduler to be removed
from individual net unittests.

Motivation for ScopedTaskEnvironment can be found in:
https://docs.google.com/document/d/1QabRo8c7D9LsYY3cEcaPQbOCLo8Tu-6VLykYXyl3Pkk/edit

BUG=708584

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

Broken in 40a227b7, we currently don't build for subrepo (and the script
fails).

BUG=695182

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

BUG=704259

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

It was written assuming its only caller would be Blink, but that's no longer the
case. Separate the Blink-specific stuff into a new Blink documentation file.

BUG=NONE
R=haraken@chromium.org

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

https://skia.googlesource.com/skia.git/+log/eb7a693e0f6d..ec8f8b064340

$ git log eb7a693e0..ec8f8b064 --date=short --no-merges --format='%ad %ae %s'
2017-05-11 brianosman Fix GPU text colors when color spaces are involved
2017-05-11 kjlubick Isolate CIPD assets for RPI tasks
2017-05-10 brianosman Remove SkCrossContextImageData and all support code
2017-05-11 mtklein hone down the SK_FORCE_RASTER_PIPELINE_BLITTER test bot

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=stani@chromium.org

Change-Id: I4329db0e7904a7a2691f3a769a34b925f01e70c2
Reviewed-on: https://chromium-review.googlesource.com/503488Reviewed-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@{#471069}

The context menu of the shelf will cover part of the icon in the shelf.
And the coordinate adjustment from |MENU_ANCHOR_BOTTOMCENTER| is not suit
for left and right shelf.

changes in this cl,
1.Adjust the position of the context menu opened by shelf icon. Make it
align to top edge of the shelf.
Context menu of touch events should be put in the middle area.

BUG=710601

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

BUG=721465
TBR=kkhorimoto@chromium.org

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

BUG=None

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

https://swiftshader.googlesource.com/SwiftShader.git/+log/f34d1ac..9ed48ba

Fixes LTO causing hard crash on Linux official builds.

BUG=720933

TBR=kbr@chromium.org

TEST=bots

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel

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

BUG=skia:3191

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

Test was disabled due to flakyness.  Re-enabling to diagnose current situation.

TBR=sullivan@chromium.org
BUG=713260

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

BUG=719448

CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_cronet_tester

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

Bug: 700092
Change-Id: I7fd398d1e25b77493eef609cbf818747216cd140
Reviewed-on: https://chromium-review.googlesource.com/479931
Commit-Queue: Jeff Carpenter <jeffcarp@chromium.org>
Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Cr-Commit-Position: refs/heads/master@{#471061}

BUG=676220
R=peter@chromium.org

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

CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

We were recording the first character of the selection to use as
a hint for triggering.  We don't currently have permission to send
this to Ranker so it's being removed for now, and the histogram being
marked obsolete.

We don't yet have permission to send any content signals to Ranker
so removing some promising ones like word-length from the send-to-ranker
list but keeping the underlying data collection for now.

Remove the entire blacklist and obsoleting the associated histogram.

BUG=720193

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

BUG=717538,715704

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

Memory-mapped files cause crashes if there are I/O problems. By ensuring
that the file is fully realized on disk, we eliminate a disk-full condition
from causing a crash down the road.

BUG=715523, 717331

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

Revert of [PageLoadMetrics] Reenable AdsMetrics and handle case where navigation aborts (patchset #7 id:340001 of https://codereview.chromium.org/2861433005/ )

Reason for revert:
Test is flaky, see https://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_rel_ng/builds/452154

Original issue's description:
> [PageLoadMetrics] Reenable AdsMetrics and handle case where navigation aborts
>
> It turns out that a frame can abort and continue to load resources even if it
> has no previous navigation. This happens if a doc.write overwrites the frame
> navigation while the navigation is still provisional.
>
> This CL now labels all finished frame navigations (regardless of success) and
> removes an incorrect dcheck.
>
> CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
>
> BUG=708570
>
> Review-Url: https://codereview.chromium.org/2861433005
> Cr-Commit-Position: refs/heads/master@{#470701}
> Committed: https://chromium.googlesource.com/chromium/src/+/6f3a10ddea2ee602ff61267ce3ed6a60c8e6ff71

TBR=bmcquade@chromium.org,csharrison@chromium.org,jkarlin@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=708570

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

This assertion adds no value, and can trigger warnings (and errors if
you build with -Werror). I think it should be removed. If it is really
important, a comment where the variable is defined would suffice.

Patch submitted upstream:

https: //github.com/gperftools/gperftools/pull/885
Change-Id: Ife6de127928bfdd0c3861b65cdeb8e5ccbfee16d
Reviewed-on: https://chromium-review.googlesource.com/503034Reviewed-by: Will Harris <wfh@chromium.org>
Commit-Queue: Mostyn Bramley-Moore <mostynb@opera.com>
Cr-Commit-Position: refs/heads/master@{#471054}

BUG=676220
R=peter@chromium.org

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

Revert of Change TBR on import CLs to jeffcarp. (patchset #1 id:1 of https://codereview.chromium.org/2827093004/ )

Reason for revert:
Switching TBR back to qyearsley

Original issue's description:
> Change TBR on import CLs to jeffcarp.
>
> Review-Url: https://codereview.chromium.org/2827093004
> Cr-Commit-Position: refs/heads/master@{#466348}
> Committed: https://chromium.googlesource.com/chromium/src/+/91d68416a8808790a8d721a5ebe0accdafbcc066

TBR=jeffcarp@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.

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

Make Metalayer mode visibility dependent on the container side
setting.

BUG=b:37997351
TEST=Manual ("Metalayer mode" visible only when screenshots
  are enabled on the container side).

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

This change updates OWNERS files for device sensors-related directories
to point to the Blink>Sensor and Blink>Sensor>DeviceOrientation
components and device-dev@chromium.org team mailing list.

BUG=None

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

BUG=677365
R=tbansal@chromium.org

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

The folder will be used to host benchmarks for measuring perf of Chrome with
the network service enabled.

Eventually we won't need this folder when the network service is enabled by
default and the old network-in-browser-process code path is removed.

BUG=717738,598073
R=nednguyen@google.com

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

BUG=714769

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

Reland of build: Enable optimize_for_size unconditionally. (patchset #1 id:1 of https://codereview.chromium.org/2881503002/ )

Reason for revert:
Relanding now that the msan issue is fixed: https://codereview.chromium.org/2876693003

Original issue's description:
> Revert of build: Enable optimize_for_size unconditionally. (patchset #1 id:1 of https://codereview.chromium.org/2864383003/ )
>
> Reason for revert:
> Causes failures on a MSAN bot.
> https://uberchromegw.corp.google.com/i/chromium.memory/builders/Linux%20ChromiumOS%20MSan%20Tests/builds/440
>
>
> Original issue's description:
> > build: Enable optimize_for_size unconditionally.
> >
> > This change causes us to favor size over speed on Linux and Mac,
> > which aligns the build config for those platforms with that of the
> > other supported platforms, and should reduce the binary size impact
> > of enabling ThinLTO. This change is expected to reduce binary size
> > for Linux official builds by about 15%.
> >
> > There may be unacceptable perf regressions associated with this
> > change, but the perf bots should at least let us know where those
> > regressions are. I plan to monitor the Linux and Mac perf bots once
> > it lands.
> >
> > BUG=660216
> > R=thakis@chromium.org
> >
> > Review-Url: https://codereview.chromium.org/2864383003
> > Cr-Commit-Position: refs/heads/master@{#470606}
> > Committed: https://chromium.googlesource.com/chromium/src/+/c1269ce7fec8568a1789e07b2b5bb3b630a67f5f
>
> TBR=thakis@chromium.org,pcc@chromium.org
> # Skipping CQ checks because original CL landed less than 1 days ago.
> NOPRESUBMIT=true
> NOTREECHECKS=true
> NOTRY=true
> BUG=660216
>
> Review-Url: https://codereview.chromium.org/2881503002
> Cr-Commit-Position: refs/heads/master@{#470793}
> Committed: https://chromium.googlesource.com/chromium/src/+/2a6f440fa84a584d916254f9a4c4e4d83c53435f

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

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

The libtxc_dxtn.so library supports loading textures in S3TC
format. This change disables code in mesa that loads libtxc_dxtn.so
if msan is enabled, as the library will be loaded from a system
directory, which may cause spurious msan reports because msan expects
most libraries in the process to be compiled with msan.

For some reason, msan only appears to break when compiling with -Os,
as revealed by a recent attempt [0] to enable -Os on Linux and Mac.

[0] https://codereview.chromium.org/2864383003/

BUG=660216
R=senorblanco@chromium.org,thakis@chromium.org

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

No behavior change.

BUG=675877

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

https://pdfium.googlesource.com/pdfium.git/+log/4da1e7623c52..dbc3d3e1e0b2

$ git log 4da1e7623..dbc3d3e1e --date=short --no-merges --format='%ad %ae %s'
2017-05-11 dsinclair Fixup Initialize spelling
2017-05-11 npm Stop rendering if a span length overflowed in AGG
2017-05-11 npm LibOpenJPEG: undefined shift in opj_t1_dec_clnpass

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

BUG=720605

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

Replaces these APIs with IncomingBrokerClientInvitation usage.
This new API more accurately reflects the intended design of
the system and forces developers to select a transport protocol
when joining a Mojo embedder process graph.

BUG=696031
TBR=jam@chromium.org

Change-Id: I90460badec6622447432b0bb76b202c8a14270ed
Reviewed-on: https://chromium-review.googlesource.com/502548Reviewed-by: Ken Rockot <rockot@chromium.org>
Reviewed-by: Jay Civelli <jcivelli@chromium.org>
Commit-Queue: Ken Rockot <rockot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#471041}