GitLab

https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/6f812e132d25..5300616351f5


git log 6f812e132d25..5300616351f5 --date=short --no-merges --format='%ad %ae %s'
2018-08-02 jbudorick@chromium.org bot_update: expose patch_refs as an option to ensure_checkout.


Created with:
  gclient setdep -r src/third_party/depot_tools@5300616351f5

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.



BUG=chromium:865882
TBR=agable@chromium.org

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

R=mek@chromium.org

Bug: 868578
Change-Id: I1c90f993f3e71d543b414c37a1b92bae5948b766
Reviewed-on: https://chromium-review.googlesource.com/1155483
Commit-Queue: Marijn Kruisselbrink <mek@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580214}

Instrument GLES2DecoderImpl::DoCommandsImpl to
- record in a crash key all of the GL commands that have been flushed
- flush at the end of every time that it is called

This will severely regress performance.

What we hope to learn:
- There's a good chance this will make the bug vanish (egregious glFlush
  calls have been known to do this), in which case we'll learn nothing.
  - But maybe we can add less egregious glFlush calls to avoid the issue
    in Chrome 69
- If elevated crash rates persist, hopefully this will point to a particular
  call or sequence of calls that is causing the crash

Bug: 863817
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: I462fab8a6d86ee3896dcd9c0c3c4aa68f9de0a69
Reviewed-on: https://chromium-review.googlesource.com/1159831
Commit-Queue: ccameron <ccameron@chromium.org>
Reviewed-by: Kenneth Russell <kbr@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580213}

event_modules_factory.cc indirectly includes gamepad.mojom-blink.h, but
there's no dependency between them. That causes a flaky build failure:
https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8939294599854040784/+/steps/compile/0/logs/raw_io.output_failure_summary_/0

Change-Id: I509f61c46957b3d3646c7c679113454148785df7
Reviewed-on: https://chromium-review.googlesource.com/1160825Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Commit-Queue: Yuki Shiino <yukishiino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580212}

It is flaky

TBR=bradnelson@chromium.org

Bug: 870309
Change-Id: I8a55fdaae7091b0dc32247527d50a35510e7a4bf
Reviewed-on: https://chromium-review.googlesource.com/1160542Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Christos Froussios <cfroussios@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580211}

For profiles with unified consent enabled, the sync state is completely
ignored when deciding whether UKM is enabled. The decision whether UKM
is allowed for a profile with user consent enabled is based only on the
URL-keyed anonymized data collection enabled preference.

This CL changes the UKM logic to check whether URL-keyed anonymized data
collections is enabled for profiles where unified consent is enabled.

This CL also updates the ProfileSyncServiceHarness to set/unset the
state of the UnifiedConsentGiven flag when the state of sync everything
changes.

Bug: 861777
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I974a3eea2caa424efe44c1dd8b7c6f2bb24f156e
Reviewed-on: https://chromium-review.googlesource.com/1152744Reviewed-by: Brian White <bcwhite@chromium.org>
Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
Reviewed-by: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Olivier Robin <olivierrobin@chromium.org>
Commit-Queue: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580210}

In preparation of supporting compression of log files, introduce
an interface named LogFileWriter, which will allow
WebRtcEventLogManager to write either compressed or uncompressed
log files.

Bug: 775415
Change-Id: I4bc30fae88bba2e071d72aabfdf9fe05a5148af0
Reviewed-on: https://chromium-review.googlesource.com/1143472
Commit-Queue: Elad Alon <eladalon@chromium.org>
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580209}

If for some reason drivefs fails to start or crashes we want to do
limited number of retries and then give up.

BUG=chromium:845388

Change-Id: Ife550b76ba060adb2b3abc1a7674ae9b396e2677
Reviewed-on: https://chromium-review.googlesource.com/1158674Reviewed-by: Sam McNally <sammc@chromium.org>
Commit-Queue: Sergei Datsenko <dats@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580208}

