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

Side by Side Diff: chrome/common/favicon/favicon_url_parser.h

Issue 15388002: Supporting high dpi favicons in Instant Extended. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Addressing Sky's comments Created 7 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 2013 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_FAVICON_FAVICON_URL_PARSER_H_
6 #define CHROME_COMMON_FAVICON_FAVICON_URL_PARSER_H_
7
8 #include <string>
9
10 #include "ui/base/layout.h"
11
12 namespace chrome {
13
14 struct ParsedFaviconPath {
15 // Whether the URL has the "iconurl" parameter.
16 bool is_icon_url;
James Hawkins 2013/07/02 23:23:04 nit: Add blank line after membar var and before ne
pedro (no code reviews) 2013/07/03 00:16:56 Done.
17 // The URL from which the favicon is being requested.
18 std::string url;
19 // The size of the requested favicon in dip.
20 int size_in_dip;
21 // The scale factor of the requested favicon.
22 ui::ScaleFactor scale_factor;
23 // The index of the first character (relative to the path) where the the URL
24 // from which the favicon is being requested is located.
25 size_t path_index;
26 };
27
28 // Parses |path|, which should be in the format described at the top of the
29 // file "chrome/browser/ui/webui/favicon_source.h". |icon_types| indicates
30 // which icon types are supported. Returns true if |path| could be parsed.
31 // The result of the parsing will be stored in a ParsedFaviconPath struct.
32 bool ParseFaviconPath(const std::string& path,
33 int icon_types,
34 ParsedFaviconPath* parsed);
35
36 } // namespace chrome
37
38 #endif // CHROME_COMMON_FAVICON_FAVICON_URL_PARSER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698