GitLab

https://chromium.googlesource.com/chromiumos/chromite.git/+log/74d7d171a121..bec9b446c412

$ git log 74d7d171a..bec9b446c --date=short --no-merges --format='%ad %ae %s'
2018-04-17 gmeinke Make parallel_emerge save install plan

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: I4e3e42dbdf5b7fcc6a583707a6ae30a5cedb474a
Reviewed-on: https://chromium-review.googlesource.com/1032071
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@{#554319}

See go/closure-compile-gn for details.

This CL converts the Closure Compilation of chrome://settings from GYP to
GN, and fixes a number of type-checking issues that were somehow
suppressed due to the way the compiler used to be run.

A local closure compile can be run by setting closure_compile = true in
args.gn, and then running ninja -C out/Default webui_closure_compile.
A more localized compile may look like:
ninja -C out/Default chrome/browser/resources/settings:closure_compile

Bug: 632206
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ib1957d6e0d7bddf2f108197c60c1ca9d9b8f74ca
Reviewed-on: https://chromium-review.googlesource.com/1023520
Commit-Queue: calamity <calamity@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554318}

When the app was unloaded, it seems loaded right before sending
the onLaunched event. Therefore asynchronously adding the listener will
miss the first event, being unable to respond for the user action in
this case.
https://developer.chrome.com/apps/app_lifecycle
The issue was introduced by:
https://chromium-review.googlesource.com/c/chromium/src/+/1018624

Bug: 837251
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: I7d613f9e877a39b3db0e92919a83d05bc45a0e06
Reviewed-on: https://chromium-review.googlesource.com/1031752
Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
Reviewed-by: Yuki Awano <yawano@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554317}

As Crostini is not supported on guest and ephemeral profiles, we change
IsExperimentalCrostiniUIEnabled() into IsCrostiniUIAllowedForProfile()
so we can avoid creating the various controllers and builders, as well
as not showing UI in the settings page. This makes us less likely to
assume a CrostiniRegistryService exists when one actually doesn't.

This is a reland of https://crrev.com/c/1029934, which happened to land
at the same time as https://crrev.com/c/1013824 causing a compile fail.

TBR=xiyuan

Bug: 836187
Change-Id: Ifefe467d1135388fa1a25c2fed22d34c09108ddb
Reviewed-on: https://chromium-review.googlesource.com/1032131Reviewed-by: Timothy Loh <timloh@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554316}

TokenizedChunkQueue is a deprecated optimization where it reduced number
of tasks being posted from BackgroundHTMLParserThread to the main thread.

Now that we actually run the BackgroundHTMLParser on the main thread,
we no longer need this optimization, we can queue the speculation anytime
we want.

Bonus: Now that we have move semantics, just use CompactHTMLToken directly, not via unique_ptr.

Bug: 689702
Change-Id: I0a219b0baba79a4293d8e8bd09300837c1c6bf4c
Reviewed-on: https://chromium-review.googlesource.com/1029552Reviewed-by: Charlie Harrison <csharrison@chromium.org>
Reviewed-by: Yoav Weiss <yoav@yoav.ws>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Kouhei Ueno <kouhei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554315}

Bug: 822507
Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation
Change-Id: Ieee1606155667561c88fc8360eeafb4067361e5d
Reviewed-on: https://chromium-review.googlesource.com/1032112Reviewed-by: calamity <calamity@chromium.org>
Commit-Queue: Nicholas Verne <nverne@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554314}

This reverts commit 673c3aa4.

The original fix missed to solve other flaky places. The new flaky test seems to
be due to not waiting long enough as well. This CL is to fix them.

Original CL description:

> The flaky failure of the test seems to be due to not waiting long enough after
> resuming work again. This Cl changes the workaround logic to actually wait until
> the javascript work again (or let the test fail by timing out otherwise).