ResetFirstAfterBootTest.RollbackUnavailable has been flaky. Disabling.

TBR=xiyuan@chromium.org

Bug: 870284
Change-Id: I7226ae73ca9c4cf440de3fbeae28886309d907d9
Reviewed-on: https://chromium-review.googlesource.com/1160866Reviewed-by: Tommy Martino <tmartino@chromium.org>
Commit-Queue: Tommy Martino <tmartino@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580207}

If host is already gone after we've found a target and is processing/
dispatching the event now, early return and drop the event.

Bug: 814674
Change-Id: Ifa1b0aac45669413f954a843bf395aae229b969c
Reviewed-on: https://chromium-review.googlesource.com/1129856Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Commit-Queue: Ria Jiang <riajiang@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580206}

This allows controllers to start datatypes in ephemeral mode,
which is only supported by ModelTypeController.

Bug: 866814
Change-Id: I20630c2ed52872baf0ecd23cccbd08f505e8835b
Reviewed-on: https://chromium-review.googlesource.com/1151299
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580205}

The (Fake)SigninManager should always be initialized before using. This
is made more critical by upcoming changes* that will cause the test to
fail if initialization is not performed.

*https://chromium-review.googlesource.com/c/chromium/src/+/1154985

Bug: 806781
Change-Id: I06989a895f86b3ca9b1b84760b54ecafdaafdb03
Reviewed-on: https://chromium-review.googlesource.com/1160643
Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580204}

This is a reland of 1e910c7f.
It is safe to reland as a fix for the test has landed downstream.

Original change's description:
> [iOS] Accessibility adjustments
>
> This CL adds a delay for the voice over announcement of the InProduct
> Help tips. This delay is added to prevent the announcement to be
> cancelled by the element being focused, when the tab is presented.
> It also adds an accessibility label to the empty space where the omnibox
> should be if the toolbar was visible.
>
> Bug: 866936
> Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
> Change-Id: Ia75dd56772203e39b8d0bf1e94bef9cfacd87fdd
> Reviewed-on: https://chromium-review.googlesource.com/1158236
> Commit-Queue: edchin <edchin@chromium.org>
> Reviewed-by: edchin <edchin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#579886}

TBR: edchin@chromium.org
Bug: 866936
Change-Id: I0c96e39827d4fb8f815664b10414314cad42a20c
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Reviewed-on: https://chromium-review.googlesource.com/1160661
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Reviewed-by: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580203}

Like legacy layout, skip list marker while finding the first letter.

Bug: 725277
Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
Change-Id: I467c766f8e362f7e12b8d9c7b0251fee02e1b0a2
Reviewed-on: https://chromium-review.googlesource.com/1156111
Commit-Queue: cathie chen <cathiechen@tencent.com>
Reviewed-by: Koji Ishii <kojii@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580202}

This reverts commit d01b7c9e.

Reason for revert: https://crrev.com/c/1159374 just landed, fixing the assertion error. Thus relanding the initial change.

