Remove extension automation support that was used only by CEEE.

chrome_frame/chrome_tab.idl

Index: chrome_frame/chrome_tab.idl
diff --git a/chrome_frame/chrome_tab.idl b/chrome_frame/chrome_tab.idl
index 3ec0236682a62f99ea8073306a0b7f3ee23648b9..166e9ef28ba29e050670fed5c4a08318fad053e9 100644
--- a/chrome_frame/chrome_tab.idl
+++ b/chrome_frame/chrome_tab.idl
@@ -11,7 +11,7 @@ import "ocidl.idl";
 
 [
   object,
-  uuid(FB243E4B-8AC2-4840-95F2-91B9AF9CFF10),
+  uuid(25248626-A306-4857-A13C-4AD84FF15E10),

[amit, 2011/04/05 18:00:24]

Tommi pointed out that any existing users embedding GCF will break with this
change.  Maybe we keep the methods (perhaps rename them) and return E_NOTIMPL?

[Jói, 2011/04/05 18:47:10]

OK, will address this and upload a new version soon.

Just to confirm:  This is only an issue for IE, right?  Just removing the
Firefox methods should be fine, it will generate an error only for those who try
to use them.

[amit, 2011/04/05 18:50:17]

Yes, that's perfect. Also, please double check that the other artifacts like
typelib are still compatible.
Thanks!

[Jói, 2011/04/05 19:24:45]

I reverted the IID change here and on the dispinterface, and left the three
deprecated methods in (they return E_NOTIMPL in chrome_frame_activex_base.h).

   dual,
   nonextensible,
   helpstring("IChromeFrame Interface"),
@@ -66,18 +66,6 @@ interface IChromeFrame : IDispatch {
   [propput, id(11)]
   HRESULT useChromeNetwork([in] VARIANT_BOOL newVal);
 
-  [id(12), hidden]
-  // This method is available only when the control is in privileged mode.
-  HRESULT installExtension([in] BSTR crx_path);
-
-  [id(13), hidden]
-  // This method is available only when the control is in privileged mode.
-  HRESULT loadExtension([in] BSTR extension_path);
-
-  [id(14), hidden]
-  // This method is available only when the control is in privileged mode.
-  HRESULT getEnabledExtensions();
-
   [id(15)]
   // This method bootstraps the BHO if it is not already loaded.
   HRESULT registerBhoIfNeeded();
@@ -85,20 +73,7 @@ interface IChromeFrame : IDispatch {
 
 [
   object,
-  uuid(851bedec-4b2c-4959-abc0-a53403117257),
-  local,  // This interface should never be marshaled.
-  hidden,
-]
-// Internal implementation interface. Not intended as an API. May change
-// frequently, don't treat this as frozen.
-interface IChromeFrameInternal : IUnknown {
-  // This method is available only when the control is in privileged mode.
-  HRESULT getSessionId([out, retval] int* session_id);
-};
-
-[
-  object,
-  uuid(B1B52A4D-B22E-489f-8CCD-1CF9166FA90E),
+  uuid(E98FDFD9-312B-444a-A640-8E88F3CC08B8),
   oleautomation,
   nonextensible,
   hidden,
@@ -111,10 +86,6 @@ interface IChromeFramePrivileged : IUnknown {
   HRESULT GetWantsPrivileged([out] boolean *wants_privileged);
   // The profile name we want to use.
   HRESULT GetChromeProfileName([out] BSTR *profile_name);
-  // The comma-separated list of extension API functions you wish to automate.
-  // Return S_FALSE to leave the default, which is to not automate any 
-  // functions.
-  HRESULT GetExtensionApisToAutomate([out] BSTR *extension_apis);
   // Called when an automation version mismatch occurs. Returns S_OK if
   // a dialog should be showed to the user by this CF instance, S_FALSE if
   // not.
@@ -131,8 +102,6 @@ typedef enum {
   CF_EVENT_DISPID_ONLOADERROR,
   CF_EVENT_DISPID_ONMESSAGE,
   CF_EVENT_DISPID_ONPRIVATEMESSAGE,
-  CF_EVENT_DISPID_ONEXTENSIONREADY,
-  CF_EVENT_DISPID_ONGETENABLEDEXTENSIONSCOMPLETE,
   CF_EVENT_DISPID_ONCHANNELERROR,
   CF_EVENT_DISPID_ONCLOSE,
   CF_EVENT_DISPID_ONREADYSTATECHANGED = DISPID_READYSTATECHANGE,
@@ -146,7 +115,7 @@ typedef enum {
 library ChromeTabLib {
   importlib("stdole2.tlb");
 
-  [uuid(388B5D64-CE67-415b-9B0F-745C56E868E7)]
+  [uuid(812C601D-D787-49f2-BF05-6A460B44D2B3)]

[Sigurður Ásgeirsson, 2011/04/05 16:57:16]

I don't think this is necessary. As the interface here is a dispinterface.
There's no harm, that I can think of, however.

[Jói, 2011/04/05 17:04:17]

Yeah, I didn't think so either.  Just going by the book, so to speak :)

[Jói, 2011/04/05 19:24:45]

To be on the safe side (to leave the type library compatible with any clients
out there) I reverted the change of IID.

   dispinterface DIChromeFrameEvents {
    properties:
     // None.
@@ -168,17 +137,6 @@ library ChromeTabLib {
     // This event is only fired when the control is in privileged mode.
     void onprivatemessage([in] IDispatch* event, [in] BSTR target);
 
-    [id(CF_EVENT_DISPID_ONEXTENSIONREADY)]
-    // This event is only fired when the control is in privileged mode.
-    // response is one of AutomationMsg_ExtensionResponseValues.
-    void onextensionready([in] BSTR path, [in] long response);
-
-    [id(CF_EVENT_DISPID_ONGETENABLEDEXTENSIONSCOMPLETE)]
-    // This event is only fired when the control is in privileged mode.
-    // extension_paths is an array of BSTRs of the base directories of
-    // enabled extensions.
-    void ongetenabledextensionscomplete([in] SAFEARRAY(BSTR) extension_paths);
-
     [id(CF_EVENT_DISPID_ONCHANNELERROR)]
     // This event is fired when there is an error in communication channel with
     // Chrome and Automation must be reconnected to continue.