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

Side by Side Diff: chrome/browser/banners/app_banner_data_fetcher_desktop.h

Issue 1161233005: Implement app banner info bars on desktop. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@stop-icon-overgeneration
Patch Set: Reuploading Created 5 years, 6 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
(Empty)
1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file.
4
5 #ifndef CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_DESKTOP_H_
6 #define CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_DESKTOP_H_
7
8 #include "chrome/browser/banners/app_banner_data_fetcher.h"
9
10 namespace extensions {
11 class BookmarkAppHelper;
12 class Extension;
13 } // namespace extensions
14
15 namespace infobars {
benwells 2015/06/03 06:13:55 Nit: I think this namespace / forward decl isn't u
dominickn (DO NOT USE) 2015/06/03 07:10:36 Done. It's used in the cc file, so I've moved it t
16 class InfoBar;
17 } // namespace infobars
18
19 namespace banners {
20
21 // Fetches data required to show a banner for the URL currently shown by the
22 // WebContents. Extends the regular fetch to support desktop web apps.
23 class AppBannerDataFetcherDesktop : public AppBannerDataFetcher {
24 public:
25 AppBannerDataFetcherDesktop(content::WebContents* web_contents,
26 base::WeakPtr<Delegate> weak_delegate,
27 int ideal_icon_size);
28
29 void ShowBanner(const SkBitmap* icon, const base::string16& title) override;
30
31 // Callback for finishing bookmark app creation
32 void FinishCreateBookmarkApp(const extensions::Extension* extension,
33 const WebApplicationInfo& web_app_info);
34
35 protected:
36 ~AppBannerDataFetcherDesktop() override;
37
38 private:
39 scoped_ptr<extensions::BookmarkAppHelper> bookmark_app_helper_;
40 DISALLOW_COPY_AND_ASSIGN(AppBannerDataFetcherDesktop);
41 };
42
43 } // namespace banners
44
45 #endif // CHROME_BROWSER_BANNERS_APP_BANNER_DATA_FETCHER_DESKTOP_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698