third_party/WebKit/Source/core/timing/PerformanceResourceTiming.idl - Issue 2105713002: Render process changes for ResourceTiming sizes.

Side by Side Diff: third_party/WebKit/Source/core/timing/PerformanceResourceTiming.idl

Issue 2105713002: Render process changes for ResourceTiming sizes. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@resource_timing_sizes_browser_process

Patch Set: Fixes from kinuko initial comments Created 4 years, 5 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 unified diff | Download patch

« no previous file with comments | « third_party/WebKit/Source/core/timing/PerformanceResourceTiming.cpp ('k') | third_party/WebKit/Source/platform/exported/WebURLLoaderTestDelegate.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')

OLD	NEW
1 /*	1 /*

2 * Copyright (C) 2012 Google Inc. All rights reserved.	2 * Copyright (C) 2012 Google Inc. All rights reserved.

3 *	3 *

4 * Redistribution and use in source and binary forms, with or without	4 * Redistribution and use in source and binary forms, with or without

5 * modification, are permitted provided that the following conditions are	5 * modification, are permitted provided that the following conditions are

6 * met:	6 * met:

7 *	7 *

8 * * Redistributions of source code must retain the above copyright	8 * * Redistributions of source code must retain the above copyright

9 * notice, this list of conditions and the following disclaimer.	9 * notice, this list of conditions and the following disclaimer.

10 * * Redistributions in binary form must reproduce the above	10 * * Redistributions in binary form must reproduce the above

(...skipping 28 matching lines...) Expand all Loading...
39 readonly attribute DOMHighResTimeStamp redirectEnd;	39 readonly attribute DOMHighResTimeStamp redirectEnd;

40 readonly attribute DOMHighResTimeStamp fetchStart;	40 readonly attribute DOMHighResTimeStamp fetchStart;

41 readonly attribute DOMHighResTimeStamp domainLookupStart;	41 readonly attribute DOMHighResTimeStamp domainLookupStart;

42 readonly attribute DOMHighResTimeStamp domainLookupEnd;	42 readonly attribute DOMHighResTimeStamp domainLookupEnd;

43 readonly attribute DOMHighResTimeStamp connectStart;	43 readonly attribute DOMHighResTimeStamp connectStart;

44 readonly attribute DOMHighResTimeStamp connectEnd;	44 readonly attribute DOMHighResTimeStamp connectEnd;

45 readonly attribute DOMHighResTimeStamp secureConnectionStart;	45 readonly attribute DOMHighResTimeStamp secureConnectionStart;

46 readonly attribute DOMHighResTimeStamp requestStart;	46 readonly attribute DOMHighResTimeStamp requestStart;

47 readonly attribute DOMHighResTimeStamp responseStart;	47 readonly attribute DOMHighResTimeStamp responseStart;

48 readonly attribute DOMHighResTimeStamp responseEnd;	48 readonly attribute DOMHighResTimeStamp responseEnd;

49 // TODO(foolip): The spec has tranferSize/encodedBodySize/decodedBodySize	49 readonly attribute unsigned short transferSize;
	Kunihiko Sakamoto 2016/06/30 10:19:49 The spec says these fields are unsigned long long. The spec says these fields are unsigned long long. Adam Rice 2016/06/30 10:29:58 Wow. I don't know how I managed to get that wrong. Show quoted text On 2016/06/30 10:19:49, Kunihiko Sakamoto wrote: > The spec says these fields are unsigned long long. Wow. I don't know how I managed to get that wrong. Thanks for catching it. Fixed.
50 // and a serializer:	50 readonly attribute unsigned short encodedBodySize;

51 // readonly attribute unsigned short transferSize;	51 readonly attribute unsigned short decodedBodySize;

52 // readonly attribute unsigned short encodedBodySize;	52 // TODO(foolip): The spec has a serializer:

53 // readonly attribute unsigned short decodedBodySize;

54 // serializer = {inherit, attribute};	53 // serializer = {inherit, attribute};

55 };	54 };

OLD	NEW