Sync core/timing/ interfaces with their specs, or lack thereof

Source/core/timing/Performance.idl

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2012 Intel Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 11 matching lines @@
  * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
  * OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
  * SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
  * LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
  * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
  * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
-// See: http://www.w3.org/TR/navigation-timing/
+// https://w3c.github.io/hr-time/#the-performance-interface
+
+// TODO(philipj): This interface should be [Exposed=(Window,Worker)]. Doing that
+// would allow the WorkerPerformance interface to be merged into this.
+// TODO(philipj): None of the optional DOMString arguments in this interface
+// should have a default value.
 interface Performance : EventTarget {
-    readonly attribute PerformanceNavigation navigation;
-    readonly attribute PerformanceTiming timing;
-    readonly attribute MemoryInfo memory;
+    DOMHighResTimeStamp now();
 
-    [MeasureAs=UnprefixedPerformanceTimeline] sequence<PerformanceEntry> getEntries();
-    [MeasureAs=UnprefixedPerformanceTimeline] sequence<PerformanceEntry> getEntriesByType(DOMString entryType);
-    [MeasureAs=UnprefixedPerformanceTimeline] sequence<PerformanceEntry> getEntriesByName(DOMString name, optional DOMString entryType = null);
+    // Performance Timeline
+    // https://w3c.github.io/performance-timeline/#the-performance-interface
+    // TODO(philipj): getEntries() should take an optional FilterOptions argument.
+    [MeasureAs=UnprefixedPerformanceTimeline] PerformanceEntryList getEntries();
+    [MeasureAs=UnprefixedPerformanceTimeline] PerformanceEntryList getEntriesByType(DOMString entryType);
+    [MeasureAs=UnprefixedPerformanceTimeline] PerformanceEntryList getEntriesByName(DOMString name, optional DOMString entryType = null);
 
+    // Resource Timing
+    // https://w3c.github.io/resource-timing/#extensions-performance-interface
+    // TODO(philipj): Unprefix thes APIs.

[Kunihiko Sakamoto, 2015/07/10 06:37:55]

s/thes/these/

[philipj_slow, 2015/07/10 08:18:14]

Done.

     [MeasureAs=PrefixedPerformanceClearResourceTimings] void webkitClearResourceTimings();
     [MeasureAs=PrefixedPerformanceSetResourceTimingBufferSize] void webkitSetResourceTimingBufferSize(unsigned long maxSize);
-
-    [MeasureAs=PerformanceFrameTiming,RuntimeEnabled=FrameTimingSupport] void clearFrameTimings();
-    [MeasureAs=PerformanceFrameTiming,RuntimeEnabled=FrameTimingSupport] void setFrameTimingBufferSize(unsigned long maxSize);
-
     attribute EventHandler onwebkitresourcetimingbufferfull;
 
+    // Navigation Timing
+    // http://www.w3.org/TR/navigation-timing/#sec-window.performance-attribute
+    // TODO(philipj): The Navigation Timing Rec. defines the (not partial)
+    // Performance interface, while https://w3c.github.io/navigation-timing/ has
+    // changed the interface in an incompatible way, so there is no current spec
+    // for this:

[Kunihiko Sakamoto, 2015/07/10 06:37:55]

IIUC https://w3c.github.io/navigation-timing/ is called "Navigation Timing 2"
that is a separate spec from (and intended to eventually replace)
Navigation Timing (http://www.w3.org/TR/navigation-timing/).

So I don't think there is no "current spec" for them.

(igrigorik@ correct me if I'm wrong)

[philipj_slow, 2015/07/10 08:18:14]

I've removed the "there is no spec" bit and just link to
https://github.com/w3c/navigation-timing/issues/22

+    [Measure] readonly attribute PerformanceTiming timing;
+    [Measure] readonly attribute PerformanceNavigation navigation;
+
+    // User Timing
+    // https://w3c.github.io/user-timing/#extensions-performance-interface
+    [MeasureAs=UnprefixedUserTiming, RaisesException] void mark(DOMString markName);
+    [MeasureAs=UnprefixedUserTiming] void clearMarks(optional DOMString markName = null);
+
+    [MeasureAs=UnprefixedUserTiming, RaisesException] void measure(DOMString measureName, optional DOMString startMark = null, optional DOMString endMark = null);
+    [MeasureAs=UnprefixedUserTiming] void clearMeasures(optional DOMString measureName = null);
+
+    // Frame Timing
+    // https://w3c.github.io/frame-timing/#extensions-performance-interface
+    [RuntimeEnabled=FrameTimingSupport, MeasureAs=PerformanceFrameTiming] void clearFrameTimings();
+    [RuntimeEnabled=FrameTimingSupport, MeasureAs=PerformanceFrameTiming] void setFrameTimingBufferSize(unsigned long maxSize);
     [RuntimeEnabled=FrameTimingSupport] attribute EventHandler onframetimingbufferfull;
 
-    // See http://www.w3.org/TR/2012/CR-user-timing-20120726/
-    [RaisesException,MeasureAs=UnprefixedUserTiming] void mark(DOMString markName);
-    [MeasureAs=UnprefixedUserTiming] void clearMarks(optional DOMString markName = null);
-
-    [RaisesException,MeasureAs=UnprefixedUserTiming] void measure(DOMString measureName, optional DOMString startMark = null, optional DOMString endMark = null);
-    [MeasureAs=UnprefixedUserTiming] void clearMeasures(optional DOMString measureName = null);
-
-    // See http://www.w3.org/TR/hr-time/ for details.
-    double now();
+    // TODO(philipj): There is no spec for the Memory Info API, see blink-dev:
+    // https://groups.google.com/a/chromium.org/d/msg/blink-dev/g5YRCGpC9vs/b4OJz71NmPwJ
+    [Measure] readonly attribute MemoryInfo memory;
 };