Bug: 832191
Change-Id: I2e36af87b341d6593dc2f10095f57742e37d5aad
Reviewed-on: https://chromium-review.googlesource.com/1031790Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Gyuyoung Kim <gyuyoung.kim@lge.com>
Cr-Commit-Position: refs/heads/master@{#554313}

As std::mem_fun is deprecated in C++11 and removed in C++17, we need to
update it to compile the code in C++17.

Bug: 752720
Change-Id: I8f1c6bb8d6b302ec20799b6d2d636f7ac4d62e2b
Reviewed-on: https://chromium-review.googlesource.com/1032111Reviewed-by: Ryo Hashimoto <hashimoto@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554312}

Scheduler classes are being renamed and relocated as part of scheduler
architecture refactoring.

The header file is moved to its new location: scheduler/public, since
the file is referred from Blink.

Bug: 826203
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I1c28c5c59ea8a2efd42eb896d89cd7539df4f609
Reviewed-on: https://chromium-review.googlesource.com/1029794Reviewed-by: Alexander Timin <altimin@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Yuta Kitamura <yutak@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554311}

https://chromium.googlesource.com/angle/angle.git/+log/b221486a8dfd..467c15f970b9

$ git log b221486a8..467c15f97 --date=short --no-merges --format='%ad %ae %s'
2018-04-24 jiawei.shao Use ShaderMap in ProgramD3D - Part II
2018-04-17 jmadill Optimize ValidateDrawAttribs: Part 3.
2018-04-24 geofflang Don't test for device loss in SwapBuffers and MakeCurrent.
2018-04-26 courtneygo Cleanup for "Add angle library apk"

Created with:
  roll-dep src/third_party/angle
BUG=chromium:621240


The AutoRoll server is located here: https://angle-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.


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

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

Bug: 792546
Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo
Change-Id: Ib8712b436e3099ad3e2379ce1835648f19fb380e
Reviewed-on: https://chromium-review.googlesource.com/1022975
Commit-Queue: Łukasz Anforowicz <lukasza@chromium.org>
Reviewed-by: Nick Carter <nick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554309}

There were race conditions related to the asynchronous task that is
posted in AutofillManager. Due to a recent refactor putting
StartUploadProcess in an isolated function, we can enforce the run loop
more consistently. Gets rids of RunLoop-related functions that were
exposed in the TestAutofillManager.

Bug: None
Change-Id: I778fe050bba261a178e44f5792f02239560fa303
Reviewed-on: https://chromium-review.googlesource.com/1030789
Commit-Queue: Mathieu Perreault <mathp@chromium.org>
Reviewed-by: Sebastien Seguin-Gagnon <sebsg@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554308}

Now that a fix for crbug.com/647286 has landed in r552619, the additional icons
added for 20x20 sizes in the omnibox can be merged back into the same icon file.
Note that only the 20x20 HTTP icon (http.icon) is merged, because scaling down
the existing 32x32 HTTP icon for use at 20x20 looks blurry. All other icons in
this patch look fine scaled.

Screenshots -
https://drive.google.com/file/d/1ROQ7L7fpb04wQ1MJZvx3mcKYnaFd5rwY/view?usp=sharing

Bug: 801583, 647286
Change-Id: I5a7adedbe0358aaa321210cd601a88a782b51a76
Reviewed-on: https://chromium-review.googlesource.com/1023357
Commit-Queue: Patti <patricialor@chromium.org>
Reviewed-by: Evan Stade <estade@chromium.org>
Reviewed-by: Peter Kasting <pkasting@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554307}

In low disk space scenarios, Cryptohome erases whole user profiles.
We count the number of user profiles, deleted during cleanup.

BUG=835114

Change-Id: I8921a4d4c7c8e843c7408b8e1eb7d78b1e8d0785
Reviewed-on: https://chromium-review.googlesource.com/1029753
Commit-Queue: Alexey Baskakov <loyso@chromium.org>
Reviewed-by: Mark Pearson <mpearson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554306}

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/97ce05851cb5..179dd1e01695

$ git log 97ce05851..179dd1e01 --date=short --no-merges --format='%ad %ae %s'
2018-04-26 tkent input_api.DEFAULT_BLACK_LIST should not block third_party/blink.

Created with:
  roll-dep src/third_party/depot_tools
BUG=chromium:836555


The AutoRoll server is located here: https://depot-tools-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=agable@chromium.org

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

