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

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

Issue 9224002: Make WebUI objects not derive from WebUI. WebUI objects own the controller. This is the ownership... (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: sync to head to clear linux_chromeos browsertest failures Created 8 years, 11 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 | Annotate | Revision Log
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_PUBLIC_BROWSER_WEB_CONTENTS_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 6 #define CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
7 #pragma once 7 #pragma once
8 8
9 #include "base/basictypes.h" 9 #include "base/basictypes.h"
10 #include "base/process_util.h" 10 #include "base/process_util.h"
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
102 // Gets the current RenderViewHost for this tab. 102 // Gets the current RenderViewHost for this tab.
103 virtual RenderViewHost* GetRenderViewHost() const = 0; 103 virtual RenderViewHost* GetRenderViewHost() const = 0;
104 104
105 // Returns the currently active RenderWidgetHostView. This may change over 105 // Returns the currently active RenderWidgetHostView. This may change over
106 // time and can be NULL (during setup and teardown). 106 // time and can be NULL (during setup and teardown).
107 virtual RenderWidgetHostView* GetRenderWidgetHostView() const = 0; 107 virtual RenderWidgetHostView* GetRenderWidgetHostView() const = 0;
108 108
109 // The TabContentsView will never change and is guaranteed non-NULL. 109 // The TabContentsView will never change and is guaranteed non-NULL.
110 virtual TabContentsView* GetView() const = 0; 110 virtual TabContentsView* GetView() const = 0;
111 111
112 // Create a WebUI page for the given url. In most cases, this doesn't need to
113 // be called by embedders since content will create its own WebUI objects as
114 // necessary. However if the embedder wants to create its own WebUI object and
115 // keep track of it manually, it can use this.
116 virtual WebUI* CreateWebUI(const GURL& url) = 0;
117
112 // Returns the committed WebUI if one exists, otherwise the pending one. 118 // Returns the committed WebUI if one exists, otherwise the pending one.
113 // Callers who want to use the pending WebUI for the pending navigation entry 119 // Callers who want to use the pending WebUI for the pending navigation entry
114 // should use GetWebUIForCurrentState instead. 120 // should use GetWebUIForCurrentState instead.
115 virtual WebUI* GetWebUI() const = 0; 121 virtual WebUI* GetWebUI() const = 0;
116 virtual WebUI* GetCommittedWebUI() const = 0; 122 virtual WebUI* GetCommittedWebUI() const = 0;
117 123
118 // Tab navigation state ------------------------------------------------------ 124 // Tab navigation state ------------------------------------------------------
119 125
120 // Returns the current navigation properties, which if a navigation is 126 // Returns the current navigation properties, which if a navigation is
121 // pending may be provisional (e.g., the navigation could result in a 127 // pending may be provisional (e.g., the navigation could result in a
(...skipping 230 matching lines...) Expand 10 before | Expand all | Expand 10 after
352 // to see what it should do. 358 // to see what it should do.
353 virtual bool FocusLocationBarByDefault() = 0; 359 virtual bool FocusLocationBarByDefault() = 0;
354 360
355 // Focuses the location bar. 361 // Focuses the location bar.
356 virtual void SetFocusToLocationBar(bool select_all) = 0; 362 virtual void SetFocusToLocationBar(bool select_all) = 0;
357 }; 363 };
358 364
359 } // namespace content 365 } // namespace content
360 366
361 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_ 367 #endif // CONTENT_PUBLIC_BROWSER_WEB_CONTENTS_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698