- 28 Apr, 2018 16 commits
-
-
Kevin Bailey authored
The code has already been modified to assume this option if enabled. This CL actually removes the option. Bug: 780835 Change-Id: I6bc84f7eb7763136d1effc40659f65297396b7b8 Reviewed-on: https://chromium-review.googlesource.com/1033574 Commit-Queue: Kevin Bailey <krb@chromium.org> Reviewed-by:
Justin Donnelly <jdonnelly@chromium.org> Cr-Commit-Position: refs/heads/master@{#554586}
-
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/179dd1e01695..65dbd539443e $ git log 179dd1e01..65dbd5394 --date=short --no-merges --format='%ad %ae %s' Created with: roll-dep src/third_party/depot_tools 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: I227729f4c5d0cbe9977b58433eed4a2d5930b640 Reviewed-on: https://chromium-review.googlesource.com/1033723Reviewed-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@{#554585}
-
John Budorick authored
Also fix fuchsia layout test imports. TBR=dpranke@chromium.org,kmarshall@chromium.org,sahel@chromium.org Bug: 805057,837786 Change-Id: I9b90c985e95df053c2badce8038d57bef3f7cd8d Reviewed-on: https://chromium-review.googlesource.com/1034097Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#554584}
-
David Black authored
Part of a clean up effort to move assistant out of app_list. Bug: b:77637813 Change-Id: I5634cfb5511dfaf4bc000b3a007f4edcec1caba4 Reviewed-on: https://chromium-review.googlesource.com/1031737 Commit-Queue: David Black <dmblack@google.com> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Xiaohui Chen <xiaohuic@chromium.org> Cr-Commit-Position: refs/heads/master@{#554583}
-
Thomas Guilbert authored
This fixes the crashes that have started appearing on Canary. Bug: 837836 Change-Id: I47b0b68a436fcc81970277d4cf20939d08b39bf5 Reviewed-on: https://chromium-review.googlesource.com/1034002Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Commit-Queue: Thomas Guilbert <tguilbert@chromium.org> Cr-Commit-Position: refs/heads/master@{#554582}
-
Bo Liu authored
Overall goal is to switch Crash related classes on Android to use content::ChildProcessTerminationInfo, which has more relevant termination information than is currently available. This is a set of related changes / fixes in order to make intentions and behaviors on Android clearer. * Change BrowserChildProcessHostImpl::OnChildDisconnected to always call BrowserChildProcessKilled (and BrowserChildProcessHostDelegate OnProcessCrashed), and explicitly document this behavior. This is because Android cannot distinguish between crashes and kills, and basing this decision on whether there was oom protection makes very little sense. Generally base::TerminationStatus does not convey much information. One concrete behavior change is that BrowserChildProcessKilled (and OnProcessCrashed) will now be called even when chrome app is in the background. * Corresponding to above, have CrashDumpObserver listen to BrowserChildProcessKilled instead of BrowserChildProcessCrashed. Crashed was never called anyway. Listening to Killed allows populating additional termination info. * Stop using base::TerminationStatus in crash, and use bool was_oom_protected_status instead, which is the only useful bit of information conveyed by TerminationStatus on Android. This also forces code to stop abusing other values of TerminationStatus for other purposes, ie added an explicit bool normal_exit instead of relying on NORMAL_EXIT. * FastShutdownStarted case is treated as a normal exit. Previously it was treated as a "normal crash", although with other fields unpopulated. components/data_reduction_proxy TBR TBR=tbansal@chromium.org Bug: 693484 Change-Id: I887dcff000ccbc38bbb9e4159c1c5bb27a0c8876 Reviewed-on: https://chromium-review.googlesource.com/1025042 Commit-Queue: Bo <boliu@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Maria Khomenko <mariakhomenko@chromium.org> Reviewed-by:
Peter Wen <wnwen@chromium.org> Reviewed-by:
Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#554581}
-
Adam Langley authored
(This is a reland of 3a01c225.) Kim points out in the bug that the current logic (see https://chromium-review.googlesource.com/c/chromium/src/+/991073) doesn't allow dev tools to be focused, so it's not possible to trigger webauthn requests from the dev tools console. Nasko thought that the addition of a ContentBrowserClient API was too large a hammer too. I've not been able to come up with any cleaner solution however, but this change perhaps makes the hammer more reasonable: In order to address the devtools issue, this change switches the focus check to requiring that the tab be active in the focused window. (This also matches what the cryptotoken extension does.) This means that it has to chrome/ logic, rather than content/ logic, but we already had to add an API to ContentBrowserClient for this and we can reuse that to solve this problem too. This change also moves the final focus check when registering to the point after the press, but before any permissions bubble. This is because it turns out that we have five different UI focus behaviours across four platforms and doing the focus check after trying to close the bubble was a mistake. Additionally, this change adds an interactive_ui_test to ensure that we don't regress. TBR=thestig,nasko Bug: 827266,836203 Change-Id: I954164d79e8dd375060fcc780a800904fe547c12 Reviewed-on: https://chromium-review.googlesource.com/1033336 Commit-Queue: Adam Langley <agl@chromium.org> Reviewed-by:
Balazs Engedy <engedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#554580}
-
Jian Li authored
In DownloadController.requestFileAccess, we should check if storage permission is already granted before requesting it. Bug: 836397 Change-Id: Idadcdfc3b055696b8f7194e5d63bdee54a8d4959 Reviewed-on: https://chromium-review.googlesource.com/1029116Reviewed-by:
Ted Choc <tedchoc@chromium.org> Commit-Queue: Jian Li <jianli@chromium.org> Cr-Commit-Position: refs/heads/master@{#554579}
-
David Benjamin authored
There are only two places left which use the notion of an "error domain", //components/error_page, and //ios. These two use them for different purposes and so they should define their own error domains. In particular, using net::kErrorDomain for NSError in iOS was never appropriate, since that string isn't globally-unique. See also: https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/ErrorHandlingCocoa/ErrorObjectsDomains/ErrorObjectsDomains.html#//apple_ref/doc/uid/TP40001806-CH202-SW1 error_page_generator.mm required some care as it then took an NSError domain and passed it into //components/error_page. //components/error_page will actually DCHECK if given an unknown error domain and, in another codepath, assume all unknown error domains are HTTP error codes! Fortunately, following up the call chain a bit, ErrorPageGenerator is only called with a web::NetErrorFromError error code, so this doesn't come up. I've switched that to recognize the //net NSErrorDomain and, if not, default to net::ERR_FAILED with a NOTREACHED() to mirror the prior DCHECK. But probably that interface should be revised a bit. Bug: none Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I693b7e2a004bd91f852a0718b93e40b3b30f66ea Reviewed-on: https://chromium-review.googlesource.com/1017604 Commit-Queue: David Benjamin <davidben@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#554578}
-
Carlos Knippschild authored
The existing script to migrate the DB schema for Prefeching of Offline Pages from version 1 to 2 had references to code that could change as the schema evolved. This is very unsafe as changes to any exterior value should not change the migration code. This CL addresses that by making it not refer to any outside values. Bug: 701939 Change-Id: Ia0e1aff10a7fb381d9caeb747d056ff12cba2d1f Reviewed-on: https://chromium-review.googlesource.com/772975 Commit-Queue: Carlos Knippschild <carlosk@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Reviewed-by:
Yafei Duan <romax@chromium.org> Cr-Commit-Position: refs/heads/master@{#554577}
-
Wez authored
Launchpad is a library provided in the Fuchsia SDK. Currently, its header files and link library are provided both in the sysroot and in an SDK package following Fucshia's standard layout (https://fuchsia.googlesource.com/docs/+/master/development/sdk/layout.md). This updates callers of the library to depend on the library through the package form so it can be removed from the sysroot. Also adds a fuchsia_sdk_lib_pkg() template, for use by SDK packages provided as pre-compiled libs. This is a rebase and extension of jamesr@'s https://chromium-review.googlesource.com/c/chromium/src/+/1026511 Bug: 707030 Change-Id: I79c6af4d63d991fe0e8e0a3285d9de815f68ce03 Reviewed-on: https://chromium-review.googlesource.com/1032854 Commit-Queue: Wez <wez@chromium.org> Reviewed-by:
James Robinson <jamesr@chromium.org> Reviewed-by:
Jochen Eisinger <jochen@chromium.org> Cr-Commit-Position: refs/heads/master@{#554576}
-
Daniel Cheng authored
For some reason, they aren't getting captured when they're captured inside the conditional block that eventually calls DumpWithoutCrashing(). Bug: 728219 Change-Id: Ic8ec936bdac4eadfe65da836dc033ae9e7c245e5 Reviewed-on: https://chromium-review.googlesource.com/1033997 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Commit-Queue: Alex Moshchuk <alexmos@chromium.org> Reviewed-by:
Alex Moshchuk <alexmos@chromium.org> Cr-Commit-Position: refs/heads/master@{#554575}
-
Rune Lillesveen authored
When we had style sharing, we could not share ComputedStyle between elements which had pseudo ::before or ::after which relied on attr() content values from the originating element. This is not necessary for MatchedPropertiesCache since we don't share the pseudo style cache like we would do for shared ComputedStyle objects. This gets rid of a dependency on mutable parent ComputedStyle. Bug: 813068 Change-Id: I4b03e5b05c9bca595188d6506fdd2907e854596a Reviewed-on: https://chromium-review.googlesource.com/1032940Reviewed-by:
Philip Rogers <pdr@chromium.org> Commit-Queue: Rune Lillesveen <futhark@chromium.org> Cr-Commit-Position: refs/heads/master@{#554574}
-
https://chromium.googlesource.com/angle/angle.git/+log/f1bcd017c95c..696434382359 $ git log f1bcd017c..696434382 --date=short --no-merges --format='%ad %ae %s' 2018-04-27 geofflang D3D11: Reorder VAO synchronization. 2018-04-24 geofflang D3D: Don't test for device loss as frequenty in tight loops. Created with: roll-dep src/third_party/angle BUG=chromium:837002,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: I8a622a7902ca0130d43619a3319c7a5d32b140b5 Reviewed-on: https://chromium-review.googlesource.com/1034003 Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#554573}
-
https://chromium.googlesource.com/chromiumos/chromite.git/+log/6b3c651c4ed2..7c0757b099e5 $ git log 6b3c651c4..7c0757b09 --date=short --no-merges --format='%ad %ae %s' 2018-04-27 levarum Revert "Copy ndk-translation artifacts when uprevving android" 2018-04-24 dgarrett som: Remove Gardener SOM tree. 2018-04-13 bmgordon cbuildbot: Use previous build state for chroot reuse 2018-04-25 dgarrett config_lib: Define LUCI_BUILDER constants. 2018-04-25 jclinton appengine/cq_stats: Set the Python SSL version to latest 2018-04-26 jclinton appengine: Update the SDK version to support new SSL 2018-04-26 jclinton lib/clactions*: Refactor into separate clactions_metrics for ts_mon Created with: roll-dep src/third_party/chromite BUG=chromium:None,chromium:829665,chromium:824550,chromium:826372,chromium:826372,chromium:826372 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: Ic622485c6d32417602f5aff7bc2cc02047e49f9a Reviewed-on: https://chromium-review.googlesource.com/1033960 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@{#554572}
-
George Burgess IV authored
This was previously split out to support defining different profiles for Chrome and Chromium. Since we no longer need to do this, it's simplest to have this all inline. (In particular, this refactor makes using this profile on Linux like, 2 lines of diff.) No functionality change is intended. Bug: 805098 Test: `rgrep fprofile-sample-use out | wc -l` gives a high number Change-Id: Ice635b71942f10ef1aed9e2fd9872ac06f598803 Reviewed-on: https://chromium-review.googlesource.com/1024637Reviewed-by:
agrieve <agrieve@chromium.org> Commit-Queue: George Burgess <gbiv@chromium.org> Cr-Commit-Position: refs/heads/master@{#554571}
-
- 27 Apr, 2018 24 commits
-
-
https://chromium.googlesource.com/catapult.git/+log/6761f26bd9da..32f02b5d358b $ git log 6761f26bd..32f02b5d3 --date=short --no-merges --format='%ad %ae %s' 2018-04-27 estevenson [Dashboard] Add M67 to speed releasing dashboard. Created with: roll-dep src/third_party/catapult BUG=chromium:765703 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: I479f2280e69e53010f02a3f557aefd6f7a23a920 Reviewed-on: https://chromium-review.googlesource.com/1033993 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@{#554570}
-
Oksana Zhuravlova authored
Bug: 799482 Change-Id: I925b822f11ac2464ef25cfb77175e9beeb3fa57b Reviewed-on: https://chromium-review.googlesource.com/1033925 Commit-Queue: Daniel Cheng <dcheng@chromium.org> Reviewed-by:
Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#554569}
-
John Budorick authored
Accidentally reenabled while disabling tests on Linux. TBR=dougarnett@chromium.org,timbrown@chromium.org Bug: 831838,837822 Change-Id: I6d0f126e73ff8eac0ed1b48c2eaa149a2be70e04 Reviewed-on: https://chromium-review.googlesource.com/1034014Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#554568}
-
Nicholas Verne authored
This runs the uninstall flow for crostini, which now includes stopping the concierge service then unloading the cros-termina component. Bug: 822507 Change-Id: I31cc40a3ad6732a55ff3f9ae47468751d996c6cf Reviewed-on: https://chromium-review.googlesource.com/1032353 Commit-Queue: Nicholas Verne <nverne@chromium.org> Reviewed-by:
Ben Wells <benwells@chromium.org> Reviewed-by:
Timothy Loh <timloh@chromium.org> Cr-Commit-Position: refs/heads/master@{#554567}
-
Eric Orth authored
Useful to override to create specialized mocks or fakes that only care about a portion of the overall interface. Especially useful for mocks or fakes owned outside the network service code as the mocks can create an interface dependency on only the overridden methods. Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Id03bdcaab6d991fb1a79ddcaf4eaf0612fe64d52 Reviewed-on: https://chromium-review.googlesource.com/1026418 Commit-Queue: Eric Orth <ericorth@chromium.org> Reviewed-by:
Yuri Wiitala <miu@chromium.org> Reviewed-by:
Helen Li <xunjieli@chromium.org> Cr-Commit-Position: refs/heads/master@{#554566}
-
Jun Choi authored
DeviceOperation is an interface for Ctap{Register, Sign}Operation and U2f{Register, Sign}Operation, and will be a branching point for separation of logic between CTAP and U2F requests. CtapRegisterOperation is an implementation of DeviceOperation and is owned by MakeCredentialTask to handle dispatching CTAP MakeCredential request to an authenticator. Bug: 798573 Change-Id: I290ab7b3f686d4418d783723a0ab510f27362462 Reviewed-on: https://chromium-review.googlesource.com/1028792 Commit-Queue: Jun Choi <hongjunchoi@chromium.org> Reviewed-by:Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#554565}
-
https://skia.googlesource.com/skia.git/+log/5cb5c74efdac..34a388edbd27 $ git log 5cb5c74ef..34a388edb --date=short --no-merges --format='%ad %ae %s' 2018-04-27 csmartdalton ccpr: Remove canMakeClipProcessor 2018-04-27 liyuqian Reland "Reland "Reland "Exercise the threaded backend in test bots""" 2018-04-27 brianosman Add ASYNC trace macros 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: Id312399ace42628fc464aecb6415d5f2dad79707 Reviewed-on: https://chromium-review.googlesource.com/1033717Reviewed-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@{#554564}
-
Matthew Jones authored
A bug in TabModelObserver is causing a discrepancy in behavior between phone and tablet (which the tests depend on). This patch disables the tests on tablet while the issue is resolved. Bug: 837809 Change-Id: I344255ad86651da484cab20569239e851b376d9d Reviewed-on: https://chromium-review.googlesource.com/1034006 Commit-Queue: Matthew Jones <mdjones@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Reviewed-by:
Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#554563}
-
Matt Menke authored
In a followup CL, invalid UTF-8 characters will never be unescaped by the text methods. Bug: 829868 Change-Id: I71df885005f5faa9f72ac5ff6cfb2789c64ae846 Reviewed-on: https://chromium-review.googlesource.com/1024771Reviewed-by:
John Abd-El-Malek <jam@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Reviewed-by:
Varun Khaneja <vakh@chromium.org> Reviewed-by:
Julian Pastarmov <pastarmovj@chromium.org> Reviewed-by:
Taiju Tsuiki <tzik@chromium.org> Reviewed-by:
Matt Giuca <mgiuca@chromium.org> Commit-Queue: Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#554562}
-
Chris Harrelson authored
M66 has launched, so we should stop running extra tests that might slow down the CQ. Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_layout_ng;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Iee9c064ea3fcdcb896ac268dbef94fad91853c0a Reviewed-on: https://chromium-review.googlesource.com/1034009Reviewed-by:
Steve Kobes <skobes@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#554561}
-
braveyao authored
In some recent testings it looks like there may be multiple PERMANENT capture error reported before the track is ended due to the first error report. So it's necesary to log the permanent error only once for one session. Bug: 831756 Change-Id: Icf969f0a1f1d644bd362811cf8658cabb7d3a3a1 Reviewed-on: https://chromium-review.googlesource.com/1033818Reviewed-by:
Zijie He <zijiehe@chromium.org> Commit-Queue: Weiyong Yao <braveyao@chromium.org> Cr-Commit-Position: refs/heads/master@{#554560}
-
Kristian H. Kristensen authored
The underlay only works for opaque buffers, so make sure we don't promote anything with alpha. Bug: 789288 Change-Id: Id39b2220cec7258699668f09b149a5d62a0cec41 Reviewed-on: https://chromium-review.googlesource.com/1026872Reviewed-by:
David Reveman <reveman@chromium.org> Commit-Queue: Kristian H. Kristensen <hoegsberg@chromium.org> Cr-Commit-Position: refs/heads/master@{#554559}
-
Ahmed Fakhry authored
BUG=836026 Change-Id: Icf5733c284cfd8cc0fddc121ab781d30de65535a Reviewed-on: https://chromium-review.googlesource.com/1025222 Commit-Queue: Ahmed Fakhry <afakhry@chromium.org> Reviewed-by:
Peter Kasting <pkasting@chromium.org> Cr-Commit-Position: refs/heads/master@{#554558}
-
https://pdfium.googlesource.com/pdfium.git/+log/3241bb3e98c0..65c255ad2076 $ git log 3241bb3e9..65c255ad2 --date=short --no-merges --format='%ad %ae %s' 2018-04-27 tsepez Fix illegal cast in core/fxge/win32/fx_win32_gdipext.cpp 2018-04-27 thestig Remove CJBig2_HuffmanTable::InitCodes(). 2018-04-27 tsepez Replace void* with CFX_GifPalette* in fxcodec 2018-04-27 hnakashima Fix focused day disappears in XFA MonthCalendar. 2018-04-27 hnakashima Update caret after changes that could move lines in XFA edit. 2018-04-27 thestig Change CJBig2_HuffmanTable to use struct JBig2HuffmanCode. 2018-04-27 thestig Adjust CJBig2_Context::HuffmanAssignCode(). 2018-04-27 tsepez Remove arguments from CPDFSDK_FormFillEnvironment methods (part 2) 2018-04-27 thestig Simplify CJBig2_Context::DecodeSymbolIDHuffmanTable(). 2018-04-27 thestig Use std::function in CJBig2_GRDProc. 2018-04-27 hnakashima Fix backspace on 1st character of a line erases the line break. 2018-04-27 thestig Disambiguate methods in CJBig2_GRDProc. 2018-04-27 thestig Simplify CJBig2_Context::FindReferredSegmentByTypeAndIndex(). 2018-04-27 tsepez Avoid potential duplicate unique_ptr to CPDF_Document from CPDFXA_Context. 2018-04-27 tsepez Remove argument from CPDFSDK_FormFillEnvironment::GetCurrentPage 2018-04-27 thestig Add DEPS include_rules for xfa/fwl. 2018-04-27 thestig Fix undefined behavior in AGG. 2018-04-27 thestig Do validation earlier in CPDF_SampledFunc::v_Init(). (try 2) Created with: roll-dep src/third_party/pdfium BUG=chromium:836361,chromium:709781,chromium:836361,chromium:746232 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: I3f93d2671a6cc6f854854c61eea6104ae84cc21e Reviewed-on: https://chromium-review.googlesource.com/1033831 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@{#554557}
-
Karandeep Bhatia authored
This CL plumbs the ResourceContext to the WebRequestInfo structure through WebRequestProxyingURLLoaderFactory. This is necessary to fix NTPInterceptionWebRequestAPITest.NTPRendererRequestsHidden for the network servicification path. BUG=797461, 721414 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: I2957c1ece0f8f7b44f15156aeb5e554b3a91ff01 Reviewed-on: https://chromium-review.googlesource.com/1029549Reviewed-by:
Ken Rockot <rockot@chromium.org> Commit-Queue: Karan Bhatia <karandeepb@chromium.org> Cr-Commit-Position: refs/heads/master@{#554556}
-
Dan Sanders authored
This fails on Mac when run in a headless content_shell. Also fixes an incorrect thread post bug in VdaVideoDecoder. Bug: 522298 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: I09d397ee108920fc35e8aadc45d9b47840eb8686 Reviewed-on: https://chromium-review.googlesource.com/1033834Reviewed-by:
Dale Curtis <dalecurtis@chromium.org> Reviewed-by:
Frank Liberato <liberato@chromium.org> Commit-Queue: Dan Sanders <sandersd@chromium.org> Cr-Commit-Position: refs/heads/master@{#554555}
-
Min Qin authored
In incognito mode, we shouldn't write to the metadata cache. BrowserContext::GetPath() will return the path of the normal profile in incognito mode. BUG=778425 Change-Id: If1cd5d819d0b754010f60f776226c34d92e383f4 Reviewed-on: https://chromium-review.googlesource.com/1033540Reviewed-by:
Xing Liu <xingliu@chromium.org> Commit-Queue: Min Qin <qinmin@chromium.org> Cr-Commit-Position: refs/heads/master@{#554554}
-
Andrey Kosyakov authored
Bug: 836732 Change-Id: Ia8b1fbfca1cb1a30289337571e8db6a3163751ac Reviewed-on: https://chromium-review.googlesource.com/1033721Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#554553}
-
Emircan Uysaler authored
This CL makes sure that we produce a keyframe for every 100 frames when MediaRecorder uses HW encoder. Bug: 837450 Change-Id: Ifa1a674012085aa122a8e907ca1f88cb8a949e55 Reviewed-on: https://chromium-review.googlesource.com/1033181 Commit-Queue: Emircan Uysaler <emircan@chromium.org> Reviewed-by:
Miguel Casas <mcasas@chromium.org> Cr-Commit-Position: refs/heads/master@{#554552}
-
Jay Civelli authored
Reenables the contentc browser test GetUserMediaVideoCaptureBrowserTest. RecoverFromCrashInVideoCaptureProcess with the network service as it seems to be passing now. Tbr: dpranke@chromium.org Bug: 729848 Cq-Include-Trybots: master.tryserver.chromium.linux:linux_mojo Change-Id: Iac6246c6f7205dee0791da39f1e08da39a62af47 Reviewed-on: https://chromium-review.googlesource.com/1033955Reviewed-by:
Jay Civelli <jcivelli@chromium.org> Commit-Queue: Jay Civelli <jcivelli@chromium.org> Cr-Commit-Position: refs/heads/master@{#554551}
-
Chris Harrelson authored
Filters which move pixels may move content which is originally outside of the cull rect or ancestor clip rects back into view. Therefore we should adjust the painting root to prevent ancestor clips and cull rects from clipping or culling out the source of the filter. Bug: 804670 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I6e1032ecef5fcb5eabb6080bd2f17d8c479dc1e8 Reviewed-on: https://chromium-review.googlesource.com/1033347Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Commit-Queue: Chris Harrelson <chrishtr@chromium.org> Cr-Commit-Position: refs/heads/master@{#554550}
-
Esmael El-Moslimany authored
Bug: 836988 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I9d8123653fd77bc9142459d666a0e9697569cd94 Reviewed-on: https://chromium-review.googlesource.com/1029127Reviewed-by:
Demetrios Papadopoulos <dpapad@chromium.org> Commit-Queue: Esmael El-Moslimany <aee@chromium.org> Cr-Commit-Position: refs/heads/master@{#554549}
-
Steven Holte authored
This avoids the implicit recording on destruction and the need to have ukm_recorder to construct an entry, and no longer needs to be part of the UkmRecorder API. Since we now are DCHECK'ing against recording metrics not in ukm.xml we no longer need to have this API behind a friend whitelist. TBR=charleszhao,mlamouri,chrishtr Bug: 754892 Change-Id: I944ae698bd5ad4146411d5e11b945e21c1b52a98 Reviewed-on: https://chromium-review.googlesource.com/1027112 Commit-Queue: Steven Holte <holte@chromium.org> Reviewed-by:
Robert Kaplow <rkaplow@chromium.org> Reviewed-by:
Timothy Dresser <tdresser@chromium.org> Reviewed-by:
Filip Gorski <fgorski@chromium.org> Reviewed-by:
Bryan McQuade <bmcquade@chromium.org> Cr-Commit-Position: refs/heads/master@{#554548}
-
Sunny Sachanandani authored
The original motivation for this code was to workaround early swap ack on freon, but since freon isn't a thing anymore and we have a display compositor that actually performs the swap, it should be ok to get rid of it. This should improve throughput similar to main frame before activation. R=brianderson BUG=311213 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I5108c74a57ba201d25999989de8da4c7f81a5176 Reviewed-on: https://chromium-review.googlesource.com/1031892Reviewed-by:
Brian Anderson <brianderson@chromium.org> Commit-Queue: Sunny Sachanandani <sunnyps@chromium.org> Cr-Commit-Position: refs/heads/master@{#554547}
-