- 13 Jun, 2018 40 commits
 - 
- 
Kevin McNee authored
Gesture pinch events have |needs_wheel_event| and |zoom_disabled| as browser side implementation details. There is no need to include these fields when sending the event to the renderer. Bug: 787924 Change-Id: I2a2e4f9376f384801bb2ca73b5a0e20d589e92e1 Reviewed-on: https://chromium-review.googlesource.com/1087433Reviewed-by:
David Bokan <bokan@chromium.org> Reviewed-by:
Ken Buchanan <kenrb@chromium.org> Commit-Queue: Kevin McNee <mcnee@chromium.org> Cr-Commit-Position: refs/heads/master@{#567043}
 - 
Saman Sami authored
Android does not support partial swaps. Instead, when an empty swap is requested we're supposed to commit overlay planes. Bug: 850546 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: If77156732b4f43872855a8ad676c42a5e9842678 Reviewed-on: https://chromium-review.googlesource.com/1099266Reviewed-by:
kylechar <kylechar@chromium.org> Commit-Queue: Saman Sami <samans@chromium.org> Cr-Commit-Position: refs/heads/master@{#567042}
 - 
Zhiheng Vincent Li authored
Move CastCommandLineHelper.java from internal to upstream and let CastBrowserHelper use it to initialize command line. Bug: b/110021629 Test: cast_shell_junit_tests Change-Id: I9940e95e26be072bc1762e60f6c6ae541277d34c Reviewed-on: https://chromium-review.googlesource.com/1097548Reviewed-by:
Simeon Anfinrud <sanfin@chromium.org> Commit-Queue: Zhiheng(Vincent) Li <vincentli@google.com> Cr-Commit-Position: refs/heads/master@{#567041}
 - 
pdfium-chromium-autoroll authored
https://pdfium.googlesource.com/pdfium.git/+log/4c48b107c6d1..571053a44f87 git log 4c48b107c6d1..571053a44f87 --date=short --no-merges --format='%ad %ae %s' 2018-06-13 hnakashima@chromium.org Mark modified content streams as dirty in CPDF_PageObjectHolder. 2018-06-13 npm@chromium.org Remove CFX_FixedBufGrow from cpdf_renderstatus.cpp 2018-06-13 npm@chromium.org Remove CFX_FixedBufGrow from cpdf_colorspace.cpp 2018-06-13 npm@chromium.org Remove almost all usages of CFX_FixedBufGrow from cpdf_dibsource.cpp 2018-06-13 npm@chromium.org Make CJBig2_Context::GetHuffmanTable return const 2018-06-13 npm@chromium.org Remove CFX_FixedBufGrow from fx_codec_icc.cpp 2018-06-13 art-snake@yandex-team.ru Merge the offset calculation logic within CPDF_HintsTable. Created with: gclient setdep -r src/third_party/pdfium@571053a44f87 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: I28c83054cdeb3c8977f25384a0a9fdcfaba05834 Reviewed-on: https://chromium-review.googlesource.com/1099815Reviewed-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@{#567040}
 - 
depot-tools-chromium-autoroll authored
https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/e09b6845cf72..31d539486159 git log e09b6845cf72..31d539486159 --date=short --no-merges --format='%ad %ae %s' 2018-06-13 recipe-roller@chromium.org Roll recipe dependencies (trivial). Created with: gclient setdep -r src/third_party/depot_tools@31d539486159 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: I36cd8ade79f1655bde894b6200092132072e55a8 Reviewed-on: https://chromium-review.googlesource.com/1099537Reviewed-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@{#567039}
 - 
John Budorick authored
This reverts commit 8dbc0c82. Reason for revert: broke https://ci.chromium.org/buildbot/chromium.chrome/Google%20Chrome%20Linux%20x64/32416 Original change's description: > Remove absolute paths from build files > > This CL removes all absolute paths to src and all usages of the out directory > name from all ninja files on most Linux builds. > > This means it's possible to mount your src directory at different locations and > still have incremental builds work. You can also rename your out directory. > > BUG=None > R=dpranke,caitkp,rockot > > Change-Id: I5ff5b0d2d60110755dde64e876c7f1a09d814b1a > Reviewed-on: https://chromium-review.googlesource.com/1091457 > Reviewed-by: Cait Phillips <caitkp@chromium.org> > Reviewed-by: Ken Rockot <rockot@chromium.org> > Reviewed-by: Dirk Pranke <dpranke@chromium.org> > Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> > Cr-Commit-Position: refs/heads/master@{#567020} TBR=caitkp@chromium.org,rockot@chromium.org,dpranke@chromium.org,thomasanderson@chromium.org Change-Id: Iec959adc2e8c82dd191f382c0cfb197aea406fc3 No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: None Reviewed-on: https://chromium-review.googlesource.com/1099979Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#567038}
 - 
Nico Weber authored
Bug: 852512 Change-Id: Ie118dcafd7dd46f62f03dbbd95091db24cb7ef26 Reviewed-on: https://chromium-review.googlesource.com/1099646Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: Nico Weber <thakis@chromium.org> Cr-Commit-Position: refs/heads/master@{#567037}
 - 
Adolfo Victoria authored
Move the proto parsing utilities into the WeeklyTime classes to make them reusable by policies that use the WeeklyTime protos. Added unit tests for the proto parsing. BUG=chromium:849506 TEST=WeeklyTimeInterval and WeeklyTime unit tests Change-Id: Ic6b327e9b75f6bf9f2c1d1d1a949ed30ec8f18f7 Reviewed-on: https://chromium-review.googlesource.com/1097698 Commit-Queue: Adolfo Higueros <adokar@google.com> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#567036}
 - 
John Budorick authored
password_manager_test_base includes chrome_password_manager_client but wasn't previously depending on it, meaning that the latter's mojo dependencies sometimes weren't built in time. Change-Id: I6b40d771d5744bd3e5975e47857e88b974a1d43a Reviewed-on: https://chromium-review.googlesource.com/1099493Reviewed-by:
Roger McFarlane <rogerm@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#567035}
 - 
John Budorick authored
Failing on Mac10.10. TBR=thakis@chromium.org Change-Id: I906d7ffe323f10630a21ec93f953b40c33fb0182 Bug: 852536 Reviewed-on: https://chromium-review.googlesource.com/1099715Reviewed-by:
John Budorick <jbudorick@chromium.org> Commit-Queue: John Budorick <jbudorick@chromium.org> Cr-Commit-Position: refs/heads/master@{#567034}
 - 
Allen Vicencio authored
Implements GetShares using an in-memory map to simulate a D-Bus call to SmbProvider daemon Bug: 757625 Change-Id: Icb655328ac0f4b766ecd6a05cb2d6439c15d9ac0 Reviewed-on: https://chromium-review.googlesource.com/1089851 Commit-Queue: Allen Vicencio <allenvic@chromium.org> Reviewed-by:
Zentaro Kavanagh <zentaro@chromium.org> Cr-Commit-Position: refs/heads/master@{#567033}
 - 
Morten Stenshorne authored
An infinite (or actually kTableMaxWidth, which is 1000000px) max logical width doesn't work well inside flexboxes, because LayoutFlexibleBox::ComputeInnerFlexBaseSizeForChild() will use a child's MaxPreferredLogicalWidth() as main axis extent if it's auto. Grid containers have similar issues with such tables, so avoid it for them as well. Don't apply this to deprecated flexbox, as that would cause two tests failures: * fast/block/block-size-integer-overflow.html * fast/deprecated-flexbox/box-size-integer-overflow.html Bug: 850566 Change-Id: I9984459801ac229575e8177a9d683088e723db6c Reviewed-on: https://chromium-review.googlesource.com/1095220 Commit-Queue: Morten Stenshorne <mstensho@chromium.org> Reviewed-by:
Christian Biesinger <cbiesinger@chromium.org> Reviewed-by:
Manuel Rego Casasnovas <rego@igalia.com> Cr-Commit-Position: refs/heads/master@{#567032}
 - 
Steven Bennetts authored
Bug: 829272 Cq-Include-Trybots: master.tryserver.chromium.linux:closure_compilation Change-Id: I79f0be1b2c5751cf62a5c4cfed1dad30fe1f6358 Reviewed-on: https://chromium-review.googlesource.com/1096285 Commit-Queue: Steven Bennetts <stevenjb@chromium.org> Reviewed-by:
Scott Chen <scottchen@chromium.org> Cr-Commit-Position: refs/heads/master@{#567031}
 - 
Tien-Ren Chen authored
This CL fixes two bugs with background-attachment:fixed computation: 1. Since CL 827671 we switch root layer in PaintInfo whenever the current transform node changes. This has a side effect that the painting root layer is no longer guaranteed to be a containing block ancestor of the current layout object, and AccumulatedScrollOffsetForFixedBackground() may skip over the painting root all the way to the root. This CL adds the skipping check to ensure we only collect scroll offsets up to the painting root. 2. FixedAttachmentPositioningArea() converts the viewport rect to the space of the current paint context. Prior to SPv175 the border box space of the painting root is always equivalent to the space of the current paint context, so FixedAttachmentPositioningArea() did just that. In SPv175 the painting root may not always create a paint offset translation in case of a squashed layer, thus can induce an offset from its transform node. This CL ensures the paint offset taken into account. BUG=529963,849854 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel;master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2 Change-Id: Ib2f84a619235ab4ec6a9bfc695b57aae9d41b808 Reviewed-on: https://chromium-review.googlesource.com/1098313 Commit-Queue: Tien-Ren Chen <trchen@chromium.org> Reviewed-by:
Xianzhu Wang <wangxianzhu@chromium.org> Cr-Commit-Position: refs/heads/master@{#567030}
 - 
Ryan Tseng authored
Using mixins Bug: 840631 Change-Id: I3f71eeb780897338bc6137594e8a0c76b77d6538 Reviewed-on: https://chromium-review.googlesource.com/1099969Reviewed-by:
Andrii Shyshkalov <tandrii@chromium.org> Commit-Queue: Ryan Tseng <hinoka@chromium.org> Cr-Commit-Position: refs/heads/master@{#567029}
 - 
Justin Cohen authored
iOS 12.0 introduced "Unable to build chain to root certificate." error. This error is mapped to CERT_STATUS_INVALID and CERT_STATUS_AUTHORITY_INVALID. Bug: 843236, 852534 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I90e4f9f97084c2ce06d4560b7e2d695f9b5454ae Reviewed-on: https://chromium-review.googlesource.com/1094700 Commit-Queue: Justin Cohen <justincohen@chromium.org> Reviewed-by:
Ryan Sleevi <rsleevi@chromium.org> Cr-Commit-Position: refs/heads/master@{#567028}
 - 
dpapad authored
Only a few WebUI pages need eval() and Function(). Explicitly allow those functions on such pages, instead of having it as the default for all WebUI pages. Specifically those pages are: chrome://appcache-internals chrome://components chrome://conflicts chrome://domain-reliability-internals chrome://download-internals crhome://dummyurl (only exists in tests) chrome://flags chrome://flash chrome://gpu chrome://indexeddb-internals chrome://invalidations chrome://nacl chrome://net-internals chrome://ntp-tiles-internals chrome://serviceworker-internals chrome://signin-internals chrome://snippets-internals chrome://supervised-user-internals chrome://sync-internals chrome://system which either use third_party/jstemplate/ in prod, or use Mock4JS during testing causing them to depend on eval/Function. Note that this CL does not affect iOS WebUI, since it currently provides no mechanism for overriding CSP for certain pages. Bug: 525224 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: Iaa3cbbca52a35dbd60e4d6fa42c5f324869859d9 Reviewed-on: https://chromium-review.googlesource.com/1065497 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by:
Nasko Oskov <nasko@chromium.org> Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Reviewed-by:
Scott Violet <sky@chromium.org> Cr-Commit-Position: refs/heads/master@{#567027}
 - 
Fabio Tirelo authored
Credit card assets already have a border, so adding a solid border around the icon on the UI is not needed. Bug: 852379 Change-Id: Ifb11db8a27e4914b9c7f373ae4bc35d2780358b2 Reviewed-on: https://chromium-review.googlesource.com/1099760Reviewed-by:
Evan Stade <estade@chromium.org> Commit-Queue: Fabio Tirelo <ftirelo@chromium.org> Cr-Commit-Position: refs/heads/master@{#567026}
 - 
Nektarios Paisios authored
Without the label, screen readers just say "button". Please advise on how to internationalize this. R=keishi@chromium.org Bug: 835365 Change-Id: I9fc4586df9d69387470b6c96313fb1eabdc66ab5 Tested: manually with input type=date Reviewed-on: https://chromium-review.googlesource.com/1094305Reviewed-by:
Aaron Leventhal <aleventhal@chromium.org> Reviewed-by:
Jay Civelli <jcivelli@chromium.org> Reviewed-by:
Kentaro Hara <haraken@chromium.org> Reviewed-by:
Keishi Hattori <keishi@chromium.org> Reviewed-by:
Nektarios Paisios <nektar@chromium.org> Commit-Queue: Nektarios Paisios <nektar@chromium.org> Cr-Commit-Position: refs/heads/master@{#567025}
 - 
Dmitry Gozman authored
- Collect DOM storages from all targets. - Instantiate InspectorDOMStorage agent for local roots, make it work with InspectedFrames. - Add a test which dumps storage security origins. - Do some origin checks on the frontend to avoid listing "opaque" security origins which do not support storage. - Check storage capabilities on the backend to avoid crasing renderer later. Bug: 850978, 844377 Change-Id: Ib4e2b444473638d896eb716fb7d9c587f5560335 Reviewed-on: https://chromium-review.googlesource.com/1096283 Commit-Queue: Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#567024}
 - 
Steven Valdez authored
https://boringssl.googlesource.com/boringssl/+log/5267ef7b4aa6385c0351ae10ce3092828cca4909..6ff2ba80b758025fdb137d9357dcbba0a9c04dbf BUG=none Change-Id: I0ed82f0919a02f8e8f57f67ff4fbdef8ea74f998 Reviewed-on: https://chromium-review.googlesource.com/1099422Reviewed-by:
David Benjamin <davidben@chromium.org> Commit-Queue: David Benjamin <davidben@chromium.org> Cr-Commit-Position: refs/heads/master@{#567023}
 - 
Sergiy Byelozyorov authored
R=dpranke@chromium.org Bug: 831171 Change-Id: I3e0614d5cdd8fcfc4e7377c4865e4b8bb9deafda Reviewed-on: https://chromium-review.googlesource.com/1095614Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org> Cr-Commit-Position: refs/heads/master@{#567022}
 - 
Daniel Bratell authored
In jumbo experiments in chrome/browser/ui, one of the problems is that there are many kBackgroundColor variables in anonymous namespaces that become the same anonymous namespaces in jumbo. This patch gives those constants unique names, matching their individual use. Bug: 773275 Change-Id: I8746a88a32f64a0645fd88e850e39d22bc635e19 Reviewed-on: https://chromium-review.googlesource.com/1078814 Commit-Queue: Daniel Bratell <bratell@opera.com> Reviewed-by:
Elly Fong-Jones <ellyjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#567021}
 - 
Tom Anderson authored
This CL removes all absolute paths to src and all usages of the out directory name from all ninja files on most Linux builds. This means it's possible to mount your src directory at different locations and still have incremental builds work. You can also rename your out directory. BUG=None R=dpranke,caitkp,rockot Change-Id: I5ff5b0d2d60110755dde64e876c7f1a09d814b1a Reviewed-on: https://chromium-review.googlesource.com/1091457Reviewed-by:
Cait Phillips <caitkp@chromium.org> Reviewed-by:
Ken Rockot <rockot@chromium.org> Reviewed-by:
Dirk Pranke <dpranke@chromium.org> Commit-Queue: Thomas Anderson <thomasanderson@chromium.org> Cr-Commit-Position: refs/heads/master@{#567020}
 - 
Roger Tawa authored
Captions: Use Enable/Control instead of Enables/Controls for consistency. Description: Use enabled/disabled instead of "set to true" and "set to false". Bug: 834939 Change-Id: Ie6552e87a9977046164874e981dde8e0725b1b6c Reviewed-on: https://chromium-review.googlesource.com/1088872 Commit-Queue: Roger Tawa <rogerta@chromium.org> Reviewed-by:
Maksim Ivanov <emaxx@chromium.org> Cr-Commit-Position: refs/heads/master@{#567019}
 - 
Maks Orlovich authored
We can do better than just dropping the DB, by dropping just the older versions of the duplicate cookie --- this is how M65 would behave when loading the store, so it preserves old behavior. This is probably too late for Chrome, but may help other embedders. Bug: 835990 Change-Id: I25bf87d46574346a47c05b35152fa3c42f9aa7a9 Reviewed-on: https://chromium-review.googlesource.com/1097204Reviewed-by:
Victor Costan <pwnall@chromium.org> Commit-Queue: Maks Orlovich <morlovich@chromium.org> Cr-Commit-Position: refs/heads/master@{#567018}
 - 
Albert Chaulk authored
//extensions/common/api/schema.gni defines extensions_api_uncompiled_sources During review, this variable was added to the common build file, but the error only occurs for clean builds, otherwise you would get a working build with flaky runtime behaviour The compiler attempts to find the common subdir between chromecast/common/extensions_api and extensions/common/api and puts the generated file in the root Merge-With: eureka-internal/171973 Bug: 852454 Test: Local clean build works Change-Id: Ic18a4a94c8c52b19a33750772fe855ce486eeec8 Reviewed-on: https://chromium-review.googlesource.com/1099373 Commit-Queue: Albert Chaulk <achaulk@google.com> Reviewed-by:
Luke Halliwell <halliwell@chromium.org> Reviewed-by:
Daniel Nicoara <dnicoara@chromium.org> Cr-Commit-Position: refs/heads/master@{#567017}
 - 
Peter Collingbourne authored
This allows compilers targeting the MS ABI to select the correct inheritance model for the member function pointer types ReaderFactory and WriterFactory using the complete type of Disassembler. It will allow us to enable the new Clang flag -fcomplete-member-pointers globally. Bug: 847724 Change-Id: Iead5fd97da5f67c336d8904a7a305a34df37065b Reviewed-on: https://chromium-review.googlesource.com/1098470 Commit-Queue: Peter Collingbourne <pcc@chromium.org> Reviewed-by:
Samuel Huang <huangs@chromium.org> Cr-Commit-Position: refs/heads/master@{#567016}
 - 
Ted Choc authored
The goal is to enable a simple data model without as much boiler plate code. It is initially optimized for frequent value changes to avoid object creation where not needed. BUG= Change-Id: I9ae02a322acefa3e6a9e6fce96fbe51e0aa2053b Reviewed-on: https://chromium-review.googlesource.com/1092797 Commit-Queue: Ted Choc <tedchoc@chromium.org> Reviewed-by:
David Trainor <dtrainor@chromium.org> Reviewed-by:
Bernhard Bauer <bauerb@chromium.org> Reviewed-by:
Matthew Jones <mdjones@chromium.org> Cr-Commit-Position: refs/heads/master@{#567015}
 - 
Chris Blume authored
viz_unittests currently do not build with Vulkan enabled. One of the unit tests is GL related and can probably just have a NOTREACHED(). The other unit test might benefit from a proper Vulkan code path. But the immediate goal is to just get Vulkan enabled by default quickly. We can later circle back and add a proper Vulkan code path to the overlay test if it would be correct. BUG=852531 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel Change-Id: I02a5bd1224575b452e92e3c147ab0c8d0efef76b Reviewed-on: https://chromium-review.googlesource.com/1099755Reviewed-by:
Antoine Labour <piman@chromium.org> Commit-Queue: Chris Blume <cblume@chromium.org> Cr-Commit-Position: refs/heads/master@{#567014}
 - 
Ben Wagner authored
Skia is cleaning up several headers. This is exposing several missing includes in Chromium code. Add the missing includes. Change-Id: I94fcd73d6cf067aacf8d8680e91e141ba57328f8 Reviewed-on: https://chromium-review.googlesource.com/1099429Reviewed-by:
Scott Violet <sky@chromium.org> Commit-Queue: Ben Wagner <bungeman@chromium.org> Cr-Commit-Position: refs/heads/master@{#567013}
 - 
Will Harris authored
Due to an issue with the spreadsheet, some rows were not handling negative enum values correctly so these values are corrected in this CL. In addition, the following names were also added based on metrics seen on Canary: util_win chrome_printing cdm profile_import file_util patch_service pdf_compositor BUG=767906 Change-Id: I8b599a38f7728d35d150efb06c9d9d47a6a6685b Reviewed-on: https://chromium-review.googlesource.com/1099366Reviewed-by:
Alexei Svitkine <asvitkine@chromium.org> Commit-Queue: Will Harris <wfh@chromium.org> Cr-Commit-Position: refs/heads/master@{#567012}
 - 
Sarah Hu authored
Add the following screens: 1. ShowErrorScreen 2. ShowSigninUI Bug: 828058 Change-Id: Id632fd49b4e1575ad5443b5f315067bdbe08efd9 Reviewed-on: https://chromium-review.googlesource.com/1098457Reviewed-by:
Jacob Dufault <jdufault@chromium.org> Commit-Queue: Xiaoyin Hu <xiaoyinh@chromium.org> Cr-Commit-Position: refs/heads/master@{#567011}
 - 
bsheedy authored
It turns out that UiDevice.getCurrentPackageName can return null, so handle that case to avoid NullPointerExceptions. TBR=mthiesse@chromium.org Change-Id: I8cb220d5df1d0f110b2bded7473f20ba7d859872 Reviewed-on: https://chromium-review.googlesource.com/1099838Reviewed-by:
Brian Sheedy <bsheedy@chromium.org> Reviewed-by:
Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#567010}
 - 
Weidong Guo authored
Changes: In old position system, dragging an item outside folder to a full page or dragging an item from another page to a full page will push last item on that page to next page. This is not allowed when apps grid gap is enabled to avoid breaking the next page's layout. Bug: 851563 Test: AppsGridGapTest.MoveItemToPreviousFullPageNotAllowed Change-Id: I2630436fe6c3aba9e2332668b5ab972b6dfec9ec Reviewed-on: https://chromium-review.googlesource.com/1095688Reviewed-by:
Xiyuan Xia <xiyuan@chromium.org> Commit-Queue: Weidong Guo <weidongg@chromium.org> Cr-Commit-Position: refs/heads/master@{#567009}
 - 
angle-chromium-autoroll authored
https://chromium.googlesource.com/angle/angle.git/+log/f462ac1be283..9327981248e7 git log f462ac1be283..9327981248e7 --date=short --no-merges --format='%ad %ae %s' 2018-06-13 oetuaho@nvidia.com Remove unnecessary checks when traversing nodes Created with: gclient setdep -r src/third_party/angle@9327981248e7 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=cwallez@chromium.org Change-Id: I97184ca916426fa98cb62f34853e4b243e4005b5 Reviewed-on: https://chromium-review.googlesource.com/1098944Reviewed-by:
angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: angle-chromium-autoroll <angle-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#567008}
 - 
Rohit Rao authored
BUG=805182 Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet Change-Id: I56014e0e6db3a8b1439d6306dee0041a3916eb31 Reviewed-on: https://chromium-review.googlesource.com/1086275Reviewed-by:
Sergio Collazos <sczs@chromium.org> Commit-Queue: Rohit Rao <rohitrao@chromium.org> Cr-Commit-Position: refs/heads/master@{#567007}
 - 
Becky Zhou authored
Padding added in two CL makes the back arrow start padding 2 times as expected. This CL removes the padding added specifically in search mode. Bug: 852410 Change-Id: I7194da772de3c8a07648fea1151185c19e93cab1 Reviewed-on: https://chromium-review.googlesource.com/1099770Reviewed-by:
Theresa <twellington@chromium.org> Commit-Queue: Becky Zhou <huayinz@chromium.org> Cr-Commit-Position: refs/heads/master@{#567006}
 - 
Alexei Filippov authored
Make it sorted for real. Fixes one source of flakiness. BUG=848398 Change-Id: I14e40f509e6363e06eeb5ed21bc239b59269bff8 Reviewed-on: https://chromium-review.googlesource.com/1095479Reviewed-by:
Dmitry Gozman <dgozman@chromium.org> Reviewed-by:
Arthur Sonzogni <arthursonzogni@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#567005}
 - 
Yi Gu authored
Upon posting removing keyframe models events from main we are using the name compositor_animation_ids_ which is misleading. This is a remaining issue from the renaming patch we had before. Bug: 807667 Change-Id: I985fff4d5073b69e8f3637b6273a6c72b7884e83 Reviewed-on: https://chromium-review.googlesource.com/1097582Reviewed-by:
Stephen McGruer <smcgruer@chromium.org> Commit-Queue: Yi Gu <yigu@chromium.org> Cr-Commit-Position: refs/heads/master@{#567004}
 
 -