Home | Search | chromium - Builders
Login

Builder Nougat Phone Tester Build 6844 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revision7a0ab14f53bb9b6346969ec10d376d51aff8eceb
Got Revision7a0ab14f53bb9b6346969ec10d376d51aff8eceb

Execution:

Steps and Logfiles:

Show:
  1. ( 374 ms ) setup_build
    running recipe: "chromium"
  2. ( 1 secs ) remove .cipd
  3. ( 4 mins 19 secs ) bot_update
    [130GB/295GB used (44%)]
  4. ( 0 ) ensure_goma
  5. ( 967 ms ) swarming.py --version
    0.13
  6. ( 44 secs ) gclient runhooks
  7. ( 1 secs ) read test spec (chromium.android.json)
    path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.android.json
  8. ( 4 secs ) clean local files
  9. ( 1 secs ) tree truth steps
  10. ( 5 secs ) build directory
  11. ( 29 mins 38 secs ) extract build
  12. ( 4 secs ) find isolated tests
  13. ( 0 ) test_pre_run
  14. ( 13 mins 54 secs ) chrome_public_test_vr_apk-pixelxl-cardboard-nougat on Android device Pixel 1 XL
    Run on OS: 'Android'
    Pending time: 25s
    Shard duration: 0:13:24.391040
  15. ( 12 mins 41 secs ) chrome_public_test_vr_apk-pixelxl-ddview-nougat on Android device Pixel 1 XL
    Run on OS: 'Android'
    Pending time: 33s
    Shard duration: 0:26:03.134770
  16. ( 2 secs ) chrome_public_test_vr_apk-pixelxl-ddview-nougat-donenabled on Android device Pixel 1 XL
    Run on OS: 'Android'
    Pending time: 0:01:09.480920
    Shard duration: 0:04:50.109230
  17. ( 864 ms ) Tests statistics
     
    Stats
    Total shards: 3
    Total runtime: 0:43:21.261069
     
  18. ( 400 ms ) test_report

Build Properties:

