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

Unified Diff: components/arc/set_wallpaper_delegate.h

Issue 2264743002: cheets: implement cros side of WallpaperManagerService. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Add a static instance to SetWallpaperDelegate and null check to ArcIntentHelper. Created 4 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: components/arc/set_wallpaper_delegate.h
diff --git a/components/arc/set_wallpaper_delegate.h b/components/arc/set_wallpaper_delegate.h
index 28e5ed038c5248a219b2b3f78e1294c8d4a7cd5a..43c8ea90de9172230ae502ff61c3e336a9bc9bc5 100644
--- a/components/arc/set_wallpaper_delegate.h
+++ b/components/arc/set_wallpaper_delegate.h
@@ -6,18 +6,26 @@
#define COMPONENTS_ARC_SET_WALLPAPER_DELEGATE_H_
#include <stdint.h>
-
Luis Héctor Chávez 2016/09/06 20:05:46 keep this empty line: https://google.github.io/sty
Muyuan 2016/09/06 23:38:00 Done.
#include <vector>
namespace arc {
+class ArcBridgeService;
Luis Héctor Chávez 2016/09/06 20:05:46 unused?
Muyuan 2016/09/06 23:38:00 done.
+
// Delegate to allow setting the wallpaper.
+// TODO(muyuanli): This class will be removed once ArcIntentHelper migrates to
+// WallpaperManager API.
class SetWallpaperDelegate {
public:
virtual ~SetWallpaperDelegate() = default;
+ static SetWallpaperDelegate* GetInstance() { return instance_; }
+
// Sets an image represented in JPEG format as the wallpaper.
- virtual void SetWallpaper(std::vector<uint8_t> jpeg_data) = 0;
+ virtual void SetWallpaperJPEG(const std::vector<uint8_t>& jpeg_data) = 0;
+
+ protected:
+ static SetWallpaperDelegate* instance_;
};
} // namespace arc

Powered by Google App Engine
This is Rietveld 408576698