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

Side by Side Diff: content/public/browser/resource_request_info.h

Issue 1413543005: Use FrameTreeNode ID as frameId in extension APIs (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address comments, add WebNavigationApiTest.CrossProcessIframe test Created 5 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_
6 #define CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_ 6 #define CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_
7 7
8 #include "base/basictypes.h" 8 #include "base/basictypes.h"
9 #include "content/common/content_export.h" 9 #include "content/common/content_export.h"
10 #include "content/public/common/resource_type.h" 10 #include "content/public/common/resource_type.h"
(...skipping 58 matching lines...) Expand 10 before | Expand all | Expand 10 after
69 69
70 // The IPC route identifier of the RenderFrame. 70 // The IPC route identifier of the RenderFrame.
71 // TODO(jam): once all navigation and resource requests are sent between 71 // TODO(jam): once all navigation and resource requests are sent between
72 // frames and RenderView/RenderViewHost aren't involved we can remove this and 72 // frames and RenderView/RenderViewHost aren't involved we can remove this and
73 // just use GetRouteID above. 73 // just use GetRouteID above.
74 virtual int GetRenderFrameID() const = 0; 74 virtual int GetRenderFrameID() const = 0;
75 75
76 // True if GetRenderFrameID() represents a main frame in the RenderView. 76 // True if GetRenderFrameID() represents a main frame in the RenderView.
77 virtual bool IsMainFrame() const = 0; 77 virtual bool IsMainFrame() const = 0;
78 78
79 // True if GetParentRenderFrameID() represents a main frame in the RenderView. 79 // True if parent frame of frame represents a main frame in the RenderView.
Charlie Reis 2015/11/03 00:23:49 nit: True if the parent frame of the frame that se
robwu 2015/11/03 10:03:02 Done (using "the frame's parent" to fit the commen
80 virtual bool ParentIsMainFrame() const = 0; 80 virtual bool ParentIsMainFrame() const = 0;
81 81
82 // Routing ID of parent frame of frame that sent this resource request.
83 // -1 if unknown / invalid.
84 virtual int GetParentRenderFrameID() const = 0;
85
86 // Returns the associated resource type. 82 // Returns the associated resource type.
87 virtual ResourceType GetResourceType() const = 0; 83 virtual ResourceType GetResourceType() const = 0;
88 84
89 // Returns the process type that initiated this request. 85 // Returns the process type that initiated this request.
90 virtual int GetProcessType() const = 0; 86 virtual int GetProcessType() const = 0;
91 87
92 // Returns the associated referrer policy. 88 // Returns the associated referrer policy.
93 virtual blink::WebReferrerPolicy GetReferrerPolicy() const = 0; 89 virtual blink::WebReferrerPolicy GetReferrerPolicy() const = 0;
94 90
95 // Returns the associated visibility state at the time the request was started 91 // Returns the associated visibility state at the time the request was started
(...skipping 28 matching lines...) Expand all
124 // Whether this is a download. 120 // Whether this is a download.
125 virtual bool IsDownload() const = 0; 121 virtual bool IsDownload() const = 0;
126 122
127 protected: 123 protected:
128 virtual ~ResourceRequestInfo() {} 124 virtual ~ResourceRequestInfo() {}
129 }; 125 };
130 126
131 } // namespace content 127 } // namespace content
132 128
133 #endif // CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_ 129 #endif // CONTENT_PUBLIC_BROWSER_RESOURCE_REQUEST_INFO_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698