Home | Search | chromium - Builders
Login

Builder chromeos-amd64-generic-rel Build 16424 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revisionf7c234b0272dff6ce746e9eaac813497986913c3
Got Revisionf7c234b0272dff6ce746e9eaac813497986913c3

Execution:

Steps and Logfiles:

Show:
  1. ( 336 ms ) setup_build
    running recipe: "chromium"
  2. ( 533 ms ) remove .cipd
  3. ( 45 secs ) bot_update
    [84GB/295GB used (28%)]
  4. ( 695 ms ) swarming.py --version
    0.13
  5. ( 2 mins 46 secs ) gclient runhooks
  6. ( 385 ms ) read test spec (chromium.chromiumos.json)
    path: /b/swarming/w/ir/cache/builder/src/testing/buildbot/chromium.chromiumos.json
  7. ( 46 secs ) generate_build_files
  8. ( 1 mins 31 secs ) compile
  9. ( 4 mins 49 secs ) isolate tests
  10. ( 578 ms ) find isolated tests
  11. ( 2 secs ) base_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:50.170890
  12. ( 1 secs ) cacheinvalidation_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:04.141810
  13. ( 1 secs ) capture_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:29.212210
  14. ( 2 secs ) cc_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:02:03.679570
  15. ( 1 mins 21 secs ) cros_vm_sanity_test
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:55.679540
  16. ( 1 secs ) crypto_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:02.553970
  17. ( 1 secs ) display_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:14.108160
  18. ( 1 secs ) google_apis_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:59.137160
  19. ( 1 secs ) ipc_tests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:02:01.242120
  20. ( 1 secs ) jingle_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:54.725800
  21. ( 1 secs ) latency_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:05.092660
  22. ( 1 secs ) midi_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 58s
  23. ( 1 secs ) mojo_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:12.079440
  24. ( 1 secs ) ozone_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:15.875020
  25. ( 1 secs ) pdf_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:37.044110
  26. ( 1 secs ) printing_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:13.616980
  27. ( 1 secs ) sql_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:01:03.716970
  28. ( 1 secs ) url_unittests
    Run on OS: 'Ubuntu-14.04'
    Shard duration: 0:02:35.826550
  29. ( 313 ms ) Tests statistics
     
    Stats
    Total shards: 18
    Total runtime: 0:24:28.720852
     

Build Properties:

NameValueSource
$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 "swarm630-c4" setup_build
branch "refs/heads/master" setup_build
buildbucket {"build": {"bucket": "luci.chromium.ci", "created_by": "user:luci-scheduler@appspot.gserviceaccount.com", "created_ts": 1531413807801790, "id": "8941189951029090256", "project": "chromium", "tags": ["builder:chromeos-amd64-generic-rel", "buildset:commit/git/f7c234b0272dff6ce746e9eaac813497986913c3", "buildset:commit/gitiles/chromium.googlesource.com/chromium/src/+/f7c234b0272dff6ce746e9eaac813497986913c3", "gitiles_ref:refs/heads/master", "scheduler_invocation_id:9106620013418092896", "scheduler_job_id:chromium/chromeos-amd64-generic-rel", "user_agent:luci-scheduler"]}, "hostname": "cr-buildbucket.appspot.com"} setup_build
buildername "chromeos-amd64-generic-rel" setup_build
buildnumber 16424 setup_build
mastername "chromium.chromiumos" setup_build
path_config "generic" setup_build
recipe "chromium" setup_build
repository "https://chromium.googlesource.com/chromium/src.git" setup_build
revision "f7c234b0272dff6ce746e9eaac813497986913c3" setup_build
got_angle_revision "5598148b761380773c4e650b5b67f47553b0f090" bot_update
got_buildtools_revision "0dd5c6f980d22be96b728155249df2da355989d9" bot_update
got_nacl_revision "e6ce828ef60c4c1438867b535efbbb5d9a177c0e" bot_update
got_revision "f7c234b0272dff6ce746e9eaac813497986913c3" bot_update
got_revision_cp "refs/heads/master@{#574598}" 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 "58c79f66dd3afef0ec2d77053e44f26b998b84b1" bot_update
got_webrtc_revision_cp "refs/heads/master@{#23950}" bot_update
swarm_hashes_refs/heads/master(at){#574598}_without_patch {"base_unittests": "1d3d2fbf31c074bd0acd6f2cbade2fa1d8fc8161", "cacheinvalidation_unittests": "dac56fbe8c2f2940065c7342e3fe12c0d71aae3a", "capture_unittests": "ec6754af5f8f87c343638d2628e02c56fe01c9b4", "cc_unittests": "0541af930fd76674a1b18d61c1cd0cddd807c0e6", "cros_vm_sanity_test": "4b45dec0b06cb050b945c72c534e6c53f7c6e119", "crypto_unittests": "810d0e5e5e0f80a398d3ecc8776b287a10416f6c", "display_unittests": "40da57884cda1f4dafd43f4d3a57659992f84790", "google_apis_unittests": "0566d2987ddf44f118b14e3896318c6105b46af1", "ipc_tests": "dba3377b5876bfcd636fe0e6bf9abf4ffc0156ec", "jingle_unittests": "b17da6e36ba64fb7aa3cd146279fefbc88176a93", "latency_unittests": "1cab41278849763ebcb10ef339d0a4b4a033d6fe", "midi_unittests": "bbd6f3440c44c5133df98d970a1d1e1a62200eda", "mojo_unittests": "cdf0425b110cf68d8e0fb071a46aa16fe29118ee", "ozone_unittests": "7b721a025bab84ef5727cce28b8483011dbc0aee", "pdf_unittests": "18b4d4c3e929e0739db1581722e1ebb2836637b7", "printing_unittests": "f735b93cab7a6f336adc62fdff69888deb98969d", "sql_unittests": "c27f1006d6b6171310edf966bbfb9daa5a6e199d", "url_unittests": "054f08fcef27baa5eb20cd3bd987fd0e4e3c5ab5"} isolate tests
swarm_hashes {"base_unittests": "1d3d2fbf31c074bd0acd6f2cbade2fa1d8fc8161", "cacheinvalidation_unittests": "dac56fbe8c2f2940065c7342e3fe12c0d71aae3a", "capture_unittests": "ec6754af5f8f87c343638d2628e02c56fe01c9b4", "cc_unittests": "0541af930fd76674a1b18d61c1cd0cddd807c0e6", "cros_vm_sanity_test": "4b45dec0b06cb050b945c72c534e6c53f7c6e119", "crypto_unittests": "810d0e5e5e0f80a398d3ecc8776b287a10416f6c", "display_unittests": "40da57884cda1f4dafd43f4d3a57659992f84790", "google_apis_unittests": "0566d2987ddf44f118b14e3896318c6105b46af1", "ipc_tests": "dba3377b5876bfcd636fe0e6bf9abf4ffc0156ec", "jingle_unittests": "b17da6e36ba64fb7aa3cd146279fefbc88176a93", "latency_unittests": "1cab41278849763ebcb10ef339d0a4b4a033d6fe", "midi_unittests": "bbd6f3440c44c5133df98d970a1d1e1a62200eda", "mojo_unittests": "cdf0425b110cf68d8e0fb071a46aa16fe29118ee", "ozone_unittests": "7b721a025bab84ef5727cce28b8483011dbc0aee", "pdf_unittests": "18b4d4c3e929e0739db1581722e1ebb2836637b7", "printing_unittests": "f735b93cab7a6f336adc62fdff69888deb98969d", "sql_unittests": "c27f1006d6b6171310edf966bbfb9daa5a6e199d", "url_unittests": "054f08fcef27baa5eb20cd3bd987fd0e4e3c5ab5"} find isolated tests

Blamelist:

  1. Sergey Ulanov (sergeyuohnoyoudont@chromium.org)
  2. Becky Zhou (huayinzohnoyoudont@chromium.org)
  3. Nico Weber (thakisohnoyoudont@chromium.org)
  4. David Grogan (dgroganohnoyoudont@chromium.org)
  5. Nick Burris (nburrisohnoyoudont@google.com)
  6. Rayan Kanso (rayankansohnoyoudont@chromium.org)
  7. Mike Reed (reedohnoyoudont@google.com)
  8. Peter E Conn (peconnohnoyoudont@chromium.org)
  9. Peter Kasting (pkastingohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 16:43:27 UTC
Start Thursday, 12-Jul-18 16:43:29 UTC
End Thursday, 12-Jul-18 16:58:44 UTC
Pending 2 secs
Execution 15 mins 14 secs

All Changes:

  1. [Fuchsia] Address CR comments in sys::Runner implementation

    Changed by Sergey Ulanov - sergeyuohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:39:02 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision f7c234b0272dff6ce746e9eaac813497986913c3

    Comments

    [Fuchsia] Address CR comments in sys::Runner implementation
    
    When addressing comments on https://crrev.com//c//1114377 I hit CQ+1
    prematurely and landed the change without my changes that addresed
    review comments. Landing them separately.
    
    Bug: 856416
    Change-Id: I1b691d824f381e9ec0a75ff7ce90b2f45171c13a
    Reviewed-on: https://chromium-review.googlesource.com/1133968
    Reviewed-by: Sergey Ulanov <sergeyu@chromium.org>
    Reviewed-by: Wez <wez@chromium.org>
    Commit-Queue: Sergey Ulanov <sergeyu@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574598}

    Changed files

    • webrunner/BUILD.gn
    • webrunner/app/component_controller_impl.cc
    • webrunner/app/component_controller_impl.h
    • webrunner/app/main.cc
    • webrunner/app/web_content_runner.cc
    • webrunner/app/web_content_runner.h
  2. [Feed] Scroll and focus handling for the FeedNewTabPage

    Changed by Becky Zhou - huayinzohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:36:01 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision b54d1975f16b7efae2408a1428fbde7207556a29

    Comments

    [Feed] Scroll and focus handling for the FeedNewTabPage
    
    + Implement NewTabPageLayout.ScrollDelegate to the FeedNewTabPage
    + Refactor snap scroll logic to a new class to make it reusable
    + Add top padding for the FeedNewTabPage to display correctly on tablet
    + Fix a small bug that search box is not enabled after auto snap scroll
    
    Bug: 860028, 860032, 862470
    Change-Id: I5bda9696a4515281378c1effe47b6ad47fdd4c49
    Reviewed-on: https://chromium-review.googlesource.com/1133594
    Reviewed-by: Theresa <twellington@chromium.org>
    Commit-Queue: Becky Zhou <huayinz@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574597}

    Changed files

    • chrome/android/feed/core/java/src/org/chromium/chrome/browser/feed/FeedNewTabPage.java
    • chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPage.java
    • chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageLayout.java
    • chrome/android/java/src/org/chromium/chrome/browser/ntp/NewTabPageView.java
    • chrome/android/java/src/org/chromium/chrome/browser/ntp/SnapScrollHelper.java
    • chrome/android/java/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerView.java
    • chrome/android/java_sources.gni
    • chrome/android/javatests/src/org/chromium/chrome/browser/ntp/cards/NewTabPageRecyclerViewTest.java
    • chrome/android/javatests/src/org/chromium/chrome/browser/suggestions/TileGroupTest.java
  3. Remove most android-specific hard_timeout and shard overrides.

    Changed by Nico Weber - thakisohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:35:49 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1a64d6771d8b24f079f24e41ebaf14f672747084

    Comments

    Remove most android-specific hard_timeout and shard overrides.
    
    These were added during android swarming bring-up.  We shouldn't use
    per-bot exceptions for this.
    
    Also remove a random sharding setting for base_unittests on ToTMac
    that doesn't look necessary.
    
    Bug: 843511
    Change-Id: Ia20771786f1b4106cd86d24d446e0a66f319dbff
    Reviewed-on: https://chromium-review.googlesource.com/1134531
    Reviewed-by: John Budorick <jbudorick@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574596}

    Changed files

    • testing/buildbot/chromium.android.fyi.json
    • testing/buildbot/chromium.android.json
    • testing/buildbot/chromium.clang.json
    • testing/buildbot/chromium.fyi.json
    • testing/buildbot/chromium.memory.json
    • testing/buildbot/chromium.swarm.json
    • testing/buildbot/test_suite_exceptions.pyl
  4. [css-tables] Make table's max-content honor captions' min-content

    Changed by David Grogan - dgroganohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:34:30 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision dff5931455e0302289a3cc9027fd3f98cfe22fca

    Comments

    [css-tables] Make table's max-content honor captions' min-content
    
    Table's min-content already honored it.
    
    We'd apparently never been bitten before by tables occasionally having
    min-content > max-content. A new DCHECK in LayoutNG exposed it.
    
    Bug: 857185
    Cq-Include-Trybots: luci.chromium.try:linux_layout_tests_layout_ng
    Change-Id: I49009a4988fbf38c1bf745a4f028e4e8b050804b
    Reviewed-on: https://chromium-review.googlesource.com/1121251
    Commit-Queue: David Grogan <dgrogan@chromium.org>
    Reviewed-by: Christian Biesinger <cbiesinger@chromium.org>
    Reviewed-by: Morten Stenshorne <mstensho@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574595}

    Changed files

    • third_party/WebKit/LayoutTests/TestExpectations
    • third_party/WebKit/LayoutTests/fast/table/caption-min-greater-than-max-crash.html
    • third_party/WebKit/LayoutTests/fast/table/spans-min-greater-than-max-crash.html
    • third_party/blink/renderer/core/layout/layout_table.cc
    • third_party/blink/renderer/core/layout/ng/ng_block_layout_algorithm.cc
  5. Settings WebUI: Fix extensions url on cleanup result page

    Changed by Nick Burris - nburrisohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 16:33:17 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 7cde465a809facc9ecb61350aaa5c504a36d9174

    Comments

    Settings WebUI: Fix extensions url on cleanup result page
    
    The cleaner success string now has HTML in it, and the JS i18n
    function strips it off. i18nAdvanced allows us to let the <a> through to
    the string, but the way the cleanup result page works is it simply sets
    the card text variable ("title") to the result message string. This
    would result in the raw string (with HTML) getting displayed, so the
    title variable needs to also be set to be the inner-h-t-m-l.
    
    Bug: 853339
    Cq-Include-Trybots: luci.chromium.try:closure_compilation
    Change-Id: I8ea98e9868f83fc14a674841072cc2915c7b4a48
    Reviewed-on: https://chromium-review.googlesource.com/1134191
    Commit-Queue: Nick Burris <nburris@google.com>
    Reviewed-by: proberge <proberge@chromium.org>
    Reviewed-by: Demetrios Papadopoulos <dpapad@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574594}

    Changed files

    • chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.html
    • chrome/browser/resources/settings/chrome_cleanup_page/chrome_cleanup_page.js
    • chrome/test/data/webui/settings/chrome_cleanup_page_test.js
  6. [Background Fetch] Add DB corruption handler to data manager observer.

    Changed by Rayan Kanso - rayankansohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:31:27 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 96544636e959581575b9c4f1ac2ecd1c5f40296d

    Comments

    [Background Fetch] Add DB corruption handler to data manager observer.
    
    Bug: 850512
    Change-Id: Iddd2773f63fd7aa0368e980a347c1bef4312f23d
    Reviewed-on: https://chromium-review.googlesource.com/1128839
    Commit-Queue: Rayan Kanso <rayankans@chromium.org>
    Reviewed-by: Peter Beverloo <peter@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574593}

    Changed files

    • content/browser/background_fetch/background_fetch_context.cc
    • content/browser/background_fetch/background_fetch_context.h
    • content/browser/background_fetch/background_fetch_data_manager.cc
    • content/browser/background_fetch/background_fetch_data_manager.h
    • content/browser/background_fetch/background_fetch_data_manager_observer.h
    • content/browser/background_fetch/background_fetch_data_manager_unittest.cc
    • content/browser/background_fetch/background_fetch_test_data_manager.cc
    • content/browser/background_fetch/storage/database_task.cc
    • content/browser/background_fetch/storage/database_task.h
    • content/browser/background_fetch/storage/delete_registration_task.cc
    • content/browser/background_fetch/storage/get_settled_fetches_task.cc
    • content/browser/background_fetch/storage/mark_registration_for_deletion_task.cc
    • content/browser/background_fetch/storage/start_next_pending_request_task.cc
  7. suppress until skia roll

    Changed by Mike Reed - reedohnoyoudont@google.com
    Changed at Thursday, 12-Jul-18 16:30:15 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision dcf8a49d5e3d2440dfc3f8e3126fa266fd25a33b

    Comments

    suppress until skia roll
    
    Bug: 863042
    Change-Id: I97a48047d0b6546055c255538fc31ddcc0a07078
    Reviewed-on: https://chromium-review.googlesource.com/1135288
    Commit-Queue: Mike Reed <reed@chromium.org>
    Commit-Queue: Florin Malita <fmalita@chromium.org>
    Reviewed-by: Florin Malita <fmalita@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574592}

    Changed files

    • third_party/WebKit/LayoutTests/TestExpectations
  8. 🤝 Add flag to force showing TWA disclosure.

    Changed by Peter E Conn - peconnohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:28:53 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 967fc76c423a5e86fc2e04b4655aa463d2ae7c12

    Comments

    🤝 Add flag to force showing TWA disclosure.
    
    Add a flag to force showing the disclosure for developer convenience.
    Also trigger even when there isn't an active tab yet.
    
    Bug: 813100
    Change-Id: I3ccd820d58f82040723a8f8a72097262ba374b82
    Reviewed-on: https://chromium-review.googlesource.com/1116922
    Reviewed-by: Benoit L <lizeb@chromium.org>
    Reviewed-by: Bernhard Bauer <bauerb@chromium.org>
    Commit-Queue: Peter Conn <peconn@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574591}

    Changed files

    • chrome/android/java/src/org/chromium/chrome/browser/ChromeSwitches.java
    • chrome/android/java/src/org/chromium/chrome/browser/browserservices/TrustedWebActivityDisclosure.java
    • chrome/android/java/src/org/chromium/chrome/browser/webapps/WebappActivity.java
  9. Once TabStrip doesn't need it, there's no reason to expose GetTopOffset().

    Changed by Peter Kasting - pkastingohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 16:28:33 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1b2b9af8ace2bbc3384147241601d4293f0c2259

    Comments

    Once TabStrip doesn't need it, there's no reason to expose GetTopOffset().
    
    Bug: none
    Change-Id: Ia78dd54f4364e7010ddf3ce16b01e5f5f3c764e4
    Reviewed-on: https://chromium-review.googlesource.com/1134499
    Commit-Queue: Peter Kasting <pkasting@chromium.org>
    Reviewed-by: Allen Bauer <kylixrd@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574590}

    Changed files

    • chrome/browser/ui/views/tabs/new_tab_button.cc
    • chrome/browser/ui/views/tabs/new_tab_button.h