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

Unified Diff: chrome/browser/signin/screenlock_bridge.h

Issue 562393002: Remove unused chrome.screenlockPrivate methods (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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
Index: chrome/browser/signin/screenlock_bridge.h
diff --git a/chrome/browser/signin/screenlock_bridge.h b/chrome/browser/signin/screenlock_bridge.h
index 5391ca37c9189bc308b9a7833dced19743aaa274..379f447488d743b9624c3dd5efd569a0e82c03db 100644
--- a/chrome/browser/signin/screenlock_bridge.h
+++ b/chrome/browser/signin/screenlock_bridge.h
@@ -15,9 +15,6 @@
#include "base/strings/string16.h"
#include "base/values.h"
-namespace gfx {
-class Image;
-}
class Profile;
@@ -50,21 +47,14 @@ class ScreenlockBridge {
// Sets the icon as chrome://theme resource URL.
void SetIconAsResourceURL(const std::string& url);
- // Sets the icon as a gfx::Image. The image will be converted to set of data
- // URLs for each icon representation. Use |SetIconAsResourceURL| instead of
- // this.
- // TODO(tbarzic): Remove this one once easy unlock app stops using
- // screenlockPrivate.showCustomIcon.
- void SetIconAsImage(const gfx::Image& image);
-
- // Sets the icon size. Has to be called if |SetIconAsResourceURL| was used
- // to set the icon. For animated icon, this should be set to a single frame
- // size, not the animation resource size.
+ // Sets the icon size. If not called, the icon will not be visible.
+ // For animated icon, this should be set to a single frame size, not the
+ // animation resource size.
void SetSize(size_t icon_width, size_t icon_height);
// If the icon is supposed to be animated, sets the animation parameters.
- // If set, it expects that the resource set using |SetIcon*| methods
- // contains horizontally arranged ordered list of animation frames.
+ // If set, it expects that the resource set using |SetIconAsResourceURL|
+ // method contains horizontally arranged ordered list of animation frames.
// Note that the icon size set in |SetSize| should be a single frame size.
// |resource_width|: Total animation resource width.
// |frame_length_ms|: Time for which a single animation frame is shown.
@@ -84,7 +74,6 @@ class ScreenlockBridge {
private:
std::string icon_resource_url_;
- scoped_ptr<gfx::Image> icon_image_;
size_t width_;
size_t height_;

Powered by Google App Engine
This is Rietveld 408576698