Conductor: Add onActivityLowMemory

[lenguyenthanh]

Some time we need to listen to onLowMemory callback in our Controller.
For example when I'm working with Mapbox, it requires to call onLowMemory:

 @Override
    public void onLowMemory() {
        super.onLowMemory();
        mapView.onLowMemory();
    }

If you ok with this feature, I'm happy to do a PR.

[adipascu]

You have access to this functionality in the Application class

[JakeWharton]

You can propagate information like this using LocalBroadcastManager, an
event bus, RxJava, or manual callbacks that you register. I don't think
this library needs to be responsible for everything when it arguably has
too many callbacks already and other mechanisms exist for passing along
data to controllers.

On Mon, Jun 6, 2016, 10:18 AM Leonardo Ferrari [email protected]
wrote:

But he needs this callback inside some Controller.
I think another callback wouldn't hurt.

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bluelinelabs/Conductor/issues/59#issuecomment-223972549,
or mute the thread
https://github.com/notifications/unsubscribe/AAEEEXOucGbfNij0c-iG7zAaQdBM82_rks5qJCw6gaJpZM4IuFdj
.

[adipascu]

@leonardo2204 Extra callbacks hurt, I'd even like some removed if possible.

[lenguyenthanh]

IMHO, it's more convenient for us if we have this callback (do not need to do some extra steps like event -bus or RxJava). It wouldn't hurt much too, because people can decide to use it or not. Btw it's just my opinion, it's not really important thought.

[EricKuck]

This should be easy enough to add to the application layer to not be required at the library layer.