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

Side by Side Diff: chrome/common/extensions/manifest_handlers/synthesize_browser_action_handler.h

Issue 324393002: Extension Toolbar redesign, part 1 (overflow) (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Missing gypi change Created 6 years, 5 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
(Empty)
1 // Copyright 2014 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_COMMON_EXTENSIONS_MANIFEST_HANDLERS_SYNTHESIZE_BROWSER_ACTION_HAN DLER_H_
6 #define CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_SYNTHESIZE_BROWSER_ACTION_HAN DLER_H_
7
8 #include <string>
9
10 #include "extensions/common/manifest_handler.h"
11
12 namespace extensions {
13
14 // Ensures that an extension that doesn't specify any icon in its manifest, gets
15 // a browser action icon, that we can hang UI off of.
16 // TODO(finnur): Combine the logic, as suggested in review 324393002.
17 class SynthesizeBrowserActionHandler : public ManifestHandler {
18 public:
19 SynthesizeBrowserActionHandler();
20 virtual ~SynthesizeBrowserActionHandler();
21
22 virtual bool Parse(Extension* extension, base::string16* error) OVERRIDE;
23 virtual bool AlwaysParseForType(Manifest::Type type) const OVERRIDE;
24
25 private:
26 virtual const std::vector<std::string> Keys() const OVERRIDE;
27
28 DISALLOW_COPY_AND_ASSIGN(SynthesizeBrowserActionHandler);
29 };
30
31 } // namespace extensions
32
33 #endif // CHROME_COMMON_EXTENSIONS_MANIFEST_HANDLERS_SYNTHESIZE_BROWSER_ACTION_ HANDLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698