Bug 1130096 - Convert embedding/components/commandhandler/ to Gecko style. r=mccr8
This commit is contained in:
@@ -7,27 +7,23 @@
|
||||
#include "nsIControllerCommand.h"
|
||||
#include "nsControllerCommandTable.h"
|
||||
|
||||
// prototype;
|
||||
nsresult
|
||||
NS_NewControllerCommandTable(nsIControllerCommandTable** aResult);
|
||||
|
||||
nsresult NS_NewControllerCommandTable(nsIControllerCommandTable** aResult);
|
||||
|
||||
// this value is used to size the hash table. Just a sensible upper bound
|
||||
#define NUM_COMMANDS_LENGTH 32
|
||||
|
||||
#define NUM_COMMANDS_LENGTH 32
|
||||
|
||||
nsControllerCommandTable::nsControllerCommandTable()
|
||||
: mCommandsTable(NUM_COMMANDS_LENGTH)
|
||||
, mMutable(true)
|
||||
: mCommandsTable(NUM_COMMANDS_LENGTH)
|
||||
, mMutable(true)
|
||||
{
|
||||
}
|
||||
|
||||
|
||||
nsControllerCommandTable::~nsControllerCommandTable()
|
||||
{
|
||||
}
|
||||
|
||||
NS_IMPL_ISUPPORTS(nsControllerCommandTable, nsIControllerCommandTable, nsISupportsWeakReference)
|
||||
NS_IMPL_ISUPPORTS(nsControllerCommandTable, nsIControllerCommandTable,
|
||||
nsISupportsWeakReference)
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::MakeImmutable(void)
|
||||
@@ -37,7 +33,8 @@ nsControllerCommandTable::MakeImmutable(void)
|
||||
}
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::RegisterCommand(const char * aCommandName, nsIControllerCommand *aCommand)
|
||||
nsControllerCommandTable::RegisterCommand(const char* aCommandName,
|
||||
nsIControllerCommand* aCommand)
|
||||
{
|
||||
NS_ENSURE_TRUE(mMutable, NS_ERROR_FAILURE);
|
||||
|
||||
@@ -46,14 +43,13 @@ nsControllerCommandTable::RegisterCommand(const char * aCommandName, nsIControll
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::UnregisterCommand(const char * aCommandName, nsIControllerCommand *aCommand)
|
||||
nsControllerCommandTable::UnregisterCommand(const char* aCommandName,
|
||||
nsIControllerCommand* aCommand)
|
||||
{
|
||||
NS_ENSURE_TRUE(mMutable, NS_ERROR_FAILURE);
|
||||
|
||||
nsDependentCString commandKey(aCommandName);
|
||||
|
||||
if (!mCommandsTable.Get(commandKey, nullptr)) {
|
||||
return NS_ERROR_FAILURE;
|
||||
}
|
||||
@@ -62,66 +58,65 @@ nsControllerCommandTable::UnregisterCommand(const char * aCommandName, nsIContro
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::FindCommandHandler(const char * aCommandName, nsIControllerCommand **outCommand)
|
||||
nsControllerCommandTable::FindCommandHandler(const char* aCommandName,
|
||||
nsIControllerCommand** aResult)
|
||||
{
|
||||
NS_ENSURE_ARG_POINTER(outCommand);
|
||||
NS_ENSURE_ARG_POINTER(aResult);
|
||||
|
||||
*outCommand = nullptr;
|
||||
*aResult = nullptr;
|
||||
|
||||
nsCOMPtr<nsIControllerCommand> foundCommand;
|
||||
mCommandsTable.Get(nsDependentCString(aCommandName), getter_AddRefs(foundCommand));
|
||||
if (!foundCommand) return NS_ERROR_FAILURE;
|
||||
mCommandsTable.Get(nsDependentCString(aCommandName),
|
||||
getter_AddRefs(foundCommand));
|
||||
if (!foundCommand) {
|
||||
return NS_ERROR_FAILURE;
|
||||
}
|
||||
|
||||
foundCommand.forget(outCommand);
|
||||
foundCommand.forget(aResult);
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
|
||||
|
||||
/* boolean isCommandEnabled (in wstring command); */
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::IsCommandEnabled(const char * aCommandName, nsISupports *aCommandRefCon, bool *aResult)
|
||||
nsControllerCommandTable::IsCommandEnabled(const char* aCommandName,
|
||||
nsISupports* aCommandRefCon,
|
||||
bool* aResult)
|
||||
{
|
||||
NS_ENSURE_ARG_POINTER(aResult);
|
||||
|
||||
*aResult = false;
|
||||
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
if (!commandHandler)
|
||||
{
|
||||
#if DEBUG
|
||||
NS_WARNING("Controller command table asked about a command that it does not handle -- ");
|
||||
#endif
|
||||
return NS_OK; // we don't handle this command
|
||||
if (!commandHandler) {
|
||||
NS_WARNING("Controller command table asked about a command that it does "
|
||||
"not handle");
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
return commandHandler->IsCommandEnabled(aCommandName, aCommandRefCon, aResult);
|
||||
return commandHandler->IsCommandEnabled(aCommandName, aCommandRefCon,
|
||||
aResult);
|
||||
}
|
||||
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::UpdateCommandState(const char * aCommandName, nsISupports *aCommandRefCon)
|
||||
nsControllerCommandTable::UpdateCommandState(const char* aCommandName,
|
||||
nsISupports* aCommandRefCon)
|
||||
{
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
if (!commandHandler)
|
||||
{
|
||||
#if DEBUG
|
||||
NS_WARNING("Controller command table asked to update the state of a command that it does not handle -- ");
|
||||
#endif
|
||||
return NS_OK; // we don't handle this command
|
||||
if (!commandHandler) {
|
||||
NS_WARNING("Controller command table asked to update the state of a "
|
||||
"command that it does not handle");
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
return NS_ERROR_NOT_IMPLEMENTED;
|
||||
}
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::SupportsCommand(const char * aCommandName, nsISupports *aCommandRefCon, bool *aResult)
|
||||
nsControllerCommandTable::SupportsCommand(const char* aCommandName,
|
||||
nsISupports* aCommandRefCon,
|
||||
bool* aResult)
|
||||
{
|
||||
NS_ENSURE_ARG_POINTER(aResult);
|
||||
|
||||
@@ -129,7 +124,6 @@ nsControllerCommandTable::SupportsCommand(const char * aCommandName, nsISupports
|
||||
|
||||
*aResult = false;
|
||||
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
|
||||
@@ -137,55 +131,50 @@ nsControllerCommandTable::SupportsCommand(const char * aCommandName, nsISupports
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
/* void doCommand (in wstring command); */
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::DoCommand(const char * aCommandName, nsISupports *aCommandRefCon)
|
||||
nsControllerCommandTable::DoCommand(const char* aCommandName,
|
||||
nsISupports* aCommandRefCon)
|
||||
{
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
if (!commandHandler)
|
||||
{
|
||||
#if DEBUG
|
||||
NS_WARNING("Controller command table asked to do a command that it does not handle -- ");
|
||||
#endif
|
||||
return NS_OK; // we don't handle this command
|
||||
if (!commandHandler) {
|
||||
NS_WARNING("Controller command table asked to do a command that it does "
|
||||
"not handle");
|
||||
return NS_OK;
|
||||
}
|
||||
|
||||
return commandHandler->DoCommand(aCommandName, aCommandRefCon);
|
||||
}
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::DoCommandParams(const char *aCommandName, nsICommandParams *aParams, nsISupports *aCommandRefCon)
|
||||
nsControllerCommandTable::DoCommandParams(const char* aCommandName,
|
||||
nsICommandParams* aParams,
|
||||
nsISupports* aCommandRefCon)
|
||||
{
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
if (!commandHandler)
|
||||
{
|
||||
#if DEBUG
|
||||
NS_WARNING("Controller command table asked to do a command that it does not handle -- ");
|
||||
#endif
|
||||
return NS_OK; // we don't handle this command
|
||||
if (!commandHandler) {
|
||||
NS_WARNING("Controller command table asked to do a command that it does "
|
||||
"not handle");
|
||||
return NS_OK;
|
||||
}
|
||||
return commandHandler->DoCommandParams(aCommandName, aParams, aCommandRefCon);
|
||||
}
|
||||
|
||||
|
||||
NS_IMETHODIMP
|
||||
nsControllerCommandTable::GetCommandState(const char *aCommandName, nsICommandParams *aParams, nsISupports *aCommandRefCon)
|
||||
nsControllerCommandTable::GetCommandState(const char* aCommandName,
|
||||
nsICommandParams* aParams,
|
||||
nsISupports* aCommandRefCon)
|
||||
{
|
||||
// find the command
|
||||
nsCOMPtr<nsIControllerCommand> commandHandler;
|
||||
FindCommandHandler(aCommandName, getter_AddRefs(commandHandler));
|
||||
if (!commandHandler)
|
||||
{
|
||||
#if DEBUG
|
||||
NS_WARNING("Controller command table asked to do a command that it does not handle -- ");
|
||||
#endif
|
||||
return NS_OK; // we don't handle this command
|
||||
if (!commandHandler) {
|
||||
NS_WARNING("Controller command table asked to do a command that it does "
|
||||
"not handle");
|
||||
return NS_OK;
|
||||
}
|
||||
return commandHandler->GetCommandStateParams(aCommandName, aParams, aCommandRefCon);
|
||||
return commandHandler->GetCommandStateParams(aCommandName, aParams,
|
||||
aCommandRefCon);
|
||||
}
|
||||
|
||||
static PLDHashOperator
|
||||
@@ -205,7 +194,7 @@ nsControllerCommandTable::GetSupportedCommands(uint32_t* aCount,
|
||||
char*** aCommands)
|
||||
{
|
||||
char** commands =
|
||||
static_cast<char **>(moz_xmalloc(sizeof(char *) * mCommandsTable.Count()));
|
||||
static_cast<char**>(moz_xmalloc(sizeof(char*) * mCommandsTable.Count()));
|
||||
*aCount = mCommandsTable.Count();
|
||||
*aCommands = commands;
|
||||
|
||||
@@ -217,13 +206,11 @@ nsresult
|
||||
NS_NewControllerCommandTable(nsIControllerCommandTable** aResult)
|
||||
{
|
||||
NS_PRECONDITION(aResult != nullptr, "null ptr");
|
||||
if (! aResult)
|
||||
if (!aResult) {
|
||||
return NS_ERROR_NULL_POINTER;
|
||||
}
|
||||
|
||||
nsControllerCommandTable* newCommandTable = new nsControllerCommandTable();
|
||||
if (! newCommandTable)
|
||||
return NS_ERROR_OUT_OF_MEMORY;
|
||||
|
||||
NS_ADDREF(newCommandTable);
|
||||
*aResult = newCommandTable;
|
||||
return NS_OK;
|
||||
|
||||
Reference in New Issue
Block a user