Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(149)

Unified Diff: third_party/WebKit/Source/platform/network/ResourceTimingInfo.h

Issue 2429063002: Implement nextHopProtocol in PerformanceResourceTiming and PerformanceNavigationTiming.
Patch Set: Created 4 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/platform/network/ResourceTimingInfo.h
diff --git a/third_party/WebKit/Source/platform/network/ResourceTimingInfo.h b/third_party/WebKit/Source/platform/network/ResourceTimingInfo.h
index 789cb5daa2829951674c1b832fe316a5b8f16088..3ac479b241ea6ccc530121c888fd7bdf90e11742 100644
--- a/third_party/WebKit/Source/platform/network/ResourceTimingInfo.h
+++ b/third_party/WebKit/Source/platform/network/ResourceTimingInfo.h
@@ -67,6 +67,11 @@ class PLATFORM_EXPORT ResourceTimingInfo {
void setInitiatorType(const AtomicString& type) { m_type = type; }
const AtomicString& initiatorType() const { return m_type; }
+ /*void setNextHopProtocol(const AtomicString& nextHopProtocol) {
panicker 2016/10/19 00:25:11 why is this commented out? Remove if not needed
sunjian 2016/10/21 00:19:50 Done.
+ m_nextHopProtocol = nextHopProtocol;
+ }*/
+ const AtomicString& nextHopProtocol() const { return m_nextHopProtocol; }
+
void setOriginalTimingAllowOrigin(
const AtomicString& originalTimingAllowOrigin) {
m_originalTimingAllowOrigin = originalTimingAllowOrigin;
@@ -83,6 +88,7 @@ class PLATFORM_EXPORT ResourceTimingInfo {
void setFinalResponse(const ResourceResponse& response) {
m_finalResponse = response;
+ m_nextHopProtocol = response.alpnNegotiatedProtocol();
}
const ResourceResponse& finalResponse() const { return m_finalResponse; }
@@ -107,12 +113,14 @@ class PLATFORM_EXPORT ResourceTimingInfo {
const double time,
bool isMainResource)
: m_type(type),
+ m_nextHopProtocol(""),
m_initialTime(time),
m_transferSize(0),
m_isMainResource(isMainResource),
m_hasCrossOriginRedirect(false) {}
AtomicString m_type;
+ AtomicString m_nextHopProtocol;
AtomicString m_originalTimingAllowOrigin;
double m_initialTime;
double m_loadFinishTime;
@@ -132,6 +140,7 @@ struct CrossThreadResourceTimingInfoData {
CrossThreadResourceTimingInfoData() {}
String m_type;
+ String m_nextHopProtocol;
String m_originalTimingAllowOrigin;
double m_initialTime;
double m_loadFinishTime;

Powered by Google App Engine
This is Rietveld 408576698