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

Unified Diff: content/renderer/manifest/manifest_parser.h

Issue 2625753002: Correct the comment for ParseIconPurpose in manifest_parser.h (Closed)
Patch Set: Created 3 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: content/renderer/manifest/manifest_parser.h
diff --git a/content/renderer/manifest/manifest_parser.h b/content/renderer/manifest/manifest_parser.h
index 3487b81b40fcfb7d3ed5e67727d0f6ee2ca49dbb..610566b4b8ddfe4ec1b52d93cb894cb11be38af2 100644
--- a/content/renderer/manifest/manifest_parser.h
+++ b/content/renderer/manifest/manifest_parser.h
@@ -134,9 +134,8 @@ class CONTENT_EXPORT ManifestParser {
// Parses the 'purpose' field of an icon, as defined in:
// https://w3c.github.io/manifest/#dfn-steps-for-processing-a-purpose-member-of-an-image
// Returns a vector of Manifest::Icon::IconPurpose with the successfully
- // parsed icon purposes, an empty vector if the field was not present or not
- // of type "string", and a vector containing Manifest::Icon::IconPurpose::Any
- // for all other parsing exceptions.
+ // parsed icon purposes, and a vector with Manifest::Icon::IconPurpose::Any if
+ // the parsing failed.
std::vector<Manifest::Icon::IconPurpose> ParseIconPurpose(
const base::DictionaryValue& icon);
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698