Changeset 48164 in vbox for trunk/src/VBox/Main

Timestamp:

Aug 29, 2013 4:35:14 PM (11 years ago)

Author:

vboxsync

Message:

Main: phase out the use of the XPCOM glue (wasn't ever used), and rename tstVBoxAPILinux to tstVBoxAPIXPCOM to match the content
libs/xpcom: strip down the standalone XPCOM glue code and disable building it as so far no one uses it

Location:

trunk/src/VBox/Main

Files:

• glue/initterm.cpp (modified) (3 diffs)
• src-server/xpcom/server.cpp (modified) (2 diffs)
• testcase/Makefile.kmk (modified) (3 diffs)
• testcase/makefile.tstVBoxAPIXPCOM (moved) (moved from trunk/src/VBox/Main/testcase/makefile.tstVBoxAPILinux ) (3 diffs)
• testcase/tstVBoxAPIXPCOM.cpp (moved) (moved from trunk/src/VBox/Main/testcase/tstVBoxAPILinux.cpp ) (3 diffs)

trunk/src/VBox/Main/glue/initterm.cpp

@@ -24 +24 @@
 
 # include <stdlib.h>
-
-  /* XPCOM_GLUE is defined when the client uses the standalone glue
-   * (i.e. dynamically picks up the existing XPCOM shared library installation).
-   * This is not the case for VirtualBox XPCOM clients (they are always
-   * distributed with the self-built XPCOM library, and therefore have a binary
-   * dependency on it) but left here for clarity.
-   */
-# if defined(XPCOM_GLUE)
-#  include <nsXPCOMGlue.h>
-# endif
 
 # include <nsIComponentRegistrar.h>
@@ -355 +345 @@
     LogFlowFunc(("component registry  : \"%s\"\n", szCompReg));
     LogFlowFunc(("XPTI data file      : \"%s\"\n", szXptiDat));
-
-#if defined (XPCOM_GLUE)
-    XPCOMGlueStartup(nsnull);
-#endif
 
     static const char *kAppPathsToProbe[] =
@@ -584 +570 @@
                 Assert(wasInited == true);
                 NOREF(wasInited);
-
-# if defined (XPCOM_GLUE)
-                XPCOMGlueShutdown();
-# endif
             }
         }

trunk/src/VBox/Main/src-server/xpcom/server.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2004-2012 Oracle Corporation
+ * Copyright (C) 2004-2013 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -20 +20 @@
 
 #include <nsIComponentRegistrar.h>
-
-#ifdef XPCOM_GLUE
-# include <nsXPCOMGlue.h>
-#endif
 
 #include <nsEventQueueUtils.h>

trunk/src/VBox/Main/testcase/Makefile.kmk

@@ -28 +28 @@
         tstAPI \
         $(if $(VBOX_OSE),,tstOVF) \
-        $(if $(VBOX_WITH_XPCOM),tstVBoxAPILinux,tstVBoxAPIWin) \
+        $(if $(VBOX_WITH_XPCOM),tstVBoxAPIXPCOM,tstVBoxAPIWin) \
         $(if $(VBOX_WITH_RESOURCE_USAGE_API),tstCollector,) \
         $(if $(VBOX_WITH_GUEST_CONTROL),tstGuestCtrlParseBuffer,) \
@@ -55 +55 @@
 samplesXPCOM_MODE = a+r,u+w
 samplesXPCOM_INST = $(INST_SDK)bindings/xpcom/samples/
-samplesXPCOM_SOURCES = tstVBoxAPILinux.cpp makefile.tstVBoxAPILinux=>Makefile
+samplesXPCOM_SOURCES = tstVBoxAPIXPCOM.cpp makefile.tstVBoxAPIXPCOM=>Makefile
 
 
@@ -89 +89 @@
 
 #
-# tstVBoxAPILinux
+# tstVBoxAPIXPCOM
 #
 # We only build the testcase here to make sure it builds.
 # It comes with a custom makefile which should be tested as well!
 #
-tstVBoxAPILinux_TEMPLATE = VBOXMAINCLIENTTSTEXE
-tstVBoxAPILinux_SOURCES  = tstVBoxAPILinux.cpp
+tstVBoxAPIXPCOM_TEMPLATE = VBOXMAINCLIENTTSTEXE
+tstVBoxAPIXPCOM_SOURCES  = tstVBoxAPIXPCOM.cpp
 
 

trunk/src/VBox/Main/testcase/makefile.tstVBoxAPIXPCOM

@@ -1 +1 @@
 #
-# tstVBoxAPILinux makefile
+# tstVBoxAPIXPCOM makefile
 #
 #
-# Copyright (C) 2006-2010 Oracle Corporation
+# Copyright (C) 2006-2013 Oracle Corporation
 #
 # This file is part of VirtualBox Open Source Edition (OSE), as
@@ -36 +36 @@
 # Link with the public XPCOM libraries
 #
-tstVBoxAPILinux: tstVBoxAPILinux.o
+tstVBoxAPIXPCOM: tstVBoxAPIXPCOM.o
         g++ -g -o $@ $^ \
                 $(PATH_BIN)/VBoxXPCOM.so \
@@ -42 +42 @@
                 -ldl -lpthread
 
-tstVBoxAPILinux.o: tstVBoxAPILinux.cpp
-        g++ -c -DRT_OS_LINUX -g -fshort-wchar $(addprefix -I, $(INCS_XPCOM)) $(addprefix -D, $(DEFS_XPCOM)) -o $@ tstVBoxAPILinux.cpp
+tstVBoxAPIXPCOM.o: tstVBoxAPIXPCOM.cpp
+        g++ -c -g -fshort-wchar $(addprefix -I, $(INCS_XPCOM)) $(addprefix -D, $(DEFS_XPCOM)) -o $@ tstVBoxAPIXPCOM.cpp
 
 clean:
-        rm -f tstVBoxAPILinux tstVBoxAPILinux.o
+        rm -f tstVBoxAPIXPCOM tstVBoxAPIXPCOM.o

trunk/src/VBox/Main/testcase/tstVBoxAPIXPCOM.cpp

@@ -67 +67 @@
  * Include the XPCOM headers
  */
-
-#if defined(XPCOM_GLUE)
-#include <nsXPCOMGlue.h>
-#endif
-
 #include <nsMemory.h>
 #include <nsString.h>
@@ -472 +467 @@
      * What we do is just follow the required steps to get an instance
      * of our main interface, which is IVirtualBox.
-     */
-#if defined(XPCOM_GLUE)
-    XPCOMGlueStartup(nsnull);
-#endif
-
-    /*
+     *
      * Note that we scope all nsCOMPtr variables in order to have all XPCOM
      * objects automatically released before we call NS_ShutdownXPCOM at the
@@ -578 +568 @@
      */
     NS_ShutdownXPCOM(nsnull);
-#if defined(XPCOM_GLUE)
-    XPCOMGlueShutdown();
-#endif
     RTPrintf("Done!\n");
     return 0;