Magic-Mirror/gecko-dev
1
0
Fork 0
mirror of https://github.com/mozilla/gecko-dev.git synced 2025-03-03 07:01:19 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Naked hostname isn't very effective for identifying dialog origin. b=334893 r=mento sr=dveditz

Browse Source
This commit is contained in:
joshmoz%gmail.com 2006-07-28 22:06:39 +00:00
parent 3204e26161
commit 7ad51b9919
4 changed files with 25 additions and 29 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
dom/src/base/nsGlobalWindow.cpp
View File

@ -3254,8 +3254,7 @@ nsGlobalWindow::SetTextZoom(float aZoom)
// static
void
nsGlobalWindow::MakeScriptDialogTitle(const nsAString &aInTitle,
nsAString &aOutTitle)
nsGlobalWindow::MakeScriptDialogTitle(nsAString &aOutTitle)
{
aOutTitle.Truncate();
@ -3293,10 +3292,14 @@ nsGlobalWindow::MakeScriptDialogTitle(const nsAString &aInTitle,
nsCAutoString prepath;
fixedURI->GetPrePath(prepath);
aOutTitle = NS_ConvertUTF8toUTF16(prepath);
if (!aInTitle.IsEmpty()) {
aOutTitle.Append(NS_LITERAL_STRING(" - ") + aInTitle);
}
NS_ConvertUTF8toUTF16 ucsPrePath(prepath);
const PRUnichar *formatStrings[] = { ucsPrePath.get() };
nsXPIDLString tempString;
nsContentUtils::FormatLocalizedString(nsContentUtils::eCOMMON_DIALOG_PROPERTIES,
"ScriptDlgHeading",
formatStrings, NS_ARRAY_LENGTH(formatStrings),
tempString);
aOutTitle = tempString;
}
}
}
@ -3308,28 +3311,18 @@ nsGlobalWindow::MakeScriptDialogTitle(const nsAString &aInTitle,
}
if (aOutTitle.IsEmpty()) {
// We didn't find a host so use the generic title modifier. Load
// the string to be prepended to titles for script
// confirm/alert/prompt boxes.
const nsAFlatString & flatTitle = PromiseFlatString(aInTitle);
const PRUnichar *formatStrings[] = { flatTitle.get() };
// We didn't find a host so use the generic heading
nsXPIDLString tempString;
nsContentUtils::FormatLocalizedString(
nsContentUtils::eCOMMON_DIALOG_PROPERTIES,
"ScriptDlgTitle",
formatStrings, NS_ARRAY_LENGTH(formatStrings),
tempString);
nsContentUtils::GetLocalizedString(nsContentUtils::eCOMMON_DIALOG_PROPERTIES,
"ScriptDlgGenericHeading",
tempString);
aOutTitle = tempString;
}
// Just in case
if (aOutTitle.IsEmpty()) {
NS_WARNING("could not get ScriptDlgTitle string from string bundle");
aOutTitle.AssignLiteral("[Script] ");
aOutTitle.Append(aInTitle);
NS_WARNING("could not get ScriptDlgGenericHeading string from string bundle");
aOutTitle.AssignLiteral("[Script]");
}
}
@ -3358,7 +3351,7 @@ nsGlobalWindow::Alert(const nsAString& aString)
EnsureReflowFlushAndPaint();
nsAutoString title;
MakeScriptDialogTitle(EmptyString(), title);
MakeScriptDialogTitle(title);
return prompter->Alert(title.get(), PromiseFlatString(*str).get());
}
@ -3383,7 +3376,7 @@ nsGlobalWindow::Confirm(const nsAString& aString, PRBool* aReturn)
EnsureReflowFlushAndPaint();
nsAutoString title;
MakeScriptDialogTitle(EmptyString(), title);
MakeScriptDialogTitle(title);
return prompter->Confirm(title.get(),
PromiseFlatString(aString).get(), aReturn);
@ -3394,6 +3387,8 @@ nsGlobalWindow::Prompt(const nsAString& aMessage, const nsAString& aInitial,
const nsAString& aTitle, PRUint32 aSavePassword,
nsAString& aReturn)
{
// We don't use "aTitle" because we ignore the 3rd (title) argument to
// prompt(). IE and Opera ignore it too. See Mozilla bug 334893.
SetDOMStringToNull(aReturn);
nsresult rv;
@ -3418,7 +3413,7 @@ nsGlobalWindow::Prompt(const nsAString& aMessage, const nsAString& aInitial,
EnsureReflowFlushAndPaint();
nsAutoString title;
MakeScriptDialogTitle(aTitle, title);
MakeScriptDialogTitle(title);
rv = prompter->Prompt(title.get(), PromiseFlatString(aMessage).get(), nsnull,
aSavePassword, PromiseFlatString(aInitial).get(),

3
dom/src/base/nsGlobalWindow.h
View File

@ -461,8 +461,7 @@ protected:
nsresult CheckSecurityLeftAndTop(PRInt32* left, PRInt32* top);
static PRBool CanSetProperty(const char *aPrefName);
static void MakeScriptDialogTitle(const nsAString &aInTitle,
nsAString &aOutTitle);
static void MakeScriptDialogTitle(nsAString &aOutTitle);
// Helper for window.find()
nsresult FindInternal(const nsAString& aStr, PRBool caseSensitive,

3
toolkit/locales/en-US/chrome/global/commonDialogs.properties
View File

@ -12,4 +12,5 @@ No=&No
Save=&Save
Revert=&Revert
DontSave=&Don't Save
ScriptDlgTitle=[JavaScript Application] %S
ScriptDlgGenericHeading=[JavaScript Application]
ScriptDlgHeading=The page at %S says:

3
xpfe/global/resources/locale/en-US/commonDialogs.properties
View File

@ -12,4 +12,5 @@ No=&No
Save=&Save
Revert=&Revert
DontSave=&Don't Save
ScriptDlgTitle=[JavaScript Application] %S
ScriptDlgGenericHeading=[JavaScript Application]
ScriptDlgHeading=The page at %S says: