Refactor: Make PyAuto InstallExtension() take a string. Delete dead code.

chrome/common/automation_messages_internal.h

Index: chrome/common/automation_messages_internal.h
diff --git a/chrome/common/automation_messages_internal.h b/chrome/common/automation_messages_internal.h
index 4502a24a0b847b61ab0f5852fafc14e423b4f4ae..568eb8959a452d4cf2f8168a2205b1fae61360fc 100644
--- a/chrome/common/automation_messages_internal.h
+++ b/chrome/common/automation_messages_internal.h
@@ -1128,11 +1128,6 @@ IPC_SYNC_MESSAGE_CONTROL1_1(AutomationMsg_GetMetricEventDuration,
 IPC_MESSAGE_ROUTED1(AutomationMsg_RequestGoToHistoryEntryOffset,
                     int)   // numbers of entries (negative or positive)
 
-// Silently install the extension in the given crx file.

[kkania, 2011/08/03 16:01:56]

Deprecate this instead, as seen below. Otherwise you change the line number and
break the perf tests and chromedriver.

-IPC_SYNC_MESSAGE_CONTROL1_1(AutomationMsg_InstallExtension,
-                            FilePath /* full path to crx file */,
-                            AutomationMsg_ExtensionResponseValues)
-
 // DEPRECATED MESSAGE - But we must leave this comment and message so as
 // not to perturb line numbers (see comment at top of file re __LINE__).
 IPC_SYNC_MESSAGE_CONTROL1_1(AutomationMsg_DeprecatedMessageTwo,
@@ -1300,12 +1295,12 @@ IPC_SYNC_MESSAGE_CONTROL2_2(AutomationMsg_SendJSONRequest,
                             std::string /* JSON response */,
                             bool /* success */)
 
-// Installs an extension from the crx file and returns its id.
-// On error, |extension handle| will be 0.
-IPC_SYNC_MESSAGE_CONTROL2_1(AutomationMsg_InstallExtensionAndGetHandle,
-                            FilePath     /* full path to crx file */,
-                            bool         /* with UI */,
-                            int          /* extension handle */)
+// Installs an extension from a crx file or unpacked extension folder
+// and returns its id. On error, |extension handle| will be 0.
+IPC_SYNC_MESSAGE_CONTROL2_1(AutomationMsg_InstallExtension,
+                            std::string /* full path to crx or unpacked dir */,

[kkania, 2011/08/03 16:01:56]

hmm, in chromedriver we just started depending on this message a few days ago
for installing extensions. We used this instead of creating a JSON one because
then our users would need to wait for 2 months to get the feature.

Changing this will break older chromedriver's ability to work with newer
chromes, and will break newer chromedriver's ability to work with the current
stable/beta builds until this change makes it there.

you need to modify chrome/test/webdriver/automation.cc too, which was modified a
few days ago to use this func.

I see that using a string makes it easier for python clients. However, it also
makes it slightly more difficult for c++ clients. Now we have to convert from
file paths <-> std::string.

Sorry to twist your arm a bit, but do you think you can make a new JSON call
instead of changing this? If you want, I'd be more than happy to do it myself.

[dtu, 2011/08/03 19:31:53]

I can move the string -> FilePath conversion into pyautolib.cc so it doesn't
affect this signature. Is the rename of this message okay, or do you need to
maintain the original name?

[kkania, 2011/08/03 20:25:34]

Renaming is an ok approach too.

+                            bool        /* with UI */,
+                            int         /* extension handle */)
 
 // Waits for the next extension test result. Sets |test result| as the
 // received result and |message| as any accompanying message with the