GitLab

Internal change: 154750107

BUG=715224

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

Decreased the number of missing header files by 1.

BUG=661774

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

Revert of [scheduler] Move some task types to suspendable task runner. (patchset #1 id:1 of https://codereview.chromium.org/2808273003/ )

Reason for revert:
Causing telemetry test failures:
https://bugs.chromium.org/p/chromium/issues/detail?id=717078&can=2&start=0&num=100&q=owner%3Ame&colspec=ID%20Pri%20M%20Stars%20ReleaseBlock%20Component%20Status%20Owner%20Summary%20OS%20Modified&groupby=&sort=&hotlist_id=

Original issue's description:
> [scheduler] Move some task types to suspendable task runner.
>
> Move several safe-looking task types to suspendable task runner.
> More task types will be moved to suspendable task runner, but
> we'll start with this set and will monitor any problems
> (there were some perfbot failures when all task queues were
> moved to suspendable task runner).
>
> R=haraken@chromium.org
> CC=skyostil@chromium.org
> BUG=702160
>
> Review-Url: https://codereview.chromium.org/2808273003
> Cr-Original-Commit-Position: refs/heads/master@{#464025}
> Committed: https://chromium.googlesource.com/chromium/src/+/78a049660074003b8e1e602b0e7783ca7f1f4ee0
> Review-Url: https://codereview.chromium.org/2808273003
> Cr-Commit-Position: refs/heads/master@{#467633}
> Committed: https://chromium.googlesource.com/chromium/src/+/c18764a33a595baab6ba35a1447e51c7c74b6e30

TBR=haraken@chromium.org,skyostil@chromium.org,altimin@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=702160

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

Last set of fixed tests in preparation for enforcing that mock host resolver isn't modified in a test body.

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

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

The 'Source map detected' infobar is created when a script with a sourcemap is
updated. When Persistence is enabled, this may happen each time the user
refreshes the page, resulting in many duplicate infobars.

This CL checks for an existing sourceMapInfobar before creating a new one, or
removes it if the updated script has no sourcemap.

BUG=716340

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

This CL adds SwapMetricsObserverMac which records macOS specific swap
metrics. Metrics includes:
- # of pages swapins/swapouts per seconds
- # of pages compressed/decompressed per seconds

These stats come from host_statistics64().

BUG=617492

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

This was seen on Sony Xperia Z5 (Adreno 430).

BUG=696126
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/2855583003
Cr-Commit-Position: refs/heads/master@{#468507}

The interface taking a raw pointer is deprecated and being removed.

BUG=690937

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

This CL relands the CL described below that made an Android bot
flakily fail content_browsertests.  I could not repro the issue on
an emulator locally, but it seems like this test stresses too much
(see e.g. https://crbug.com/653864 where the test started failing
flakily for no apparent reason).

In any case, landing this CL while disabling the test makes sense
since the new functionality is stressed in a number of other
tests in WebRtcMediaRecorderTest.

Also BUG=716691

Original CL description ------------------------------------------------
MediaRecorder: enable encode acceleration for VP8 in Android

This is a refry of https://crrev.com/2801803002 (as pointed out
in https://crrev.com/2836013006) that enables only VP8
hardware encoding in Android.

Tested via webrtc_media_recorder_browsertest.cc [1]

[1] https://cs.chromium.org/chromium/src/content/browser/webrtc/webrtc_media_recorder_browsertest.cc?q=webrtc+mediarecorder&sq=package:chromium&dr=CSs&l=27

BUG=638664

Review-Url: https://codereview.chromium.org/2852583002
Cr-Commit-Position: refs/heads/master@{#467879}
Committed: https://chromium.googlesource.com/chromium/src/+/6089c73618b3b5010bdb81441c6f5f58936f9e58

patch from issue 2852583002 at patchset 40001 (http://crrev.com/2852583002#ps40001)

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

TBR=dgrogan@chromium.org
BUG=679905

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

BUG=708056

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

Terminate the renderer if an excessive number of calls are made,
This is indicative of script in an infinite loop or being malicious.
It's better to crash intentionally than by running the system OOM
and interfering with everything else running on the system.

BUG=706432

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

More details in the bug.

TEST=manual
BUG=714265

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

BUG=708742

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

Decreased the number of missing header files by 18.

BUG=661774

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

1. Uses OnceCallback instead of Callback.
2. Changes the close callback to use a const& to avoid copying the string.
3. Adds an alias for base::OnceCallback<void(const base::Optional<std::string>&)>:
   WebShareTargetPickerCallback.

BUG=716956

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

TBR=staraz@chromium.org
BUG=709080

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

That significantly improves the startup performance: if a URL is loaded using
commandline flag, starting the network service process early could reduce the
timeToOnLoad number (from navigation start to onload event) by 0.5 ~ 1 second.

BUG=598073

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

Revert of Use new SafeBrowsing redirect tracking code in CWS pings. (patchset #6 id:140001 of https://codereview.chromium.org/2779643002/ )

Reason for revert:
Reverting in hopes of fixing 717315. See it for details.

Original issue's description:
> Use new SafeBrowsing redirect tracking code in CWS pings.
>
> The SafeBrowsing code tracks more types of redirects than the /net code which strictly adheres to spec and so can drop various types of redirects which can obscure the referrer chain.
>
> BUG=685905
>
> Review-Url: https://codereview.chromium.org/2779643002
> Cr-Commit-Position: refs/heads/master@{#468389}
> Committed: https://chromium.googlesource.com/chromium/src/+/39d1ffe7ba2c6fc8ebe6245472e0191b6f29aa9c

TBR=jialiul@chromium.org,rdevlin.cronin@chromium.org,robertshield@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=685905

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

Changes to the Infobar spec and how they lay out controls are coming.
Split off radio buttons into a separate layout that InfoBarControls
use and can be added by other layouts to prevent conflating changes
between them.

Also fixes up how the promo dialog (which used the RadioButtons)
assigns widths to some of its children.

Screenshots:
https://drive.google.com/corp/drive/u/0/folders/0B7c8ZkXVwskDYkxiQWNwS1JzR2c

BUG=712836,712833

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

The previous ignore logic was incorrect as it was both calling
create and set index.  Thus the zero tabs case was resulting in
switch tabs always.

Adds tests to catch regressions going forward.

BUG=717062

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

Last two usages in RFH:
- media interface registry
- java interface provider bridge

both replaced by ad-hoc InterfaceProvider impls

R=rockot@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

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

This CL fixes a problem when checking for if timezone autodetection should
startfor signin screen, when unset preference was treated as "false".

BUG=622555

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

Added trailing comments displaying the property api pointed to by all
the non-null entries of the CSSDescriptorIndices table in
CSSPropertyDescriptor.cpp. This improves the readability of the table.

Changes to generated files:
https://gist.github.com/BugsNash/44fa796f876d5b4727931a4b63cec386/revisions

BUG=668012

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

Depending on selected call frame, we only use one specific
context per debugger model to evaluate in while on pause.

BUG=716306

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

Previously, GpuSurfaceTracker would add accelerated widgets and java
Surfaces via separate API calls.  Now, a java surface is required
when registering an accelerated widget (ANativeWindow) on Android.

BUG=710230
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/2791723003
Cr-Commit-Position: refs/heads/master@{#468488}

Generated externs instruct maintainers to reference this doc to
regenerate them, so add the expected instructions here.

R=dbeam@chromium.org
BUG=none

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

Curry up the errors from argument parsing to signature parsing in order
to indicate to scripts which argument threw the error.  Update the
APISignature unittests to test the new behavior.

BUG=653596

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

This CL also fixes the log which coverts an EID to a hex string.

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

Currently, if a pref is changed and the pref service is enabled, the
entire value is sent to the pref service and other clients. For large
prefs used by extensions or content settings this can result
milliseconds of busy time on the UI thread.

ScopedDictionaryPrefUpdate tracks which components of a pref are changed
so only the changes need to be sent to the pref service and other pref
clients.

Bug: 654988
Change-Id: Ied5dc6d0fadaa04817330fddce08efdc8b1d0fce
Reviewed-on: https://chromium-review.googlesource.com/476370Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Commit-Queue: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#468484}

https://pdfium.googlesource.com/pdfium.git/+log/a0a69233cafa..40baddef7fda

$ git log a0a69233c..40baddef7 --date=short --no-merges --format='%ad %ae %s'
2017-05-01 tsepez Remove more |new|s, part 6

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/58c56c18..447fde34

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/2850283002
Cr-Commit-Position: refs/heads/master@{#468482}

Revert of Reland RedirectNonUINonIOBrowserThreads to TaskScheduler by default on trunk (patchset #1 id:1 of https://codereview.chromium.org/2856583002/ )

Reason for revert:
Possible failure: https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.chromiumos%2FLinux_ChromiumOS_Tests__dbg__1_%2F25521%2F%2B%2Frecipes%2Fsteps%2Fbrowser_tests%2F0%2Flogs%2FCaptivePortalWindowTest.OnRedirected%2F0

Original issue's description:
> Reland RedirectNonUINonIOBrowserThreads to TaskScheduler by default on trunk
>
> This reverts commit f07732ec
> (https://codereview.chromium.org/2791933004)
> and reapplies b6d0c9a0
> (https://codereview.chromium.org/2690183002/).
>
> BUG=653916, 707986
> TBR=jam@chromium.org, rkaplow@chromium.org
> Previously reviewed at https://codereview.chromium.org/2690183002/
>
> Review-Url: https://codereview.chromium.org/2856583002
> Cr-Commit-Position: refs/heads/master@{#468376}
> Committed: https://chromium.googlesource.com/chromium/src/+/3e27cb6e501ae55d614b56d566487590518ec223

TBR=jam@chromium.org,rkaplow@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=653916, 707986

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

BUG=635633
TEST=Open chrome:md-settings, look at the DevTools console, no more warnings.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Remove a host_resolver()->ClearRules call in a test body and replace it with an AddRule call that happens in SetUpOnMainThread.

This allows us to enforce that mock host resolver isn't modified in a test body.

BUG=713847

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

The minidump uploader attempts to sanitize multipart boundary lines present in
minidumps, but was not checking for invalid characters. Only alphanumeric
characters, plus dash, are valid.

BUG=712420
TEST=MinidumpUploadCallableTest
R=gsennton@chromium.org

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

BUG=640835
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

Cr-Commit-Position: refs/heads/master@{#468476}

Before the change, an update check error would terminate the update but leave the
components unhandled. Therefore, for an external observer, it would appear that the
components were stuck on checking for updates.

BUG=687231

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

BUG=690179

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