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

Side by Side Diff: content/browser/web_contents/web_contents_impl.h

Issue 1352813006: Move WebUI ownership from the RenderFrameHostManager to the RenderFrameHost. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Addressing some of the current CR comments (before moving on to eliminate the pending WebUI). 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_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 5 #ifndef CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 6 #define CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 418 matching lines...) Expand 10 before | Expand all | Expand 10 after
429 RenderFrameHost* GetGuestByInstanceID( 429 RenderFrameHost* GetGuestByInstanceID(
430 RenderFrameHost* render_frame_host, 430 RenderFrameHost* render_frame_host,
431 int browser_plugin_instance_id) override; 431 int browser_plugin_instance_id) override;
432 GeolocationServiceContext* GetGeolocationServiceContext() override; 432 GeolocationServiceContext* GetGeolocationServiceContext() override;
433 void EnterFullscreenMode(const GURL& origin) override; 433 void EnterFullscreenMode(const GURL& origin) override;
434 void ExitFullscreenMode() override; 434 void ExitFullscreenMode() override;
435 bool ShouldRouteMessageEvent( 435 bool ShouldRouteMessageEvent(
436 RenderFrameHost* target_rfh, 436 RenderFrameHost* target_rfh,
437 SiteInstance* source_site_instance) const override; 437 SiteInstance* source_site_instance) const override;
438 void EnsureOpenerProxiesExist(RenderFrameHost* source_rfh) override; 438 void EnsureOpenerProxiesExist(RenderFrameHost* source_rfh) override;
439 scoped_ptr<WebUIImpl> CreateWebUIForRenderFrameHost(const GURL& url) override;
439 #if defined(OS_WIN) 440 #if defined(OS_WIN)
440 gfx::NativeViewAccessible GetParentNativeViewAccessible() override; 441 gfx::NativeViewAccessible GetParentNativeViewAccessible() override;
441 #endif 442 #endif
442 443
443 // RenderViewHostDelegate ---------------------------------------------------- 444 // RenderViewHostDelegate ----------------------------------------------------
444 RenderViewHostDelegateView* GetDelegateView() override; 445 RenderViewHostDelegateView* GetDelegateView() override;
445 bool OnMessageReceived(RenderViewHost* render_view_host, 446 bool OnMessageReceived(RenderViewHost* render_view_host,
446 const IPC::Message& message) override; 447 const IPC::Message& message) override;
447 // RenderFrameHostDelegate has the same method, so list it there because this 448 // RenderFrameHostDelegate has the same method, so list it there because this
448 // interface is going away. 449 // interface is going away.
(...skipping 165 matching lines...) Expand 10 before | Expand all | Expand 10 after
614 RenderViewHost* render_view_host) override; 615 RenderViewHost* render_view_host) override;
615 void UpdateRenderViewSizeForRenderManager() override; 616 void UpdateRenderViewSizeForRenderManager() override;
616 void CancelModalDialogsForRenderManager() override; 617 void CancelModalDialogsForRenderManager() override;
617 void NotifySwappedFromRenderManager(RenderFrameHost* old_host, 618 void NotifySwappedFromRenderManager(RenderFrameHost* old_host,
618 RenderFrameHost* new_host, 619 RenderFrameHost* new_host,
619 bool is_main_frame) override; 620 bool is_main_frame) override;
620 void NotifyMainFrameSwappedFromRenderManager( 621 void NotifyMainFrameSwappedFromRenderManager(
621 RenderViewHost* old_host, 622 RenderViewHost* old_host,
622 RenderViewHost* new_host) override; 623 RenderViewHost* new_host) override;
623 NavigationControllerImpl& GetControllerForRenderManager() override; 624 NavigationControllerImpl& GetControllerForRenderManager() override;
624 scoped_ptr<WebUIImpl> CreateWebUIForRenderManager(const GURL& url) override;
625 NavigationEntry* GetLastCommittedNavigationEntryForRenderManager() override; 625 NavigationEntry* GetLastCommittedNavigationEntryForRenderManager() override;
626 bool FocusLocationBarByDefault() override; 626 bool FocusLocationBarByDefault() override;
627 void SetFocusToLocationBar(bool select_all) override; 627 void SetFocusToLocationBar(bool select_all) override;
628 bool IsHidden() override; 628 bool IsHidden() override;
629 int GetOuterDelegateFrameTreeNodeID() override; 629 int GetOuterDelegateFrameTreeNodeID() override;
630 630
631 // NotificationObserver ------------------------------------------------------ 631 // NotificationObserver ------------------------------------------------------
632 632
633 void Observe(int type, 633 void Observe(int type,
634 const NotificationSource& source, 634 const NotificationSource& source,
(...skipping 695 matching lines...) Expand 10 before | Expand all | Expand 10 after
1330 // Adds/removes a callback called on creation of each new WebContents. 1330 // Adds/removes a callback called on creation of each new WebContents.
1331 static void AddCreatedCallbackForTesting(const CreatedCallback& callback); 1331 static void AddCreatedCallbackForTesting(const CreatedCallback& callback);
1332 static void RemoveCreatedCallbackForTesting(const CreatedCallback& callback); 1332 static void RemoveCreatedCallbackForTesting(const CreatedCallback& callback);
1333 1333
1334 DISALLOW_COPY_AND_ASSIGN(FriendZone); 1334 DISALLOW_COPY_AND_ASSIGN(FriendZone);
1335 }; 1335 };
1336 1336
1337 } // namespace content 1337 } // namespace content
1338 1338
1339 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 1339 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698