Home | Search | chromium - Builders
Login

Builder Linux Tests (dbg)(1)(32) Build 51997 Canonical Ubuntu

Results:

Success

Trigger Info:

Projectchromium
Revision3a30357690bfd391e2ddab461c6bd48e2c8a5580
Got Revision3a30357690bfd391e2ddab461c6bd48e2c8a5580

Execution:

Steps and Logfiles:

Show:
  1. ( 106751 days 23 hrs ) Failed to fetch step information from LogDog
    Log stream has no annotation entries

Build Properties:

NameValueSource

Blamelist:

  1. Findit (findit-for-meohnoyoudont@appspot.gserviceaccount.com)
  2. Peter Kasting (pkastingohnoyoudont@chromium.org)
  3. Yuwei Huang (yuweihohnoyoudont@chromium.org)

Timing:

Create Saturday, 11-Aug-18 06:10:53 UTC
Start Saturday, 11-Aug-18 06:10:59 UTC
End Saturday, 11-Aug-18 07:09:23 UTC
Pending 6 secs
Execution 58 mins 23 secs

All Changes:

  1. Revert "Add vector of intervals range checking to Time Restrictions policy"

    Changed by Findit - findit-for-meohnoyoudont@appspot.gserviceaccount.com
    Changed at Saturday, 11-Aug-18 05:19:12 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 3a30357690bfd391e2ddab461c6bd48e2c8a5580

    Comments

    Revert "Add vector of intervals range checking to Time Restrictions policy"
    
    This reverts commit b3392d988ae23dc52d9b386af47dc6a53a2d22c8.
    
    Reason for revert:
    
    Findit (https://goo.gl/kROfz5) identified CL at revision 582403 as the
    culprit for failures in the build cycles as shown on:
    https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyRAsSDVdmU3VzcGVjdGVkQ0wiMWNocm9taXVtL2IzMzkyZDk4OGFlMjNkYzUyZDliMzg2YWY0N2RjNmE1M2EyZDIyYzgM
    
    Sample Failed Build: https://ci.chromium.org/buildbot/chromium.memory/Linux%20Chromium%20OS%20ASan%20LSan%20Tests%20%281%29/28575
    
    Sample Failed Step: unit_tests
    
    Original change's description:
    > Add vector of intervals range checking to Time Restrictions policy
    > 
    > * Add function GetEndOfCurrent interval to obtain the last end of the
    > interval which contains the current time.
    > * Add function MaterializeIntervalsToLocalTimezone, which materializes
    > the given intervals to the current local timezone.
    > * Add function TimeIsInsideCrosSettingsIntervals, which checks if the
    > current time is inside the intervals set in cros settings. It then
    > returns the localized string of the end of the interval that contains
    > the current time.
    > 
    > BUG=chromium:852860
    > TEST=added unittests
    > 
    > Change-Id: Ife32e31c8a929827fd14ab9f108c49eacb9c32c3
    > Reviewed-on: https://chromium-review.googlesource.com/1132676
    > Reviewed-by: Maksim Ivanov <emaxx@chromium.org>
    > Commit-Queue: Maksim Ivanov <emaxx@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#582403}
    
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    BUG=chromium:852860
    
    Change-Id: I1663fc559003d938270da4b8fd7e85719b8e9e01
    Reviewed-on: https://chromium-review.googlesource.com/1172003
    Cr-Commit-Position: refs/heads/master@{#582430}

    Changed files

    • chrome/browser/chromeos/BUILD.gn
    • chrome/browser/chromeos/policy/device_auto_update_time_restrictions_utils.cc
    • chrome/browser/chromeos/policy/device_auto_update_time_restrictions_utils.h
    • chrome/browser/chromeos/policy/device_auto_update_time_restrictions_utils_unittest.cc
    • chrome/browser/chromeos/policy/weekly_time/time_utils.cc
    • chrome/browser/chromeos/policy/weekly_time/time_utils.h
    • chrome/browser/chromeos/policy/weekly_time/time_utils_unittest.cc
    • chrome/browser/chromeos/policy/weekly_time/weekly_time.h
    • chrome/browser/chromeos/policy/weekly_time/weekly_time_interval.h
  2. Revert "Remove unnecessary tab key press from Focus_AdvanceFocus test"

    Changed by Peter Kasting - pkastingohnoyoudont@chromium.org
    Changed at Saturday, 11-Aug-18 05:15:47 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 6df1590559bd965c71b181b4d948e0ea3318189a

    Comments

    Revert "Remove unnecessary tab key press from Focus_AdvanceFocus test"
    
    This reverts commit 5eeadd143332f6fc6c26af924d8a38d5f49b06a2.
    
    Reason for revert: Seems to have caused failures on mac_cocoa_rel, see e.g. https://logs.chromium.org/logs/chromium/buildbucket/cr-buildbucket.appspot.com/8938549477439492256/+/steps/interactive_ui_tests/0/logs/WebViewFocusInteractiveTest.Focus_AdvanceFocus/0
    
    Original change's description:
    > Remove unnecessary tab key press from Focus_AdvanceFocus test
    > 
    > The associated bug which required this extra tab is fixed.
    > 
    > Bug: 147644
    > Change-Id: Ied21533accc82613053e119aa2c5c64fc9de8343
    > Reviewed-on: https://chromium-review.googlesource.com/1170986
    > Reviewed-by: Ehsan Karamad <ekaramad@chromium.org>
    > Commit-Queue: Kevin McNee <mcnee@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#582240}
    
    TBR=ekaramad@chromium.org,mcnee@chromium.org
    
    Change-Id: I6c84e6950d7f74ae5b15eec084a536115e635ceb
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 147644
    Reviewed-on: https://chromium-review.googlesource.com/1172002
    Reviewed-by: Peter Kasting <pkasting@chromium.org>
    Commit-Queue: Peter Kasting <pkasting@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#582429}

    Changed files

    • chrome/browser/apps/guest_view/web_view_interactive_browsertest.cc
  3. [CRD iOS] Make ChromotingSession own the AudioStub

    Changed by Yuwei Huang - yuweihohnoyoudont@chromium.org
    Changed at Saturday, 11-Aug-18 04:59:12 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 206d44a2086dc771c898b5d3f9da0bcd95f6c0a7

    Comments

    [CRD iOS] Make ChromotingSession own the AudioStub
    
    Previously AudioStub is owned by the the (Remoting|Jni)Client class and
    being passed to ChromotingSession as a WeakPtr. The AudioStub is not
    used after it is passed to ChromotingSession and it is a burden for the
    *Client class to manage its lifetime.
    
    This CL fixes this by making ChromotingSession own the AudioStub.
    
    Bug: 868088
    Change-Id: I067f321b3aab7110df7bb85a8b8c519ea889865e
    Reviewed-on: https://chromium-review.googlesource.com/1171840
    Reviewed-by: Joe Downing <joedow@chromium.org>
    Commit-Queue: Yuwei Huang <yuweih@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#582428}

    Changed files

    • remoting/client/audio/audio_playback_stream.cc
    • remoting/client/audio/audio_playback_stream.h
    • remoting/client/chromoting_session.cc
    • remoting/client/chromoting_session.h
    • remoting/client/jni/jni_client.cc
    • remoting/client/jni/jni_client.h
    • remoting/ios/session/remoting_client.mm