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

Unified Diff: content/browser/frame_host/render_frame_host_impl.h

Issue 2146803004: Create a content::FrameOwnerProperties struct for IPC transport of WebFrameOwnerProperties (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Transport Created 4 years, 5 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: content/browser/frame_host/render_frame_host_impl.h
diff --git a/content/browser/frame_host/render_frame_host_impl.h b/content/browser/frame_host/render_frame_host_impl.h
index fe6afddd09d44dde202d18b06ab1d68ebf5ac1f2..68daf9833cc79d4c7ce719034065fb3b6fe390a8 100644
--- a/content/browser/frame_host/render_frame_host_impl.h
+++ b/content/browser/frame_host/render_frame_host_impl.h
@@ -39,7 +39,6 @@
#include "net/http/http_response_headers.h"
#include "services/shell/public/cpp/interface_registry.h"
#include "third_party/WebKit/public/platform/WebInsecureRequestPolicy.h"
-#include "third_party/WebKit/public/web/WebFrameOwnerProperties.h"
#include "third_party/WebKit/public/web/WebTextDirection.h"
#include "third_party/WebKit/public/web/WebTreeScopeType.h"
#include "ui/accessibility/ax_node_data.h"
@@ -66,6 +65,10 @@ class ListValue;
}
namespace blink {
+struct WebFrameOwnerProperties;
nasko 2016/07/18 21:59:15 Why do we still need this? The browser side should
raymes 2016/07/19 00:22:00 It's still used right now in OnCreateChildFrame bu
+}
+
+namespace blink {
namespace mojom {
class WebBluetoothService;
}
@@ -95,6 +98,7 @@ class WebBluetoothServiceImpl;
struct ContentSecurityPolicyHeader;
struct ContextMenuParams;
struct FileChooserParams;
+struct FrameOwnerProperties;
struct GlobalRequestID;
struct FileChooserParams;
struct Referrer;
@@ -680,9 +684,8 @@ class CONTENT_EXPORT RenderFrameHostImpl
void OnDidAssignPageId(int32_t page_id);
void OnDidChangeSandboxFlags(int32_t frame_routing_id,
blink::WebSandboxFlags flags);
- void OnDidChangeFrameOwnerProperties(
- int32_t frame_routing_id,
- const blink::WebFrameOwnerProperties& properties);
+ void OnDidChangeFrameOwnerProperties(int32_t frame_routing_id,
+ const FrameOwnerProperties& properties);
void OnUpdateTitle(const base::string16& title,
blink::WebTextDirection title_direction);
void OnUpdateEncoding(const std::string& encoding);

Powered by Google App Engine
This is Rietveld 408576698