From 8b264ece4dde99afd8020013695e521ca634d5b8 Mon Sep 17 00:00:00 2001 From: Makoto Kato Date: Thu, 24 May 2018 20:46:18 +0900 Subject: [PATCH] Bug 1455006 - Use Looper to get main thread instead of main thread name. r=jchen Actually, Gecko uses thread name "main" to find main thread. But Android has Looper object to get main loop. So we should use it instead. MozReview-Commit-ID: 9oVqftqLZmh --HG-- extra : rebase_source : 84593b794f9055739a10a08ca2b4fa737043100c --- .../base/java/org/mozilla/gecko/GeckoJavaSampler.java | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/mobile/android/base/java/org/mozilla/gecko/GeckoJavaSampler.java b/mobile/android/base/java/org/mozilla/gecko/GeckoJavaSampler.java index 319eccec1117..314cf36099b0 100644 --- a/mobile/android/base/java/org/mozilla/gecko/GeckoJavaSampler.java +++ b/mobile/android/base/java/org/mozilla/gecko/GeckoJavaSampler.java @@ -5,6 +5,7 @@ package org.mozilla.gecko; +import android.os.Looper; import android.os.SystemClock; import android.util.Log; import android.util.SparseArray; @@ -78,14 +79,7 @@ public class GeckoJavaSampler { mSamplePos = 0; // Find the main thread - Set threadSet = Thread.getAllStackTraces().keySet(); - for (Thread t : threadSet) { - if (t.getName().compareToIgnoreCase("main") == 0) { - sMainThread = t; - break; - } - } - + sMainThread = Looper.getMainLooper().getThread(); if (sMainThread == null) { Log.e(LOGTAG, "Main thread not found"); return;