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

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: Removed InitializeWebUI Created 5 years, 2 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 (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 416 matching lines...) Expand 10 before | Expand all | Expand 10 after
427 RenderFrameHost* GetGuestByInstanceID( 427 RenderFrameHost* GetGuestByInstanceID(
428 RenderFrameHost* render_frame_host, 428 RenderFrameHost* render_frame_host,
429 int browser_plugin_instance_id) override; 429 int browser_plugin_instance_id) override;
430 GeolocationServiceContext* GetGeolocationServiceContext() override; 430 GeolocationServiceContext* GetGeolocationServiceContext() override;
431 void EnterFullscreenMode(const GURL& origin) override; 431 void EnterFullscreenMode(const GURL& origin) override;
432 void ExitFullscreenMode() override; 432 void ExitFullscreenMode() override;
433 bool ShouldRouteMessageEvent( 433 bool ShouldRouteMessageEvent(
434 RenderFrameHost* target_rfh, 434 RenderFrameHost* target_rfh,
435 SiteInstance* source_site_instance) const override; 435 SiteInstance* source_site_instance) const override;
436 void EnsureOpenerProxiesExist(RenderFrameHost* source_rfh) override; 436 void EnsureOpenerProxiesExist(RenderFrameHost* source_rfh) override;
437 scoped_ptr<WebUIImpl> CreateWebUIForRenderFrameHost(const GURL& url) override;
437 #if defined(OS_WIN) 438 #if defined(OS_WIN)
438 gfx::NativeViewAccessible GetParentNativeViewAccessible() override; 439 gfx::NativeViewAccessible GetParentNativeViewAccessible() override;
439 #endif 440 #endif
440 441
441 // RenderViewHostDelegate ---------------------------------------------------- 442 // RenderViewHostDelegate ----------------------------------------------------
442 RenderViewHostDelegateView* GetDelegateView() override; 443 RenderViewHostDelegateView* GetDelegateView() override;
443 bool OnMessageReceived(RenderViewHost* render_view_host, 444 bool OnMessageReceived(RenderViewHost* render_view_host,
444 const IPC::Message& message) override; 445 const IPC::Message& message) override;
445 // RenderFrameHostDelegate has the same method, so list it there because this 446 // RenderFrameHostDelegate has the same method, so list it there because this
446 // interface is going away. 447 // interface is going away.
(...skipping 168 matching lines...) Expand 10 before | Expand all | Expand 10 after
615 RenderViewHost* render_view_host) override; 616 RenderViewHost* render_view_host) override;
616 void UpdateRenderViewSizeForRenderManager() override; 617 void UpdateRenderViewSizeForRenderManager() override;
617 void CancelModalDialogsForRenderManager() override; 618 void CancelModalDialogsForRenderManager() override;
618 void NotifySwappedFromRenderManager(RenderFrameHost* old_host, 619 void NotifySwappedFromRenderManager(RenderFrameHost* old_host,
619 RenderFrameHost* new_host, 620 RenderFrameHost* new_host,
620 bool is_main_frame) override; 621 bool is_main_frame) override;
621 void NotifyMainFrameSwappedFromRenderManager( 622 void NotifyMainFrameSwappedFromRenderManager(
622 RenderViewHost* old_host, 623 RenderViewHost* old_host,
623 RenderViewHost* new_host) override; 624 RenderViewHost* new_host) override;
624 NavigationControllerImpl& GetControllerForRenderManager() override; 625 NavigationControllerImpl& GetControllerForRenderManager() override;
625 scoped_ptr<WebUIImpl> CreateWebUIForRenderManager(const GURL& url) override;
626 NavigationEntry* GetLastCommittedNavigationEntryForRenderManager() override; 626 NavigationEntry* GetLastCommittedNavigationEntryForRenderManager() override;
627 bool FocusLocationBarByDefault() override; 627 bool FocusLocationBarByDefault() override;
628 void SetFocusToLocationBar(bool select_all) override; 628 void SetFocusToLocationBar(bool select_all) override;
629 bool IsHidden() override; 629 bool IsHidden() override;
630 int GetOuterDelegateFrameTreeNodeID() override; 630 int GetOuterDelegateFrameTreeNodeID() override;
631 631
632 // NotificationObserver ------------------------------------------------------ 632 // NotificationObserver ------------------------------------------------------
633 633
634 void Observe(int type, 634 void Observe(int type,
635 const NotificationSource& source, 635 const NotificationSource& source,
(...skipping 695 matching lines...) Expand 10 before | Expand all | Expand 10 after
1331 // Adds/removes a callback called on creation of each new WebContents. 1331 // Adds/removes a callback called on creation of each new WebContents.
1332 static void AddCreatedCallbackForTesting(const CreatedCallback& callback); 1332 static void AddCreatedCallbackForTesting(const CreatedCallback& callback);
1333 static void RemoveCreatedCallbackForTesting(const CreatedCallback& callback); 1333 static void RemoveCreatedCallbackForTesting(const CreatedCallback& callback);
1334 1334
1335 DISALLOW_COPY_AND_ASSIGN(FriendZone); 1335 DISALLOW_COPY_AND_ASSIGN(FriendZone);
1336 }; 1336 };
1337 1337
1338 } // namespace content 1338 } // namespace content
1339 1339
1340 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_ 1340 #endif // CONTENT_BROWSER_WEB_CONTENTS_WEB_CONTENTS_IMPL_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698