Original change's description:
> Revert "[Bluetooth][WinRT] Implement Reading and Writing of Descriptors"
> 
> This reverts commit f61df6f9.
> 
> Reason for revert: Multiple BluetoothTestWinrt{,Only} tests failing on Win10 Tests bot after around this change.
> 
> Sample builds:
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20Tests%20x64%20%28dbg%29/2242
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20Tests%20x64%20%28dbg%29/2240
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20Tests%20x64%20%28dbg%29/2236
> https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Win10%20Tests%20x64%20%28dbg%29/2235
> 
> failures:
> BluetoothTestWinrtOnly.GattServices_ObserversCalls/0
> BluetoothTestWinrt.GetPrimaryServices/1
> BluetoothRemoteGattServiceTestWinrt.SimulateGattServiceRemove/1
> BluetoothTestWinrt.GetPrimaryServicesByUUID/1
> BluetoothTestWinrtOnly.BluetoothGattConnection_DisconnectGatt_Cleanup/0
> BluetoothTestWinrt.GetGattServices_and_GetGattService/1
> BluetoothRemoteGattServiceTestWinrt.GetCharacteristicsByUUID/1
> BluetoothTestWinrtOnly.GattServicesDiscovered_Success/0
> 
> Original change's description:
> > [Bluetooth][WinRT] Implement Reading and Writing of Descriptors
> > 
> > This change implements reading and writing of Remote Gatt Descriptors
> > for WinRT. Furthermore, it enables corresponding tests.
> > 
> > Bug: 821766
> > Change-Id: I3ac07969c3fbf3806b71543ae79807c95efe78fe
> > Reviewed-on: https://chromium-review.googlesource.com/1156703
> > Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
> > Reviewed-by: Reilly Grant <reillyg@chromium.org>
> > Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#579838}
> 
> TBR=reillyg@chromium.org,ortuno@chromium.org,jdoerrie@chromium.org
> 
> Change-Id: Icb716ed05f9b8bec2a5b1bdd33692006314a99e7
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 821766
> Reviewed-on: https://chromium-review.googlesource.com/1159961
> Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
> Commit-Queue: Kinuko Yasuda <kinuko@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#580062}

TBR=kinuko@chromium.org,reillyg@chromium.org,ortuno@chromium.org,jdoerrie@chromium.org

Change-Id: I7c6455ea83413a3527ce59bf41fbf42f2bb80208
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 821766
Reviewed-on: https://chromium-review.googlesource.com/1160902Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580201}

This reverts commit 67ca5a07.

Reason for revert: https://crrev.com/c/1159374 just landed, fixing the assertion error. Thus relanding the initial change.

Original change's description:
> Revert "[Bluetooth][WinRT] Implement Obtaining Gatt Descriptors"
> 
> This reverts commit 20f64257.
> 
> Reason for revert: Suspected of breaking device_unittests on Win10 Tests x64 (dbg)
> 
> Original change's description:
> > [Bluetooth][WinRT] Implement Obtaining Gatt Descriptors
> > 
> > This change implements obtainining Remote Gatt Descriptors for WinRT by
> > hooking up the appropriate logic into GattDiscovererWinrt. Furthermore,
> > appropriate tests are enabled.
> > 
> > Bug: 821766
> > Change-Id: I20475478367c2ab6b18940b4271de70ec138f676
> > Reviewed-on: https://chromium-review.googlesource.com/1156392
> > Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
> > Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
> > Reviewed-by: Reilly Grant <reillyg@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#579830}
> 
> TBR=reillyg@chromium.org,ortuno@chromium.org,jdoerrie@chromium.org
> 
> Change-Id: If5128ebf0c9dd1f585bd98b9729a57340f4493a0
> No-Presubmit: true
> No-Tree-Checks: true
> No-Try: true
> Bug: 821766, 870208
> Reviewed-on: https://chromium-review.googlesource.com/1159943
> Reviewed-by: Christos Froussios <cfroussios@chromium.org>
> Commit-Queue: Christos Froussios <cfroussios@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#580106}

TBR=reillyg@chromium.org,ortuno@chromium.org,cfroussios@chromium.org,jdoerrie@chromium.org

Change-Id: I8a4e456fe6d3b63c9d9b02c15ecdd35ac61a7fa7
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 821766, 870208
Reviewed-on: https://chromium-review.googlesource.com/1160901Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580200}

Also remove unused indices in desktop_power_stories.json.

Bug: chromium:810616
Change-Id: Id12e4ed6bf8433de5a7c1a08e5e86b3ad1b56bb0
Reviewed-on: https://chromium-review.googlesource.com/1160122Reviewed-by: Juan Antonio Navarro Pérez <perezju@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Vovo Yang <vovoy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580199}

