ResourceTiming should not depend on Document

Source/core/timing/PerformanceBase.h

 /*
  * 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 28 matching lines @@
 #include "wtf/RefCounted.h"
 #include "wtf/RefPtr.h"
 #include "wtf/text/WTFString.h"
 
 namespace blink {
 
 class Document;
 class ExceptionState;
 class PerformanceTiming;
 class ResourceTimingInfo;
+class SecurityOrigin;
 class UserTiming;
 
 using PerformanceEntryVector = HeapVector<Member<PerformanceEntry>>;
 
 class CORE_EXPORT PerformanceBase : public RefCountedGarbageCollectedEventTargetWithInlineData<PerformanceBase> {
     REFCOUNTED_GARBAGE_COLLECTED_EVENT_TARGET(PerformanceBase);
 public:
     virtual ~PerformanceBase();
 
     virtual const AtomicString& interfaceName() const override;
@@ skipping 10 matching lines @@
     void webkitClearResourceTimings();
     void webkitSetResourceTimingBufferSize(unsigned);
 
     DEFINE_ATTRIBUTE_EVENT_LISTENER(webkitresourcetimingbufferfull);
 
     void clearFrameTimings();
     void setFrameTimingBufferSize(unsigned);
 
     DEFINE_ATTRIBUTE_EVENT_LISTENER(frametimingbufferfull);
 
-    void addResourceTiming(const ResourceTimingInfo&, Document*);
+    void addResourceTiming(const ResourceTimingInfo&, const SecurityOrigin*);

[kinuko, 2015/06/12 05:44:54]

nit: if we don't expect this could be null consider making this const ref

[Kunihiko Sakamoto, 2015/06/12 06:03:21]

Done.

 
     void addRenderTiming(Document*, unsigned, double, double);
 
     void addCompositeTiming(Document*, unsigned, double);
 
     void mark(const String& markName, ExceptionState&);
     void clearMarks(const String& markName);
 
     void measure(const String& measureName, const String& startMark, const String& endMark, ExceptionState&);
     void clearMeasures(const String& measureName);
@@ skipping 13 matching lines @@
     PerformanceEntryVector m_resourceTimingBuffer;
     unsigned m_resourceTimingBufferSize;
     double m_timeOrigin;
 
     Member<UserTiming> m_userTiming;
 };
 
 } // namespace blink
 
 #endif // PerformanceBase_h