diff options
author | Ben Wagner <bungeman@google.com> | 2015-01-12 17:06:21 -0500 |
---|---|---|
committer | Ben Wagner <bungeman@google.com> | 2015-01-12 18:13:27 -0500 |
commit | c2d39573ca95994d9a44db8aea4887a67f84e5ac (patch) | |
tree | ab7b13f2c5f0279f6cb9d9a2b8bdde0257d0c0a8 /core/jni | |
parent | 8d2e8e2bf7341aa8d4b5c0ea5ce038e745275324 (diff) | |
download | frameworks_base-c2d39573ca95994d9a44db8aea4887a67f84e5ac.zip frameworks_base-c2d39573ca95994d9a44db8aea4887a67f84e5ac.tar.gz frameworks_base-c2d39573ca95994d9a44db8aea4887a67f84e5ac.tar.bz2 |
Remove dead code from AssetStreamAdapter.
After "Use at least SkStreamAsset for SkTypefaces." there is now dead
code in AssetStreamAdapter. Remove the code and update the users.
Change-Id: I17a9d82dd6ca4d87a951c1570dc91de7f2fc54f2
Diffstat (limited to 'core/jni')
-rw-r--r-- | core/jni/android/graphics/BitmapFactory.cpp | 3 | ||||
-rw-r--r-- | core/jni/android/graphics/Movie.cpp | 4 | ||||
-rw-r--r-- | core/jni/android/graphics/Utils.cpp | 12 | ||||
-rw-r--r-- | core/jni/android/graphics/Utils.h | 23 |
4 files changed, 5 insertions, 37 deletions
diff --git a/core/jni/android/graphics/BitmapFactory.cpp b/core/jni/android/graphics/BitmapFactory.cpp index 37f33a9..672008d 100644 --- a/core/jni/android/graphics/BitmapFactory.cpp +++ b/core/jni/android/graphics/BitmapFactory.cpp @@ -523,8 +523,7 @@ static jobject nativeDecodeAsset(JNIEnv* env, jobject clazz, jlong native_asset, Asset* asset = reinterpret_cast<Asset*>(native_asset); // since we know we'll be done with the asset when we return, we can // just use a simple wrapper - SkAutoTUnref<SkStreamRewindable> stream(new AssetStreamAdaptor(asset, - AssetStreamAdaptor::kNo_OwnAsset, AssetStreamAdaptor::kNo_HasMemoryBase)); + SkAutoTUnref<SkStreamRewindable> stream(new AssetStreamAdaptor(asset)); return doDecode(env, stream, padding, options); } diff --git a/core/jni/android/graphics/Movie.cpp b/core/jni/android/graphics/Movie.cpp index 1430c7e..134551d 100644 --- a/core/jni/android/graphics/Movie.cpp +++ b/core/jni/android/graphics/Movie.cpp @@ -84,9 +84,7 @@ static void movie_draw(JNIEnv* env, jobject movie, jlong canvasHandle, static jobject movie_decodeAsset(JNIEnv* env, jobject clazz, jlong native_asset) { android::Asset* asset = reinterpret_cast<android::Asset*>(native_asset); if (asset == NULL) return NULL; - SkAutoTUnref<SkStreamRewindable> stream (new android::AssetStreamAdaptor(asset, - android::AssetStreamAdaptor::kNo_OwnAsset, - android::AssetStreamAdaptor::kNo_HasMemoryBase)); + SkAutoTUnref<SkStreamRewindable> stream(new android::AssetStreamAdaptor(asset)); SkMovie* moov = SkMovie::DecodeStream(stream.get()); return create_jmovie(env, moov); } diff --git a/core/jni/android/graphics/Utils.cpp b/core/jni/android/graphics/Utils.cpp index a134d4b..3c471bd 100644 --- a/core/jni/android/graphics/Utils.cpp +++ b/core/jni/android/graphics/Utils.cpp @@ -19,21 +19,11 @@ using namespace android; -AssetStreamAdaptor::AssetStreamAdaptor(Asset* asset, OwnAsset ownAsset, - HasMemoryBase hasMemoryBase) +AssetStreamAdaptor::AssetStreamAdaptor(Asset* asset) : fAsset(asset) - , fMemoryBase(kYes_HasMemoryBase == hasMemoryBase ? - asset->getBuffer(false) : NULL) - , fOwnAsset(ownAsset) { } -AssetStreamAdaptor::~AssetStreamAdaptor() { - if (kYes_OwnAsset == fOwnAsset) { - delete fAsset; - } -} - bool AssetStreamAdaptor::rewind() { off64_t pos = fAsset->seek(0, SEEK_SET); if (pos == (off64_t)-1) { diff --git a/core/jni/android/graphics/Utils.h b/core/jni/android/graphics/Utils.h index bde87f0..c0b9410 100644 --- a/core/jni/android/graphics/Utils.h +++ b/core/jni/android/graphics/Utils.h @@ -28,22 +28,7 @@ namespace android { class AssetStreamAdaptor : public SkStreamRewindable { public: - // Enum passed to constructor. If set to kYes_OwnAsset, - // the passed in Asset will be deleted upon destruction. - enum OwnAsset { - kYes_OwnAsset, - kNo_OwnAsset, - }; - - // Enum passed to constructor. If set to kYes_HasMemoryBase, - // getMemoryBase will return the Asset's buffer. - enum HasMemoryBase { - kYes_HasMemoryBase, - kNo_HasMemoryBase, - }; - - AssetStreamAdaptor(Asset*, OwnAsset, HasMemoryBase); - ~AssetStreamAdaptor(); + AssetStreamAdaptor(Asset*); virtual bool rewind(); virtual size_t read(void* buffer, size_t size); @@ -51,13 +36,9 @@ public: virtual size_t getLength() const; virtual bool isAtEnd() const; - virtual const void* getMemoryBase() { return fMemoryBase; } - virtual SkStreamRewindable* duplicate() const; private: - Asset* fAsset; - const void* const fMemoryBase; - const OwnAsset fOwnAsset; + Asset* fAsset; }; /** |