BluetoothDevice::DeviceUUIDs::UpdateDeviceUUIDs() makes use of
std::set_union in order to merge advertised uuids and service uuids.
However, std::set_union requires sorted ranges as input, which wasn't
necessarily guaranteed before this change. When using a
std::unordered_set the order of elements is not specified and
std::set_union is not guaranteed to work correctly. Using a
base::flat_set for UUIDSet instead fixes this issue. In order to prevent
future regressions this change also replaces the usage of std::set_union
with base::STLSetUnion performing an explicit IsSorted() check in Debug
builds.

Bug: 821766, 870208
Change-Id: Id25711e6acf77ded9eebb6c80db5bcb2ed8a63e0
Reviewed-on: https://chromium-review.googlesource.com/1159374Reviewed-by: Giovanni Ortuño Urquidi <ortuno@chromium.org>
Commit-Queue: Jan Wilken Dörrie <jdoerrie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580198}

It broke because of the test that the response RP ID hash was actually
the hash of the RP ID. (Which isn't true when using an AppID hash.)

Obviously we were missing test coverage here, so add that now that it's
easy to do with VirtualFidoDevice. (Confirmed that the test failed
before the rest of these changes and passes now.)

Change-Id: Ibd3b39339500e24f00c4f6c12ab4405ae0596107
Reviewed-on: https://chromium-review.googlesource.com/1159522Reviewed-by: Jun Choi <hongjunchoi@chromium.org>
Commit-Queue: Adam Langley <agl@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580197}

Make proxy.pac resolution for WebSockets work for Windows and Mac when
the --winhttp-proxy-resolver flag is supplied. These platforms don't
support passing WebSocket URLs to the proxy.pac file. Convert ws and wss
URLs to http and https before passing them to proxy.pac. This matches
the behaviour of the vendor browsers on these platforms.

Add a WebSocket end-to-end test to verify that WebSockets can be
accessed via a proxy specified in proxy.pac.

Since preflighting proxy credentials makes the tests really complicated,
add an extra mode to testserver.py for a vanilla proxy that doesn't
require auth.

BUG=862121

Change-Id: I63b318b7f6d3083a6c21f5811bb27966d17cb02a
Reviewed-on: https://chromium-review.googlesource.com/1143093
Commit-Queue: Adam Rice <ricea@chromium.org>
Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
Reviewed-by: Ian Clelland <iclelland@chromium.org>
Reviewed-by: Eric Roman <eroman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580196}

The only code outside of net/ / services/network that does direct proxy
resolutions appears to be the two PPAPI consumers. This CL updates them
both to use the new API.

Bug: 810950
Cq-Include-Trybots: luci.chromium.try:linux_mojo
Change-Id: I643605dfe2d8b67626ea8b84ee793b2c88b66e12
Reviewed-on: https://chromium-review.googlesource.com/1136875Reviewed-by: Eric Roman <eroman@chromium.org>
Reviewed-by: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Raymes Khoury <raymes@chromium.org>
Reviewed-by: John Abd-El-Malek <jam@chromium.org>
Commit-Queue: Matt Menke <mmenke@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580195}

Adds call to SigninManager::SignIn(...) as part of TestSuccessfulSignin
helper method. This is needed to simulate real signin and becomes
critical with upcoming changes* to FakeSigninManager.

*https://chromium-review.googlesource.com/c/chromium/src/+/1154985

Bug: 806781
Change-Id: I973859ef52bcddd978429dbb4ce9d913224c7b0d
Reviewed-on: https://chromium-review.googlesource.com/1160492
Commit-Queue: Chrome Cunningham (In Paris) <chcunningham@chromium.org>
Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580194}

This integration tests now test TitleWasChanged, which is not navigation
or load callback.

Bug: None
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: Ic78b2758ac2ba92dece1034dc92328ce7b30cd1d
Reviewed-on: https://chromium-review.googlesource.com/1159332Reviewed-by: Kurt Horimoto <kkhorimoto@chromium.org>
Commit-Queue: Eugene But <eugenebut@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580193}

