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

Unified Diff: third_party/WebKit/Source/core/dom/Document.h

Issue 2516473002: Cross-origin iframes: collect data under hypothetical loading strategies (Closed)
Patch Set: Created 4 years, 1 month 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/dom/Document.h
diff --git a/third_party/WebKit/Source/core/dom/Document.h b/third_party/WebKit/Source/core/dom/Document.h
index fb3246d9d7b5a0fca90587ecaca1ba190b6c5148..358828949ca45847e9bcb1856d6beca9f6d2fc1e 100644
--- a/third_party/WebKit/Source/core/dom/Document.h
+++ b/third_party/WebKit/Source/core/dom/Document.h
@@ -237,20 +237,16 @@ enum CreateElementFlags {
// See https://crbug.com/635105.
// Logged to UMA, don't re-arrange entries without creating a new histogram.
enum WouldLoadReason {
+ Invalid,
Created,
+ WouldLoad3ScreensAway,
+ WouldLoad2ScreensAway,
+ WouldLoad1ScreenAway,
+ WouldLoadVisible,
// If outer and inner frames aren't in the same process we can't determine
// if the inner frame is visible, so just load it.
// TODO(dgrogan): Revisit after https://crbug.com/650433 is fixed.
- WouldLoadOutOfProcess,
- // The next five indicate frames that are probably used for cross-origin
- // communication.
- WouldLoadDisplayNone,
- WouldLoadZeroByZero,
- WouldLoadAboveAndLeft,
- WouldLoadAbove,
- WouldLoadLeft,
- // We have to load documents in visible frames.
- WouldLoadVisible,
+ WouldLoadNoParent,
WouldLoadReasonEnd
};
@@ -1298,8 +1294,8 @@ class CORE_EXPORT Document : public ContainerNode,
bool isInMainFrame() const;
- void maybeRecordLoadReason(WouldLoadReason);
- WouldLoadReason wouldLoadReason() { return m_wouldLoadReason; }
+ void recordDeferredLoadReason(WouldLoadReason);
+ WouldLoadReason deferredLoadReason() { return m_wouldLoadReason; }
PropertyRegistry* propertyRegistry();
« no previous file with comments | « no previous file | third_party/WebKit/Source/core/dom/Document.cpp » ('j') | third_party/WebKit/Source/core/dom/Document.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698