diff options
author | Yuli Huang <yuli@google.com> | 2011-11-28 20:05:27 +0800 |
---|---|---|
committer | Yuli Huang <yuli@google.com> | 2011-12-02 00:57:02 +0800 |
commit | 11571bf7436900b513dd3696a90f546b25839591 (patch) | |
tree | 37320f29530b7e641f7243c64afd0e8504b0491f /src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java | |
parent | 5a7fe75b7c95c2612b4a27ebaaa8d324fac6a849 (diff) | |
download | android_packages_apps_Snap-11571bf7436900b513dd3696a90f546b25839591.tar.gz android_packages_apps_Snap-11571bf7436900b513dd3696a90f546b25839591.tar.bz2 android_packages_apps_Snap-11571bf7436900b513dd3696a90f546b25839591.zip |
Refactor to remove obsolete design and improve code consistency.
1. Remove isValid()/validate() in Filter; EffectAction would push
filters into the stack when being notified by notifyChanged().
2. Make RedEyeAction consistent with DoodleAction by ignoring
out-of-bounds operations.
3. Improve consistency by encapsulate effect-tool creation/removal in
one class (EffectToolFactory).
4. Minimize variable visibility in EffectAction subclasses.
5. Fix unfinished doodle lost when exiting Doodle effect by multi-touch.
Change-Id: I8d3b127c5fe8777eb45b7c5ae680f6a30a99635a
Diffstat (limited to 'src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java')
-rw-r--r-- | src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java b/src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java index 0ec4a0205..d1fd61171 100644 --- a/src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java +++ b/src/com/android/gallery3d/photoeditor/actions/DocumentaryAction.java @@ -31,12 +31,8 @@ public class DocumentaryAction extends EffectAction { } @Override - public void doBegin() { - notifyFilterChanged(new DocumentaryFilter(), true); + public void prepare() { + notifyChanged(new DocumentaryFilter()); notifyDone(); } - - @Override - public void doEnd() { - } } |