This CL adds tests to the AdTracker to verify that resources loaded
by an ad script are marked properly ad ads during loaded.  This
does not work properly for images currently due to crbug.com/848916.

Bug: 848916, 863446
Change-Id: I6643ec24c16ec7dceb9c60fd150046b86cb008ac
Reviewed-on: https://chromium-review.googlesource.com/1140196Reviewed-by: Josh Karlin <jkarlin@chromium.org>
Reviewed-by: Daniel Cheng <dcheng@chromium.org>
Commit-Queue: Eric Robinson <ericrobinson@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580192}

It is flaky on mac_chromium_rel_ng

Bug: 870259
No-try: True
No-presubmit: True
Change-Id: I73ac22cce3d7d03ce2f36c90d79e8a829fe28100
Reviewed-on: https://chromium-review.googlesource.com/1160543Reviewed-by: Christos Froussios <cfroussios@chromium.org>
Commit-Queue: Christos Froussios <cfroussios@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580191}

When a unified consent service is disabled while unified consent is
given, unified consent is revoked.

This can happen when a synced preference is turned off on another
machine or MetricsReporting is disabled on another profile.

Bug: 867480
Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I2dfe0127a86ccb718f4f6414f42c7c22eb8dc117
Reviewed-on: https://chromium-review.googlesource.com/1151324
Commit-Queue: Thomas Tangl <tangltom@chromium.org>
Reviewed-by: Mihai Sardarescu <msarda@chromium.org>
Reviewed-by: Robert Kaplow (slow) <rkaplow@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580190}

We do this for the control methods to the delegate to simplify tests,
by migrating away from ProxyModelTypeControllerDelegate.

Instead, ForwardingModelTypeControllerDelegate is introduced, currently
used in tests only but with the goal to be adopted more broadly (see
crbug.com/867801).

A proxy processor is still in use, and that still requires some nested
loops, but that's left out for future patches.

Bug: 855010,867801
Change-Id: I1933355fee2093bf006830cde334c644f4391020
Reviewed-on: https://chromium-review.googlesource.com/1160642
Commit-Queue: Mikel Astiz <mastiz@chromium.org>
Reviewed-by: Marc Treib <treib@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580189}

After this CL, WTF::Bind refuses to bind STACK_ALLOCATED objects.
And to do that, this adds a marker to STACK_ALLOCATED macro in
allocator.h, and adds WTF::IsStackAllocatedType<> to type_traits.h
that detects the marker.

Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel
Change-Id: Ibee83e3f457619af0e117584dae3bc4a7a33a51c
Reviewed-on: https://chromium-review.googlesource.com/1160424Reviewed-by: Kentaro Hara <haraken@chromium.org>
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580188}

https://pdfium.googlesource.com/pdfium.git/+log/649929af2140..b07deb3fc1f5


git log 649929af2140..b07deb3fc1f5 --date=short --no-merges --format='%ad %ae %s'
2018-08-02 art-snake@yandex-team.ru Rework of CPDF_Parser::GetLastObjNum.


Created with:
  gclient setdep -r src/third_party/pdfium@b07deb3fc1f5

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: Idb9f2e95b3550c107a6c459192f8c8dd1d5d51f0
Reviewed-on: https://chromium-review.googlesource.com/1160601Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#580187}

Summary of changes available at:
https://chromium.googlesource.com/v8/v8/+log/8319a20f..7b323896

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;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;luci.chromium.try:android_optional_gpu_tests_rel

TBR=hablich@chromium.org,v8-waterfall-sheriff@grotations.appspotmail.com

