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

Unified Diff: chrome/browser/tab_contents/web_contents.h

Issue 19746: Remove old web app code it's no longer needed. Simplifies startup a little. (Closed) Base URL: svn://chrome-svn/chrome/trunk/src/
Patch Set: '' Created 11 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/tab_contents/web_contents.h
===================================================================
--- chrome/browser/tab_contents/web_contents.h (revision 9030)
+++ chrome/browser/tab_contents/web_contents.h (working copy)
@@ -12,7 +12,7 @@
#include "chrome/browser/tab_contents/render_view_host_manager.h"
#include "chrome/browser/shell_dialogs.h"
#include "chrome/browser/tab_contents/tab_contents.h"
-#include "chrome/browser/web_app.h"
+#include "chrome/common/gears_api.h"
class AutofillManager;
class InterstitialPageDelegate;
@@ -32,8 +32,7 @@
class WebContents : public TabContents,
public RenderViewHostDelegate,
public RenderViewHostManager::Delegate,
- public SelectFileDialog::Listener,
- public WebApp::Observer {
+ public SelectFileDialog::Listener {
public:
// If instance is NULL, then creates a new process for this view. Otherwise
// initialize with a process already created for a different WebContents.
@@ -88,7 +87,6 @@
virtual void Destroy();
virtual WebContents* AsWebContents() { return this; }
virtual SiteInstance* GetSiteInstance() const;
- virtual SkBitmap GetFavIcon();
virtual std::wstring GetStatusText() const;
virtual bool NavigateToPendingEntry(bool reload);
virtual void Stop();
@@ -113,13 +111,6 @@
// Web apps ------------------------------------------------------------------
- // Sets the WebApp for this WebContents.
- void SetWebApp(WebApp* web_app);
- WebApp* web_app() { return web_app_.get(); }
-
- // Return whether this tab contents was created to contain an application.
- bool IsWebApplication() const;
-
// Tell Gears to create a shortcut for the current page.
void CreateShortcut();
@@ -431,17 +422,6 @@
// Send webkit specific settings to the renderer.
void UpdateWebPreferences();
- // Return whether the optional web application is active for the current URL.
- // Call this method to check if web app properties are in effect.
- //
- // Note: This method should be used for presentation but not security. The app
- // is always active if the containing window is a web application.
- bool IsWebApplicationActive() const;
-
- // WebApp::Observer method. Invoked when the set of images contained in the
- // web app changes. Notifies the delegate our favicon has changed.
- virtual void WebAppImagesChanged(WebApp* web_app);
-
// Called when the user dismisses the shortcut creation dialog. 'success' is
// true if the shortcut was created.
void OnGearsCreateShortcutDone(const GearsShortcutData& shortcut_data,
@@ -566,9 +546,6 @@
net::LoadState load_state_;
std::wstring load_state_host_;
- // Non-null if we're displaying content for a web app.
- scoped_refptr<WebApp> web_app_;
-
DISALLOW_COPY_AND_ASSIGN(WebContents);
};

Powered by Google App Engine
This is Rietveld 408576698