Home | Search | chromium - Builders
Login

Builder Mac FYI Experimental Retina Release (AMD) Build 5926 Apple OSX

Results:

Success

Trigger Info:

Projectchromium
Revision703e2b8a5b805ecd4fba4bffb76095125330cec4
Got Revision703e2b8a5b805ecd4fba4bffb76095125330cec4

Execution:

Steps and Logfiles:

Show:
  1. ( 331 ms ) setup_build
    running recipe: "chromium"
  2. ( 217 ms ) report builders
     
    running tester 'Mac FYI Experimental Retina Release (AMD)' on master 'chromium.gpu.fyi'
  3. ( 1 mins 15 secs ) bot_update
    [74GB/232GB used (31%)]
  4. ( 733 ms ) swarming.py --version
    0.13
  5. ( 168 ms ) ensure_installed
  6. ( 13 secs ) gclient runhooks
  7. ( 1 secs ) read test spec (chromium.gpu.fyi.json)
    path: /b/s/w/ir/cache/builder/src/testing/buildbot/chromium.gpu.fyi.json
  8. ( 369 ms ) lookup builder GN args
     
    build_angle_gles1_conform_tests = true
    dcheck_always_on = true
    ffmpeg_branding = "Chrome"
    internal_gles2_conform_tests = true
    is_component_build = false
    is_debug = false
    proprietary_codecs = true
    strip_absolute_paths_from_debug_symbols = true
    symbol_level = 1
    use_goma = true
    goma_dir = "/b/s/w/ir/cache/goma/client"
  9. ( 400 ms ) build directory
  10. ( 408 ms ) find isolated tests
  11. ( 3 secs ) angle_unittests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 56s
  12. ( 38 secs ) tab_capture_end2end_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 34s
  13. ( 1 mins 3 secs ) gl_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:01:01.047101
  14. ( 1 secs ) gl_unittests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 8s
  15. ( 1 secs ) gles2_conform_test on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:04:32.812272
  16. ( 2 secs ) gpu_unittests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 16s
  17. ( 18 secs ) services_unittests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 15s
  18. ( 2 secs ) angle_end2end_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Max shard duration: 13s (shard #1)
    Min shard duration: 12s (shard #2)
  19. ( 2 mins ) context_lost_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:01:56.231784
    Total tests: 10
    * Passed: 10 (10 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  20. ( 30 secs ) depth_capture_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 28s
    Total tests: 3
    * Passed: 3 (3 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  21. ( 56 secs ) gpu_process_launch_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 54s
    Total tests: 15
    * Passed: 15 (15 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  22. ( 30 secs ) hardware_accelerated_feature_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 28s
    Total tests: 2
    * Passed: 2 (2 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  23. ( 27 secs ) info_collection_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 25s
    Total tests: 1
    * Passed: 1 (1 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  24. ( 1 mins 7 secs ) maps_pixel_test on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:01:05.685802
    Total tests: 1
    * Passed: 1 (1 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  25. ( 4 mins 37 secs ) pixel_test on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:04:30.085131
    Total tests: 57
    * Passed: 54 (54 expected, 0 unexpected)
    * Skipped: 3 (3 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  26. ( 1 mins 9 secs ) screenshot_sync_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:01:05.348391
    Total tests: 4
    * Passed: 4 (4 expected, 0 unexpected)
    * Skipped: 0 (0 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  27. ( 1 mins 23 secs ) trace_test on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Shard duration: 0:01:21.034978
    Total tests: 36
    * Passed: 18 (18 expected, 0 unexpected)
    * Skipped: 18 (18 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  28. ( 10 mins 25 secs ) webgl2_conformance_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Max shard duration: 0:10:12.110160 (shard #19)
    Min shard duration: 0:07:03.243137 (shard #8)
    Total tests: 2781
    * Passed: 2716 (2716 expected, 0 unexpected)
    * Skipped: 65 (65 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  29. ( 7 mins 16 secs ) webgl_conformance_tests on ATI GPU on Mac Retina on Mac-10.13.6
    Run on OS: 'Mac-10.13.6'
    Max shard duration: 0:07:03.710255 (shard #1)
    Min shard duration: 0:02:23.014617 (shard #0)
    Total tests: 926
    * Passed: 919 (919 expected, 0 unexpected)
    * Skipped: 7 (7 expected, 0 unexpected)
    * Failed: 0 (0 expected, 0 unexpected)
    * Flaky: 0 (0 expected, 0 unexpected)
     
  30. ( 264 ms ) Tests statistics
     
    Stats
    Total shards: 42
    Total runtime: 3:02:04.156807
     

Build Properties:

NameValueSource
$recipe_engine/path {"cache_dir": "/b/s/w/ir/cache", "temp_dir": "/b/s/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "build89-m9" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1539352415804793, "id": "8932865717203934208", "project": "chromium", "tags": ["builder:Mac FYI Experimental Retina Release (AMD)", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/703e2b8a5b805ecd4fba4bffb76095125330cec4", "gitiles_ref:refs/heads/master", "parent_buildername:GPU FYI Mac Builder", "parent_buildnumber:23983", "scheduler_invocation_id:9098295779243925712", "scheduler_job_id:chromium/Mac FYI Experimental Retina Release (AMD)", "user_agent:luci-scheduler", "user_agent:recipe"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Mac FYI Experimental Retina Release (AMD)" setup_build
buildnumber 5926 setup_build
mastername "chromium.gpu.fyi" setup_build
parent_buildername "GPU FYI Mac Builder" setup_build
parent_got_angle_revision "671809e62e807eb0f890a5d53eee1f8cb5fb825e" setup_build
parent_got_buildtools_revision "2dff9c9c74e9d732e6fe57c84ef7fd044cc45d96" setup_build
parent_got_nacl_revision "2ebbdab4bfb1ed9dfe52ce756b34829207dbe25f" setup_build
parent_got_revision "703e2b8a5b805ecd4fba4bffb76095125330cec4" setup_build
parent_got_revision_cp "refs/heads/master@{#599196}" setup_build
parent_got_swarming_client_revision "486c9b53c4d54dd4b95bb6ce0e31160e600dfc11" setup_build
parent_got_v8_revision "f70aaa8ab2e8815505a6145c745e50d8328cd28c" setup_build
parent_got_v8_revision_cp "refs/heads/7.1.302@{#1}" setup_build
parent_got_webrtc_revision "55d1af14751ad10e0cae741240f013ea67039f45" setup_build
parent_got_webrtc_revision_cp "refs/heads/master@{#25138}" setup_build
parent_mastername "chromium.gpu.fyi" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
revision "703e2b8a5b805ecd4fba4bffb76095125330cec4" setup_build
swarm_hashes {"angle_end2end_tests": "379553b02756189637d37b439177a6493f142e5a", "angle_unittests": "3af1f03ce0f689a6914fbfe10a0d0f7900ff2353", "browser_tests": "d8fcb3df0a09eddc2dd1a940328c5cb1e3201c6b", "gl_tests": "5e3afd9fee2bb870f96a649b169d97664658dbeb", "gl_unittests": "27b7e3cf53323642d98ad131638a2e8ce0e0449b", "gles2_conform_test": "84074cd2dddae6dd82d64da00c040b7eecdf74d4", "gpu_unittests": "df5eee88c64d4f2a6269bcfbdf38bda0f3d9ced7", "services_unittests": "3bce9f9b8aa7f80ba4086d21bcae900ea0f5696d", "swiftshader_unittests": "887bfd2c07de007e59ffcd2eab31a08bee292560", "telemetry_gpu_integration_test": "c5453cd4df6f02303dad312faf2381fd5d728eb9"} setup_build
got_angle_revision "671809e62e807eb0f890a5d53eee1f8cb5fb825e" bot_update
got_buildtools_revision "2dff9c9c74e9d732e6fe57c84ef7fd044cc45d96" bot_update
got_nacl_revision "2ebbdab4bfb1ed9dfe52ce756b34829207dbe25f" bot_update
got_revision "703e2b8a5b805ecd4fba4bffb76095125330cec4" bot_update
got_revision_cp "refs/heads/master@{#599196}" bot_update
got_swarming_client_revision "486c9b53c4d54dd4b95bb6ce0e31160e600dfc11" bot_update
got_v8_revision "f70aaa8ab2e8815505a6145c745e50d8328cd28c" bot_update
got_v8_revision_cp "refs/heads/7.1.302@{#1}" bot_update
got_webrtc_revision "55d1af14751ad10e0cae741240f013ea67039f45" bot_update
got_webrtc_revision_cp "refs/heads/master@{#25138}" bot_update
swarm_hashes {} find isolated tests

Blamelist:

  1. Dmitry Gozman (dgozmanohnoyoudont@chromium.org)
  2. Marc Treib (treibohnoyoudont@chromium.org)
  3. Stephane Zermatten (szermattohnoyoudont@chromium.org)
  4. Peng Huang (penghuangohnoyoudont@chromium.org)
  5. Ben Kelly (wanderviewohnoyoudont@chromium.org)
  6. Shu Chen (shuchenohnoyoudont@google.com)
  7. Findit (findit-for-meohnoyoudont@appspot.gserviceaccount.com)
  8. Colin Blundell (blundellohnoyoudont@chromium.org)

Timing:

Create Friday, 12-Oct-18 13:53:35 UTC
Start Friday, 12-Oct-18 13:53:39 UTC
End Friday, 12-Oct-18 14:29:49 UTC
Pending 3 secs
Execution 36 mins 10 secs

All Changes:

  1. Restore FrameLoadRequest::HrefTranslate

    Changed by Dmitry Gozman - dgozmanohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:42:52 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 703e2b8a5b805ecd4fba4bffb76095125330cec4

    Comments

    Restore FrameLoadRequest::HrefTranslate
    
    This was accidentaly removed as part of FrameLoadRequest cleanup.
    
    Bug: none
    Change-Id: I58bf324729d397a2f0340831fd116ab749f7a213
    Reviewed-on: https://chromium-review.googlesource.com/c/1278290
    Commit-Queue: Dave Tapuska <dtapuska@chromium.org>
    Reviewed-by: Dave Tapuska <dtapuska@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599196}

    Changed files

    • third_party/blink/renderer/core/html/html_anchor_element.cc
    • third_party/blink/renderer/core/loader/frame_load_request.h
  2. ProfileSyncService cleanup: Remove platform_sync_allowed_provider_

    Changed by Marc Treib - treibohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:42:49 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f0d63432c51741e6dc0bf48f0b84e4afb63f887f

    Comments

    ProfileSyncService cleanup: Remove platform_sync_allowed_provider_
    
    ProfileSyncService::platform_sync_allowed_provider_ was a callback that
    returned whether Android's "Master Sync" toggle is enabled, and so Sync
    shouldn't start up. However, PSS didn't know when the value changed, so
    the callback was kinda useless, and PSS could easily get into an
    inconsistent state (where there is a disable reason, but Sync is still
    running). Things were working out in practice only because the Android
    UI code also called RequestStart/Stop as necessary.
    So, since PSS needs change notifications anyway, let's make that
    explicit: This CL replaces the callback by a bool, which can be set
    explicitly.
    
    Bug: 867901
    Change-Id: I5b7e9f4657d1f565d9643984eb52ec1df26b5c90
    Reviewed-on: https://chromium-review.googlesource.com/c/1269870
    Reviewed-by: Mikel Astiz <mastiz@chromium.org>
    Commit-Queue: Marc Treib <treib@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599195}

    Changed files

    • chrome/android/java/src/org/chromium/chrome/browser/sync/ProfileSyncService.java
    • chrome/android/java/src/org/chromium/chrome/browser/sync/SyncController.java
    • chrome/browser/sync/profile_sync_service_android.cc
    • chrome/browser/sync/profile_sync_service_android.h
    • components/browser_sync/profile_sync_service.cc
    • components/browser_sync/profile_sync_service.h
  3. [Autofill Assistant] Add Mathias and I as owners

    Changed by Stephane Zermatten - szermattohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:39:47 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1b71114eba7ab4e982eaf03186b12503c8ca49c4

    Comments

    [Autofill Assistant] Add Mathias and I as owners
    
    Change-Id: I71ed01f39d1c81c62828d8e4900091e7941e7ab1
    Reviewed-on: https://chromium-review.googlesource.com/c/1278399
    Reviewed-by: Ganggui Tang <gogerald@chromium.org>
    Commit-Queue: Ganggui Tang <gogerald@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599194}

    Changed files

    • components/autofill_assistant/OWNERS
  4. Hook up skia makeYUVAPromiseTexture API

    Changed by Peng Huang - penghuangohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:39:05 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 33ca177f048a1d593c5486b46f09ac9ea91071a9

    Comments

    Hook up skia makeYUVAPromiseTexture API
    
    Know issues:
     * Skia only draws Y plane.
     * makeYUVAPromiseTexture() will release context, even if it failed.
     * Skia doesn't accept GL_RGBA8 format for a yuva plane
    
    Bug: 824382
    Cq-Include-Trybots: luci.chromium.try:android_optional_gpu_tests_rel
    Change-Id: Ic8da2b3de574ef2a0d527c79f34d383a676a3bc0
    Reviewed-on: https://chromium-review.googlesource.com/c/1262421
    Reviewed-by: Antoine Labour <piman@chromium.org>
    Commit-Queue: Peng Huang <penghuang@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599193}

    Changed files

    • components/viz/service/display/skia_output_surface.h
    • components/viz/service/display/skia_renderer.cc
    • components/viz/service/display_embedder/skia_output_surface_impl.cc
    • components/viz/service/display_embedder/skia_output_surface_impl.h
    • components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.cc
    • components/viz/service/display_embedder/skia_output_surface_impl_on_gpu.h
  5. Make DataPipeBytesConsumer support ReadableStream loading better.

    Changed by Ben Kelly - wanderviewohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:36:24 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision c43eba9311573672d0773beed07f117d08eed09f

    Comments

    Make DataPipeBytesConsumer support ReadableStream loading better.
    
    This CL fixes some edge condition interactions when being loaded as
    a ReadableStream.  In particular, it:
    
    1. Properly distinguishes between the end of the DataPipe and a closed
       DataPipe with bytes left to be read.  A ReadableStream that is not
       actively draining the pipe could get closed too early.
    2. Responses must support explicit completion in order to handle error
       conditions properly.  This CL makes DataPipeBytesConsumer wait
       for an explicit signal before closing.
    3. Service worker navigation preload is updated to provide the explicit
       completion signals.
    
    Bug: 894815
    Change-Id: I8cff3de94aa2dcbc8deb4a9601a95c13b8ab94d9
    Reviewed-on: https://chromium-review.googlesource.com/c/1272715
    Commit-Queue: Ben Kelly <wanderview@chromium.org>
    Reviewed-by: Yutaka Hirano <yhirano@chromium.org>
    Reviewed-by: Marijn Kruisselbrink <mek@chromium.org>
    Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599192}

    Changed files

    • content/browser/service_worker/service_worker_browsertest.cc
    • third_party/blink/renderer/core/fetch/bytes_consumer.h
    • third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer.cc
    • third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer.h
    • third_party/blink/renderer/core/fetch/data_pipe_bytes_consumer_test.cc
    • third_party/blink/renderer/core/fetch/fetch_data_loader.cc
    • third_party/blink/renderer/core/fetch/fetch_data_loader.h
    • third_party/blink/renderer/modules/service_worker/fetch_event.cc
    • third_party/blink/renderer/modules/service_worker/fetch_event.h
    • third_party/blink/renderer/modules/service_worker/fetch_respond_with_observer.cc
  6. Avoid using gypi_to_gn.py.

    Changed by Shu Chen - shuchenohnoyoudont@google.com
    Changed at Friday, 12-Oct-18 13:25:50 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision b85f543af1e343404dde1a2c51067b2ed5ac08cb

    Comments

    Avoid using gypi_to_gn.py.
    
    Change-Id: I9bac8317577feaadfa00ec88661175b7338ced4a
    Reviewed-on: https://chromium-review.googlesource.com/c/1267596
    Reviewed-by: Nico Weber <thakis@chromium.org>
    Commit-Queue: Shu Chen <shuchen@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599191}

    Changed files

    • third_party/google_input_tools/README.chromium
    • third_party/google_input_tools/builder.py
    • third_party/google_input_tools/closure.gni
    • third_party/google_input_tools/inputview.gni
    • third_party/google_input_tools/inputview.gypi
    • third_party/google_input_tools/update.py
    • ui/keyboard/BUILD.gn
  7. Revert "Files app: Select My files when there are no volumes"

    Changed by Findit - findit-for-meohnoyoudont@appspot.gserviceaccount.com
    Changed at Friday, 12-Oct-18 13:17:58 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 402571f89e1aff1627ccf491110764c74029e3de

    Comments

    Revert "Files app: Select My files when there are no volumes"
    
    This reverts commit 5cd8c123e452c0b76fbbcaf53e82d86ca7d54930.
    
    Reason for revert:
    
    Findit (https://goo.gl/kROfz5) identified CL at revision 599026 as the
    culprit for flakes in the build cycles as shown on:
    https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vNWNkOGMxMjNlNDUyYzBiNzZmYmJjYWY1M2U4MmQ4NmNhN2Q1NDkzMAw
    
    Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20ChromiumOS%20MSan%20Tests/9016
    
    Sample Failed Step: viz_browser_tests
    
    Sample Flaky Test: FileDisplay/FilesAppBrowserTest.Test/fileDisplayWithoutVolumesThenMountDrive_DriveFs
    
    Original change's description:
    > Files app: Select My files when there are no volumes
    > 
    > Make Files app select "My files" when there are no available volumes,
    > this to allow Files app to behave properly when volumes subsequently
    > become available.
    > 
    > Change DirectoryModel.onVolumeInfoListUpdated_ method to check for
    > non-null |displayRoot| before trying to change to |displayRoot|. This
    > fixes the error "Cannot read property 'getParent' of null" when Drive
    > volume becomes available before Downloads volume, which is the default
    > volume/root.
    > 
    > Change FakeDriveFs to unbind the two mojo bindings if they're bound, so
    > DriveFsTestVolume can re-mount itself.
    > 
    > Test: browser_tests --gtest_filter="*/fileDisplayWithoutVolumesThenMount*"
    > Bug: 893161, 884967
    > Change-Id: Ic813b25261530495c11c9f641a92f6e07f883702
    > Reviewed-on: https://chromium-review.googlesource.com/c/1272418
    > Reviewed-by: Noel Gordon <noel@chromium.org>
    > Reviewed-by: Sam McNally <sammc@chromium.org>
    > Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#599026}
    
    Change-Id: I883485c8fc1bdcc22dba93cc4b03b7c157dcb5f2
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 893161, 884967, 889703
    Reviewed-on: https://chromium-review.googlesource.com/c/1278063
    Cr-Commit-Position: refs/heads/master@{#599190}

    Changed files

    • chrome/browser/chromeos/file_manager/file_manager_browsertest.cc
    • chrome/browser/chromeos/file_manager/file_manager_browsertest_base.cc
    • chromeos/components/drivefs/fake_drivefs.cc
    • ui/file_manager/file_manager/foreground/js/directory_model.js
    • ui/file_manager/file_manager/foreground/js/file_manager.js
    • ui/file_manager/integration_tests/file_manager/file_display.js
  8. Start using IdentityTestEnvironment in ProfileSyncServiceBundle

    Changed by Colin Blundell - blundellohnoyoudont@chromium.org
    Changed at Friday, 12-Oct-18 13:10:51 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 863a437d0fdbbba19ad5446420b72d2f91748a72

    Comments

    Start using IdentityTestEnvironment in ProfileSyncServiceBundle
    
    ProfileSyncServiceBundle owns and exposes signin-related objects for
    use by sync tests. These signin-related objects include ones that are
    deprecated by IdentityManager and no longer used directly in the
    production code being tested; however, up until now it has not been
    feasible to convert this test utility to using IdentityTestEnvironment,
    as doing so would have required converting the widespread usage of this
    utility in one go.
    
    We recently introduced an IdentityTestEnvironment usage mode wherein
    it can be supplied with IdentityManager's dependencies from the
    external client, precisely to enable incremental conversion in use cases
    like this. This CL exploits this usage mode to introduce
    IdentityTestEnvironment in ProfileSyncServiceBundle.h and provide an
    example of its usage to eliminate a direct usage of the deprecated
    classes. Once the conversion is complete, ProfileSyncServiceBundle can
    then be changed to *only* hold an IdentityTestEnvironment object.
    
    Bug: 886600, 886599
    Change-Id: I205c3567fc0a9bcd6f7b613c00a247c302d71be5
    Reviewed-on: https://chromium-review.googlesource.com/c/1273065
    Commit-Queue: Colin Blundell <blundell@chromium.org>
    Reviewed-by: Marc Treib <treib@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#599189}

    Changed files

    • components/browser_sync/BUILD.gn
    • components/browser_sync/profile_sync_service_autofill_unittest.cc
    • components/browser_sync/profile_sync_test_util.cc
    • components/browser_sync/profile_sync_test_util.h