GitLab

https://pdfium.googlesource.com/pdfium.git/+log/58854942e06d..d3a3cc24a034

$ git log 58854942e..d3a3cc24a --date=short --no-merges --format='%ad %ae %s'
2017-05-16 thestig Handle when XFA parser error handlers cannot format error messages.
2017-05-16 dsinclair Add formcalc lexer tests.
2017-05-16 dsinclair Update formcalc return types
2017-05-16 dsinclair Do not walk off end of formcalc string
2017-05-16 dsinclair Minor xfa_lexer.cpp cleanup

Created with:
  roll-dep src/third_party/pdfium
BUG=708428,721533


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

Revert of Make /deep/ as no-op and remove ::shadow in dynamic profile (patchset #9 id:160001 of https://codereview.chromium.org/2778983006/ )

Reason for revert:
(from the right account this time) Completely breaks Chrome OS login UI

Original issue's description:
> Make /deep/ as no-op and remove ::shadow in dynamic profile
>
> Intent to Remove: https://groups.google.com/a/chromium.org/forum/#!topic/blink-dev/HX5Y8Ykr5Ns
>
> To make this CL small one, and easy to be reverted, all tests which depend on /deep/
> (or ::shadow) were either removed or updated in another CL:
> https://bugs.chromium.org/p/chromium/issues/detail?id=715034.
>
> This CL only touched the small part so that users can't use /deep/ or ::shadow
> in CSS dynamic profile in M60. The further internal clean up is needed in other CLs.
>
> BUG=489954
>
> Review-Url: https://codereview.chromium.org/2778983006
> Cr-Commit-Position: refs/heads/master@{#471684}
> Committed: https://chromium.googlesource.com/chromium/src/+/a7ab8a110bd6b5339c03a34a5a6bb9a419a49e60

TBR=achuith@chromium.org,alemate@chromium.org,kochi@chromium.org,hayato@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=489954

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

Add _env_version attribute for each cros component update check.
Remove dir field in configuration and use name to be dir name.

BUG=chromium:690521
TEST=Run on a chromebook to download escpr successfully.

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

Traffic type will be used in data saver site-breakdown code to ignore
non user initiated traffic.

BUG=718656

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

replace is2D() method with attribute in CSSTransformValue.

https://drafts.css-houdini.org/css-typed-om-1/#csstransformvalue

BUG=545318

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

It caused double-click-copy to copy the path along with the period.

TBR=estevenson

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

Doesn't fix the original issue of notifications failing after reconnect
https://crbug.com/680099 correctly.

Introduces new issue:
startCharacteristicNotifications returns 'Operation failed' after disconnect
https://crbug.com/719581

This reverts commit db614933.
originally from https://codereview.chromium.org/2625013003/

TBR=ortuno@chromium.org,rkc@google.com,scheib@chromium.org,perja@opera.com
BUG=680099,719581

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

The support in Chromium is still under development. Adding test to make
sure we don't accidentally enable them before we can actually handle
them.

BUG=707128
TEST=This CL addes more test cases.

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

This CL gets the DataSaver setting on the main thread and pass it to
WebWorkerFetchContext (WorkerFetchContextImpl, ServiceWorkerFetchContextImpl).
WorkerFetchContext::AddAdditionalRequestHeaders() will check the flag on the
worker thread and set "Save-Data" header if enabled.

BUG=443374,722774

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

This CL:

 - Moves about 80 [ Skip ] lines from TestExpectations into NeverFixTests.
 - Sorts the expectations lines for imported web-platform-tests.
 - Changes the comment at the top of NeverFixTests to suggest that the file
   can also be used for manual tests that we never want to run automatically.

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

Two unit tests are crashing a lot on Android, disabling those
tests in background_loader_offliner_unittest.cc for Android
only.

BUG=722556
TBR=dewittj@chromium.org

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

Revert of [PageLoadMetrics] Enable AdsPageLoadMetrics by default (patchset #1 id:1 of https://codereview.chromium.org/2887773003/ )

Reason for revert:
Reverting based on findit-for-me notification

Original issue's description:
> [PageLoadMetrics] Enable AdsPageLoadMetrics by default
>
> Now that we've addressed all of the known causes of crashes, it's time to
> reenable.
>
> BUG=708570
>
> Review-Url: https://codereview.chromium.org/2887773003
> Cr-Commit-Position: refs/heads/master@{#472248}
> Committed: https://chromium.googlesource.com/chromium/src/+/6c08596b40d0fb9e1d66ca96b65b2d1a707fcb26

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

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

Certain editing commands shouldn't be enabled when they are user-triggered and
the selection doesn't have focus. This patch adds checking of
FrameSelection::SelectionHasFocus() in EnabledInEditableText to fix
the behavior of these commands:

BackwardDelete
DeleteBackward
DeleteBackwardByDecomposingPreviousCharacter
DeleteForward
DeleteToBeginningOfLine
DeleteToBeginningOfParagraph
DeleteToEndOfLine
DeleteToEndOfParagraph
DeleteToMark
DeleteWordBackward
DeleteWordForward
ForwardDelete
IgnoreSpelling
InsertBacktab
InsertHTML
InsertLineBreak
InsertNewline
InsertParagraph
InsertTab
InsertText
MoveBackward
MoveDown
MoveForward
MoveLeft
MovePageDown
MovePageUp
MoveParagraphBackward
MoveParagraphForward
MoveRight
MoveToBeginningOfDocument
MoveToBeginningOfLine
MoveToBeginningOfParagraph
MoveToBeginningOfSentence
MoveToEndOfDocument
MoveToEndOfLine
MoveToEndOfParagraph
MoveToEndOfSentence
MoveToLeftEndOfLine
MoveToLeftEndOfLineAndModifySelection
MoveToRightEndOfLine
MoveToRightEndOfLineAndModifySelection
MoveUp
MoveWordBackward
MoveWordForward
MoveWordLeft
MoveWordRight
Yank
YankAndSelect

BUG=713607, 722925
TEST=editing/selection/arrow_key_with_unfocused_selection.html

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

renamed TaskRunner::RunsTasksOnCurrentThread() to
TaskRunner::RunsTasksInCurrentSequence() in //chrome

BUG=665062
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.win:win10_chromium_x64_rel_ng

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

To allow changes to where fields are stored, ComputedStyle code should
not refer directly to a group, as that code will break when we change
groups. This patch removes references to visual_data_ in ComputedStyle,
replacing with generated or handwritten getters. This patch does not
remove references within diffing functions as those will soon be
generated.

BUG=710938

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

Revert of Rename AXObject to AXObjectImpl in modules/ and web/ (patchset #9 id:160001 of https://codereview.chromium.org/2858493002/ )

Reason for revert:
Landed wrong patchset.

Original issue's description:
> Rename AXObject to AXObjectImpl in modules/ and web/
>
> Rename AXObject to AXObjectImpl in modules/ and web/, and introduce an
> empty abstract parent class in core/. This is pre-work to remove usages
> of AXObjectImpl from web/, which will use AXObject instead.
>
> This patch does not rename the usages in core/ since these should use
> the new abstract object.
>
> BUG=715382
>
> Review-Url: https://codereview.chromium.org/2858493002
> Cr-Commit-Position: refs/heads/master@{#471663}
> Committed: https://chromium.googlesource.com/chromium/src/+/234813ae8d52f622a1664e11365642751e8f26ba

TBR=aboxhall@chromium.org,dmazzoni@chromium.org,haraken@chromium.org,mkwst@chromium.org,slangley@chromium.org
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=715382

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

This file does not exist under src/util/ anymore and thus should not be listed in the GN file.

Checked on Linux with the following command line:

$ gn check --force gn-ninja-out/local-dbg --args='is_debug=true
is_component_build=true symbol_level=1' //third_party/leveldatabase/*

BUG=502447

Change-Id: I9ce11e7f575b82345b95def661e5bb11396ba124
Reviewed-on: https://chromium-review.googlesource.com/503707Reviewed-by: Joshua Bell <jsbell@chromium.org>
Commit-Queue: Thiago Farina <tfarina@chromium.org>
Cr-Commit-Position: refs/heads/master@{#472280}

I got back some preliminary UMA data from Canary experiment that
confirm the timestamp drift relative to the timing of v-sync signal
which makes BeginImplFrameLatency2 UMA to be all over the place with
a distribution that is spread evenly in the entire 0 - 16667 range.

This happens because D3D V-sync signal is generated based on v-blank
event for a display that contains contains the window (the current
display), but the timestamp is obtained from DWM which is based on
the most recent v-blank timing for the primary monitor. So if a
secondary monitor frequency is even slightly different that causes
v-sync / RAF timestamp drift that is clearly visible on some websites
like vsynctester.com.

One possible solution is to capture the timestamp when v-blank event
is received, but that seems to be a bit less smooth than the DWM
timestamp. So the compromise is to use DWM timing only when running on
a primary monitor; otherwise use the v-blank wake-up timestamp.
I've verified that this fixes BeginImplFrameLatency2 UMA distribution on
my setup where the secondary monitor refresh rate seems to differ from
the primary monitor by about 0.15 Hz.

BUG=467617,680639
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/2874833003
Cr-Commit-Position: refs/heads/master@{#472279}

BUG=708175
TBR=pwnall@chromium.org
NOTRY=true

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

Add tests for certificates that pass X509CertificateBytes parsing, but fail in CertVerifyProc. Fix iOS.

BUG=711077,671420

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

This CL removes both android_webview_apk and android_webview_test_apk
build targets as we are deprecating those two names.

Please refer the crbug for more info, and the new target names are
webview_instrumentation_apk and webview_instrumentation_test_apk
correspondingly.

New build names are WebViewInstrumentation.apk and
WebViewInstrumentationTest.apk

BUG=701938

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

BUG=722320,722307

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

BUG=596760

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

The code related to update-webgl-conformance-tests hasn't been
touched in a long time, so I suspect it's unused.

BUG=677561

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

Setting the "WindowsTargetPlatformVersion" XML element seems to
be necessary in VS2017. Single-file compilations do not work
without it. It does not cause any problems in older VS versions.

R=brettw@chromium.org
BUG=

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

V8 CL rolled into V8 few days ago.

BUG=chromium:719008
TBR=dgozman@chromium.org

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

BUG=582836
TBR=kojii@chromium.org
NOTRY=true

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

BUG=705115

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

1- TransferBufferManager is the only implementation. Devirtualize.
2- Refcounting is not needed, move to single-ownership.
3- Separating Initialize from the constructor is not needed, so merge.

Bug: None
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: I06eff971424c8400e48fdd75068614086c56a1c3
Reviewed-on: https://chromium-review.googlesource.com/506607Reviewed-by: Zhenyao Mo <zmo@chromium.org>
Commit-Queue: Antoine Labour <piman@chromium.org>
Cr-Commit-Position: refs/heads/master@{#472268}

Currently, the default behavior when resizing ElementsPanel will switch from
vertical to horizontal mode when it's width is < 680px. At super small widths,
this makes the `Styles` and `Computed` views both tall and cramped for space.

This CL adds another rule when the width is < 415px, giving the `Computed` view
its own separate tab. A screenshot of DevTools at this width is in the crbug.

BUG=619064

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

Port https://sites.google.com/a/chromium.org/dev/developers/design-documents/threading/suble-threading-bugs-and-patterns-to-avoid-them to in-repo documentation.  This is a straight port is almost no content editing.

BUG=

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

Build: https://build.chromium.org/p/chromium.infra.cron/builders/rebaseline-o-matic/builds/693016

https://chromium.googlesource.com/chromium/src/+/79799bced99fd

BUG=719535
TBR=robertphillips@google.com

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

audio-testing.js no longer exists so remove references to this file.
These files weren't using anything from audio-testing.js anyway.

BUG=704967
TEST=none

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

Change-Id: I76afae213d8dfbaa61c76b33384b083cd5d3c4bc
Reviewed-on: https://chromium-review.googlesource.com/506464Reviewed-by: Philip Rogers <pdr@chromium.org>
Commit-Queue: Daniel Cheng <dcheng@chromium.org>
Cr-Commit-Position: refs/heads/master@{#472263}

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

Previously InterfaceFactoryImpl instances are using SrongBinding such
that they will only be destroyed on connection error. However, we could
hit an issue when MediaService is being destructed, the MojoMediaClient
is already destroyed, and an InterfaceFactoryImpl still tries to access
the MojoMediaClient.

This CL ensures that when MediaService is being destructed, all
InterfaceFactoryImpl will also be destroyed to avoid the issue.

BUG=721965

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

This patch makes sure the following names follows the JavaScript naming rules in the use_new_js_bindings mode:
- method names;
- parameter names;
- struct/union field names;
- namespaces.

BUG=718614

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

This patch changes the names of functions as follows:
PaintOpBuffer::approximateBytesUsed -> bytes_used
PaintOpBuffer::approximateOpCount -> size

Also changes related functions in other classes where appropriate.
Also changes the return type of new size function to be size_t.

R=enne@chromium.org, danakj@chromium.org
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2

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

R=thakis@chromium.org
TBR=mseaborn@chromium.org
BUG=607968

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

This fixes an issue where extraneous flags were being set on some
devices and causing the flag check to fail.

BUG=719013

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