GitLab

The canvas element simply mirrors the video element by grabbing frames from it.

Removing these 3 pages should reduce benchmark_duration for tough_video_cases by ~17%.

BUG=710253

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

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}

This roll includes:
  - [inspector_protocol] added StringUtil::toDouble method as requirement [1]

[1] https://codereview.chromium.org/2843223005/

BUG=712610
R=dgozman@chromium.org

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

When I moved the multi window swap interval logic, I changed a condition
by mistake. With that change the swap interval is reset to 1 for the
first swap in a new swap generation. I suspect that caused the input
latency regression although I can't reproduce it in local testing.

R=kbr
BUG=714902

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: If7f7ae4f824b39dade4cd9d81e68ce47cfc8f38d
Reviewed-on: https://chromium-review.googlesource.com/487875Reviewed-by: Kenneth Russell <kbr@chromium.org>
Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org>
Cr-Commit-Position: refs/heads/master@{#467877}

https://pdfium.googlesource.com/pdfium.git/+log/f452f56021f2..66568bcd683d

$ git log f452f5602..66568bcd6 --date=short --no-merges --format='%ad %ae %s'
2017-04-26 thestig Assert CPDF_Pattern always has a valid CPDF_Document pointer.
2017-04-27 tsepez Zero 4 byte variable before reading 3 bytes into it
2017-04-27 tsepez Add barcode test skeleton
2017-04-27 thestig Remove CBC_HighLevelEncoder::illegalCharacter().

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

Since we started using LayoutOpportunity iterator for positioning BFC
fragments, we can't longer rely on the assumption that
FindLayoutOpportunityForFragment will be used for floats only.

List of changes:
- Change FindLayoutOpportunityForFragment to check if fragment's block
  size fits into a layout opportunity.
- Change NGInlineLayoutAlgorithm to avoid adding font's descent to
  line_bottom if the text only includes object replacement characters
  I need it to fix floats-wrap-top-below-002l-ref.xht.

BUG=635619

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

The vertical_align_ field in ComputedStyle currently has generated
storage, but not copy and compare. This patch makes it use generated
copy and compare. We are able to do this now because we have added
the 'has_custom_compare_and_copy' parameter in a previous patch.

Diff of generated files:
https://gist.github.com/183fa3073f5de347a685b1834913ee2e/revisions

BUG=628043

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

Because it will be used from outside ModuleScriptLoader when we support
inline scripts.

This CL
- Renames existing ModuleScript::Create() (which is a simple wrapper of
  the constructor) to CreateForTest(),
- Moves ModuleScriptLoader::CreateModuleScript() to new
  ModuleScript::Create(), and
- Reorders the parameters to match with the spec.

BUG=594639, 715369

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

Like the WTF::String change, this leaves the the Find() overload that
takes a StringView, start offset, and a case sensitivity option for a
followup CL.

Bug: 709815
Change-Id: I8f98fe0b41eb99a525ca608690c8ae0bdcebf6a6
Reviewed-on: https://chromium-review.googlesource.com/489368Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#467872}

Native layer is already calculating some frame info about
top control (position + update flag). This CL passes
the values to Java layer so that it won't repeat those
operations again.

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

Revert of Only do Smart Select if one word is selected. (patchset #4 id:60001 of https://codereview.chromium.org/2838023002/ )

Reason for revert:
Null pointer crash http://crbug.com/715526

BUG=715526

Original issue's description:
> Only do Smart Select if one word is selected.
>
> There is a bug where after "Select All" Smart Select will suggest a different
> (smaller) selection. This CL checks if the selection only has one word. Since
> longpress/double tap only selects one word, if we have multiple words we can assume
> it comes from "Select All" and not do Smart Select.
>
> BUG=714106
>
> Review-Url: https://codereview.chromium.org/2838023002
> Cr-Commit-Position: refs/heads/master@{#466856}
> Committed: https://chromium.googlesource.com/chromium/src/+/03f5058e764e8b2b01d571f29426075a83f6049b

TBR=timav@chromium.org,amaralp@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=714106

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

Indicates whether an error has been encountered by the interface endpoint client. If true, method calls made on this interface will be dropped (and may already have been dropped).

BUG=695635

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

When logging a URL with a port number in the console, 'http://localhost:8000',
our linkifier incorrectly treats the '8000' port number as a line number.

Using our existing granular URL regexes, this patch makes the linkifier ignore
the URL part before the 'path' when looking for a line/column number.

BUG=701865

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

I think this used to be built by the old Windows official builders, but
I have no idea if it was ever even used, and AFAICT, it hasn't ever been
built by the current (PGO) builders, and hasn't been missed. It seems we
don't really need it (especially not for the official continuous
builders, which appear to be the only ones that currently build the
"chrome_official_builder" target).

R=dimu@google.com, dpranke@google.com

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

This enables module scripts in Blink behind a flag.

This CL implements module-script-related steps of "prepare a script"
in ScriptLoader and thus makes the module implementation code paths that
are committed previously to be executed, if the runtime flag is enabled.

This CL also reorganizes the test expectations for module-related tests.

BUG=594639

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

This property is necessary because the networking stack must show a dialog to users the first time they connect to a given host device which alerts the user that tethering will use mobile data and battery.

This change adds the new property to the networkingPrivate IDL file and pipes it through from NetworkState. Currently, the value is hard-coded to "false", but this value will be propagated through the network stack from the Tether component in a follow-up CL.

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

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

accepted previous save card prompt.

BUG=715617

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

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

I'm going to build a general persistence mechanism for //net instead of
tediously plumbing out to the profile's prefs (which gets really messy
with isolated app contexts and such), so I'm removing the old
persistence stuff until that's done.

BUG=704259

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

This field represents the CPU temperature at each thermal zone in
degrees Celsius.

BUG=chromium:709102

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

This CL also changes UnlockManager to use the new interface for ProximityMonitor.

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

BUG=715674

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

This is the next iteration of the modal dialog based on UX team's
latest proposal. Strings are still not finalized.

BUG=690020

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

Add a metric to track the type of encryption used by chromeos user
homedirectories.
This histogram is added in chromeos at https://chromium-review.googlesource.com/c/486439/

BUG=714926
TEST=./validate_format.py && ./pretty_print.py

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

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

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

For everything else we should rely on SAC::DidRequestShowFromMainThread.

Additionally, ensure that show-from-main requests are processed in synchronous
compositing mode (which commits directly to the active tree).

BUG=606395
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

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

Implement request id in PaymentDetailsInit as well as id
getter in PaymentRequest.

In order for this functionality to make sense we pass the
id to the payment app and make sure to set it in the
PaymentResponse through the requestId attribute.

Added PaymentRequestIdTest as a payment integration test for verifying
the PaymentResponse contains the free-form identifier specified in
PaymentDetailsInit.

Added PaymentRequestTest.DetailsIdIsSet unit test to verify that
PaymentDetailsInit.id is reflected in PaymentRequest.id.

Added external/wpt/payment-request/payment-request-id.https.html to test that
PaymentDetailsInit.id is reflected in PaymentRequest.id and that not providing
an id generates a UUID.

Added manual test external/wpt/payment-request/payment-request-response-id.html
for verifying the PaymentResponse contains the free-form identifier specified
in PaymentDetailsInit.

Intent to Implement and Ship thread:
https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/WaHAg3wh7jw/vWAqxSnOCgAJ

BUG=701254
TEST=PaymentRequestIdTest, DetailsIdIsSet

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

https://chromium.googlesource.com/external/github.com/catapult-project/catapult.git/+log/770ad6af2778..9c9ac13a2b02

$ git log 770ad6af2..9c9ac13a2 --date=short --no-merges --format='%ad %ae %s'
2017-04-27 nednguyen [Telemetry] Enable more useful error stack for native crash with faulthandler tool

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

Revert of Roll googletest to 1.8.0. (patchset #7 id:340001 of https://codereview.chromium.org/2779193002/ )

Reason for revert:

Findit(https://goo.gl/kROfz5) identified CL at revision 467833 as the
culprit for failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2I1MzlmYTExMjNhMTllNjQ5OWY1ODU4MmRhNTVjYTVhZjlkZjk0YTQM

Original issue's description:
> Roll googletest to 1.8.0.
>
> GoogleTest (gtest) and GoogleMock (gmock) are now hosted into the same
> googletest repository. In order to cope with this, the googletest
> repository is now sourced at third_party/googletest.
>
> The file/directory layout of Google Test is not yet considered stable.
> To minimize disruption while Google Test stabilizes, Chromium code will
> be insulated from third_party/googletest.
>
> * testing/gtest/include/gtest/ and testing/gmock/include/gmock have
>   been populated with headers that forward into the appropriate locations of
>   third_party/googletest
>
> * testing/BUILD.gn has been populated with the targets
>   //testing/gtest(:gtest_main) and //testing/gmock(:gmock_main), which
>   depend on the appropriate //third_party/googletest targets.
>
> All Chromium code should keep depending on the targets and headers
> in testing/{gtest,gmock} for now.
>
> BUG=630705
> TBR=rkc
>
> Review-Url: https://codereview.chromium.org/2779193002
> Cr-Commit-Position: refs/heads/master@{#467833}
> Committed: https://chromium.googlesource.com/chromium/src/+/b539fa1123a19e6499f58582da55ca5af9df94a4

TBR=thakis@chromium.org,dpranke@chromium.org,agl@chromium.org,rkc@chromium.org,skobes@chromium.org,pwnall@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=630705

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

This CL adds support for invoking a callback when the underlying model
has finished loading. In addition, it exposes the restul of the
initialization as a boolean method.

The result of the initialization is always posted back, even if the
result is already known.

The CL implements both the C++ and the Java bridge implementation of
this.

BUG=706309

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

Basically, it is desired to see how many feedbacks are filed from the sad
tab page relative to other sources.

BUG=715682

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

Changes:
Add a window property for set/get whether an aura window is considered activatable for pointer window activation. Modify CanFocusWindow in FocusRules to allow optional check for ui::Event.
This is a stage one implementation for crbug.com/657525.

BUG=657525
TEST=added test coverage

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

Related CLs:
https://codereview.chromium.org/2842333002/
https://codereview.chromium.org/2843223002/

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

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

Cryptohome.DircryptoMigrationEndStatus histogram should use the
DircryptoMigrationEndStatus, not DircryptoMigrationStartStatus.

BUG=715870
TEST=./validate_format.py && ./pretty_print.py

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

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/227fee38..71c1795a

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

When it will only be used for a single frame, there is no need to
fallback for complex frames. This restores behaviour for CSSPaint
module to before https://codereview.chromium.org/2833593002/.

R=ikilpatrick@chromium.org, pdr@chromium.org
TBR=senorblanco
BUG=671433

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

ProcessStatsStub is an interface to receive process resource usage information
from a ProcessStats.

ProcessResourceUsage is a structure to store the resource usage of one process.
AggregatedProcessResourceUsage is a structure to store the resource usgaes of
several processes.

ProcessStatsAgent is an interface to retrieve resource usage of one process.
CurrentProcessStatsAgent is a ProcessStatsAgent implementation to retrieve
resource usage from current process.
ForwardProcessStatsAgent receives resource usage from a third-party and forwards
the latest usage as a ProcessStatsAgent.

ProcessStatsSender is a class to regularly report resource usages from various
ProcessStatsAgent implementations to a ProcessStatsStub. It merges several
non-empty ProcessResourceUsage into one AggregatedProcessResourceUsage.
This is typical useful on Windows: we need to calculate the resource usages from
both network process and desktop process.

R=sergeyu@chromium.org, jamiewalch@chromium.org
BUG=650926

Review-Url: https://codereview.chromium.org/2775983003
Cr-Original-Commit-Position: refs/heads/master@{#467753}
Committed: https://chromium.googlesource.com/chromium/src/+/5928f440f84065ab45f848dcd135e3c2f52ca5bb
Review-Url: https://codereview.chromium.org/2775983003
Cr-Commit-Position: refs/heads/master@{#467844}

This CL hides the "Site setting" action when notifications are shown by WebAPKs.
The "additional settings in the app" link will be removed from the notification
settings for WebAPKs in the System UI in Android O. Before Android O, the
settings gear to open app-specific settings shown on the rear of a notification
following a long-press will be gone as well.

BUG=654539

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

BUG=661774

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

https://chromium.googlesource.com/external/khronosgroup/webgl.git/+log/40d75ac..f7157c2

BUG=696223,715696

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