Index: Source/core/loader/ProgressTracker.h |
diff --git a/Source/core/loader/ProgressTracker.h b/Source/core/loader/ProgressTracker.h |
index 5e7188fa8885bec9cfbf4e0338c398988a41aa73..d3dcd2cfffaed91dba26f59f8b0ee6510dd4c847 100644 |
--- a/Source/core/loader/ProgressTracker.h |
+++ b/Source/core/loader/ProgressTracker.h |
@@ -66,6 +66,12 @@ private: |
void reset(); |
+ // This bare frame pointer is safe; it refers to the |
Mads Ager (chromium)
2014/09/19 06:37:24
Add FIXME: Move ProgressTracker to the oilpan heap
sof
2014/09/19 11:56:43
FIXME: Oilpan: added.
|
+ // owning frame (via FrameLoader.) |
+ // |
+ // ProgressTracker is finalized by FrameLoader (which in turn is |
+ // finalized by LocalFrame.) Its lifetime thus guarantees that |
+ // m_frame cannot be used to access a dead LocalFrame. |
LocalFrame* m_frame; |
bool m_inProgress; |
long long m_totalPageAndResourceBytesToLoad; |