diff options
Diffstat (limited to 'chrome/browser/ui/webui/options/chromeos/user_image_source.cc')
-rw-r--r-- | chrome/browser/ui/webui/options/chromeos/user_image_source.cc | 41 |
1 files changed, 5 insertions, 36 deletions
diff --git a/chrome/browser/ui/webui/options/chromeos/user_image_source.cc b/chrome/browser/ui/webui/options/chromeos/user_image_source.cc index 6ad87bee..0d25528 100644 --- a/chrome/browser/ui/webui/options/chromeos/user_image_source.cc +++ b/chrome/browser/ui/webui/options/chromeos/user_image_source.cc @@ -19,31 +19,15 @@ namespace { -// Animated key is used in user image URL requests to specify that -// animated version of user image is required. Without that key -// non-animated version of user image should be returned. -const char kKeyAnimated[] = "animated"; - // Parses the user image URL, which looks like // "chrome://userimage/user@host?key1=value1&...&key_n=value_n", -// to user email and optional parameters. +// to user email. void ParseRequest(const GURL& url, - std::string* email, - bool* is_image_animated) { + std::string* email) { DCHECK(url.is_valid()); *email = net::UnescapeURLComponent(url.path().substr(1), (net::UnescapeRule::URL_SPECIAL_CHARS | net::UnescapeRule::SPACES)); - std::string url_spec = url.possibly_invalid_spec(); - url::Component query = url.parsed_for_possibly_invalid_spec().query; - url::Component key, value; - *is_image_animated = false; - while (ExtractQueryKeyValue(url_spec.c_str(), &query, &key, &value)) { - if (url_spec.substr(key.begin, key.len) == kKeyAnimated) { - *is_image_animated = true; - break; - } - } } } // namespace @@ -53,13 +37,10 @@ namespace options { base::RefCountedMemory* UserImageSource::GetUserImage( const std::string& email, - bool is_image_animated, ui::ScaleFactor scale_factor) const { const chromeos::User* user = chromeos::UserManager::Get()->FindUser(email); if (user) { - if (user->has_animated_image() && is_image_animated) { - return new base::RefCountedBytes(user->animated_image()); - } else if (user->has_raw_image()) { + if (user->has_raw_image()) { return new base::RefCountedBytes(user->raw_image()); } else if (user->image_is_stub()) { return ResourceBundle::GetSharedInstance(). @@ -93,26 +74,14 @@ void UserImageSource::StartDataRequest( int render_frame_id, const content::URLDataSource::GotDataCallback& callback) { std::string email; - bool is_image_animated = false; GURL url(chrome::kChromeUIUserImageURL + path); - ParseRequest(url, &email, &is_image_animated); - callback.Run(GetUserImage(email, is_image_animated, ui::SCALE_FACTOR_100P)); + ParseRequest(url, &email); + callback.Run(GetUserImage(email, ui::SCALE_FACTOR_100P)); } std::string UserImageSource::GetMimeType(const std::string& path) const { // We need to explicitly return a mime type, otherwise if the user tries to // drag the image they get no extension. - std::string email; - bool is_image_animated = false; - - GURL url(chrome::kChromeUIUserImageURL + path); - ParseRequest(url, &email, &is_image_animated); - - if (is_image_animated) { - const chromeos::User* user = chromeos::UserManager::Get()->FindUser(email); - if (user && user->has_animated_image()) - return "image/gif"; - } return "image/png"; } |