timba
timba

Reputation: 261

AdvertisingIdClient getAdvertisingIdInfo hangs forever

I'm trying to get advertising ID from Google Play services API. Here is a sample code:

...
import com.google.android.gms.ads.identifier.AdvertisingIdClient;
import com.google.android.gms.common.GooglePlayServicesNotAvailableException;
import com.google.android.gms.common.GooglePlayServicesRepairableException;
...

public class MyActivity extends Activity {

    @Override
    protected void onStart() {
        super.onStart();
        Thread thr = new Thread(new Runnable() {
            @Override
            public void run() {
            try {
                Context ctx = MyActivity.this.getApplicationContext();
                AdvertisingIdClient.Info adInfo = AdvertisingIdClient.getAdvertisingIdInfo(ctx);
            } catch (IllegalStateException e) {
                e.printStackTrace();
            } catch (GooglePlayServicesRepairableException e) {
                e.printStackTrace();
            } catch (IOException e) {
                e.printStackTrace();
            } catch (GooglePlayServicesNotAvailableException e) {
                e.printStackTrace();
            }
        });

        thr.start();
        synchronized (thr) {
            try {
                thr.join();
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }
}

When I'm calling getAdvertisingIdInfo method, application hangs forever (no matter with debugger or not).

I'm using Windows ADT 22.3, Android SDK API 19, Google Play SDK rev. 16, Android 4.4.2 Nexus devices. I'm integrating API as described here: https://developer.android.com/google/play-services/id.html

What could be a reason?

Upvotes: 12

Views: 8574

Answers (2)

Sojan P R
Sojan P R

Reputation: 546

Unfortunately the getAdvertisingIdInfo call needs to be done from a background thread, you should not block the main thread while invoking it. Seems like there is no option to get the AdId synchronously.

---- EDIT----

I could achieve this by running on a new thread. Here is what finally work for me. It is not hanging anymore (May not be an ideal)

getGAIDSync(){
 final CountDownLatch  latch  = new CountDownLatch(1);
 new Thread(new Runnable() {
            @Override
            public void run() {               
                getGoogleAdsIDAsyncTask.execute().get(5000, TimeUnit.MilliSecond);
               latch.countDown();                
    }}).start();
    try {
        latch.await(5000,TimeUnit.MILLISECONDS);
    } catch (InterruptedException e) {
         e.printStackTrace();
    }

}

Upvotes: 1

timba
timba

Reputation: 261

I found the reason. It shouldn't block onStart() handler because blocked context blocks Play API in ID settings obtaining. Fixed code looks like this:

@Override
protected void onStart() {
    super.onStart();
    Thread thr = new Thread(new Runnable() {
        @Override
        public void run() {
            try {
                Context ctx = MyActivity.this.getApplicationContext();
                AdvertisingIdClient.Info adInfo = AdvertisingIdClient.getAdvertisingIdInfo(ctx);
                finished(adInfo);
            } catch (...) {
                // All exceptions blocks
            }

            finished(null);
        }
    });

    thr.start();
}

private void finished(final AdvertisingIdClient.Info adInfo){
    if(adInfo!=null){
        // In case you need to use adInfo in UI thread
        runOnUiThread(new Runnable() {
            @Override
            public void run() {
                // Do some stuff with adInfo
            }
        });
    }
}

It would be helpful if official instructions had such usage comments.

Upvotes: 14

Related Questions