New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 663175 link

Starred by 4 users

Issue metadata

Status: Fixed
Owner:
Closed: Aug 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 3
Type: Launch-OWP
Launch-Accessibility: ----
Launch-Exp-Leadership: ----
Launch-Leadership: ----
Launch-Legal: ----
Launch-M-Approved: ----
Launch-M-Target: ----
Launch-Privacy: ----
Launch-Security: ----
Launch-Test: ----
Launch-UI: ----
Rollout-Type: ----



Sign in to add a comment

Implement and ship nextHopProtocol in Resource Timing

Project Member Reported by panicker@google.com, Nov 8 2016

Issue description

Change description:
nextHopProtocol returns the network protocol used to fetch the resource (as identified by the ALPN Protocol ID (RFC).

Changes to API surface:
adds field to Resource Timing: nextHopProtocol

Links:
https://w3c.github.io/resource-timing/#dom-performanceresourcetiming-nexthopprotocol



 
Cc: y...@yoav.ws
Labels: -M-56
Owner: ----
Status: Available (was: Assigned)
Owner: shaseley@google.com
Status: Assigned (was: Available)
Project Member

Comment 6 by bugdroid1@chromium.org, Jun 24 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/0d7335cd221d21f07fbc8dce72e437167e08ca7e

commit 0d7335cd221d21f07fbc8dce72e437167e08ca7e
Author: shaseley <shaseley@google.com>
Date: Sat Jun 24 00:27:44 2017

Implement nextHopProtocol in PerformanceResourceTiming and PerformanceNavigationTiming.

BUG= 663175 

Review-Url: https://codereview.chromium.org/2932003002
Cr-Commit-Position: refs/heads/master@{#482091}

[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/content/child/web_url_loader_impl.cc
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/content/child/weburlresponse_extradata_impl.cc
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/content/child/weburlresponse_extradata_impl.h
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/content/renderer/render_frame_impl.cc
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_attributes_exist.html
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/external/wpt/navigation-timing/nav2_test_attributes_values.html
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/external/wpt/resource-timing/test_resource_timing-expected.txt
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/platform/mac/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/platform/win/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/virtual/service-worker-navigation-preload-disabled/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/bindings/core/v8/BUILD.gn
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/BUILD.gn
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.cpp
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceNavigationTiming.h
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.cpp
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.h
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceResourceTiming.idl
[add] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/core/timing/PerformanceResourceTimingTest.cpp
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/modules/exported/BUILD.gn
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/platform/exported/WebURLResponse.cpp
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/platform/loader/fetch/ResourceResponse.cpp
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/Source/platform/loader/fetch/ResourceResponse.h
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/public/BUILD.gn
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/public/platform/DEPS
[modify] https://crrev.com/0d7335cd221d21f07fbc8dce72e437167e08ca7e/third_party/WebKit/public/platform/WebURLResponse.h

If this is shipping in M61, please create a chromestatus.com entry for it.

Comment 9 by shaseley@google.com, Aug 30 2017

Status: Fixed (was: Assigned)
Project Member

Comment 10 by bugdroid1@chromium.org, Dec 13 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5830b993a3529bf00e2149afc0c0b3d268d78d36

commit 5830b993a3529bf00e2149afc0c0b3d268d78d36
Author: Hiroshige Hayashizaki <hiroshige@chromium.org>
Date: Wed Dec 13 21:21:38 2017

Always initialize ResourceResponse::connection_info_

In a ResourceResponse constructor, connection_info_ is not initialized.
This CL initialize it with the same value as the default constructor.

Bug: 794651,  663175 
Change-Id: I895ed2e7e436ed5a2390da1f54e70ce3b7af26b3
Reviewed-on: https://chromium-review.googlesource.com/825249
Commit-Queue: Hiroshige Hayashizaki <hiroshige@chromium.org>
Reviewed-by: Nate Chapin <japhet@chromium.org>
Cr-Commit-Position: refs/heads/master@{#523880}
[modify] https://crrev.com/5830b993a3529bf00e2149afc0c0b3d268d78d36/third_party/WebKit/Source/platform/loader/fetch/ResourceResponse.cpp

Sign in to add a comment