NameValueSource
$build/goma {"jobs": 150} setup_build
$recipe_engine/path {"cache_dir": "/b/swarming/w/ir/cache", "temp_dir": "/b/swarming/w/ir/tmp/rt"} setup_build
$recipe_engine/runtime {"is_experimental": false, "is_luci": true} setup_build
bot_id "swarm376-c4" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1531397277993720, "id": "8941207283789345008", "project": "chromium", "tags": ["builder:Nougat Phone Tester", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/7a0ab14f53bb9b6346969ec10d376d51aff8eceb", "gitiles_ref:refs/heads/master", "parent_buildername:Android arm64 Builder (dbg)", "parent_buildnumber:31821", "scheduler_invocation_id:9106637346041889872", "scheduler_job_id:chromium/Nougat Phone Tester", "user_agent:luci-scheduler", "user_agent:recipe"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "Nougat Phone Tester" setup_build
buildnumber 6844 setup_build
mastername "chromium.android" setup_build
parent_buildername "Android arm64 Builder (dbg)" setup_build
parent_got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" setup_build
parent_got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" setup_build
parent_got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" setup_build
parent_got_revision "7a0ab14f53bb9b6346969ec10d376d51aff8eceb" setup_build
parent_got_revision_cp "refs/heads/master@{#574521}" setup_build
parent_got_swarming_client_revision "9a518d097dca20b7b00ce3bdfc5d418ccc79893a" setup_build
parent_got_v8_revision "8b17db68f75a95d69debfa85b70aea3ec0766aaf" setup_build
parent_got_v8_revision_cp "refs/heads/6.9.352@{#1}" setup_build
parent_got_webrtc_revision "5795489e83adba65cb480d57e71f52bae1f5ee1b" setup_build
parent_got_webrtc_revision_cp "refs/heads/master@{#23944}" setup_build
parent_mastername "chromium.android" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
revision "7a0ab14f53bb9b6346969ec10d376d51aff8eceb" setup_build
got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" bot_update
got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" bot_update
got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" bot_update
got_revision "7a0ab14f53bb9b6346969ec10d376d51aff8eceb" bot_update
got_revision_cp "refs/heads/master@{#574521}" bot_update
got_swarming_client_revision "9a518d097dca20b7b00ce3bdfc5d418ccc79893a" bot_update
got_v8_revision "8b17db68f75a95d69debfa85b70aea3ec0766aaf" bot_update
got_v8_revision_cp "refs/heads/6.9.352@{#1}" bot_update
got_webrtc_revision "5795489e83adba65cb480d57e71f52bae1f5ee1b" bot_update
got_webrtc_revision_cp "refs/heads/master@{#23944}" bot_update
swarm_hashes {"android_webview_unittests": "0ab40d612dcf6fc75ffbbc5e4757bc8b2d9365f1", "angle_unittests": "77932599ba11affae28003593c031956173d98b2", "base_unittests": "f4946cd0c4ec0236b8a6e07f41eaaffd8f206460", "blink_common_unittests": "dc2547ae9b6fa9e0f9ee8cb03237ec45ff261cf0", "blink_heap_unittests": "b3a17c5928f7177826d9326062a0741e51a7f79e", "blink_platform_unittests": "902cec152e936aee4f949af3fbc59e5732b7d6c2", "boringssl_crypto_tests": "90affe09b2078c9b8ea220413f37a264fdf3d70b", "boringssl_ssl_tests": "b91084f11b383643ec2b893f44e5b1222f87b754", "breakpad_unittests": "bd1bb2693b5782df3f98d72ee1c00244e169df63", "cacheinvalidation_unittests": "30f08533a863b9da5c508b7f8983c042df0878d7", "capture_unittests": "ad147f23a6f8460e5a3a4472bf0337058d1a0624", "cast_unittests": "393c09f6066732f5733a0995311c70bbb636ca4a", "cc_unittests": "83d15b957a557d7b9a4daf109608d8d6041b4e20", "chrome_public_test_apk": "6c1d6b5b8c3b7939c509f029fe4d282f4b6a0ec0", "chrome_public_test_vr_apk": "8ddedef80c96dc4ed6631f60c7ec83eb552f02ee", "chrome_sync_shell_test_apk": "6ecf6b8fc0205c0f56959e6cbaadfe48940d1e1a", "components_browsertests": "f32a856d273b3b368c920039a6b72c6768d73ceb", "components_unittests": "4b945be674ffeb7236433fee2e8ad63d7ac0542a", "content_browsertests": "47b541487807e1215f2edc3b72a8b6e12e2d128e", "content_shell_test_apk": "f28c3b7e006b3ada8e1f4fe5756f7592d33984f3", "content_unittests": "a0d36501e24f487a82aa6c4cffe586e63af5e34a", "crypto_unittests": "c9d61a8d7a7379b77d8b9fa8247ee1f49bc974b8", "device_unittests": "f87e7d55fa435c1bf13e3892c403b8baee87e509", "display_unittests": "b13a4e43318904c793245b7f607a0f1f2ce31a3c", "events_unittests": "fcd69be308632f57a8f92af7c5791fddf81fb9e1", "gcm_unit_tests": "b1920c72484785a66b3fac55b18e38c1cc7d88d0", "gfx_unittests": "a9c551a83e573102c3dc4158b2baf9b577001ccf", "gin_unittests": "d1dddb2036ba5776c3bfbbe36e2e0f86d2696171", "gl_tests": "71e46bd07ec32750918704ee74c694e49a7cf2d5", "gl_unittests": "7b7772c366022f5337d6f4f02a9c8eb0f8b31075", "google_apis_unittests": "ece1d4c6b4b479127a0e8e92924480f44111c591", "gpu_unittests": "047ff2378bfc6e636a3580b19813846d04bba761", "ipc_tests": "66fde598bd5f043f0e173ada6c7010374faf5199", "jingle_unittests": "59280b2e6cc9f52b73f4f574019328486b03e3ac", "latency_unittests": "c97b8f1881c9f179b44fb325297a88b0263823f1", "libjingle_xmpp_unittests": "11456a4a8e0a9c35b93a863dd385f80f87cc6cb9", "media_blink_unittests": "5cdc8f9013c8e1c761f674d9b362411df470af55", "media_service_unittests": "21b2c3c34e0ea779709ff9bcbebe03c6a5c0629a", "media_unittests": "f7721a979858f2466f51413374b704431b174aeb", "midi_unittests": "1db91250aca4fa76b2fac81db9f08c7540fa137e", "mojo_test_apk": "74a3a31bfbe2e78657455ed8b6c6a6c83618d656", "mojo_unittests": "5e26ab51d372e6999d9c6ffd26c49a6796c347aa", "net_unittests": "646ab869d165db340f96465df535cbf77f75065f", "sandbox_linux_unittests": "356c492745d718d89479327cef0222c5c5ea832d", "services_unittests": "b84308f9327d87b30d315e4f6b21a9dab61cae3f", "shell_dialogs_unittests": "94e17848fdf08b1c31b0c9fd10d1e8d63c2ad28f", "skia_unittests": "ea9017cde326756847cb776c961ab319615798a1", "sql_unittests": "c293740aca64ddc2bf12443f90ea8b193bd05da9", "storage_unittests": "68f53852344d70fef9289a39cf59f5f93b34e054", "ui_android_unittests": "d099b6913fe5e86518076f31170eacdd13164efb", "ui_base_unittests": "7e08122c6f4402bdedcc9a97245ab96690ebd67a", "ui_touch_selection_unittests": "fb700124223311116785f5ac1ce260d2a85ff07e", "unit_tests": "240afa4bd6302dd4c5d711ce886aa52473d504d1", "url_unittests": "7df1d3948e6064a3e54b3c3fa2f5637b6d3e54e7", "viz_unittests": "623f88cb03e6d3049417f8269ab743f8d2501984", "vr_common_unittests": "2f7cfc289c89a3bb4758e913572d2920f9051ba6", "vr_pixeltests": "20bd4bfbae2a233ab01ce01fec8d4d9356754b6c", "webkit_unit_tests": "372bf5afa2bb9286e42e999167c5731c6a4b5e56", "webview_instrumentation_test_apk": "53c8318e13fc53e27814153747feb935bbfaa247", "wtf_unittests": "752a7c79e1779c3b952b83ce215a01d7a01cbac6"} find isolated tests

Blamelist:

  1. Tatsuhisa Yamaguchi (yamaguchiohnoyoudont@chromium.org)
  2. Rohit Rao (rohitraoohnoyoudont@chromium.org)
  3. Mark Cogan (marqohnoyoudont@google.com)
  4. Mike West (mkwstohnoyoudont@chromium.org)
  5. Stefan Teodorescu (faneohnoyoudont@google.com)
  6. Han Leon (leon.hanohnoyoudont@intel.com)
  7. Thomas Tangl (tangltomohnoyoudont@chromium.org)
  8. depot-tools-chromium-autoroll (depot-tools-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com)
  9. Mike West (mkwstohnoyoudont@chromium.org)
  10. yoshiki iguchi (yoshikiohnoyoudont@chromium.org)
  11. Colin Blundell (blundellohnoyoudont@chromium.org)
  12. Matt Falkenhagen (falkenohnoyoudont@chromium.org)
  13. Raphael Kubo da Costa (CET) (raphael.kubo.da.costaohnoyoudont@intel.com)
  14. Guido Urdaneta (guidouohnoyoudont@chromium.org)
  15. Matt Falkenhagen (falkenohnoyoudont@chromium.org)
  16. rajendrant (rajendrantohnoyoudont@chromium.org)
  17. Luciano Pacheco (lucmultohnoyoudont@chromium.org)
  18. Aleks Totic (atoticohnoyoudont@chromium.org)
  19. Makoto Shimazu (shimazuohnoyoudont@chromium.org)
  20. Anita Woodruff (awdfohnoyoudont@chromium.org)
  21. Hitoshi Yoshida (periaohnoyoudont@chromium.org)
  22. Luciano Pacheco (lucmultohnoyoudont@chromium.org)
  23. Alan Cutter (alancutterohnoyoudont@chromium.org)
  24. Marc Treib (treibohnoyoudont@chromium.org)
  25. Dominic Mazzoni (dmazzoniohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 12:07:57 UTC
Start Thursday, 12-Jul-18 12:08:13 UTC
End Thursday, 12-Jul-18 13:11:31 UTC
Pending 15 secs
Execution 1 hrs 3 mins

All Changes:

  1. Always align slider bubbles to the bottom of the desktop.

    Changed by Tatsuhisa Yamaguchi - yamaguchiohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 10:06:16 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 7a0ab14f53bb9b6346969ec10d376d51aff8eceb

    Comments

    Always align slider bubbles to the bottom of the desktop.
    
    The slider was aligned towards the system tray icon view in the shelf
    by the bubble's bottom edge. It meant that the slider bubble was shown
    apart from the bottom of the desktop when the shelf layout is either
    right or left. It was not consistent with popup notification, causing
    view overlap between notifications and a slider bubble.
    
    Bug: 862496
    Change-Id: Ie6db618f8b744ecc4de52ee7a4866ca9e9ecd26c
    Reviewed-on: https://chromium-review.googlesource.com/1134643
    Reviewed-by: Tetsui Ohkubo <tetsui@chromium.org>
    Reviewed-by: Yoshiki Iguchi <yoshiki@chromium.org>
    Commit-Queue: Tatsuhisa Yamaguchi <yamaguchi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574521}

    Changed files

    • ash/system/unified/unified_slider_bubble_controller.cc
  2. [ios] Adds icons and hides headers on the main Settings page.

    Changed by Rohit Rao - rohitraoohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 10:00:36 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 5f5527a3ed5e88090ce316b8afac85c39cbb0ba8

    Comments

    [ios] Adds icons and hides headers on the main Settings page.
    
    Screenshot: https://drive.google.com/file/d/16aS0MP0LwdGOq8-IJgtIbq4ozXy44meT/view?usp=sharing
    
    BUG=805213,805214
    
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
    Change-Id: Iccdd87d8f95fd80b51fcdcfa2caa691c7fcd78b4
    Reviewed-on: https://chromium-review.googlesource.com/1133323
    Reviewed-by: edchin <edchin@chromium.org>
    Commit-Queue: Rohit Rao <rohitrao@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574520}

    Changed files

    • ios/chrome/browser/ui/settings/BUILD.gn
    • ios/chrome/browser/ui/settings/cells/settings_switch_item.mm
    • ios/chrome/browser/ui/settings/resources/BUILD.gn
    • ios/chrome/browser/ui/settings/resources/settings_about_chrome.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_about_chrome.imageset/settings_about_chrome.png
    • ios/chrome/browser/ui/settings/resources/settings_about_chrome.imageset/settings_about_chrome@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_about_chrome.imageset/settings_about_chrome@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_article_suggestions.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_article_suggestions.imageset/settings_article_suggestions.png
    • ios/chrome/browser/ui/settings/resources/settings_article_suggestions.imageset/settings_article_suggestions@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_article_suggestions.imageset/settings_article_suggestions@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_autofill_forms.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_autofill_forms.imageset/settings_autofill_forms.png
    • ios/chrome/browser/ui/settings/resources/settings_autofill_forms.imageset/settings_autofill_forms@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_autofill_forms.imageset/settings_autofill_forms@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_bandwidth.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_bandwidth.imageset/settings_bandwidth.png
    • ios/chrome/browser/ui/settings/resources/settings_bandwidth.imageset/settings_bandwidth@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_bandwidth.imageset/settings_bandwidth@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_content_settings.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_content_settings.imageset/settings_content_settings.png
    • ios/chrome/browser/ui/settings/resources/settings_content_settings.imageset/settings_content_settings@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_content_settings.imageset/settings_content_settings@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_debug.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_debug.imageset/settings_debug.png
    • ios/chrome/browser/ui/settings/resources/settings_debug.imageset/settings_debug@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_debug.imageset/settings_debug@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_passwords.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_passwords.imageset/settings_passwords.png
    • ios/chrome/browser/ui/settings/resources/settings_passwords.imageset/settings_passwords@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_passwords.imageset/settings_passwords@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_privacy.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_privacy.imageset/settings_privacy.png
    • ios/chrome/browser/ui/settings/resources/settings_privacy.imageset/settings_privacy@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_privacy.imageset/settings_privacy@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_search_engine.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_search_engine.imageset/settings_search_engine.png
    • ios/chrome/browser/ui/settings/resources/settings_search_engine.imageset/settings_search_engine@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_search_engine.imageset/settings_search_engine@3x.png
    • ios/chrome/browser/ui/settings/resources/settings_voice_search.imageset/Contents.json
    • ios/chrome/browser/ui/settings/resources/settings_voice_search.imageset/settings_voice_search.png
    • ios/chrome/browser/ui/settings/resources/settings_voice_search.imageset/settings_voice_search@2x.png
    • ios/chrome/browser/ui/settings/resources/settings_voice_search.imageset/settings_voice_search@3x.png
    • ios/chrome/browser/ui/settings/settings_collection_view_controller.mm
  3. [iOS] Tweak tab grid transition animations.

    Changed by Mark Cogan - marqohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 09:57:57 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision e96b9e2a058ddc4573deba9c0159077c8a89b45d

    Comments

    [iOS] Tweak tab grid transition animations.
    
    This CL adjusts the tab grid transition animations.
    
    (A) The timing (curve, delay, and duration) of some of the sub-animations is tweaked.
    
    (B) Because -viewWillAppear: is called inside the animation, some of the
        setup for the animations needs to be done before that method is called;
        the cleanest mechanism seemed to be to make these changes when the view
        disappeared. This is mostly setting the toolbar alpha to zero so it can be
        faded in during the animation.
    
    
    Cq-Include-Trybots: luci.chromium.try:ios-simulator-full-configs;master.tryserver.chromium.mac:ios-simulator-cronet
    Change-Id: I8c84f5726a0ce49e13c5fd32ab91295ffbb00e57
    Reviewed-on: https://chromium-review.googlesource.com/1127994
    Commit-Queue: Mark Cogan <marq@chromium.org>
    Reviewed-by: edchin <edchin@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574519}

    Changed files

    • ios/chrome/browser/ui/tab_grid/tab_grid_view_controller.mm
    • ios/chrome/browser/ui/tab_grid/transitions/grid_transition_animation.mm
  4. Disabling ViewIDTest.Basic.

    Changed by Mike West - mkwstohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:52:33 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 8fc8ad7e2561f48248fee640b72a808f1592634a

    Comments

    Disabling ViewIDTest.Basic.
    
    This test is consistently failing on:
    
    * 10.11 as of https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/28018
    * 10.12 as of https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.12%20Tests/14366
    * 10.13 as of https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.13%20Tests/4145
    
    TBR=mkwst@chromium.org,kpaulhamus@chromium.org
    
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 862945
    Change-Id: I9156862b472f2ad993f9da7d709b75baaddc2908
    Reviewed-on: https://chromium-review.googlesource.com/1134780
    Reviewed-by: Mike West <mkwst@chromium.org>
    Commit-Queue: Mike West <mkwst@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574518}

    Changed files

    • chrome/browser/ui/cocoa/view_id_util_browsertest.mm
  5. Fix description of NetworkPredictionOptions policy

    Changed by Stefan Teodorescu - faneohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 09:46:16 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 3d8142d8da58d317cc159294a6b9c6139e7173b1

    Comments

    Fix description of NetworkPredictionOptions policy
    
    Removed unnecessary preference names from description.
    
    Bug: 858829
    Change-Id: Ic75e9af5ebca5e56c20e796e1991a217ad427246
    Reviewed-on: https://chromium-review.googlesource.com/1131115
    Reviewed-by: Lutz Justen <ljusten@chromium.org>
    Commit-Queue: Stefan Teodorescu <fane@google.com>
    Cr-Commit-Position: refs/heads/master@{#574517}

    Changed files

    • components/policy/resources/policy_templates.json
  6. [ServiceWorker] Remove native struct ServiceWorkerProviderHostInfo

    Changed by Han Leon - leon.hanohnoyoudont@intel.com
    Changed at Thursday, 12-Jul-18 09:45:56 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d00d598cb4ae00510a30326c3db46f9b0506e36d

    Comments

    [ServiceWorker] Remove native struct ServiceWorkerProviderHostInfo
    
    This CL removes native struct ServiceWorkerProviderHostInfo and the
    corresponding typemapping for it, instead, directly uses mojom struct
    ServiceWorkerProviderHostInfo everywhere.
    
    BUG=789854
    
    Change-Id: I5aa15afb38d5658e25ba40dacae5873cff4caa62
    Reviewed-on: https://chromium-review.googlesource.com/1132847
    Commit-Queue: Han Leon <leon.han@intel.com>
    Reviewed-by: Matt Falkenhagen <falken@chromium.org>
    Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
    Reviewed-by: Makoto Shimazu <shimazu@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574516}

    Changed files

    • content/browser/service_worker/service_worker_dispatcher_host.cc
    • content/browser/service_worker/service_worker_dispatcher_host.h
    • content/browser/service_worker/service_worker_dispatcher_host_unittest.cc
    • content/browser/service_worker/service_worker_object_host_unittest.cc
    • content/browser/service_worker/service_worker_provider_host.cc
    • content/browser/service_worker/service_worker_provider_host.h
    • content/browser/service_worker/service_worker_provider_host_unittest.cc
    • content/browser/service_worker/service_worker_test_utils.cc
    • content/browser/service_worker/service_worker_test_utils.h
    • content/common/BUILD.gn
    • content/common/service_worker/service_worker_provider.mojom
    • content/common/service_worker/service_worker_provider.typemap
    • content/common/service_worker/service_worker_provider_host_info.cc
    • content/common/service_worker/service_worker_provider_host_info.h
    • content/common/service_worker/service_worker_provider_struct_traits.cc
    • content/common/service_worker/service_worker_provider_struct_traits.h
    • content/common/typemaps.gni
    • content/renderer/service_worker/service_worker_network_provider.cc
  7. [unified-consent] Enable sync everything only when sync is active

    Changed by Thomas Tangl - tangltomohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:41:14 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision eef2c0be5716d8857f9ff7dce265e2569c8d333e

    Comments

    [unified-consent] Enable sync everything only when sync is active
    
    UnifiedConsentService is observing changes of the sync service,
    so the sync data types are only enabled when sync is started up
    and active.
    
    Bug: 862139, 819909
    Change-Id: I4c006e0c1cc7b263f113865da0c128620e1817ae
    Reviewed-on: https://chromium-review.googlesource.com/1131744
    Commit-Queue: Thomas Tangl <tangltom@chromium.org>
    Reviewed-by: David Roger <droger@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574515}

    Changed files

    • components/unified_consent/unified_consent_service.cc
    • components/unified_consent/unified_consent_service.h
    • components/unified_consent/unified_consent_service_unittest.cc
  8. Roll src/third_party/depot_tools 32e3d1e37cd6..2b138f742b74 (1 commits)

    Changed by depot-tools-chromium-autoroll - depot-tools-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com
    Changed at Thursday, 12-Jul-18 09:40:04 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f198a1c4c492486c961c0a112247ddac9a9e3379

    Comments

    Roll src/third_party/depot_tools 32e3d1e37cd6..2b138f742b74 (1 commits)
    
    https://chromium.googlesource.com/chromium/tools/depot_tools.git/+log/32e3d1e37cd6..2b138f742b74
    
    
    git log 32e3d1e37cd6..2b138f742b74 --date=short --no-merges --format='%ad %ae %s'
    2018-07-12 sdefresne@chromium.org Avoid creating an empty netrc file in $TEMP
    
    
    Created with:
      gclient setdep -r src/third_party/depot_tools@2b138f742b74
    
    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.
    
    
    
    BUG=chromium:none
    TBR=agable@chromium.org
    
    Change-Id: I8e88cdcad6399e9a7b117a2fe36dc264a6e61c78
    Reviewed-on: https://chromium-review.googlesource.com/1134149
    Reviewed-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@{#574514}

    Changed files

    • DEPS
  9. Revert "Delete the media cache(s) when using a single combined HTTP cache."

    Changed by Mike West - mkwstohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:35:30 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision eb81f73d09c9efd4ff7eb181ce3ffe105d1549a8

    Comments

    Revert "Delete the media cache(s) when using a single combined HTTP cache."
    
    This reverts commit e226149af1b3f003b5d3f59e19ee17bef7419c42.
    
    Reason for revert: 
    
    `ProfileWithoutMediaCacheBrowserTest.DeleteMediaCache` and `ProfileWithoutMediaCacheBrowserTest.DeleteIsolatedAppMediaCache` have been failing, apparently since they were added by this CL.
    
    They were added here: https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/28016 (which failed to actually run `browser_test`) and have been failing consistently since https://ci.chromium.org/p/chromium/builders/luci.chromium.ci/Mac10.11%20Tests/28018.
    
    
    Original change's description:
    > Delete the media cache(s) when using a single combined HTTP cache.
    > 
    > This is so that I can try increasing the media cache size in experiments
    > without increasing Chrome's profile size (Except possibly for profiles
    > that have never been used to play video).
    > 
    > Bug: 789657
    > Change-Id: I45e8f8c6e346a1fa7d05b0bea3e8b001d7dc41e1
    > Reviewed-on: https://chromium-review.googlesource.com/1130162
    > Reviewed-by: Maks Orlovich <morlovich@chromium.org>
    > Commit-Queue: Matt Menke <mmenke@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#574296}
    
    TBR=mmenke@chromium.org,morlovich@chromium.org
    
    Change-Id: Ibc519750f00ad7e698d279af6baffe77b32d30be
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 789657, 862945
    Reviewed-on: https://chromium-review.googlesource.com/1134886
    Reviewed-by: Mike West <mkwst@chromium.org>
    Commit-Queue: Mike West <mkwst@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574513}

    Changed files

    • chrome/browser/profiles/profile_browsertest.cc
    • chrome/browser/profiles/profile_impl_io_data.cc
  10. Cancel long-press operation if the gesture is scrolling

    Changed by yoshiki iguchi - yoshikiohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:21:50 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d417866204188b709447dfb64ba68740a182b7a5

    Comments

    Cancel long-press operation if the gesture is scrolling
    
    This CL adds the code to cancel the long-press operation which shows
    inline setting when the user stats the scrollingo
    
    Previously that was done in Android (ag/4340201) but it didn't work
    for vertical scroll since the scroll is handled on Chrome side. With
    this CL, scroll detection is done on Chrome side and the code handles
    correctly with vertical scrolls in addition to horizontal ones.
    
    Bug: b/111100376
    Bug: 858938
    Test: Manual (inline setting does't open during vertical scroll)
    Change-Id: I84407b331dc320e6acd3a776938104ad018d46a3
    Reviewed-on: https://chromium-review.googlesource.com/1124263
    Commit-Queue: Yoshiki Iguchi <yoshiki@chromium.org>
    Reviewed-by: Eliot Courtney <edcourtney@chromium.org>
    Reviewed-by: Luis Hector Chavez <lhchavez@chromium.org>
    Reviewed-by: Greg Kerr <kerrnel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574512}

    Changed files

    • ash/system/message_center/arc/arc_notification_content_view.cc
    • ash/system/message_center/arc/arc_notification_item.h
    • ash/system/message_center/arc/arc_notification_item_impl.cc
    • ash/system/message_center/arc/arc_notification_item_impl.h
    • ash/system/message_center/arc/arc_notification_manager.cc
    • ash/system/message_center/arc/arc_notification_manager.h
    • ash/system/message_center/arc/mock_arc_notification_item.h
    • components/arc/common/notifications.mojom
    • components/arc/test/fake_notifications_instance.cc
    • components/arc/test/fake_notifications_instance.h
  11. OAuth2LoginManager: Handle consecutive refresh token updates

    Changed by Colin Blundell - blundellohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:20:45 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision c2f7998c5e85ffebc0c1f7588bcc45a73e1de4dc

    Comments

    OAuth2LoginManager: Handle consecutive refresh token updates
    
    OAuth2LoginManager currently assumes that
    OAuth2TokenService::Observer::OnRefreshTokenAvailable() will never be
    called twice consecutively for the primary account; in particular,
    it calls VerifySessionCookies(), which DCHECKS that
    |oauth2_login_verifier_| is null before instantiating that ivar.
    
    However, this assumption is no longer valid. A simple example:
    - A refresh token is found during startup, resulting in
      OnRefreshTokenAvailable() firing.
    - The user revokes their refresh token remotely.
    - Sync locally enters an error state, prompting the user to reauth
      locally.
    - The user reauths locally, resulting in OnRefreshTokenAvailable()
      firing.
    
    Other examples are possible as well.
    
    This CL extends OAuth2LoginManager to handle this case by stopping
    ongoing actions related to the current refresh token for the primary
    account when OAuth2LoginManager receives the notification that that
    refresh token has been updated.
    
    This CL is concretely necessary to unblock
    https://chromium-review.googlesource.com/c/chromium/src/+/1126861,
    which is firing the "token available" notification as part of startup
    when adding an invalid token for the primary account and triggers the
    DCHECK above on ChromeOS browsertests. Those failures were how I
    discovered this issue.
    
    Change-Id: I55ced6fe49055c6c6c4d1e7ce138a0d080ecda38
    Reviewed-on: https://chromium-review.googlesource.com/1127662
    Commit-Queue: Colin Blundell <blundell@chromium.org>
    Reviewed-by: Jacob Dufault <jdufault@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574511}

    Changed files

    • chrome/browser/chromeos/login/signin/oauth2_login_manager.cc
  12. shared worker: Simplify the default network factory logic.

    Changed by Matt Falkenhagen - falkenohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:20:42 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision ab39b893750485f84d1f607be642fe1efdc27f7f

    Comments

    shared worker: Simplify the default network factory logic.
    
    When NetworkService is off but S13nSW is on, shared worker jumped
    through hoops to get a factory bundle to the renderer. It was
    giving the renderer a NetworkService default factory and expecting the
    renderer to clear it.
    
    But we can just give a non-NetworkService direct network default factory,
    since that capability was added in r565058.
    
    Bug: 839982
    Change-Id: Ie5c4f182b87526e2f15a7b2a8b62ed03f7e5595f
    Reviewed-on: https://chromium-review.googlesource.com/1134708
    Commit-Queue: Matt Falkenhagen <falken@chromium.org>
    Reviewed-by: Hiroki Nakagawa <nhiroki@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574510}

    Changed files

    • content/browser/shared_worker/shared_worker_host.cc
    • content/renderer/shared_worker/embedded_shared_worker_stub.cc
  13. Revert "licenses.py: Skip another //tools/binary_size subdirectory."

    Changed by Raphael Kubo da Costa (CET) - raphael.kubo.da.costaohnoyoudont@intel.com
    Changed at Thursday, 12-Jul-18 09:19:39 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision cc3a2b34c667b8e2e0cde7766e9439e80fb72174

    Comments

    Revert "licenses.py: Skip another //tools/binary_size subdirectory."
    
    This reverts commit a6c69c8113a7113cc6b4a4f8de3291095d717e40.
    
    Reason for revert: https://chromium-review.googlesource.com/c/chromium/src/+/1080401 moved gvr-android-sdk into a directory we already skip, so this is no longer needed.
    
    Original change's description:
    > licenses.py: Skip another //tools/binary_size subdirectory.
    > 
    > Commit 6a15a6c4 ("SuperSize: Teach it about thin archives") added a
    > third_party directory to tools/binary_size with a .a file with some dummy
    > contents that we need to skip.
    > 
    > Change-Id: Ia1e129dc67617207ac920106e835d762887e74ac
    > Reviewed-on: https://chromium-review.googlesource.com/1078967
    > Reviewed-by: Scott Graham <scottmg@chromium.org>
    > Commit-Queue: Raphael Kubo da Costa <raphael.kubo.da.costa@intel.com>
    > Cr-Commit-Position: refs/heads/master@{#564081}
    
    TBR=raphael.kubo.da.costa@intel.com,scottmg@chromium.org,ichikawa@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Change-Id: I8a85ee225f54a0499ee18e20311dfd5b9dc280fe
    Reviewed-on: https://chromium-review.googlesource.com/1133138
    Reviewed-by: Hiroshi Ichikawa <ichikawa@chromium.org>
    Commit-Queue: Raphael Kubo da Costa (CET) <raphael.kubo.da.costa@intel.com>
    Cr-Commit-Position: refs/heads/master@{#574509}

    Changed files

    • tools/licenses.py
  14. Add AudioLogFactory mojo interface.

    Changed by Guido Urdaneta - guidouohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 09:19:34 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision ee6cfd00b7b35f8bc4ced3eecc801fa2f55a85ae

    Comments

    Add AudioLogFactory mojo interface.
    
    This interface allows a mojo client to create audio logs on
    the browser process.
    This CL also removes content::MojoLogAudioAdapter, which is moved to the
    audio service as audio::LogAdapter in follow-up CL crrev.com/c/1128974
    
    Bug: 812557
    Change-Id: I70504a4878553f1a3c2d58a29c6ab960b6f15b39
    Reviewed-on: https://chromium-review.googlesource.com/1128966
    Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
    Reviewed-by: Dale Curtis <dalecurtis@chromium.org>
    Commit-Queue: Guido Urdaneta <guidou@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574508}

    Changed files

    • content/browser/BUILD.gn
    • content/browser/media/audio_log_factory.cc
    • content/browser/media/audio_log_factory.h
    • content/browser/media/media_internals.cc
    • content/browser/media/media_internals.h
    • content/browser/media/mojo_audio_logging_adapter.cc
    • content/browser/media/mojo_audio_logging_adapter.h
    • media/mojo/interfaces/audio_logging.mojom
  15. Add service worker README.md.

    Changed by Matt Falkenhagen - falkenohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:46:41 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 75c9e38bcfe01f3a7c2637050ddcb2f342dd3034

    Comments

    Add service worker README.md.
    
    Bug: 795989
    Change-Id: I1f90fa1f71f47fa3ba884814d932bffbc67a4a8d
    Reviewed-on: https://chromium-review.googlesource.com/1134718
    Commit-Queue: Matt Falkenhagen <falken@chromium.org>
    Reviewed-by: Kenichi Ishibashi <bashi@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574507}

    Changed files

    • content/browser/service_worker/README.md
    • content/common/service_worker/README.md
    • third_party/blink/common/service_worker/README.md
    • third_party/blink/public/common/service_worker/README.md
    • third_party/blink/public/mojom/service_worker/README.md
    • third_party/blink/public/platform/modules/service_worker/README.md
    • third_party/blink/public/web/modules/service_worker/README.md
    • third_party/blink/renderer/modules/service_worker/README.md
  16. Use PLM for data saver site-breakdown

    Changed by rajendrant - rajendrantohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:45:55 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 74f3e3d131a093053a5e5f737e162c99e1c69e62

    Comments

    Use PLM for data saver site-breakdown
    
    This CL adds PLM observer for site-breakdown that records the data use
    by host. The new codepath is enabled and the old data use asriber observer
    based site-breakdown codepath is disabled, based on a feature.
    
    Bug: 836029
    Change-Id: I667d0dc6e0227c83a46d353b2feffaec9043a219
    Reviewed-on: https://chromium-review.googlesource.com/1050868
    Commit-Queue: rajendrant <rajendrant@chromium.org>
    Reviewed-by: Tarun Bansal <tbansal@chromium.org>
    Reviewed-by: Ryan Sturm <ryansturm@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574506}

    Changed files

    • chrome/browser/BUILD.gn
    • chrome/browser/page_load_metrics/observers/data_saver_site_breakdown_metrics_observer.cc
    • chrome/browser/page_load_metrics/observers/data_saver_site_breakdown_metrics_observer.h
    • chrome/browser/page_load_metrics/observers/data_saver_site_breakdown_metrics_observer_browsertest.cc
    • chrome/browser/page_load_metrics/page_load_metrics_initialize.cc
    • chrome/test/BUILD.gn
    • components/data_reduction_proxy/core/browser/data_reduction_proxy_data_use_observer.cc
    • components/data_reduction_proxy/core/common/data_reduction_proxy_features.cc
    • components/data_reduction_proxy/core/common/data_reduction_proxy_features.h
    • components/data_reduction_proxy/core/common/data_reduction_proxy_params.cc
    • components/data_reduction_proxy/core/common/data_reduction_proxy_params.h
  17. Add new methods to EntryList and tests

    Changed by Luciano Pacheco - lucmultohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:27:24 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 76bc398de31290a2d97ee6b7ea804a55d1d5451d

    Comments

    Add new methods to EntryList and tests
    
    Change the logic to add entries to My Files to account for volume being
    mounted/unmounted. This fixes some broken tests when running with the
    flag enabled by default, so test to cover this is on
    crrev.com/c/1131025.
    
    Fix test getMetadata function name.
    
    Bug: 857335
    Cq-Include-Trybots: luci.chromium.try:closure_compilation
    Change-Id: Id056b0e0d5c77a07c2561f6566bfc99111864917
    Reviewed-on: https://chromium-review.googlesource.com/1133613
    Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
    Reviewed-by: Noel Gordon <noel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574505}

    Changed files

    • ui/file_manager/file_manager/common/js/files_app_entry_types.js
    • ui/file_manager/file_manager/common/js/files_app_entry_types_unittest.html
    • ui/file_manager/file_manager/common/js/files_app_entry_types_unittest.js
    • ui/file_manager/file_manager/foreground/js/navigation_list_model.js
    • ui/file_manager/file_manager/foreground/js/navigation_list_model_unittest.js
  18. [LayoutNG] Fix static position for inline containing block

    Changed by Aleks Totic - atoticohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:24:12 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f321a196a45bce7d32257d0b93b472dba89e205a

    Comments

    [LayoutNG] Fix static position for inline containing block
    
    Bug: 740993
    Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
    Change-Id: Iacdfc2694b2522c06a2512000ed1ca7f043b7541
    Reviewed-on: https://chromium-review.googlesource.com/1134386
    Reviewed-by: Koji Ishii <kojii@chromium.org>
    Commit-Queue: Aleks Totic <atotic@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574504}

    Changed files

    • third_party/WebKit/LayoutTests/FlagExpectations/enable-blink-features=LayoutNG
    • third_party/blink/renderer/core/layout/ng/ng_out_of_flow_layout_part.cc
  19. NetS13nServiceWorker: Add more info to traces in SWNavigationLoader

    Changed by Makoto Shimazu - shimazuohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:13:43 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 670a4fd7d5c89c8904f638edda72083e41e5d016

    Comments

    NetS13nServiceWorker: Add more info to traces in SWNavigationLoader
    
    Bug: 806658
    Change-Id: Ie52bdfdd6f44354c42067f1c069996cc80b6df17
    Reviewed-on: https://chromium-review.googlesource.com/1134711
    Reviewed-by: Matt Falkenhagen <falken@chromium.org>
    Commit-Queue: Makoto Shimazu <shimazu@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574503}

    Changed files

    • content/browser/service_worker/service_worker_navigation_loader.cc
  20. [Android PageInfo] Refactor PageInfoController to enable testing

    Changed by Anita Woodruff - awdfohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 08:05:32 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision dfb08c4ccef342639df17f6a423643df1784b767

    Comments

    [Android PageInfo] Refactor PageInfoController to enable testing
    
    - Extracted PermissionParamsListBuilder from PageInfoController
    to handle all the permission-related logic when displaying
    permissions in page info. This new class requires fewer
    hard-to-unit-test dependencies so its logic can now be tested.
    
    Bug: 691898
    Change-Id: Iddbea30642a0dc31dae26110381e2dad9feb2af4
    Reviewed-on: https://chromium-review.googlesource.com/1103564
    Commit-Queue: Anita Woodruff <awdf@chromium.org>
    Reviewed-by: Tibor Goldschwendt <tiborg@chromium.org>
    Reviewed-by: Ted Choc <tedchoc@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574502}

    Changed files

    • chrome/android/java/src/org/chromium/chrome/browser/page_info/PageInfoController.java
    • chrome/android/java/src/org/chromium/chrome/browser/page_info/PermissionParamsListBuilder.java
    • chrome/android/java/src/org/chromium/chrome/browser/page_info/SystemSettingsActivityRequiredListener.java
    • chrome/android/java_sources.gni
    • chrome/android/junit/src/org/chromium/chrome/browser/page_info/PermissionParamsListBuilderUnitTest.java
  21. oilpan: Make ImageLoader::Task::script_state_ WeakPersistent

    Changed by Hitoshi Yoshida - periaohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 07:41:46 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f59c5558bcae3a45e08edc99450f00242e5e7ce0

    Comments

    oilpan: Make ImageLoader::Task::script_state_ WeakPersistent
    
    ImageLoader::Task class does not actually own the script state,
    so its reference can be weak.
    This CL makes the reference weak.
    
    Bug: 773605
    Change-Id: Ib5761227e6582a8519a9d10ffaa00226559ee3d6
    Reviewed-on: https://chromium-review.googlesource.com/1134629
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Commit-Queue: Hitoshi Yoshida <peria@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574501}

    Changed files

    • third_party/blink/renderer/core/loader/image_loader.cc
  22. Change test to use full-path-for-testing attribute selector

    Changed by Luciano Pacheco - lucmultohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 07:12:07 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 9a8460b3e49c7b9ee03dcf56af71a0d76c0f60e4

    Comments

    Change test to use full-path-for-testing attribute selector
    
    This makes this test compatible with directory tree having My Files or
    not, and makes the test more readable too.
    
    This is a preparation to enable new navigation (with My Files) by
    default.
    
    Bug: 857335
    No-presubmit: true
    Cq-Include-Trybots: luci.chromium.try:closure_compilation
    Change-Id: Ia35bc2cc7c91e0492f0158990bfc397a6b57da21
    Reviewed-on: https://chromium-review.googlesource.com/1133610
    Commit-Queue: Noel Gordon <noel@chromium.org>
    Reviewed-by: Noel Gordon <noel@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574500}

    Changed files

    • ui/file_manager/integration_tests/file_manager/context_menu.js
    • ui/file_manager/integration_tests/file_manager/create_new_folder.js
    • ui/file_manager/integration_tests/file_manager/folder_shortcuts.js
  23. Add find bar icon to hosted app windows

    Changed by Alan Cutter - alancutterohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 07:03:30 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision e35f205a1441afebe9987e606c7c876d5bfc4594

    Comments

    Add find bar icon to hosted app windows
    
    This CL moves the find icon from LocationBarView into PageActionIconContainerView
    enabling the icon to appear in hosted app window title bars.
    
    This change also fixes the find icon getting nudged by
    crrev.com/108c0687254683b3beeaa32b6e0a86f2046a867e.
    
    Before 108c0687254683b3beeaa32b6e0a86f2046a867e:
    https://bugs.chromium.org/p/chromium/issues/attachment?aid=346964&signed_aid=IuPvufZswamla90tWW5FCg==&inline=1
    Before CL:
    https://bugs.chromium.org/p/chromium/issues/attachment?aid=346965&signed_aid=Aiy5ATHUK4cWS64MyuRMdQ==&inline=1
    After CL:
    https://bugs.chromium.org/p/chromium/issues/attachment?aid=346968&signed_aid=DR3vtzLR7tlPH3cEwibshQ==&inline=1
    
    Bug: 788051, 814531
    Change-Id: I5fa60cce484bd2553131582fc39e6737da34e967
    Reviewed-on: https://chromium-review.googlesource.com/1054908
    Commit-Queue: Alan Cutter <alancutter@chromium.org>
    Reviewed-by: Trent Apted <tapted@chromium.org>
    Reviewed-by: Nico Weber <thakis@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574499}

    Changed files

    • chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.h
    • chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
    • chrome/browser/ui/find_bar/find_bar_controller.cc
    • chrome/browser/ui/location_bar/location_bar.h
    • chrome/browser/ui/page_action/page_action_icon_container.h
    • chrome/browser/ui/views/frame/browser_non_client_frame_view_ash_browsertest.cc
    • chrome/browser/ui/views/frame/browser_view_layout.cc
    • chrome/browser/ui/views/frame/hosted_app_button_container.cc
    • chrome/browser/ui/views/frame/hosted_app_button_container.h
    • chrome/browser/ui/views/frame/toolbar_button_provider.h
    • chrome/browser/ui/views/location_bar/find_bar_icon.cc
    • chrome/browser/ui/views/location_bar/find_bar_icon.h
    • chrome/browser/ui/views/location_bar/location_bar_view.cc
    • chrome/browser/ui/views/location_bar/location_bar_view.h
    • chrome/browser/ui/views/page_action/page_action_icon_container_view.cc
    • chrome/browser/ui/views/page_action/page_action_icon_container_view.h
    • chrome/browser/ui/views/toolbar/toolbar_view.cc
    • chrome/browser/ui/views/toolbar/toolbar_view.h
    • chrome/test/base/test_browser_window.h
  24. Replace ThreadChecker by SequenceChecker in /components/sync*

    Changed by Marc Treib - treibohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 06:54:48 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision fc8a763a002339d6a7821e3c69d45ad8642dafc1

    Comments

    Replace ThreadChecker by SequenceChecker in /components/sync*
    
    https://crrev.com/c/1131474 removed the unnecessary thread-affinity,
    this one adjusts the checks accordingly.
    
    Bug: 846238
    Change-Id: If4e1145712fe99f4d8236d93e4f645e69c3e7655
    Reviewed-on: https://chromium-review.googlesource.com/1131738
    Reviewed-by: Mikel Astiz <mastiz@chromium.org>
    Commit-Queue: Mikel Astiz <mastiz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574498}

    Changed files

    • components/sync/driver/glue/sync_backend_host_core.cc
    • components/sync/driver/glue/sync_backend_host_core.h
    • components/sync/driver/glue/sync_backend_host_impl.cc
    • components/sync/driver/glue/sync_backend_host_impl.h
    • components/sync/driver/sync_service_crypto.cc
    • components/sync/driver/sync_service_crypto.h
    • components/sync/engine/sync_backend_registrar.cc
    • components/sync/engine/sync_backend_registrar.h
    • components/sync/engine_impl/debug_info_event_listener.cc
    • components/sync/engine_impl/debug_info_event_listener.h
    • components/sync/engine_impl/model_type_worker.cc
    • components/sync/engine_impl/model_type_worker.h
    • components/sync/engine_impl/net/server_connection_manager.cc
    • components/sync/engine_impl/net/server_connection_manager.h
    • components/sync/engine_impl/sync_manager_impl.cc
    • components/sync/engine_impl/sync_manager_impl.h
    • components/sync/engine_impl/sync_manager_impl_unittest.cc
    • components/sync/test/engine/fake_model_worker.cc
    • components/sync/test/engine/fake_model_worker.h
    • components/sync_bookmarks/bookmark_change_processor.cc
    • components/sync_bookmarks/bookmark_change_processor.h
  25. Track changes to nodes that are in the AX tree but aren't laid out.

    Changed by Dominic Mazzoni - dmazzoniohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 06:41:00 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision d535bf95b998d1061629905139485c937272fc3b

    Comments

    Track changes to nodes that are in the AX tree but aren't laid out.
    
    Previously we only had hooks to update the children of a node in the
    AX tree when LayoutObjects were inserted or removed. That left out some
    corner cases when Nodes are inserted or removed that are part of the
    AX tree but not laid out, like nodes in the accessible / fallback content
    for a canvas.
    
    Bug: 862311
    Change-Id: I9482a9e0dc9ba45368b2fc87ea3449bdd4ef1c50
    Reviewed-on: https://chromium-review.googlesource.com/1132119
    Reviewed-by: Alice Boxhall <aboxhall@chromium.org>
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Commit-Queue: Dominic Mazzoni <dmazzoni@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574497}

    Changed files

    • third_party/WebKit/LayoutTests/accessibility/children-changed-in-canvas.html
    • third_party/blink/renderer/core/dom/element.cc
    • third_party/blink/renderer/modules/accessibility/ax_layout_object.cc
    • third_party/blink/renderer/modules/accessibility/ax_layout_object.h
    • third_party/blink/renderer/modules/accessibility/ax_node_object.cc
    • third_party/blink/renderer/modules/accessibility/ax_node_object.h