diff options
| author | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-19 07:17:23 +0000 |
|---|---|---|
| committer | xiyuan@chromium.org <xiyuan@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-11-19 07:17:23 +0000 |
| commit | 12ea22a1337a63db017338156ba56abd62beacbd (patch) | |
| tree | 81fc9e7eeb51a9605ef3dc7d2b7b074b1e2e4614 /chrome/browser/web_applications/web_app.h | |
| parent | 6bd2521cca50d0575c350234be687cf2ead92b01 (diff) | |
| download | chromium_src-12ea22a1337a63db017338156ba56abd62beacbd.zip chromium_src-12ea22a1337a63db017338156ba56abd62beacbd.tar.gz chromium_src-12ea22a1337a63db017338156ba56abd62beacbd.tar.bz2 | |
Disable create application shortcuts for internal pages
- In fact, we only allow scheme file, ftp, http and https to have
shortcuts;
- Removed GetFavIcon().isNull() for checking since we are using url
scheme now;
- Add a CommandCreateAppShortcut to BrowserTest per Ben's comments;
BUG=26743
TEST=Verify create application shortcuts menu item is disabled for all
internal pages including but not limited to new tab page, history,
downloads, view-source etc.
Review URL: http://codereview.chromium.org/404011
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@32503 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/web_applications/web_app.h')
| -rw-r--r-- | chrome/browser/web_applications/web_app.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/chrome/browser/web_applications/web_app.h b/chrome/browser/web_applications/web_app.h index 3e3c9b0..f6c9612 100644 --- a/chrome/browser/web_applications/web_app.h +++ b/chrome/browser/web_applications/web_app.h @@ -25,6 +25,9 @@ void CreateShortcut( const ShellIntegration::ShortcutInfo& shortcut_info, CreateShortcutCallback* callback); +// Returns true if given url is a valid web app url. +bool IsValidUrl(const GURL& url); + }; // namespace web_app #endif // CHROME_BROWSER_WEB_APPLICATIONS_WEB_APP_H_ |