Adds code to parse elf binary to get build id from
.note.gnu.build-id section. The build ID is needed to symbolize
heap dumps from the users.

BUG=734705

Change-Id: If4365e232c060ba96071ba1e1c43618f9807e39c
Reviewed-on: https://chromium-review.googlesource.com/1028995
Commit-Queue: Siddhartha S <ssid@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554304}

https://skia.googlesource.com/skia.git/+log/a070ed7fccd8..414be9b332f7

$ git log a070ed7fc..414be9b33 --date=short --no-merges --format='%ad %ae %s'
2018-04-26 csmartdalton Blacklist filltypespersp on IntelHD405 vulkan
2018-04-26 benjaminwagner Revert "[recipes] Use named caches for git and workdirs"
2018-04-26 swiftshader-skia-autoroll Roll third_party/externals/swiftshader/ 840f47ea8..ce8eb945c (1 commit)

Created with:
  roll-dep src/third_party/skia


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

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

Traffic annotation in Feed Image Loader is updated and annotations.xml
is updated based on that.

Change-Id: I1a0d7a6ce1ba16a10ef2e3dd6cd72f0d47b2744f
Reviewed-on: https://chromium-review.googlesource.com/1030194Reviewed-by: Filip Gorski <fgorski@chromium.org>
Reviewed-by: Gang Wu <gangwu@chromium.org>
Commit-Queue: Ramin Halavati <rhalavati@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554302}

Bug: b/74415372
Test: Manually Tested
Change-Id: I4e3e5a5b48054b6a7c59e2a02b38b0d200885fee
Reviewed-on: https://chromium-review.googlesource.com/1009243Reviewed-by: Mitsuru Oshima <oshima@chromium.org>
Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
Commit-Queue: Sara Kato <sarakato@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554301}

Code clean-up and refactor for U2f{Request, Register, Sign} and its
tests. More specifically, this CL
  - Moves all tests related to creating U2f{Register, Sign,
    BogusRegistration}Command to uf_command_constructor_unittest.cc
  - Moves all tests related to parsing/serializing responses to
    ctap_response_unittest.cc

This refactoring is prepares for
migration of U2f{Register, Sign} to U2f{Register, Sign}Operation.

Bug: 798573
Change-Id: I24ab0a2c5796571b9ff06fe062e5823ed779c725
Reviewed-on: https://chromium-review.googlesource.com/1024868
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554300}

In a recent CL, https://chromium-review.googlesource.com/c/chromium/src/+/681850,
MesasgePumpDefault is used for MessageLoop::TYPE_DEFAULT on OSX.
This requests the desktopCaptureThread to be adjusted to run as
a UI thread, otherwise the CFRunLoop won't run again after
display resolution changes.

Bug: 836978
Change-Id: Ifb460f19dca4741359d93bb73e280404d62c5a05
Reviewed-on: https://chromium-review.googlesource.com/1029113Reviewed-by: Zijie He <zijiehe@chromium.org>
Commit-Queue: Weiyong Yao <braveyao@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554299}

This patch makes |{Start,End}{Offset,Position}InCurrentContainer()| in
|TextIteratorInFlatTree| to work correctly with non-text nodes, e.g. image
alt text, leading whitespace, etc., by computing parent and child index in
|TextIteratorAlgorithm<Stargety>| instead of |TextIteratorTextState|.


This patch is a preparation of the patch[1].

[1] http://crrev.com/c/1029393 Make TextIteratorTextState to represent
relative position explicitly

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

When the accessibility object for a View is ignored, it is not getting
ignored by NVDA's virtual buffer. It is not enough to give it a role
of client and make it invisible -- it also need to be nameless.

Bug: 820350
Change-Id: I388b19ae65f1f5bf02ed6135936f0b75e2a153f8
Reviewed-on: https://chromium-review.googlesource.com/1031130
Commit-Queue: Aaron Leventhal <aleventhal@chromium.org>
Reviewed-by: Nektarios Paisios <nektar@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554297}

This reverts commit 90d40910.