Change-Id: I8f4724714cac89463e0dc536126b24030ceeb5b8
Reviewed-on: https://chromium-review.googlesource.com/1160343Reviewed-by: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Commit-Queue: V8 Autoroller <v8-ci-autoroll-builder@chops-service-accounts.iam.gserviceaccount.com>
Cr-Commit-Position: refs/heads/master@{#580186}

Remove unused base class and json files.

Bug: chromium:777664
Change-Id: I06323e435e2acb9b471a2e733f80dae92f4d94ce
Reviewed-on: https://chromium-review.googlesource.com/1160129Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#580185}

Since we start to switch to TSF from IMM32 in IME on Windows, we want
to rename InputMethodWin class to InputMethodWinImm32 to distinguish
these two input methods.

Bug: 657623
Change-Id: I70fc2f6ee319fd050e06e5937c10dbf4996681f7
Reviewed-on: https://chromium-review.googlesource.com/1159351Reviewed-by: Shu Chen <shuchen@chromium.org>
Reviewed-by: Sadrul Chowdhury <sadrul@chromium.org>
Commit-Queue: Lan Wei <lanwei@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580184}

Update the fetch interface of GeneratedCodeCache to send
response time and data instead of just raw data. This will
make it cleaner when using it and also make it consistent
with the write interface.

Bug: chromium:812168
Change-Id: Ic460bbbf15788caf04946a35b5cb3ba356cd212e
Reviewed-on: https://chromium-review.googlesource.com/1154539Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Mythri Alle <mythria@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580183}

After this CL, base::BindOnce and base::BindRepeating rejecting pointers
to ref-counted objects that have zero ref count. So that we can avoid a
pattern of racy case like below in most cases:

If a ref-counted class |Foo| posted a task that contains |this| as a
receiver like this:
  Foo::Foo() {
    base::PostTask(FROM_HERE, base::BindOnce(&Foo::Bar, this));
  }
base::BindOnce retains a reference, and PostTask may release the
reference before the construction of Foo completes, that happen when
PostTask fails or the task runs soon.
As the result, `new Foo()` may return stale pointer.

Bug: 866456
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:ios-simulator-full-configs;luci.chromium.try:linux_mojo;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel;master.tryserver.chromium.mac:ios-simulator-cronet
Change-Id: I2c7e8efcbd74fda49d183dbb782f94b0d7cf6af3
Reviewed-on: https://chromium-review.googlesource.com/1144813
Commit-Queue: Taiju Tsuiki <tzik@chromium.org>
Reviewed-by: Gabriel Charette <gab@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580182}

This reverts commit e63be647.

Reason for revert: breaking generate_build_files on chromium.perf/Win Builder Perf

Original change's description:
> Add logging and chrome_utils directories
> 
> Bug: 830892
> Change-Id: I8c121dff724dea526a7cc7ce198da51cc1b8d045
> Reviewed-on: https://chromium-review.googlesource.com/1158719
> Reviewed-by: Joe Mason <joenotcharles@chromium.org>
> Reviewed-by: Peter Kasting <pkasting@chromium.org>
> Reviewed-by: Christian Dullweber <dullweber@chromium.org>
> Commit-Queue: Oliver Li <olivierli@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#580163}

TBR=pkasting@chromium.org,joenotcharles@chromium.org,dullweber@chromium.org,olivierli@chromium.org

Change-Id: I295b58fdf9300e681b985c93b6f68b0ec0702da0
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 830892
Reviewed-on: https://chromium-review.googlesource.com/1160881Reviewed-by: Ned Nguyen <nednguyen@google.com>
Commit-Queue: Ned Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#580181}

TBR=alexmos@chromium.org

Bug: 662160
Change-Id: Ice66d0146daf565cf0119a040175641e3b6af791
Reviewed-on: https://chromium-review.googlesource.com/1160586Reviewed-by: Nasko Oskov <nasko@chromium.org>
Commit-Queue: Nasko Oskov <nasko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580180}

This reverts commit aaa73d4c.

Reason for revert: Apparently glob.glob doesn't work with expressions line {value1,value2} :(

