- 03 Jul, 2018 40 commits
- 
- 
Lei Zhang authoredIt is only used in the ProfileInfoCache sub-class. Change-Id: Iba2c64e99821866d2ec3072d0b0ef412a1353e01 Reviewed-on: https://chromium-review.googlesource.com/1124107Reviewed-by: David Roger <droger@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#572301} 
- 
Pavel Feldman authoredChange-Id: I916652f378d2d43cb89c1aa546108d4bc38d8746 Reviewed-on: https://chromium-review.googlesource.com/1121420 Commit-Queue: Pavel Feldman <pfeldman@chromium.org> Reviewed-by: Andrey Kosyakov <caseq@chromium.org> Cr-Commit-Position: refs/heads/master@{#572300} 
- 
Reza.Zakerinasab authoredThe flakiness cannot be reproduced and suspected to be a result of recent changes in Skia color management code. Removing timeout tag for the tests and monitoring the bots to see if the flakiness returns. TBR=fserb@chromium.org Bug: 851746 Change-Id: Ie9f3e650140be4b9f27e9e7963ead014cee2bf49 Reviewed-on: https://chromium-review.googlesource.com/1124986Reviewed-by: Mohammad Reza Zakerinasab <zakerinasab@chromium.org> Commit-Queue: Mohammad Reza Zakerinasab <zakerinasab@chromium.org> Cr-Commit-Position: refs/heads/master@{#572299} 
- 
spqchan authoredTBR=carlosk@chromium.org Bug: 859681 Change-Id: Ic06fbd6b18c570e210ecd5e2e7bd8aa3802fc5cb Reviewed-on: https://chromium-review.googlesource.com/1123319 Commit-Queue: Sarah Chan <spqchan@chromium.org> Reviewed-by: Sarah Chan <spqchan@chromium.org> Cr-Commit-Position: refs/heads/master@{#572298} 
- 
Simeon Anfinrud authoredThis method was only used in one place, and was the only user of first() and changes(). unique() can be confusing because its scope exits before it gets new data, so it only happens to work when we don't care about deactivations. While these methods seemed like a good idea at the time, the possibility of Observable implementations that have multiple simultaneous activations make these methods tough to define. In the meantime, duplicate-filtering can be done with Controllers, which now have that built-in to set(). Bug: None Test: cast_base_junit_tests, cast_shell_junit_tests Change-Id: I86f542bbcdcb1a643400715733ecdaae438915ba Reviewed-on: https://chromium-review.googlesource.com/1119192Reviewed-by: Luke Halliwell <halliwell@chromium.org> Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Cr-Commit-Position: refs/heads/master@{#572297} 
- 
Jun Choi authoredImplement GetAssertion operation for CTAP2 software key. The implement GetAssertionOperation does not support multiple accounts per device and never returns UserEntity as a response. Bug: 829413 Change-Id: I58a43a032fc520494ea8fdbfb1c64959a04c6b82 Reviewed-on: https://chromium-review.googlesource.com/1117369 Commit-Queue: Jun Choi <hongjunchoi@chromium.org> Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org> Reviewed-by: Jan Wilken Dörrie <jdoerrie@chromium.org> Cr-Commit-Position: refs/heads/master@{#572296} 
- 
Christian Fremerey authoredThere is an incompatibility in the serialization of shared memory handles between the Mojo libraries used in Chromium vs. the older ones used in ChromiumOS. As a workaround, this CL allows clients of the video capture service to ask it to use raw file descriptors for sharing shared memory handles instead. Note, this is only supported on platforms that use file descriptors for base::SharedMemoryHandle, i.e. Posix platforms. Design Doc: https://docs.google.com/document/d/1cmtWA8yua5sqjZC0tswVUZwJeQR-KSZBr3Mj1jUagKk/edit?usp=sharing Bug: 847598 Change-Id: Ie269231b98149cbbc104ea62c1ff522c665cf187 Reviewed-on: https://chromium-review.googlesource.com/1043646 Commit-Queue: Christian Fremerey <chfremer@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org> Reviewed-by: Emircan Uysaler <emircan@chromium.org> Cr-Commit-Position: refs/heads/master@{#572295} 
- 
Charlie Harrison authoredThis mode was added as a way to hedge our feature against failing to launch metadata on the SafeBrowsing API side. This feature has been launched for a while and the parameter did not end up being used, so just remove it and associated tests. Bug: None Change-Id: Ib2e7d633da2d98478cee17dc3cdffe4e0b998d1b Reviewed-on: https://chromium-review.googlesource.com/1124763Reviewed-by: Avi Drissman <avi@chromium.org> Commit-Queue: Charlie Harrison <csharrison@chromium.org> Cr-Commit-Position: refs/heads/master@{#572294} 
- 
Philip Rogers authoredThese were disabled in https://codereview.chromium.org/34353004 but now run. The setting, multiTargetTapNotificationEnabled, is enabled on all platforms. This patch unskips the gesture tap highlight tests on mac and updates their expectations. Bug: 857501 Change-Id: I4bb6c91cd5b75d3fe30420759073f404a62d896f Cq-Include-Trybots: luci.chromium.try:linux-blink-gen-property-trees;luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Reviewed-on: https://chromium-review.googlesource.com/1121786Reviewed-by: David Bokan <bokan@chromium.org> Commit-Queue: Philip Rogers <pdr@chromium.org> Cr-Commit-Position: refs/heads/master@{#572293} 
- 
Brian Osman authoredThis is going away in Skia (no one wants linear blending), so this lets us land some Skia changes that alter image encode/decode/conversion behavior without breaking webkit unit tests. Some canvas color space layout tests do break, because they exercise multiple code paths in Skia. The paths that depend on image encode and decode are now using nonlinear blending, but the paths that just draw (to SW canvas) are hitting code that isn't updated yet. To make it easier to finish that migration, this change suppresses all of the transparent pixel tests within those five layout tests. Bug: chromium:859102 chromium:844145 Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_slimming_paint_v2;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I331b5dfc08e53d498fdc1449519b42f67de6d1af Reviewed-on: https://chromium-review.googlesource.com/1117026Reviewed-by: Mike Klein <mtklein@chromium.org> Reviewed-by: Mohammad Reza Zakerinasab <zakerinasab@chromium.org> Reviewed-by: Florin Malita <fmalita@chromium.org> Commit-Queue: Florin Malita <fmalita@chromium.org> Cr-Commit-Position: refs/heads/master@{#572292} 
- 
Qiyu Hu authored- Ensure that CCCD is added such that we can report the value change in CCCD to Android in OnNotificationStart/Stop() - Implement SendIndication() such that Android can call it Bug: b:78593133 Test: Pass Android CtsVerifier test on server sending indication/notification Change-Id: I8b1a4be6596555003d6d447b4518f37632a45370 Reviewed-on: https://chromium-review.googlesource.com/1119079 Commit-Queue: Qiyu Hu <qiyuh@google.com> Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org> Cr-Commit-Position: refs/heads/master@{#572291} 
- 
Lei Zhang authoredFix includes in chrome_signin_client.cc. Change-Id: I22bdfd9a625ed7e57cab49f61004d2334dea066a Reviewed-on: https://chromium-review.googlesource.com/1124108Reviewed-by: David Roger <droger@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#572290} 
- 
Simeon Anfinrud authoredreactive_java.md documents the expectation that the Observable produced by map() is not activated when its source is activated and the Function that transforms the activation data returns null, but this expectation was not properly tested. Bug: None Test: cast_base_junit_tests Change-Id: If8ff1764efe6790a9eeabcfeadf28b052ce924f5 Reviewed-on: https://chromium-review.googlesource.com/1120643Reviewed-by: Luke Halliwell <halliwell@chromium.org> Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Cr-Commit-Position: refs/heads/master@{#572289} 
- 
Quan Nguyen authoredAlso includes a run of "git cl format --js" on display_manager.js, and a fix for splash screens not stretching to the full screen. Bug: 836861 Change-Id: Icfe750f9d743b9608e120fc02f8cf16228be5c8f Reviewed-on: https://chromium-review.googlesource.com/1115977 Commit-Queue: Quan Nguyen <qnnguyen@chromium.org> Reviewed-by: Xiyuan Xia <xiyuan@chromium.org> Reviewed-by: Jacob Dufault <jdufault@chromium.org> Reviewed-by: Daniel Cheng <dcheng@chromium.org> Cr-Commit-Position: refs/heads/master@{#572288} 
- 
yiyix authoredRemove the following metrics: Renderer4.CompositorThreadImplDrawDelay (99th percentile by size) Renderer4.LanguageDetection (95th percentile by size) Renderer4.SoftwareCompositorThreadImplDrawDelay (85th percentile by size) Bug: 851784 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I99947370afa55bea137f99a317983db11292e33e Reviewed-on: https://chromium-review.googlesource.com/1112203Reviewed-by: Mark Pearson <mpearson@chromium.org> Reviewed-by: Michael Martis <martis@chromium.org> Reviewed-by: Fady Samuel <fsamuel@chromium.org> Commit-Queue: Yi Xu <yiyix@chromium.org> Cr-Commit-Position: refs/heads/master@{#572287} 
- 
Simeon Anfinrud authoredNow it's possible to e.g. create a Function<Both<ADerived, BDerived>, CBase> from a BiFunction<ABase, BBase, CDerived> if: * ADerived extends ABase * BDerived extends BBase * CDerived extends CBase This will help with some refactors. Bug: None Test: cast_base_junit_tests Change-Id: I7cfa3b3cfb559510e318ac8a597a138c0e4b3e0e Reviewed-on: https://chromium-review.googlesource.com/1119056Reviewed-by: Luke Halliwell <halliwell@chromium.org> Commit-Queue: Simeon Anfinrud <sanfin@chromium.org> Cr-Commit-Position: refs/heads/master@{#572286} 
- 
Lei Zhang authoredChange-Id: I8a8c0ba1da296572c5c661a82faa15c776200671 Reviewed-on: https://chromium-review.googlesource.com/1123895Reviewed-by: Mihai Sardarescu <msarda@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org> Cr-Commit-Position: refs/heads/master@{#572285} 
- 
Bill Orr authoredThe proper fix requires some design work. The current behavior does suffer from potential raciness, but shouldn't cause real-world issues for users. BUG=859666 Change-Id: I8d502a7db25949d2508675e1d153dbf56f69e1ed Reviewed-on: https://chromium-review.googlesource.com/1124948Reviewed-by: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Bill Orr <billorr@chromium.org> Cr-Commit-Position: refs/heads/master@{#572284} 
- 
Kevin Strohbehn authoredBug: none Change-Id: I564a8acb297811c9dee27d2c6d91077774e1bba7 Reviewed-on: https://chromium-review.googlesource.com/1123372Reviewed-by: James Cook <jamescook@chromium.org> Reviewed-by: Dirk Pranke <dpranke@chromium.org> Commit-Queue: Kevin Strohbehn <ginko@google.com> Cr-Commit-Position: refs/heads/master@{#572283} 
- 
Toni Barzic authoredAdds OFFLINE_DEMO_ACCOUNT to ChromeAccountType in auth.mojom, which will be set for offline enrolled demo sessions, so ARC++ can use offline demo mode provisioning flow for this account. Currently, there is no real change in prod, given that ARC still handles demo sessions the same (as arc_kiosk), and logic to determine whether the device is in demo mode is not yet hooked up. This cl additionally updates chromeos::DemoSession to indicate whether the device is offline enrolled while in demo mode - this is hardcoded to false for now (except in tests). BUG=b:110276688 BUG=846934 TEST=None Change-Id: Ib1c49401020838e566e07f765aff3e4f4dd9aad9 Reviewed-on: https://chromium-review.googlesource.com/1105599Reviewed-by: Yury Khmel <khmel@chromium.org> Reviewed-by: Mattias Nissler <mnissler@chromium.org> Reviewed-by: Ilya Sherman <isherman@chromium.org> Reviewed-by: Alexander Alekseev <alemate@chromium.org> Reviewed-by: Yusuke Sato <yusukes@chromium.org> Reviewed-by: Aga Wronska <agawronska@chromium.org> Commit-Queue: Toni Barzic <tbarzic@chromium.org> Cr-Commit-Position: refs/heads/master@{#572282} 
- 
Chromite Chromium Autoroll authoredhttps://chromium.googlesource.com/chromiumos/chromite.git/+log/45a546a30885..da20e836abca git log 45a546a30885..da20e836abca --date=short --no-merges --format='%ad %ae %s' 2018-07-03 dgarrett@google.com chromeos_config: Move all chrome-pfq builders to swarming. 2018-07-03 dgarrett@google.com chromeos_config: Move all android-pfq builders to swarming. Created with: gclient setdep -r src/third_party/chromite@da20e836abca 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. BUG=chromium:853969,chromium:853969 TBR=chrome-os-gardeners@chromium.org Change-Id: Ic6cfa31337c5a29b332625f48f0521cd8e5b6e7b Reviewed-on: https://chromium-review.googlesource.com/1124646Reviewed-by: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Chromite Chromium Autoroll <chromite-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#572281} 
- 
Robbie McElrath authoredThis migrates IntranetRedirectDetector from using net::NetworkChangeNotifier to content::NetworkConnectionTracker, which works with the network service enabled. Bug: 859131 Change-Id: Iec4c2f7907066d030d8de11d13ac955a558a2415 Reviewed-on: https://chromium-review.googlesource.com/1123332Reviewed-by: Nico Weber <thakis@chromium.org> Commit-Queue: Robbie McElrath <rmcelrath@chromium.org> Cr-Commit-Position: refs/heads/master@{#572280} 
- 
Theresa authoredAdd tests for: * Opening EoC using the toolbar button and tapping a suggestion * Button visibility when entering and exiting the tab switcher * Button visibility when switching tabs BUG=859567 Change-Id: Ib786454fc0f23ead9d6c446e05877e9fb92638e7 Reviewed-on: https://chromium-review.googlesource.com/1123273Reviewed-by: Becky Zhou <huayinz@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#572279} 
- 
Dominic Battre authoredDisable LocalNTPInterceptionWebRequestAPITest.OneGoogleBarRequestsHidden when run in servicified network stack but not otherwise. Flakes are caused due to a bug in the servicified network stack, the test seems to work well on other platforms. Bug: 853118 Cq-Include-Trybots: luci.chromium.try:linux_mojo Change-Id: I02bf111449dd88c24c788d630e8e53f364e03265 Reviewed-on: https://chromium-review.googlesource.com/1123822Reviewed-by: Dominic Battré <battre@chromium.org> Reviewed-by: Ken Rockot <rockot@chromium.org> Commit-Queue: Dominic Battré <battre@chromium.org> Cr-Commit-Position: refs/heads/master@{#572278} 
- 
Brian Sheedy authoredThis reverts commit 65c38f54. Reason for revert: Causing DCHECK failures in VR instrumentation tests Original change's description: > GPU: Grow and shrink transfer buffer > > Automatically grow the transfer buffer when it is full and shrink it > when the full capacity is not being used. Allow choosing larger and > smaller sizes than before. > > Before we would only grow it when a single transfer request was larger > than the whole buffer; requests smaller than the whole buffer would just > block if the buffer was full. Also we would not ever shrink the buffer > until someone called Free() manually. > > Bug: 850271, 835353, 828363, 856347 > 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: If2f2235a1d5297c63663398b37e1e30791347a3e > Reviewed-on: https://chromium-review.googlesource.com/1105505 > Commit-Queue: James Darpinian <jdarpinian@chromium.org> > Reviewed-by: Antoine Labour <piman@chromium.org> > Cr-Commit-Position: refs/heads/master@{#572135} TBR=jdarpinian@chromium.org,sunnyps@chromium.org,piman@chromium.org Change-Id: I031b30234cdcfbd8a4e48c7ebb6fd23312ec10ed No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 850271, 835353, 828363, 856347, 859952 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 Reviewed-on: https://chromium-review.googlesource.com/1124822Reviewed-by: Brian Sheedy <bsheedy@chromium.org> Commit-Queue: Brian Sheedy <bsheedy@chromium.org> Cr-Commit-Position: refs/heads/master@{#572277} 
- 
Egor Pasko authoredCoding against this API has been a 2-day debugging challenge :) I also considered "moving network controller options out of browser_options into finder_options: [1]. Then quickly I was chasing the ideals of unbundling StorySet from WprArchiveInfo in order to make _possible_browser.{SetUp,CleanUp}Environment() to start/stop the WPR server and forwarding. This would have been a better API IMO. On the other hand the refactoring associated with it looked a bit involved. So here it is - re-elly partial solution, to say mildly, which injects a few more tightly-coupled classes into each other to solve one specific problem. Sorry. [1] Mention of http://crbug.com/404771 in shared_page_state.py. The catapult side of it (https://github.com/catapult-project/catapult/issues/1977) was closed, but seems not done yet. Bug: 760498 Change-Id: I44e03179f9f2ed16f8fc1934c9786ef342f83609 Reviewed-on: https://chromium-review.googlesource.com/1122862 Commit-Queue: Egor Pasko <pasko@chromium.org> Reviewed-by:Juan Antonio Navarro Pérez <perezju@chromium.org> Cr-Commit-Position: refs/heads/master@{#572276} 
- 
Theresa authoredBUG=859564 Change-Id: I928bffc0ca099121134c5df8cb2355f7b39738cd Reviewed-on: https://chromium-review.googlesource.com/1124933Reviewed-by: Becky Zhou <huayinz@chromium.org> Commit-Queue: Theresa <twellington@chromium.org> Cr-Commit-Position: refs/heads/master@{#572275} 
- 
Wez authoredThe AutoRoll server is located here: https://fuchsia-sdk-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:fuchsia_arm64_cast_audio;luci.chromium.try:fuchsia_x64_cast_audio TBR=cr-fuchsia+bot@chromium.org Change-Id: I4129f2fa7e0a9162077e6aeb6ded3c1857113638 Reviewed-on: https://chromium-review.googlesource.com/1124666Reviewed-by: Wez <wez@chromium.org> Reviewed-by: Fuchsia SDK Autoroller <fuchsia-sdk-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: Wez <wez@chromium.org> Cr-Commit-Position: refs/heads/master@{#572274} 
- 
Tarun Bansal authoredThe WPT tests check the Accept-CH-Lifetime (ACL) header functionality. Add tests for: (i) When ACL header is sent by a same-origin iframe. (ii) When ACL header is sent by a cross-origin iframe. (iii) When ACL header is sent by a subresource. (iv) When ACL header is sent on the main frame. Also, change the ACL persistence time-duration from 20 seconds to 5 seconds to make it easier to run multiple tests quickly and without waiting too long. Change-Id: I31047598bbb1e7c64cadf3e6845bf850c29aca8b Bug: 856700 Reviewed-on: https://chromium-review.googlesource.com/1119848 Commit-Queue: Tarun Bansal <tbansal@chromium.org> Reviewed-by: Yoav Weiss <yoav@yoav.ws> Cr-Commit-Position: refs/heads/master@{#572273} 
- 
kylechar authoredThere is a lot of complexity in InProcessCommandBuffer and the internal Service class makes things more complex. This CL attempts to clean this up a bit. 1. Move InProcessCommandBuffer::Service into it's own file and rename to GpuCommandService. Improve comments and modernize code slightly. 2. Make GpuCommandService a subclass of RefCountedThreadSafe, rather than all of the subclasses of GpuCommandService and then manually forwarding calls to AddRef() and Release(). This was originally done because GpuCommandService was a pure interface, however that is no longer the case. 3. Remove passing around const refs to scoped_refptrs. We can just pass the scoped_refptr by value in these cases. Bug: 832243 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;master.tryserver.blink:linux_trusty_blink_rel Change-Id: Ib6dc5212ba71e3645e2649b5db43f2c06fbb14e2 Reviewed-on: https://chromium-review.googlesource.com/1118618 Commit-Queue: kylechar <kylechar@chromium.org> Reviewed-by: Antoine Labour <piman@chromium.org> Reviewed-by: Bo <boliu@chromium.org> Cr-Commit-Position: refs/heads/master@{#572272} 
- 
Aldo Culquicondor authoredWhen entering presentation mode from 2D browsing, the content's buffer size is not set, and thus VrShellGl uses the wrong buffer size for the dialogs. Bug: 856727 Change-Id: Idf740285739e026a818b44c85fd05b34edfbd17f Reviewed-on: https://chromium-review.googlesource.com/1124864Reviewed-by: Michael Thiessen <mthiesse@chromium.org> Commit-Queue: Aldo Culquicondor <acondor@chromium.org> Cr-Commit-Position: refs/heads/master@{#572271} 
- 
Chromium WPT Sync authoredUsing wpt-import in Chromium 5a8662be. With Chromium commits locally applied on WPT: 98d4a440 "[LayoutNG] Fix hit test clipping by fragmented border with round corner" Build: https://ci.chromium.org/buildbot/chromium.infra.cron/wpt-importer/20698 Note to sheriffs: This CL imports external tests and adds expectations for those tests; if this CL is large and causes a few new failures, please fix the failures by adding new lines to TestExpectations rather than reverting. See: https://chromium.googlesource.com/chromium/src/+/master/docs/testing/web_platform_tests.md Directory owners for changes in this CL: kojii@chromium.org, ksakamoto@chromium.org: external/wpt/css/css-font-loading TBR=markdittmer No-Export: true Change-Id: Ifc94884ef14e91e48af233c7a69a4babd4d3a071 Reviewed-on: https://chromium-review.googlesource.com/1124599 Commit-Queue: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Reviewed-by: Blink WPT Bot <blink-w3c-test-autoroller@chromium.org> Cr-Commit-Position: refs/heads/master@{#572270} 
- 
Ned Nguyen authoredTBR=eyaich@chromium.org NOTRY=true Change-Id: I889a03d6149f4ea3a90be461f4a97620ae957228 Reviewed-on: https://chromium-review.googlesource.com/1124947Reviewed-by: Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#572269} 
- 
Emily Hanley authoredCleanup of perf_data_generator will be in a follow on CL NOTRY=true Bug: 757933 Cq-Include-Trybots: master.tryserver.chromium.perf:obbs_fyi Change-Id: I93f2f3f1cbc52e0ef97d578a7b689d7f9201b733 Reviewed-on: https://chromium-review.googlesource.com/1124662 Commit-Queue: Ned Nguyen <nednguyen@google.com> Reviewed-by: John Budorick <jbudorick@chromium.org> Reviewed-by: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#572268} 
- 
perfetto-chromium-autoroll authoredhttps://android.googlesource.com/platform/external/perfetto.git/+log/c56c2429b0f6..24b373394918 git log c56c2429b0f6..24b373394918 --date=short --no-merges --format='%ad %ae %s' 2018-07-03 lalitm@google.com trace_processor: fix BestIndex/Filter calls when using OR constraints Created with: gclient setdep -r src/third_party/perfetto@24b373394918 The AutoRoll server is located here: https://perfetto-chromium-roll.skia.org Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff, who should be CC'd on the roll, and stop the roller if necessary. BUG=chromium:80416541 TBR=perfetto-bugs@google.com Change-Id: I7995b1ad8f88de53e2b5ba13224db5378ef01eb0 Reviewed-on: https://chromium-review.googlesource.com/1124648 Reviewed-by: <perfetto-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Commit-Queue: <perfetto-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com> Cr-Commit-Position: refs/heads/master@{#572267} 
- 
Ned Nguyen authoredThis reverts commit 3c6537d8. Reason for revert: block other CL from landing with PRESUBMIT error: https://logs.chromium.org/v/?s=chromium%2Fbuildbucket%2Fcr-buildbucket.appspot.com%2F8942003279127111280%2F%2B%2Fsteps%2Fpresubmit%2F0%2Fstdout ** Presubmit ERRORS ** Pylint (261 files using ['--disable=cyclic-import'] on 8 cores) (8.11s) failed ************* Module contrib.orderfile.orderfile E:109,20: story_class is not callable (not-callable) W:113,19: Access to a protected member _IterAllSystemHealthStoryClasses of a client class (protected-access) R:144, 0: Too many ancestors (8/7) (too-many-ancestors) R:155, 0: Too many ancestors (8/7) (too-many-ancestors) R:189, 0: Too many ancestors (8/7) (too-many-ancestors) R:194, 0: Too many ancestors (8/7) (too-many-ancestors) R:200, 0: Too many ancestors (8/7) (too-many-ancestors) R:206, 0: Too many ancestors (8/7) (too-many-ancestors) R:212, 0: Too many ancestors (8/7) (too-many-ancestors) R:218, 0: Too many ancestors (8/7) (too-many-ancestors) Presubmit checks took 15.4s to calculate. Original change's description: > Orderfile: training and testing benchmarks. > > These benchmarks will be used to train the new production android orderfile. > > Bug: 843561 > Change-Id: Iea097d784773c98ec3076f297aac16558c2a8104 > Reviewed-on: https://chromium-review.googlesource.com/1124561 > Reviewed-by: Benoit L <lizeb@chromium.org> > Cr-Commit-Position: refs/heads/master@{#572219} TBR=pasko@chromium.org,lizeb@chromium.org,mattcary@chromium.org Change-Id: I0f22ff64f7532aaa913bb4dbd844be6cfdb7b5ef No-Presubmit: true No-Tree-Checks: true No-Try: true Bug: 843561 Reviewed-on: https://chromium-review.googlesource.com/1124702Reviewed-by: Ned Nguyen <nednguyen@google.com> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#572266} 
- 
Alexei Filippov authoredThe test was flaky because of the rounding errors. BUG=856955 Change-Id: I089ff4efa05935433c6f1cd3a89f8cbad26879d5 Reviewed-on: https://chromium-review.googlesource.com/1123768Reviewed-by: Dmitry Gozman <dgozman@chromium.org> Commit-Queue: Alexei Filippov <alph@chromium.org> Cr-Commit-Position: refs/heads/master@{#572265} 
- 
Emily Hanley authoredBug:855063 NOTRY=true TBR=nednguyen@google.com Cq-Include-Trybots: master.tryserver.chromium.perf:obbs_fyi Change-Id: Ica527284a240bf2251e2a57c26729c541577aa44 Reviewed-on: https://chromium-review.googlesource.com/1124943Reviewed-by: Emily Hanley <eyaich@chromium.org> Commit-Queue: Emily Hanley <eyaich@chromium.org> Cr-Commit-Position: refs/heads/master@{#572264} 
- 
Sandra Sun authoredThis patch implements snapping for gesture fling on main thread. The ScrollManager would implement SnapFlingClient to collect snap information, schedule animation, and execute the scroll as instructed by SnapFlingController. This patch also updates the interface of SnapCoordinator::GetPositionForPoint() for it to return an Optional FloatPoint, clearly indicating whether there exists a snap point for the current scroll. Bug: 778259 Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel;master.tryserver.blink:linux_trusty_blink_rel Change-Id: I196e8c58c238cfad2a96e3f6a1c218f2c4f77a3c Reviewed-on: https://chromium-review.googlesource.com/1083144Reviewed-by: Rick Byers <rbyers@chromium.org> Reviewed-by: David Bokan <bokan@chromium.org> Reviewed-by: Majid Valipour <majidvp@chromium.org> Commit-Queue: Sandra Sun <sunyunjia@chromium.org> Cr-Commit-Position: refs/heads/master@{#572263} 
- 
Michael Spang authoredImplement sync_file export for fences so we can submit fences with planes to DRM atomic. This does not work unless you have a fairly recent kernel (requires syncobj support, I915_EXEC_FENCE_ARRAY). Bug: 851997 Test: ozone_demo on eve with USE=vulkan and kernel 4.14 Change-Id: I5859b64690e9b45ccb95698a9ca096f0847ec51b Reviewed-on: https://chromium-review.googlesource.com/1115570Reviewed-by: Antoine Labour <piman@chromium.org> Commit-Queue: Michael Spang <spang@chromium.org> Cr-Commit-Position: refs/heads/master@{#572262} 
 
-