Reason for revert: Landed at the same time as https://chromium-review.googlesource.com/c/chromium/src/+/1013824 which used the old function.

Original change's description:
> Avoid creating Crostini UI objects on guest and ephemeral profiles
> 
> As Crostini is not supported on guest and ephemeral profiles, we change
> IsExperimentalCrostiniUIEnabled() into IsCrostiniUIAllowedForProfile()
> so we can avoid creating the various controllers and builders. This
> makes us less likely to assume a CrostiniRegistryService exists when
> one actually doesn't.
> 
> Bug: 836187
> Change-Id: I5a65ad250edcb16daf381dfd77d7a5df0a1de98f
> Reviewed-on: https://chromium-review.googlesource.com/1029934
> Commit-Queue: Timothy Loh <timloh@chromium.org>
> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#554294}

TBR=xiyuan@chromium.org,timloh@chromium.org

Change-Id: I76a977c63aaffdf4de5cc54c8cb02d9a345f2e43
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 836187
Reviewed-on: https://chromium-review.googlesource.com/1031993Reviewed-by: Timothy Loh <timloh@chromium.org>
Commit-Queue: Timothy Loh <timloh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554296}

Bug: 829893
Change-Id: I757a4a7f6d633f530e95de9d4d7a36c3e27ea14d
Reviewed-on: https://chromium-review.googlesource.com/999747Reviewed-by: Chris Mumford <cmumford@chromium.org>
Commit-Queue: Victor Costan <pwnall@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554295}

As Crostini is not supported on guest and ephemeral profiles, we change
IsExperimentalCrostiniUIEnabled() into IsCrostiniUIAllowedForProfile()
so we can avoid creating the various controllers and builders. This
makes us less likely to assume a CrostiniRegistryService exists when
one actually doesn't.

Bug: 836187
Change-Id: I5a65ad250edcb16daf381dfd77d7a5df0a1de98f
Reviewed-on: https://chromium-review.googlesource.com/1029934
Commit-Queue: Timothy Loh <timloh@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554294}

This reverts commit 50b44e68.

Reason for revert: Breaks NaCl on win7 32 bit: https://ci.chromium.org/buildbot/chromium.win/Win7%20%2832%29%20Tests/32881

Original change's description:
> Reland "win: Link with lld instead of MSVC's link.exe by default"
> 
> This is a reland of be8138a4
> 
> The issues with the first attempt were addressed in the latest
> Clang roll (#553415). ASan is excluded from using lld until
> the fix for crbug.com/837090 is rolled into Chromium.
> 
> Original change's description:
> > win: Link with lld instead of MSVC's link.exe by default
> >
> > lld is LLVM's linker. It produces PE/COFF and PDB files just like
> > link.exe, but it's significantly faster and it can also handle LLVM's
> > internal representation, which will enable us to do link-time
> > optimization and control-flow integraty checks with Clang.
> >
> > While lld is much faster at linking, it doesn't support incremental
> > links, meaning builds that only touch a few files and re-link a large
> > executable may become slower.
> >
> > This is the first attempt at switching everything over, with the
> > purpose of gathering data and finding unknown unknowns. It's likely
> > temporary until something breaks.
> >
> > is_win_fastlink is implicitly ignored when using lld, as lld without
> > fastlink is faster than link.exe with it.
> >
> > Also switch the CrWinClangLLD bots on chromium.clang to use MSVC's
> > link.exe to make sure that configuration keeps working.
> >
> > Bug: 792131
> > Change-Id: I0f115a78c33d69eadbd480f75c2a5d636e86483d
> > Reviewed-on: https://chromium-review.googlesource.com/983632
> > Commit-Queue: Nico Weber <thakis@chromium.org>
> > Reviewed-by: Nico Weber <thakis@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#551367}
> 
> Bug: 792131
> Change-Id: I85ad51904884734156a4e06859fd1ba08e519a3b
> Reviewed-on: https://chromium-review.googlesource.com/1031330
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Commit-Queue: Hans Wennborg <hans@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#554239}

TBR=thakis@chromium.org,hans@chromium.org

Change-Id: I1e873015f928603909b744e857efa1f5a44c2ba0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 792131
Reviewed-on: https://chromium-review.googlesource.com/1032010Reviewed-by: Sam McNally <sammc@chromium.org>
Commit-Queue: Sam McNally <sammc@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554293}