Original change's description:
> Do not bundle compiler-rt libraries (ASan,UBSan,Profile) for Android on i686.
> 
> This change removes 6+ MB of binaries from the build tools bundle:
> 3.7M	lib/linux/libclang_rt.asan-i686-android.so
> 2.5M	lib/linux/libclang_rt.ubsan_standalone-i686-android.so
> 80K	lib/linux/libclang_rt.profile-i686-android.a
> 
> It also can break some workflows, but as discussed in person, there is a good
> chance that no one is using those libraries on i686 arch.
> 
> 
> Bug: 866225
> Change-Id: Ifbc85e542359681f60f19f52e5bd7a8a7e1655f3
> Reviewed-on: https://chromium-review.googlesource.com/1158708
> Reviewed-by: Nico Weber <thakis@chromium.org>
> Commit-Queue: Max Moroz <mmoroz@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#579854}

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

Change-Id: Ic009c159a98be684a9dec8ea5aec364e83d7389d
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 866225
Reviewed-on: https://chromium-review.googlesource.com/1160801Reviewed-by: Max Moroz <mmoroz@chromium.org>
Commit-Queue: Max Moroz <mmoroz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580179}

A controller delegate is responsible for updating the controller, detect
gestures and produce a model to be sent to the Ui. This CL introduces 2
delegates for VrShellGl, one for production and one for testing.

This is the first step on a major refactoring on which VrShellGl
disappears in favor of a cross platform RenderLoop, composed of platform
specific delegates for each of its functions.

A revert to https://crrev.com/c/1121670 was needed to get the
chrome_public_test_vr_apk built properly.

Bug: 767282, 848494
Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;luci.chromium.try:linux_optional_gpu_tests_rel;luci.chromium.try:linux_vr;luci.chromium.try:mac_optional_gpu_tests_rel;luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Ib68f975ce4e5e39fc7ebb5cf386b400f48039f4e
Reviewed-on: https://chromium-review.googlesource.com/1151623Reviewed-by: agrieve <agrieve@chromium.org>
Reviewed-by: Amirhossein Simjour <asimjour@chromium.org>
Reviewed-by: Brian Sheedy <bsheedy@chromium.org>
Reviewed-by: Bill Orr <billorr@chromium.org>
Commit-Queue: Aldo Culquicondor <acondor@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580178}

- Presubmit still enforces that this interface is used instead of
Notification(Compat).Builders directly - the main benefit being that
it's harder to forget to initialize a notification channel, since both
implementations initialize the channel in their constructors.

- We'd like to keep this presubmit and this class may yet be used for
even greater purposes, hence removing the TODO since it's not going
away any time soon.

R=peter@chromium.org

Change-Id: Id6f4d80b37763b3a44a4a2d94dd1f6c6958a99da
Reviewed-on: https://chromium-review.googlesource.com/1160641Reviewed-by: Peter Beverloo <peter@chromium.org>
Commit-Queue: Peter Beverloo <peter@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580177}

This addresses a bug where tapping the installable ambient mini-infobar on
Chrome for Android may not correctly trigger the add to home screen dialog.
The root cause is that the user gesture requirement for calling
beforeinstallpromptevent.prompt() is checked in the browser, not the
renderer, because some platforms are only just rolling out the
requirement. As a result, calling prompt() without a user gesture
invalidates all weak pointers when it terminates, including the weak
pointer used by the mini-infobar to trigger the add to home screen dialog.

This CL fixes the bug by checking the user gesture in the renderer, and
not dispatching to the browser unless the gesture is present. This means
that the banner pipeline will not invalidate weak pointers when a user
gesture is not present.

BUG=869780
TBR=mkwst@chromium.org

Change-Id: Iee7dfc76401e7141707b2d7b2e6ac5e24297ac4b
Reviewed-on: https://chromium-review.googlesource.com/1158305Reviewed-by: Dominick Ng <dominickn@chromium.org>
Reviewed-by: Matt Giuca <mgiuca@chromium.org>
Commit-Queue: Dominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580176}