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

Unified Diff: extensions/browser/api/web_request/web_request_event_details.cc

Issue 2335133003: PlzNavigate: support the WebRequest API (Closed)
Patch Set: PlzNavigate: support the WebRequest API Created 4 years, 3 months 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: extensions/browser/api/web_request/web_request_event_details.cc
diff --git a/extensions/browser/api/web_request/web_request_event_details.cc b/extensions/browser/api/web_request/web_request_event_details.cc
index 56ec81f3944b232e6867ccef27099cc28966479b..a4c601b6ef878d10d8112c167404d112d7db56f4 100644
--- a/extensions/browser/api/web_request/web_request_event_details.cc
+++ b/extensions/browser/api/web_request/web_request_event_details.cc
@@ -154,16 +154,20 @@ void WebRequestEventDetails::SetResponseSource(const net::URLRequest* request) {
dict_.SetString(keys::kIpKey, response_ip);
}
+void WebRequestEventDetails::SetFrameData(
+ const ExtensionApiFrameIdMap::FrameData& frame_data) {
+ dict_.SetInteger(keys::kTabIdKey, frame_data.tab_id);
+ dict_.SetInteger(keys::kFrameIdKey, frame_data.frame_id);
+ dict_.SetInteger(keys::kParentFrameIdKey, frame_data.parent_frame_id);
+}
+
void WebRequestEventDetails::DetermineFrameDataOnUI() {
DCHECK_CURRENTLY_ON(content::BrowserThread::UI);
content::RenderFrameHost* rfh =
content::RenderFrameHost::FromID(render_process_id_, render_frame_id_);
ExtensionApiFrameIdMap::FrameData frame_data =
ExtensionApiFrameIdMap::Get()->GetFrameData(rfh);
-
- dict_.SetInteger(keys::kTabIdKey, frame_data.tab_id);
- dict_.SetInteger(keys::kFrameIdKey, frame_data.frame_id);
- dict_.SetInteger(keys::kParentFrameIdKey, frame_data.parent_frame_id);
+ SetFrameData(frame_data);
}
void WebRequestEventDetails::DetermineFrameDataOnIO(

Powered by Google App Engine
This is Rietveld 408576698