Adds a SettingsPageUIHandler subclass called CrostiniHandler which
- launches the CrostiniInstallerView when requested by settings
- destroys a Crostini VM/Container when requested by settings

This code doesn't really do anything yet, because it relies on Settings
UI to send requests. Settings UI section for Crostini is added in the
next CL.

Bug: 833125
Change-Id: I556077e50979d39a887629d9d5f5aa748166f193
Reviewed-on: https://chromium-review.googlesource.com/1013824
Commit-Queue: Renée Wright <rjwright@chromium.org>
Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Reviewed-by: Nicholas Verne <nverne@chromium.org>
Reviewed-by: Ben Wells <benwells@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554292}

I applied modernize-use-override in blink
http://clang.llvm.org/extra/clang-tidy/checks/modernize-use-override.html

Bug: 428099
Cq-Include-Trybots: 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
Change-Id: If7c2dd539d1c0920310c357dd1014a2c6758ffd1
Reviewed-on: https://chromium-review.googlesource.com/1029592Reviewed-by: Nico Weber <thakis@chromium.org>
Commit-Queue: Takuto Ikuta <tikuta@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554291}

Bug: 731436,805257
Change-Id: I9fe022c27489a82a306eccb7359f9af30553cbbb
Reviewed-on: https://chromium-review.googlesource.com/1031891Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
Commit-Queue: John Budorick <jbudorick@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554290}

https://chromium.googlesource.com/catapult.git/+log/846cec5c4d22..82069c8b6ea2

$ git log 846cec5c4..82069c8b6 --date=short --no-merges --format='%ad %ae %s'
2018-04-26 dtu [dashboard] Make api_auth.Authorize() not a decorator.

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: Ie72a8c8523a99ebc5b456bf54fddfd98a346b396
Reviewed-on: https://chromium-review.googlesource.com/1031748
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@{#554289}

Replace them with gfx geometry types. Add a FloatPoint -> gfx::PointF
conversion operator to assist with this. Also add an IntSize ->
gfx::Vector2d conversion operator, and make both it and the gfx::Size
conversions explicit, so that users must be cognizant they are making
a choice. DCHECK in the gfx::Size conversion that the values are not
negative for now (we may want to just clamp silently in the future
once we have a use case for it?).

R=pdr@chromium.org

Bug: 738465
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: I9daee692287744780878fcc294f8e054dc222a5a
Reviewed-on: https://chromium-review.googlesource.com/1031205
Commit-Queue: danakj <danakj@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554288}

We added their replacements to third_party/blink/tools, and we have
no code refers to them in third_party/WebKit.

Bug: 829697
Change-Id: I096512bc323cb5f16d771a1319c7581a22732e2a
Reviewed-on: https://chromium-review.googlesource.com/1029566Reviewed-by: Quinten Yearsley <qyearsley@chromium.org>
Commit-Queue: Kent Tamura <tkent@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554287}

Bug: 805271
Change-Id: I09e58d9368e02e8fd41c139440381722def67509
Reviewed-on: https://chromium-review.googlesource.com/1031750Reviewed-by: Kent Tamura <tkent@chromium.org>
Commit-Queue: Fergal Daly <fergal@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554286}

When setting the bounds of a WebContents, we shouldn't also change the
browser window size to the size of the WebContents. This can lead to
the bounds of the WebContents lying outside the browser's window, in
which case screenshots will hang.

Instead, make sure that all WebContents are contained within the browser
window size.

Bug: 832138
Change-Id: Iaeb8b36b4edceedfdf2fdd1d54c3765587c79ea7
Reviewed-on: https://chromium-review.googlesource.com/1031655Reviewed-by: Andrey Kosyakov <caseq@chromium.org>
Commit-Queue: Eric Seckler <eseckler@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554285}

# This is an automated release commit.
# Do not revert without consulting chrome-pmo@google.com.
NOAUTOREVERT=true
TBR=amineer@chromium.org

