mozilla
/
pjs
зеркало из https://github.com/mozilla/pjs.git
1
0
Форкнуть 0
Код Задачи Пакеты Проекты Релизы Вики Активность

bug 721654 - Crash in mozilla::AndroidBridge::HandleGeckoMessage, follow up to require a JNI env for nsJNIString constructor r=kats

This commit is contained in:
Brad Lassey 2012-04-16 18:31:05 -04:00
Родитель 891927a344
Коммит 5dede9de57
2 изменённых файлов: 11 добавлений и 11 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

20
widget/android/AndroidBridge.cpp
Просмотреть файл

@ -422,13 +422,13 @@ getHandlersFromStringArray(JNIEnv *aJNIEnv, jobjectArray jArr, jsize aLen,
nsString empty = EmptyString();
for (jsize i = 0; i < aLen; i+=4) {
nsJNIString name(
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i)));
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i)), aJNIEnv);
nsJNIString isDefault(
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 1)));
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 1)), aJNIEnv);
nsJNIString packageName(
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 2)));
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 2)), aJNIEnv);
nsJNIString className(
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 3)));
static_cast<jstring>(aJNIEnv->GetObjectArrayElement(jArr, i + 3)), aJNIEnv);
nsIHandlerApp* app = nsOSHelperAppService::
CreateAndroidHandlerApp(name, className, packageName,
className, aMimeType, aAction);
@ -561,7 +561,7 @@ AndroidBridge::GetMimeTypeFromExtensions(const nsACString& aFileExt, nsCString&
env->CallStaticObjectMethod(mGeckoAppShellClass,
jGetMimeTypeFromExtensions,
jstrExt));
nsJNIString jniStr(jstrType);
nsJNIString jniStr(jstrType, env);
aMimeType.Assign(NS_ConvertUTF16toUTF8(jniStr.get()));
}
@ -581,7 +581,7 @@ AndroidBridge::GetExtensionFromMimeType(const nsACString& aMimeType, nsACString&
env->CallStaticObjectMethod(mGeckoAppShellClass,
jGetExtensionFromMimeType,
jstrType));
nsJNIString jniStr(jstrExt);
nsJNIString jniStr(jstrExt, env);
aFileExt.Assign(NS_ConvertUTF16toUTF8(jniStr.get()));
}
@ -612,7 +612,7 @@ AndroidBridge::GetClipboardText(nsAString& aText)
jGetClipboardText));
if (!jstrType)
return false;
nsJNIString jniStr(jstrType);
nsJNIString jniStr(jstrType, env);
aText.Assign(jniStr);
return true;
}
@ -752,7 +752,7 @@ AndroidBridge::ShowFilePickerForExtensions(nsAString& aFilePath, const nsAString
jstring jstr = static_cast<jstring>(env->CallStaticObjectMethod(
mGeckoAppShellClass,
jShowFilePickerForExtensions, jstrFilers));
aFilePath.Assign(nsJNIString(jstr));
aFilePath.Assign(nsJNIString(jstr, env));
}
void
@ -770,7 +770,7 @@ AndroidBridge::ShowFilePickerForMimeType(nsAString& aFilePath, const nsAString&
jstring jstr = static_cast<jstring>(env->CallStaticObjectMethod(
mGeckoAppShellClass,
jShowFilePickerForMimeType, jstrFilers));
aFilePath.Assign(nsJNIString(jstr));
aFilePath.Assign(nsJNIString(jstr, env));
}
void
@ -1165,7 +1165,7 @@ AndroidBridge::GetStaticStringField(const char *className, const char *fieldName
if (!jstr)
return false;
result.Assign(nsJNIString(jstr));
result.Assign(nsJNIString(jstr, env));
return true;
}

2
widget/android/AndroidJavaWrappers.h
Просмотреть файл

@ -609,7 +609,7 @@ public:
class nsJNIString : public nsString
{
public:
nsJNIString(jstring jstr, JNIEnv *jenv = NULL);
nsJNIString(jstring jstr, JNIEnv *jenv);
};
}