Home | Search | chromium - Builders
Login

Builder Win10 Tests x64 Build 25526 Microsoft Windows

Results:

Success

Trigger Info:

Projectchromium
Revision4a7c8ee607674efabc720016992893e2226ce3a0
Got Revision4a7c8ee607674efabc720016992893e2226ce3a0

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. Yutaka Hirano (yhiranoohnoyoudont@chromium.org)
  2. Mathieu Perreault (mathpohnoyoudont@chromium.org)
  3. pdfium-chromium-autoroll (pdfium-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com)
  4. Philip Jägenstedt (foolipohnoyoudont@chromium.org)
  5. Mike West (mkwstohnoyoudont@chromium.org)

Timing:

Create Thursday, 12-Jul-18 12:19:59 UTC
Start Thursday, 12-Jul-18 12:20:05 UTC
End Thursday, 12-Jul-18 13:02:34 UTC
Pending 5 secs
Execution 42 mins 29 secs

All Changes:

  1. [Fetch] Use wrapper tracing to express references between wrappers

    Changed by Yutaka Hirano - yhiranoohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 11:53:46 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 4a7c8ee607674efabc720016992893e2226ce3a0

    Comments

    [Fetch] Use wrapper tracing to express references between wrappers
    
    This is a reland of
    https://crrev.com/8057f58cb22579bc685efa50c579325ed047d51a which has
    been reverted due to a crash issue. The crash was caused by a missing
    TraceWrapperMember in blink::Response. This CL fixes that, and adds
    a layout test to verify that.
    
    Original CL: https://chromium-review.googlesource.com/c/1124270
    
    Bug: 862440
    Change-Id: I2842b957f223cfb32fee2aa9048b550010cb2e45
    Reviewed-on: https://chromium-review.googlesource.com/1134642
    Commit-Queue: Yutaka Hirano <yhirano@chromium.org>
    Reviewed-by: Kentaro Hara <haraken@chromium.org>
    Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
    Reviewed-by: Adam Rice <ricea@chromium.org>
    Reviewed-by: Matt Falkenhagen <falken@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574527}

    Changed files

    • third_party/WebKit/LayoutTests/http/tests/fetch/chromium/body-wrapper-tracing.html
    • third_party/blink/renderer/core/fetch/body_stream_buffer.cc
    • third_party/blink/renderer/core/fetch/body_stream_buffer.h
    • third_party/blink/renderer/core/fetch/fetch_request_data.cc
    • third_party/blink/renderer/core/fetch/fetch_request_data.h
    • third_party/blink/renderer/core/fetch/fetch_response_data.cc
    • third_party/blink/renderer/core/fetch/fetch_response_data.h
    • third_party/blink/renderer/core/fetch/request.cc
    • third_party/blink/renderer/core/fetch/request.h
    • third_party/blink/renderer/core/fetch/response.cc
    • third_party/blink/renderer/core/fetch/response.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/platform/bindings/v8_private_property.h
  2. [NTP] Adding mathp@ as owner of components/search

    Changed by Mathieu Perreault - mathpohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 11:31:03 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 1bc34891c3dbe8da1ac513111db7c7d2efd56f20

    Comments

    [NTP] Adding mathp@ as owner of components/search
    
    Few reasons for this:
    - My team and myself are now owners of the feature
    - Both current owners are gone through the end of July
    - I am owners of other parts of the NTP code (resources, data sources) as well
      as server-side code; as such, I think I'm qualified.
    
    Bug: None
    Change-Id: Ic1261dacba81a420500240c5a55bc547fb8b6c14
    Reviewed-on: https://chromium-review.googlesource.com/1133767
    Reviewed-by: Jochen Eisinger <jochen@chromium.org>
    Commit-Queue: Mathieu Perreault <mathp@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574526}

    Changed files

    • components/search/OWNERS
  3. Roll src/third_party/pdfium d8882193737a..68d04f284423 (1 commits)

    Changed by pdfium-chromium-autoroll - pdfium-chromium-autorollohnoyoudont@skia-buildbots.google.com.iam.gserviceaccount.com
    Changed at Thursday, 12-Jul-18 11:28:44 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision a1f9c10c3ca23a47aa60fe21a5b990c9d7cd589a

    Comments

    Roll src/third_party/pdfium d8882193737a..68d04f284423 (1 commits)
    
    https://pdfium.googlesource.com/pdfium.git/+log/d8882193737a..68d04f284423
    
    
    git log d8882193737a..68d04f284423 --date=short --no-merges --format='%ad %ae %s'
    2018-07-12 art-snake@yandex-team.ru Reland "Avoid duplicate data buffering in CPDF_SyntaxParser::ReadStream()."
    
    
    Created with:
      gclient setdep -r src/third_party/pdfium@68d04f284423
    
    The AutoRoll server is located here: https://pdfium-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.
    
    
    TBR=dsinclair@chromium.org
    
    Change-Id: I09ab0d06005fa49756095e583c16e039d0c8df3c
    Reviewed-on: https://chromium-review.googlesource.com/1134846
    Reviewed-by: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Commit-Queue: pdfium-chromium-autoroll <pdfium-chromium-autoroll@skia-buildbots.google.com.iam.gserviceaccount.com>
    Cr-Commit-Position: refs/heads/master@{#574525}

    Changed files

    • DEPS
  4. Mark textdecoder-fatal-single-byte.any.worker.html flaky to unblock WPT import

    Changed by Philip Jägenstedt - foolipohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 11:24:10 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 3dea5439837339b97168cdbbf6a45bfa27748b16

    Comments

    Mark textdecoder-fatal-single-byte.any.worker.html flaky to unblock WPT import
    
    TBR=robertma
    
    Bug: 862938
    Change-Id: I9ceda537526da02ebadc9799d58fb042d7f339d3
    No-Try: true
    Reviewed-on: https://chromium-review.googlesource.com/1134770
    Commit-Queue: Philip Jägenstedt <foolip@chromium.org>
    Reviewed-by: Philip Jägenstedt <foolip@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574524}

    Changed files

    • third_party/WebKit/LayoutTests/TestExpectations
  5. Revert "Make NetworkConnectionTracker survive network service crashes and restarts"

    Changed by Mike West - mkwstohnoyoudont@chromium.org
    Changed at Thursday, 12-Jul-18 11:11:00 UTC
    Repository https://chromium.googlesource.com/chromium/src
    Branch
    Revision 197e9188dc5be69cb34e1ebd67f86a1146439367

    Comments

    Revert "Make NetworkConnectionTracker survive network service crashes and restarts"
    
    This reverts commit cec0800c372ef670f59999fb3f028013d36b866c.
    
    Reason for revert:
    
    Findit (https://goo.gl/kROfz5) identified this CL at revision 574314 as the culprit
    for introducing flakiness in the tests as shown on:
    https://findit-for-me.appspot.com/waterfall/flake/flake-culprit?key=ag9zfmZpbmRpdC1mb3ItbWVyQwsSDEZsYWtlQ3VscHJpdCIxY2hyb21pdW0vY2VjMDgwMGMzNzJlZjY3MGY1OTk5OWZiM2YwMjgwMTNkMzZiODY2Yww
    
    TBR=xunjieli@chromium.org
    
    Bug: 862883
    
    Original change's description:
    > Make NetworkConnectionTracker survive network service crashes and restarts
    > 
    > This CL is to mirror tbansal@'s CL
    > (https://chromium-review.googlesource.com/1058528) to make
    > NetworkConnectionTracker survice network service's crashes and restarts.
    > 
    > Bug: 821009
    > Change-Id: Iec33797c0fd0ce12a8e5fdab9fda0a710eb9a528
    > Reviewed-on: https://chromium-review.googlesource.com/1120738
    > Commit-Queue: Helen Li <xunjieli@chromium.org>
    > Reviewed-by: John Abd-El-Malek <jam@chromium.org>
    > Reviewed-by: Tarun Bansal <tbansal@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#574314}
    
    TBR=jam@chromium.org,xunjieli@chromium.org,tbansal@chromium.org
    
    Change-Id: I2524f0e04441097e06585f4f2e4cdb948a419641
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: 821009
    Reviewed-on: https://chromium-review.googlesource.com/1135006
    Reviewed-by: Mike West <mkwst@chromium.org>
    Commit-Queue: Mike West <mkwst@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#574523}

    Changed files

    • chrome/browser/browser_process_impl.cc
    • chrome/browser/net/network_connection_tracker_browsertest.cc
    • chrome/browser/net/network_quality_tracker_browsertest.cc
    • content/public/browser/network_connection_tracker.cc
    • content/public/browser/network_connection_tracker.h
    • content/public/browser/network_connection_tracker_unittest.cc