Change-Id: I0b31cdad969298c0670967616359dd617b495e3a
Reviewed-on: https://chromium-review.googlesource.com/1031930Reviewed-by: chrome-release-bot@chromium.org <chrome-release-bot@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554284}

https://pdfium.googlesource.com/pdfium.git/+log/52ab95aa3edb..575f238334d1

$ git log 52ab95aa3..575f23833 --date=short --no-merges --format='%ad %ae %s'
2018-04-27 thestig Roll DEPS for ICU to e4194dc7
2018-04-27 thestig Roll DEPS for build and buildtools.
2018-04-27 thestig Roll DEPS for Clang to abe5e4f9.

Created with:
  roll-dep src/third_party/pdfium


The AutoRoll server is located here: https://pdfium-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=dsinclair@chromium.org

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

Using wpt-import in Chromium 244bd88b.
With Chromium commits locally applied on WPT:
303956b6 "Enable WPT tests for the Generic Sensor classes"
7c13f644 "Subtract scrollbar in PerpendicularContainingBlockLogicalHeight"
a8385691 "Make web font timeouts work for preloaded fonts"


Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/16282

Note to sheriffs: This CL imports external tests and adds
expectations for those tests; if this CL is large and causes
a few new failures, please fix the failures by adding new
lines to TestExpectations rather than reverting. See:
https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md

TBR=foolip

No-Export: true
Change-Id: If5c54f1f79e8b3ffa920fe424b3f00ae4fa0a308
Reviewed-on: https://chromium-review.googlesource.com/1031400
Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554282}

We use ScopedPaintChunkProperties anywhere when there might be a
property change, but don't expect new chunks if the properties are
the same as the current. Previously we created new chunks if the
id changes which added many chunks with the same properties which
unnecessarily increased the load of CompositedLayerRasterInvalidator
and PaintChunksToCcLayer.

https://ct.skia.org/results/cluster-telemetry/tasks/chromium_perf_runs/wangxianzhu-20180425154628/html/index.html
shows great results:
- painter_memory_usage: -16.5%
- record_time_subsequence_caching_disabled: -4%
- record_time: -1.2%

Drawbacks: It combines some previously separated paint
chunks together, so full chunk invalidations are now issued on
the combined chunk, and the bounds may cover more area than
the previously separated chunks. Another drawback is about
appeared or disappeared chunks that now may also occur on bigger
chunks. Consider the big benefits, it's a good trace-off for the
drawbacks (shown in about 10 layout tests).

Bug: 803867
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
Change-Id: Iba541b24bb4ad92c112acf980d96a6ef4d186880
Reviewed-on: https://chromium-review.googlesource.com/1027209
Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org>
Reviewed-by: Philip Rogers <pdr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554281}

This is a preparation CL for the off-main-thread WebSocket.

After this CL, DedicatedWorkerHost is created from RenderFrameHostImpl, not from
RendererInterfaceBinders, so that DedicatedWorkerHost can get its parent frame's
id. This may not work for nested workers, but probably we can add an interface
for creating DedicatedWorkerHost to DedicatedWorkerHost for that case.

<Motivation>

DedicatedWorkerHost needs to know the parent frame id in order to share a user
decision on the SSL certificate warning interstitial among the parent document
and dedicated workers. When the off-main-thread WebSocket is disabled, the
decision is naturally shared among them because the dedicated workers depend on
their parent document's loader. Once the off-main-thread WebSocket is enabled,
the dedicated workers need to annotate requests as permitted using their parent
frame's id.

<Notes>

This CL renames private WebSocketManager::CreateWebSocket() overloaded by public
CreateWebSocket() to DoCreateWebSocketInternal() because the overload makes
base::BindRepeating() confused.

Bug: 825740
Change-Id: I9c2b07bea60cce315b94a4776b1d163e4330a208
Reviewed-on: https://chromium-review.googlesource.com/1025497Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Commit-Queue: Hiroki Nakagawa <nhiroki@chromium.org>
Cr-Commit-Position: refs/heads/master@{#554280}