- 29 Jan, 2018 40 commits
-
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/dbac8ca4f5d7..bc4869293d78 $ git log dbac8ca4f..bc4869293 --date=short --no-merges --format='%ad %ae %s' 2018-01-29 thestig Fix identical if/else blocks. 2018-01-29 thestig Remove duplicate assignments in sycc420_to_rgb(). 2018-01-29 thestig Roll V8 to ee5d9a0c. Created with: roll-dep src/third_party/pdfium BUG=805881,805881 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: Id658b85157c1332418f1509c8653ecab8390cfae Reviewed-on: https://chromium-review.googlesource.com/890631 Commit-Queue: <pdfium-deps-roller@chromium.org> Reviewed-by: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#532498}
-
Christopher Grant authored
As requested by the test team, this will eliminate a manual test case. BUG=806279 R=ymalik Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.linux:linux_vr;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel Change-Id: I5e08e7b8e30e65cebf680d3449d15a0b293de7b1 Reviewed-on: https://chromium-review.googlesource.com/890392Reviewed-by:
Yash Malik <ymalik@chromium.org> Commit-Queue: Christopher Grant <cjgrant@chromium.org> Cr-Commit-Position: refs/heads/master@{#532497}
-
Luna Lu authored
unsized-media sets a default size for laying out intrinsically sized images / videos. The feature is allowed for all origins by default. When disabled, any intrinsically sized images / videos will be using the default dimension (300 x 150) to prevent relayout. Bug: 623682 Change-Id: I1bd105ded2580349e2a2b177e87b70417403cb59 Reviewed-on: https://chromium-review.googlesource.com/865159Reviewed-by:
Chris Harrelson <chrishtr@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Reviewed-by:
Ian Clelland <iclelland@chromium.org> Commit-Queue: Luna Lu <loonybear@chromium.org> Cr-Commit-Position: refs/heads/master@{#532496}
-
Kevin McNee authored
In the OOPIF-based guest implementation of attachment, the contentWindow of the internal iframe element is used to identify the local frame to swap for a remote frame. When the guest is destroyed, we currently don't "reset" the internal iframe element so that it can be used for another attachment. When we try to attach, the frame is still remote, which causes GuestViewInternalCustomBindings::AttachIframeGuest to be unable to find the frame to swap. We now replace our internal iframe element upon guest destruction, so that the frame is local when we reattach. Bug: 800407 Change-Id: I4424793c7c0230d62a424765bc3c288080b2e0b9 Reviewed-on: https://chromium-review.googlesource.com/871292Reviewed-by:
Paul Meyer <paulmeyer@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#532495}
-
Brian Salomon authored
Bug: 801783 Change-Id: I78455d0a50869e7a81b9bfe8613e53bcb5973c80 Reviewed-on: https://chromium-review.googlesource.com/884441Reviewed-by:
Robert Phillips <robertphillips@google.com> Commit-Queue: Brian Salomon <bsalomon@chromium.org> Cr-Commit-Position: refs/heads/master@{#532494}
-
Xianzhu Wang authored
Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: I7cdba1e84b853eeaff281093555e8757dc1616ed Reviewed-on: https://chromium-review.googlesource.com/882396Reviewed-by:
Tien-Ren Chen <trchen@chromium.org> Commit-Queue: Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#532493}
-
danakj authored
But drop them when they become available if we're in the SUSPENDED memory state. Deleting busy resorces doesn't actually free any memory, it just moves the ownership solely into the ResourceProvider until they become available. Instead, we can keep them in the busy list, and when they are available, decide to keep them or not based on the memory state instead. This should also improve things by reducing memory footprint in the SUSPENDED memory state if OnPurgeMemory() isn't being called but new tile resources are being made and released. R=ericrk@chromium.org, vmpstr@chromium.org Bug: 730660 Cq-Include-Trybots: master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I24e1a6f38c88fa278566849174a315bff71ed375 Reviewed-on: https://chromium-review.googlesource.com/886862 Commit-Queue: danakj <danakj@chromium.org> Reviewed-by:
vmpstr <vmpstr@chromium.org> Reviewed-by:
Eric Karl <ericrk@chromium.org> Cr-Commit-Position: refs/heads/master@{#532492}
-
v8-autoroll authored
Summary of changes available at: https://chromium.googlesource.com/v8/v8/+log/c1717655..787ecbb3 Please follow these instructions for assigning/CC'ing issues: https://github.com/v8/v8/wiki/Triaging%20issues Please close rolling in case of a roll revert: https://v8-roll.appspot.com/ This only works with a Google account. CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_optional_gpu_tests_rel;master.tryserver.chromium.mac:mac_optional_gpu_tests_rel;master.tryserver.chromium.win:win_optional_gpu_tests_rel;master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=hablich@chromium.org,machenbach@chromium.org,kozyatinskiy@chromium.org,sergiyb@chromium.org Change-Id: I29e1ddf6f049f287169f92091db9ec3c3df9c503 Reviewed-on: https://chromium-review.googlesource.com/889869Reviewed-by:
v8 autoroll <v8-autoroll@chromium.org> Commit-Queue: v8 autoroll <v8-autoroll@chromium.org> Cr-Commit-Position: refs/heads/master@{#532491}
-
David 'Digit' Turner authored
This CL supports the case where the crazy linker dlsym() or dlclose() functions are passed handles that were opened from the system linker. To do that, a set of valid crazy-linker-created dlopen handles is maintained, and compared against at runtime. + Remove crazy::Set<> template, and introduce crazy::PointerSet that has better lookup performance. BUG=805861 R=agrieve@chromium.org,pasko@chromium.org,rmcilroy@chromium.org,ymalik@chromium.org Change-Id: I6b965c2dabc4ac9f5340ce028444e4195aa2b580 Reviewed-on: https://chromium-review.googlesource.com/885763 Commit-Queue: David Turner <digit@chromium.org> Reviewed-by:
Egor Pasko <pasko@chromium.org> Reviewed-by:
agrieve <agrieve@chromium.org> Cr-Commit-Position: refs/heads/master@{#532490}
-
stkhapugin@chromium.org authored
Moves the _locationBar ivar (it becomes _locationBarController) to LocationBarCoordinator. Moves popup coordinator, too. Bug: 785253, 806028 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: I4bea5a069fc2caeb76f27d204eeddd8d5b281430 Reviewed-on: https://chromium-review.googlesource.com/883504 Commit-Queue: Stepan Khapugin <stkhapugin@chromium.org> Reviewed-by:
Gauthier Ambard <gambard@chromium.org> Cr-Commit-Position: refs/heads/master@{#532489}
-
Alexander Timin authored
s/kFrameLoading_kControl/kFrameLoadingControl/ R=alexclarke@chromium.org Change-Id: I3011117f013c6c257255bc06da38b2d6833e7981 Reviewed-on: https://chromium-review.googlesource.com/891321Reviewed-by:
Alex Clarke <alexclarke@chromium.org> Commit-Queue: Alexander Timin <altimin@chromium.org> Cr-Commit-Position: refs/heads/master@{#532488}
-
James Forshaw authored
Map any generic access rights during the AC access check. Also fixes a bug in the test code which preventing testing of generic access masks. Bug: 760977 Change-Id: Iec729a40a25c187b758f61e3128930e7f4bfb02d Reviewed-on: https://chromium-review.googlesource.com/890454Reviewed-by:
Will Harris <wfh@chromium.org> Commit-Queue: James Forshaw <forshaw@chromium.org> Cr-Commit-Position: refs/heads/master@{#532487}
-
kylechar authored
There is a timer in SurfaceManager that periodically checks for old temporary reference and deletes them. This timer was always running to drive an UMA stat, even if there are no temporary references to expire. Running the timer unnecessarily caused a regression on Android in the thread_times.key_idle_power_cases benchmark. Stop the timer when there are no temporary references. This won't wake the thread unnecessary and should save power. Also replace the UMA stat, instead of tracking the number of expired temporary references per timer tick, track the reason each temporary reference was removed. This new UMA metric provides more information and isn't coupled to the implementation. Bug: 804696 Cq-Include-Trybots: master.tryserver.chromium.android:android_optional_gpu_tests_rel Change-Id: I5e4385a72ca597de29094b8352ed7908916f4cf0 Reviewed-on: https://chromium-review.googlesource.com/883775 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by:
Fady Samuel <fsamuel@chromium.org> Reviewed-by:
Steven Holte <holte@chromium.org> Cr-Commit-Position: refs/heads/master@{#532486}
-
Denis Yaroshevskiy authored
Change-Id: Ia0a8721df2c0503ea180cfdeba507173a051c62b Reviewed-on: https://chromium-review.googlesource.com/868510 Commit-Queue: Dmitry Skiba <dskiba@chromium.org> Reviewed-by:
Brett Wilson <brettw@chromium.org> Reviewed-by:
Dmitry Skiba <dskiba@chromium.org> Cr-Commit-Position: refs/heads/master@{#532485}
-
Elly Fong-Jones authored
Bug: None Change-Id: I2338cec9d47330ccf0a76de1d8894d632fea267b Reviewed-on: https://chromium-review.googlesource.com/891280Reviewed-by:
Nico Weber <thakis@chromium.org> Reviewed-by:
Mark Mentovai <mark@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#532484}
-
Nico Weber authored
R=mpearson@chromium.org Change-Id: I5508ab909d436112fdf537a7ae9b6828f862fb93 Reviewed-on: https://chromium-review.googlesource.com/889779Reviewed-by:
Mark Pearson <mpearson@chromium.org> Cr-Commit-Position: refs/heads/master@{#532483}
-
catapult-deps-roller@chromium.org authored
https://chromium.googlesource.com/catapult.git/+log/82f653f9a44e..e99634797bd6 $ git log 82f653f9a..e99634797 --date=short --no-merges --format='%ad %ae %s' 2018-01-29 perezju [Devil] Use ListProcesses for testRestartAdbd 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. CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.android:android_optional_gpu_tests_rel TBR=sullivan@chromium.org Change-Id: Ibc667a9370472f3b7116a568e7fb5f89301fbd92 Reviewed-on: https://chromium-review.googlesource.com/890390 Reviewed-by: <catapult-deps-roller@chromium.org> Commit-Queue: <catapult-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#532482}
-
Weidong Guo authored
Changes: Assign 0 to the timeout of the notification to make resolution change irreversible in Kiosk mode. BUG=794135 Change-Id: Icb17ba31c15fe25476674142db5ae317f6d4e6af Reviewed-on: https://chromium-review.googlesource.com/887224 Commit-Queue: Weidong Guo <weidongg@chromium.org> Reviewed-by:
Ahmed Fakhry <afakhry@chromium.org> Cr-Commit-Position: refs/heads/master@{#532481}
-
Mustaq Ahmed authored
Reasons for removal: - UserGestureIndicatorTest.Timeouts asserts the same check. - The removed test seems to have a sloppy time expectation. - This is blocking UserActivationV2 (with extended expiry). Bug: 804903 Change-Id: I8960e670772cb4a0bb306206e86a873e1c226f77 Reviewed-on: https://chromium-review.googlesource.com/881523Reviewed-by:
Dave Tapuska <dtapuska@chromium.org> Commit-Queue: Mustaq Ahmed <mustaq@chromium.org> Cr-Commit-Position: refs/heads/master@{#532480}
-
https://skia.googlesource.com/skia.git/+log/ba3ed69a4ee4..cca86f386cfd $ git log ba3ed69a4..cca86f386 --date=short --no-merges --format='%ad %ae %s' 2018-01-29 fmalita [skottie] Hierarchical animators 2018-01-29 egdaniel Blacklist additional perf skp on win Intel Vulkan 2018-01-29 egdaniel Remove Ops whose lazy proxies fail to instantiate on flush 2018-01-29 kjlubick Disable flaky perf benchmark 2018-01-29 angle-skia-autoroll Roll skia/third_party/externals/angle2/ ffa4cbb6f..d1434c048 (6 commits) 2018-01-29 fmalita Revert "[skottie] Cache attached assets" 2018-01-29 bsalomon Allow printf style var args for messages in REPORTER_ASSERT. 2018-01-29 fmalita [skottie] Reduce parser verbosity 2018-01-29 egdaniel Add createTextureProxy function to ProxyProvider that takes an SkImage 2018-01-29 ethannicholas fixed creating a vector from another vector in SPIR-V output 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;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 TBR=bungeman@chromium.org Change-Id: I36366d9be68d3c011f892a19dc65a8e8d13e3598 Reviewed-on: https://chromium-review.googlesource.com/890821Reviewed-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@{#532479}
-
Mirko Raus authored
With tablet style Bookmarks modified in: https://codereview.chromium.org/2837643002 Bookmarks tests need to be modified. Bug: 806217 Change-Id: I761e7fdade7a311d65b4411be2d98b3cffe071d0 Reviewed-on: https://chromium-review.googlesource.com/889338Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#532478}
-
Sylvain Defresne authored
os.path.getmtime(path) returns a float value with non-zero fractional timestamp when using APFS+. Truncate the value when comparing to the saved timestamp in remap file. Bug: 324079 Change-Id: I92e100548d96d66df0da2d72258015763a5b6d5c Reviewed-on: https://chromium-review.googlesource.com/891358Reviewed-by:
Nico Weber <thakis@chromium.org> Commit-Queue: Sylvain Defresne <sdefresne@chromium.org> Cr-Commit-Position: refs/heads/master@{#532477}
-
Michael Wasserman authored
This reverts commit 751cc857. Reason for revert: Incorrect speculative revert. The actual change causing this failure was likely http://crrev.com/c/887282 for Issue 805937 That CL actually touched the failing test, and its revert caused the bot to pass: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/25915 I'm reverting this seemingly incorrect speculative revert of my CL. Original change's description: > Revert "mus: Fix event processing for touch events in unified mode" > > This reverts commit 1423331f. > > Reason for revert: speculative attempt due to unhappy MSan ChromiumOS bot. > See also crbug.com/806638. > > Original change's description: > > mus: Fix event processing for touch events in unified mode > > > > This refines my earlier work done in: http://crrev.com/c/842467 > > > > Move event target nulling and root location use to WindowTreeClient, to ensure events reach MusUnifiedEventTargeter. > > Otherwise, WindowEventDispatcher::GetInitialEventTarget will use the touch gesture recognizer's target (returned by GetPriorityTargetInRootWindow) before performing the mirror display transformation on the event's location. > > > > Inline DispatchEventToTarget for additional clarity. > > > > Bug: 805714 > > Test: Mus touch events work in Chrome OS's unified desktop mode. > > Change-Id: Ic4844c85cc24a9f9885e7705ff27cb57b305d245 > > Reviewed-on: https://chromium-review.googlesource.com/887250 > > Commit-Queue: Michael Wasserman <msw@chromium.org> > > Reviewed-by: Scott Violet <sky@chromium.org> > > Cr-Commit-Position: refs/heads/master@{#532030} > > TBR=sky@chromium.org,msw@chromium.org > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Bug: 805714 > Change-Id: I02333546f91d0d0f495153c8556aa9820612673d > Reviewed-on: https://chromium-review.googlesource.com/890841 > Reviewed-by: Makoto Shimazu <shimazu@chromium.org> > Commit-Queue: Makoto Shimazu <shimazu@chromium.org> > Cr-Commit-Position: refs/heads/master@{#532328} TBR=sky@chromium.org,msw@chromium.org,shimazu@chromium.org Change-Id: I6c7b91429abdddb4f9db881c40c64f53141bc707 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 805714 Reviewed-on: https://chromium-review.googlesource.com/890822Reviewed-by:
Michael Wasserman <msw@chromium.org> Commit-Queue: Michael Wasserman <msw@chromium.org> Cr-Commit-Position: refs/heads/master@{#532476}
-
Adithya Srinivasan authored
A variant of this was originally generated by clusterfuzz (https://crbug.com/803352), and caught a bug in https://crrev.com/c/829613. This test tests nameditem cache invalidation for form control collections, even after their form owner is disconnected from the document. Change-Id: I2b3389926e2bed9dd9173a9f26649c5aa55526c6 Reviewed-on: https://chromium-review.googlesource.com/878661 Commit-Queue: Adithya Srinivasan <adithyas@chromium.org> Reviewed-by:
Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#532475}
-
Peter Beverloo authored
The serialization contains the p256dh and auth values of the push subscription encoded per base64url. Normally it's valid to keep the trailing equal-sign padding, but the Push API specifically forbids this. Bug: 801555 Change-Id: I370f01b6aa398ccd407502c967391ea573352381 Reviewed-on: https://chromium-review.googlesource.com/868295 Commit-Queue: Peter Beverloo <peter@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Cr-Commit-Position: refs/heads/master@{#532474}
-
Jeremy Roman authored
For consistency between the methods, have all converters take v8::Isolate*, as all of the ones that presently take v8::Local<v8::Context> implicitly assume it to be the current context anyhow. Bug: None Change-Id: I565fb54bdc7a1d5ac1796ef5d7c1c69ca83290bb Reviewed-on: https://chromium-review.googlesource.com/876563 Commit-Queue: Jeremy Roman <jbroman@chromium.org> Reviewed-by:
Devlin <rdevlin.cronin@chromium.org> Cr-Commit-Position: refs/heads/master@{#532473}
-
Emily Hanley authored
Bug: 806257 Change-Id: Ia59da0cc8e3f5892aa1d250958bec6619ecc3ab2 Reviewed-on: https://chromium-review.googlesource.com/888097Reviewed-by:
Ned Nguyen <nednguyen@google.com> Reviewed-by:
Emily Hanley <eyaich@chromium.org> Commit-Queue: Emily Hanley <eyaich@chromium.org> Cr-Commit-Position: refs/heads/master@{#532472}
-
Nico Weber authored
This CL was uploaded by git cl split. R=dmazzoni@chromium.org Bug: 177475 Change-Id: I4d9385c71c2cdb2202a768f547636bb8ecb3d298 Reviewed-on: https://chromium-review.googlesource.com/889949 Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org> Reviewed-by:
Dominic Mazzoni <dmazzoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#532471}
-
Elly Fong-Jones authored
These source files produce object files with names that conflict with views object files when both are linked into the same product, so rename them to avoid those conflicts. This is a part of the broken-up CL 874730. Bug: 804950 Change-Id: Ic8967241efbe43a5a86bc76fd951296687c24e22 Reviewed-on: https://chromium-review.googlesource.com/889735Reviewed-by:
Sidney San Martín <sdy@chromium.org> Commit-Queue: Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#532470}
-
Takuto Ikuta authored
This is follow up of https://chromium-review.googlesource.com/612493 We want to use the config around L923. strip_debug_info is important to improve link time for buildbot having slow disk throughput. This fix improves link time of some large binaries on android_n5x_swarming_rel. With this fix: https://ci.chromium.org/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/349367 build stats are taken from https://chromium-build-stats.appspot.com/ninja_log/2018/01/29/slave613-c4/ninja_log.slave613-c4.chrome-bot.20180128-192032.27201.gz/table?dedup=true - link time of android_clang_arm/libwebviewchromium.so : 1m14.193s - link time of android_clang_arm/libmonochrome.so : 36.539s Without this fix: https://ci.chromium.org/buildbot/tryserver.chromium.android/android_n5x_swarming_rel/349369 build stats are taken from https://chromium-build-stats.appspot.com/ninja_log/2018/01/29/slave153-c4/ninja_log.slave153-c4.chrome-bot.20180128-192916.30766.gz/table?dedup=true - link time of android_clang_arm/libwebviewchromium.so : 2m31.189s - link time of android_clang_arm/libmonochrome.so : 1m33.087s Change-Id: Ie180c37ff7924c7577ecbfe18da80a2bc4c20a98 Reviewed-on: https://chromium-review.googlesource.com/890687 Commit-Queue: Takuto Ikuta <tikuta@google.com> Reviewed-by:
John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#532469}
-
Simeon Anfinrud authored
This permission is unneeded now that the system settings menu is using the leanback preference library. Bug: 72498412 Test: Install Cast apk on Android TV with OPR2 and access Cast settings menu. Change-Id: Ib9955bec23394c4ca232aced20e0798d7cbad261 Reviewed-on: https://chromium-review.googlesource.com/889987Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Cr-Commit-Position: refs/heads/master@{#532468}
-
pdfium-deps-roller@chromium.org authored
https://pdfium.googlesource.com/pdfium.git/+log/6e8a3e62e171..dbac8ca4f5d7 $ git log 6e8a3e62e..dbac8ca4f --date=short --no-merges --format='%ad %ae %s' 2018-01-29 rharrison Fix typo introduced in cleanup of IsHyphen Created with: roll-dep src/third_party/pdfium BUG=805881 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: I889d9a09816c347b29c300e25a372bae61acb5db Reviewed-on: https://chromium-review.googlesource.com/891299 Reviewed-by: <pdfium-deps-roller@chromium.org> Commit-Queue: <pdfium-deps-roller@chromium.org> Cr-Commit-Position: refs/heads/master@{#532467}
-
Jayson Adams authored
The recent change to separate TabWindowController from NSWindowController also removed it from the responder chain. As a result messages like swipeWithEvent: no longer reached the BrowserWindowController BWC. This cl reinserts the TabWindowController into the responder chain, before the window's NSWindowController. Bug: 805241 Change-Id: Ifa46d3e66ad306f26d860fe196f445e76fe293a4 Reviewed-on: https://chromium-review.googlesource.com/887326 Commit-Queue: Jayson Adams <shrike@chromium.org> Reviewed-by:
Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#532466}
-
Julia Tuttle authored
This is a reland of https://chromium-review.googlesource.com/c/chromium/src/+/853172 which was rolled back as part of https://chromium-review.googlesource.com/c/chromium/src/+/879041. Bug: 799616 Change-Id: I5dae7cd41b55dd2151a06e0f708743cd19b85b41 Cq-Include-Trybots: master.tryserver.chromium.android:android_cronet_tester;master.tryserver.chromium.mac:ios-simulator-cronet Reviewed-on: https://chromium-review.googlesource.com/881401 Commit-Queue: Julia Tuttle <juliatuttle@chromium.org> Reviewed-by:
Miriam Gershenson <mgersh@chromium.org> Cr-Commit-Position: refs/heads/master@{#532465}
-
Sahel Sharify authored
I wasn't able to reproduce the failures on my local windows official build and that's why I think this might be a timing issue with other machines. Bug: 800822 Change-Id: I46d4c58e8169897b613045553dfbaf131de46089 Reviewed-on: https://chromium-review.googlesource.com/880885 Commit-Queue: Sahel Sharifymoghaddam <sahel@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Cr-Commit-Position: refs/heads/master@{#532464}
-
Julia Tuttle authored
Apps should be able to use NEL, but they shouldn't share state with the main request context. This is a reland of https://chromium-review.googlesource.com/c/chromium/src/+/875050 which was rolled back as part of https://chromium-review.googlesource.com/c/chromium/src/+/879041 Bug: 875050 Change-Id: Ibcdcf852cea1d8da16f812d175a15d4bdefd3e14 Reviewed-on: https://chromium-review.googlesource.com/881402 Commit-Queue: Julia Tuttle <juliatuttle@chromium.org> Reviewed-by:
Matt Menke <mmenke@chromium.org> Cr-Commit-Position: refs/heads/master@{#532463}
-
John Z Wu authored
One test that verifies fetching, filling, and clearing. Bug: 778764 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Icc484e63fc0787bdfcedca95a15528111e4d7dcb Reviewed-on: https://chromium-review.googlesource.com/841342 Commit-Queue: John Wu <jzw@chromium.org> Reviewed-by:
Eugene But <eugenebut@chromium.org> Cr-Commit-Position: refs/heads/master@{#532462}
-
sczs authored
Everytime a swipe to navigate gesture is started fullscreen will be disabled causing the Toolbar to show. This CL also updates the ToolbarModel state to reflect that the toolbar is being shown when the model is disabled, this includes updating y_contest_offset even when the model is disabled. ToolbarModelUnitTests have been updated accordingly. Bug: 799088 Cq-Include-Trybots: master.tryserver.chromium.mac:ios-simulator-cronet;master.tryserver.chromium.mac:ios-simulator-full-configs Change-Id: Icfe7f6caafcc6113b7c2909472fcbb1a0384cb79 Reviewed-on: https://chromium-review.googlesource.com/879126 Commit-Queue: Sergio Collazos <sczs@chromium.org> Reviewed-by:
Kurt Horimoto <kkhorimoto@chromium.org> Reviewed-by:
Justin Cohen <justincohen@chromium.org> Cr-Commit-Position: refs/heads/master@{#532461}
-
Raymond Toy authored
Move the mostly common test code into one method and use it for testing all of the valid and invalid attributes for the stereo panner. Bug: 717655 Change-Id: If6d6347d1fe303bc1c79acf141aa12ce3b04259c Reviewed-on: https://chromium-review.googlesource.com/879535Reviewed-by:
Hongchan Choi <hongchan@chromium.org> Commit-Queue: Raymond Toy <rtoy@chromium.org> Cr-Commit-Position: refs/heads/master@{#532460}
-
sangwoo.ko authored
When global FindPboard is updated, find result is cleared and next/prev buttons are disabled. But this makes users unabled to find the exact string without editing text. So enable buttons unless the text from pboard is empty. Bug: 739735 Change-Id: Iad72ae1e0a88165774fecd21fe26c3a2949c60c4 Reviewed-on: https://chromium-review.googlesource.com/888229Reviewed-by:
Robert Sesek <rsesek@chromium.org> Commit-Queue: Robert Sesek <rsesek@chromium.org> Cr-Commit-Position: refs/heads/master@{#532459}
-