Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c5a16291 authored by Chris Tate's avatar Chris Tate Committed by android-build-merger
Browse files

Merge "Setting a wallpaper must be synchronous" into oc-mr1-dev am: b00a8aa6

am: 367bbff6

Change-Id: I81b6d1356dad7db58b10a76ae049d64590ac8909
parents ef1c48fa 367bbff6
Loading
Loading
Loading
Loading
+10 −8
Original line number Diff line number Diff line
@@ -280,14 +280,6 @@ public class WallpaperManagerService extends IWallpaperManager.Stub {
                                Slog.v(TAG, "Crop done; invoking completion callback");
                            }
                            wallpaper.imageWallpaperPending = false;
                            if (wallpaper.setComplete != null) {
                                try {
                                    wallpaper.setComplete.onWallpaperChanged();
                                } catch (RemoteException e) {
                                    // if this fails we don't really care; the setting app may just
                                    // have crashed and that sort of thing is a fact of life.
                                }
                            }
                            if (sysWallpaperChanged) {
                                // If this was the system wallpaper, rebind...
                                bindWallpaperComponentLocked(mImageWallpaper, true,
@@ -311,6 +303,16 @@ public class WallpaperManagerService extends IWallpaperManager.Stub {
                            }

                            saveSettingsLocked(wallpaper.userId);

                            // Publish completion *after* we've persisted the changes
                            if (wallpaper.setComplete != null) {
                                try {
                                    wallpaper.setComplete.onWallpaperChanged();
                                } catch (RemoteException e) {
                                    // if this fails we don't really care; the setting app may just
                                    // have crashed and that sort of thing is a fact of life.
                                }
                            }
                        }
                    }
                }