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

Bug 1221449: Add null ptr check and fix variable shadowing in OSXNotificationCenter.mm. r=spohl

This commit is contained in:
Josh Aas 2015-11-07 18:23:00 +09:00
Родитель bbb0292ad2
Коммит 14354d55b1
1 изменённых файлов: 3 добавлений и 3 удалений
Показать статистику Скачать Patch файл Скачать Diff файл Показать все файлы Свернуть все файлы

6
widget/cocoa/OSXNotificationCenter.mm
Просмотреть файл

@ -247,9 +247,9 @@ OSXNotificationCenter::ShowAlertNotification(const nsAString & aImageUrl, const
nsAlertsUtils::GetSourceHostPort(aPrincipal, hostPort);
nsCOMPtr<nsIStringBundle> bundle;
nsCOMPtr<nsIStringBundleService> sbs = do_GetService(NS_STRINGBUNDLE_CONTRACTID);
nsresult rv = sbs->CreateBundle("chrome://alerts/locale/alert.properties", getter_AddRefs(bundle));
sbs->CreateBundle("chrome://alerts/locale/alert.properties", getter_AddRefs(bundle));
if (!hostPort.IsEmpty()) {
if (!hostPort.IsEmpty() && bundle) {
const char16_t* formatStrings[] = { hostPort.get() };
nsXPIDLString notificationSource;
bundle->FormatStringFromName(MOZ_UTF16("source.label"),
@ -265,7 +265,7 @@ OSXNotificationCenter::ShowAlertNotification(const nsAString & aImageUrl, const
// If this is not an application/extension alert, show additional actions dealing with permissions.
if (nsAlertsUtils::IsActionablePrincipal(aPrincipal)) {
if (NS_SUCCEEDED(rv)) {
if (bundle) {
nsXPIDLString closeButtonTitle, actionButtonTitle, disableButtonTitle, settingsButtonTitle;
bundle->GetStringFromName(MOZ_UTF16("closeButton.title"),
getter_Copies(closeButtonTitle));