From bcf05a69090f342d328f1537d1d83406b883290b Mon Sep 17 00:00:00 2001 From: Fabrice Di Meglio Date: Tue, 1 Nov 2011 17:53:34 -0700 Subject: Fix bug #5553401 TextLayoutCache is too verbose: "computeValuesWithHarfbuzz -- need to force to single run" - make single run case non verbose Change-Id: I5c3b87aeb613697233290ddecac3ca00f58f8313 --- core/jni/android/graphics/TextLayoutCache.cpp | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/core/jni/android/graphics/TextLayoutCache.cpp b/core/jni/android/graphics/TextLayoutCache.cpp index 3fdc79b3a90a..7db8abd39d68 100644 --- a/core/jni/android/graphics/TextLayoutCache.cpp +++ b/core/jni/android/graphics/TextLayoutCache.cpp @@ -444,7 +444,11 @@ void TextLayoutCacheValue::computeValuesWithHarfbuzz(SkPaint* paint, const UChar LOGD("computeValuesWithHarfbuzz -- dirFlags=%d run-count=%d paraDir=%d", dirFlags, rc, paraDir); #endif - if (!U_SUCCESS(status) || rc <= 1) { + if (U_SUCCESS(status) && rc == 1) { + // Normal case: one run, status is ok + isRTL = (paraDir == 1); + useSingleRun = true; + } else if (!U_SUCCESS(status) || rc < 1) { LOGW("computeValuesWithHarfbuzz -- need to force to single run"); isRTL = (paraDir == 1); useSingleRun = true; -- cgit v1.2.3-59-g8ed1b