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

Side by Side Diff: content/renderer/render_frame_impl.h

Issue 2821473002: Service CreateNewWindow on the UI thread with a new mojo interface (Closed)
Patch Set: dcheng fixes + security exploit browsertest nerfing Created 3 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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_RENDERER_RENDER_FRAME_IMPL_H_ 5 #ifndef CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 6 #define CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
7 7
8 #include <stddef.h> 8 #include <stddef.h>
9 #include <stdint.h> 9 #include <stdint.h>
10 10
(...skipping 303 matching lines...) Expand 10 before | Expand all | Expand 10 after
314 314
315 // The focused node changed to |node|. If focus was lost from this frame, 315 // The focused node changed to |node|. If focus was lost from this frame,
316 // |node| will be null. 316 // |node| will be null.
317 void FocusedNodeChanged(const blink::WebNode& node); 317 void FocusedNodeChanged(const blink::WebNode& node);
318 318
319 // TODO(dmazzoni): the only reason this is here is to plumb it through to 319 // TODO(dmazzoni): the only reason this is here is to plumb it through to
320 // RenderAccessibilityImpl. It should use the RenderFrameObserver method, once 320 // RenderAccessibilityImpl. It should use the RenderFrameObserver method, once
321 // blink has a separate accessibility tree per frame. 321 // blink has a separate accessibility tree per frame.
322 void FocusedNodeChangedForAccessibility(const blink::WebNode& node); 322 void FocusedNodeChangedForAccessibility(const blink::WebNode& node);
323 323
324 // Virtual to allow for overriding in TestRenderFrame.
325 virtual mojom::FrameHostIPC* GetFrameHostIPC();
326
324 // A RenderView opened by this RenderFrame needs to be shown. 327 // A RenderView opened by this RenderFrame needs to be shown.
325 void ShowCreatedWindow(bool opened_by_user_gesture, 328 void ShowCreatedWindow(bool opened_by_user_gesture,
326 RenderWidget* render_widget_to_show, 329 RenderWidget* render_widget_to_show,
327 blink::WebNavigationPolicy policy, 330 blink::WebNavigationPolicy policy,
328 const gfx::Rect& initial_rect); 331 const gfx::Rect& initial_rect);
329 332
330 // Called when this frame's widget is focused. 333 // Called when this frame's widget is focused.
331 void RenderWidgetSetFocus(bool enable); 334 void RenderWidgetSetFocus(bool enable);
332 335
333 // Called when the widget receives a mouse event. 336 // Called when the widget receives a mouse event.
(...skipping 1043 matching lines...) Expand 10 before | Expand all | Expand 10 after
1377 1380
1378 HostZoomLevels host_zoom_levels_; 1381 HostZoomLevels host_zoom_levels_;
1379 EngagementOriginAndLevel engagement_level_; 1382 EngagementOriginAndLevel engagement_level_;
1380 1383
1381 mojo::AssociatedBinding<blink::mojom::EngagementClient> engagement_binding_; 1384 mojo::AssociatedBinding<blink::mojom::EngagementClient> engagement_binding_;
1382 mojo::Binding<mojom::Frame> frame_binding_; 1385 mojo::Binding<mojom::Frame> frame_binding_;
1383 mojo::AssociatedBinding<mojom::HostZoom> host_zoom_binding_; 1386 mojo::AssociatedBinding<mojom::HostZoom> host_zoom_binding_;
1384 mojo::AssociatedBinding<mojom::FrameBindingsControl> 1387 mojo::AssociatedBinding<mojom::FrameBindingsControl>
1385 frame_bindings_control_binding_; 1388 frame_bindings_control_binding_;
1386 mojom::FrameHostPtr frame_host_; 1389 mojom::FrameHostPtr frame_host_;
1390 mojom::FrameHostIPCAssociatedPtr frame_host_ipc_;
1387 1391
1388 // Indicates whether |didAccessInitialDocument| was called. 1392 // Indicates whether |didAccessInitialDocument| was called.
1389 bool has_accessed_initial_document_; 1393 bool has_accessed_initial_document_;
1390 1394
1391 AssociatedInterfaceRegistryImpl associated_interfaces_; 1395 AssociatedInterfaceRegistryImpl associated_interfaces_;
1392 std::unique_ptr<AssociatedInterfaceProviderImpl> 1396 std::unique_ptr<AssociatedInterfaceProviderImpl>
1393 remote_associated_interfaces_; 1397 remote_associated_interfaces_;
1394 1398
1395 // TODO(dcheng): Remove these members. 1399 // TODO(dcheng): Remove these members.
1396 bool committed_first_load_ = false; 1400 bool committed_first_load_ = false;
(...skipping 29 matching lines...) Expand all
1426 std::unique_ptr<PendingNavigationInfo> pending_navigation_info_; 1430 std::unique_ptr<PendingNavigationInfo> pending_navigation_info_;
1427 1431
1428 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1432 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1429 1433
1430 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1434 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1431 }; 1435 };
1432 1436
1433 } // namespace content 1437 } // namespace content
1434 1438
1435 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1439 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698