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

Side by Side Diff: third_party/WebKit/Source/core/timing/Performance.h

Issue 2539323002: PerformanceMonitor: do not maintain a vector of frames in the monitor, single ExecutionContext memb… (Closed)
Patch Set: off-by-one in depth fixed Created 4 years 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2010 Google Inc. All rights reserved. 2 * Copyright (C) 2010 Google Inc. All rights reserved.
3 * Copyright (C) 2012 Intel Inc. All rights reserved. 3 * Copyright (C) 2012 Intel Inc. All rights reserved.
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions are 6 * modification, are permitted provided that the following conditions are
7 * met: 7 * met:
8 * 8 *
9 * * Redistributions of source code must retain the above copyright 9 * * Redistributions of source code must retain the above copyright
10 * notice, this list of conditions and the following disclaimer. 10 * notice, this list of conditions and the following disclaimer.
(...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after
70 70
71 DECLARE_VIRTUAL_TRACE(); 71 DECLARE_VIRTUAL_TRACE();
72 72
73 private: 73 private:
74 explicit Performance(LocalFrame*); 74 explicit Performance(LocalFrame*);
75 75
76 // DOMWindowProperty overrides. 76 // DOMWindowProperty overrides.
77 void frameDestroyed() override; 77 void frameDestroyed() override;
78 78
79 static std::pair<String, DOMWindow*> sanitizedAttribution( 79 static std::pair<String, DOMWindow*> sanitizedAttribution(
80 const HeapHashSet<Member<Frame>>& frames, 80 ExecutionContext*,
81 bool hasMultipleContexts,
81 Frame* observerFrame); 82 Frame* observerFrame);
82 83
83 // PerformanceMonitor::Client implementation. 84 // PerformanceMonitor::Client implementation.
84 void reportLongTask(double startTime, 85 void reportLongTask(double startTime,
85 double endTime, 86 double endTime,
86 const HeapHashSet<Member<Frame>>& contextFrames) override; 87 ExecutionContext* taskContext,
88 bool hasMultipleContexts) override;
87 89
88 mutable Member<PerformanceNavigation> m_navigation; 90 mutable Member<PerformanceNavigation> m_navigation;
89 mutable Member<PerformanceTiming> m_timing; 91 mutable Member<PerformanceTiming> m_timing;
90 }; 92 };
91 93
92 } // namespace blink 94 } // namespace blink
93 95
94 #endif // Performance_h 96 #endif // Performance_h
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/frame/PerformanceMonitorTest.cpp ('k') | third_party/WebKit/Source/core/timing/Performance.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698