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

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: Move over to render frame (but not WebFrameClient yet) + a bunch of cleanups 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 1038 matching lines...) Expand 10 before | Expand all | Expand 10 after
1372 1375
1373 HostZoomLevels host_zoom_levels_; 1376 HostZoomLevels host_zoom_levels_;
1374 EngagementOriginAndLevel engagement_level_; 1377 EngagementOriginAndLevel engagement_level_;
1375 1378
1376 mojo::AssociatedBinding<blink::mojom::EngagementClient> engagement_binding_; 1379 mojo::AssociatedBinding<blink::mojom::EngagementClient> engagement_binding_;
1377 mojo::Binding<mojom::Frame> frame_binding_; 1380 mojo::Binding<mojom::Frame> frame_binding_;
1378 mojo::AssociatedBinding<mojom::HostZoom> host_zoom_binding_; 1381 mojo::AssociatedBinding<mojom::HostZoom> host_zoom_binding_;
1379 mojo::AssociatedBinding<mojom::FrameBindingsControl> 1382 mojo::AssociatedBinding<mojom::FrameBindingsControl>
1380 frame_bindings_control_binding_; 1383 frame_bindings_control_binding_;
1381 mojom::FrameHostPtr frame_host_; 1384 mojom::FrameHostPtr frame_host_;
1385 mojom::FrameHostIPCAssociatedPtr frame_host_ipc_;
1382 1386
1383 // Indicates whether |didAccessInitialDocument| was called. 1387 // Indicates whether |didAccessInitialDocument| was called.
1384 bool has_accessed_initial_document_; 1388 bool has_accessed_initial_document_;
1385 1389
1386 AssociatedInterfaceRegistryImpl associated_interfaces_; 1390 AssociatedInterfaceRegistryImpl associated_interfaces_;
1387 std::unique_ptr<AssociatedInterfaceProviderImpl> 1391 std::unique_ptr<AssociatedInterfaceProviderImpl>
1388 remote_associated_interfaces_; 1392 remote_associated_interfaces_;
1389 1393
1390 // TODO(dcheng): Remove these members. 1394 // TODO(dcheng): Remove these members.
1391 bool committed_first_load_ = false; 1395 bool committed_first_load_ = false;
(...skipping 29 matching lines...) Expand all
1421 std::unique_ptr<PendingNavigationInfo> pending_navigation_info_; 1425 std::unique_ptr<PendingNavigationInfo> pending_navigation_info_;
1422 1426
1423 base::WeakPtrFactory<RenderFrameImpl> weak_factory_; 1427 base::WeakPtrFactory<RenderFrameImpl> weak_factory_;
1424 1428
1425 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl); 1429 DISALLOW_COPY_AND_ASSIGN(RenderFrameImpl);
1426 }; 1430 };
1427 1431
1428 } // namespace content 1432 } // namespace content
1429 1433
1430 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_ 1434 #endif // CONTENT_RENDERER_RENDER_FRAME_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698