Use BRANDING settings for IE CEEE. Simplify chrome_dll_version handling.

Adding new CEEE_PRODUCT_FULLNAME key to the BRANDING file. Adapting version.h.in to define a preprocessor constant for it. Using this and the other version/product name constants in IE CEEE's .rc files.

Piggybacking on MAD's recent changes to make %NAME% a variable in the .rgs files to fairly easily get the product name from branding in there. Naming the toolband with the product name directly (as it is what shows up in IE's View/Toolbars menu), the BHO with an added "Helper" and the broker with an added "Broker".

Removing the 'chrome_dll_version' target from chrome.gyp, and skipping code generation of chrome_dll_version.rc. Instead we have a source file with that name, that includes the verson.h generated by the 'chrome_version_header' target. This is cleaner and less code generation.

Fixing 'chrome_dll_nacl64' target, it seemed like it wanted to have version information, but it didn't.

Cleaning up the way sources are specified in several of the chrome_tests.gypi targets, to use fewer source! statements just to remove source files that are Windows-only (put them in the Windows-only section instead).

BUG=none
TEST=Visible change should be that user-visible names in Chromium builds are like "Chromium Extensions Execution Environment" (with "Broker" or "Helper" appended when appropriate), i.e. they lose the "Google" part.

Review URL: http://codereview.chromium.org/5360002

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@67457 0039d316-1c4b-4281-b951-d872f2087c98

9 files changed, 34 insertions, 22 deletions

diff --git a/ceee/ie/broker/broker.h b/ceee/ie/broker/broker.h
index 8953f95c..617fc8f 100644
--- a/ceee/ie/broker/broker.h
+++ b/ceee/ie/broker/broker.h
@@ -35,7 +35,7 @@ class ATL_NO_VTABLE CeeeBroker
   BEGIN_REGISTRY_MAP(CeeeBroker)
     REGMAP_UUID("CLSID", CLSID_CeeeBroker)
     REGMAP_UUID("LIBID", LIBID_CeeeBrokerLib)
-    REGMAP_ENTRY("NAME", "Google CEEE Broker")
+    REGMAP_RESOURCE("NAME", IDS_CEEE_BROKER_NAME)
   END_REGISTRY_MAP()
 
   DECLARE_NOT_AGGREGATABLE(CeeeBroker)
diff --git a/ceee/ie/broker/broker_module.rc b/ceee/ie/broker/broker_module.rc
index 43af9fe..2bb9dc4 100644
--- a/ceee/ie/broker/broker_module.rc
+++ b/ceee/ie/broker/broker_module.rc
@@ -52,13 +52,13 @@ BEGIN
     BEGIN
         BLOCK "040904e4"
         BEGIN
-            VALUE "CompanyName", "Google Inc."
-            VALUE "FileDescription", "Internet Explorer User Broker for Google CEEE"
+            VALUE "CompanyName", COMPANY_FULLNAME_STRING
+            VALUE "FileDescription", CEEE_PRODUCT_FULLNAME_STRING " Broker"
             VALUE "FileVersion", CHROME_VERSION_STRING
             VALUE "LegalCopyright", COPYRIGHT_STRING
             VALUE "InternalName", "ceee_broker.exe"
             VALUE "OriginalFilename", "ceee_broker.exe"
-            VALUE "ProductName", "Google Chrome Extensions Execution Environment"
+            VALUE "ProductName", CEEE_PRODUCT_FULLNAME_STRING
             VALUE "ProductVersion", CHROME_VERSION_STRING
         END
     END
@@ -86,6 +86,7 @@ IDR_BROKER                REGISTRY                "broker.rgs"
 STRINGTABLE
 BEGIN
     IDS_PROJNAME            "ceee_broker"
+    IDS_CEEE_BROKER_NAME    CEEE_PRODUCT_FULLNAME_STRING " Broker"
 END
 
 #endif    // English (U.S.) resources
diff --git a/ceee/ie/broker/resource.h b/ceee/ie/broker/resource.h
index 60c6977..4d0a67e 100644
--- a/ceee/ie/broker/resource.h
+++ b/ceee/ie/broker/resource.h
@@ -15,6 +15,7 @@
 #define IDS_PROJNAME                    100
 #define IDR_BROKER_MODULE               101
 #define IDR_BROKER                      102
+#define IDS_CEEE_BROKER_NAME            103
 
 
 // Next default values for new objects
@@ -23,7 +24,7 @@
 #define _APS_NEXT_RESOURCE_VALUE        201
 #define _APS_NEXT_COMMAND_VALUE         32768
 #define _APS_NEXT_CONTROL_VALUE         201
-#define _APS_NEXT_SYMED_VALUE           103
+#define _APS_NEXT_SYMED_VALUE           104
 #endif
 #endif
 
diff --git a/ceee/ie/plugin/bho/bho.gyp b/ceee/ie/plugin/bho/bho.gyp
index 5e6249f..2316aa4 100644
--- a/ceee/ie/plugin/bho/bho.gyp
+++ b/ceee/ie/plugin/bho/bho.gyp
@@ -26,6 +26,7 @@
         '../../../../chrome_frame/chrome_frame.gyp:chrome_tab_idl',
         # For the vtable patching stuff...
         '<(DEPTH)/chrome_frame/chrome_frame.gyp:chrome_frame_ie',
+        '<(DEPTH)/chrome/chrome.gyp:chrome_version_header',
       ],
       'sources': [
         'browser_helper_object.cc',
@@ -78,7 +79,7 @@
         '../../../../chrome/common/extensions/extension_resource.h',
       ],
       'include_dirs': [
-        # For chrome_tab.h
+        # For chrome_tab.h and version.h
         '<(SHARED_INTERMEDIATE_DIR)',
       ],
       'libraries': [
diff --git a/ceee/ie/plugin/bho/browser_helper_object.h b/ceee/ie/plugin/bho/browser_helper_object.h
index 8fc6584..c568108 100644
--- a/ceee/ie/plugin/bho/browser_helper_object.h
+++ b/ceee/ie/plugin/bho/browser_helper_object.h
@@ -34,6 +34,7 @@
 #include "ceee/ie/plugin/scripting/userscripts_librarian.h"
 #include "ceee/ie/plugin/scripting/content_script_native_api.h"
 #include "ceee/ie/plugin/toolband/resource.h"
+
 #include "broker_lib.h"  // NOLINT
 #include "toolband.h"  // NOLINT
 
@@ -56,7 +57,7 @@ class ATL_NO_VTABLE BrowserHelperObject
   DECLARE_REGISTRY_RESOURCEID_EX(IDR_BROWSERHELPEROBJECT)
   BEGIN_REGISTRY_MAP(BrowserHelperObject)
     REGMAP_UUID("CLSID", CLSID_BrowserHelperObject)
-    REGMAP_RESOURCE("NAME", IDS_CEEE_NAME)
+    REGMAP_RESOURCE("NAME", IDS_CEEE_BHO_NAME)
   END_REGISTRY_MAP()
 
   DECLARE_NOT_AGGREGATABLE(BrowserHelperObject)
diff --git a/ceee/ie/plugin/scripting/script_host.cc b/ceee/ie/plugin/scripting/script_host.cc
index e2faf42..15a57eb 100644
--- a/ceee/ie/plugin/scripting/script_host.cc
+++ b/ceee/ie/plugin/scripting/script_host.cc
@@ -13,8 +13,11 @@
 
 #include "base/logging.h"
 #include "base/string_util.h"
+#include "base/stringize_macros.h"
 #include "ceee/common/com_utils.h"
 
+#include "version.h"  // NOLINT
+
 #ifndef CMDID_SCRIPTSITE_URL
 
 // These are documented in MSDN, but not declared in the platform SDK.
@@ -523,7 +526,7 @@ STDMETHODIMP ScriptHost::OnScriptErrorDebug(IActiveScriptErrorDebug* err,
   // TODO(ericdingle@chromium.org): internationalization
   int ret = ::MessageBox(
       NULL, L"A script error occured. Do you want to debug?",
-      L"Google Chrome Extensions Execution Environment",
+      TO_L_STRING(CEEE_PRODUCT_FULLNAME_STRING),
       MB_ICONERROR | MB_SETFOREGROUND | MB_TASKMODAL | MB_YESNO);
   *enter_debugger = (ret == IDYES);
   *call_on_script_err_when_continuing = FALSE;
diff --git a/ceee/ie/plugin/scripting/scripting.gyp b/ceee/ie/plugin/scripting/scripting.gyp
index 0cfa271..10e42f7 100644
--- a/ceee/ie/plugin/scripting/scripting.gyp
+++ b/ceee/ie/plugin/scripting/scripting.gyp
@@ -16,12 +16,13 @@
       'type': 'static_library',
       'dependencies': [
         'javascript_bindings',
-        '../../common/common.gyp:ie_common',
-        '../../common/common.gyp:ie_common_settings',
-        '../toolband/toolband.gyp:toolband_idl',
-        '../../../../base/base.gyp:base',
-        '../../../../ceee/common/common.gyp:ceee_common',
-        '../../../../ceee/common/common.gyp:initializing_coclass',
+        '<(DEPTH)/base/base.gyp:base',
+        '<(DEPTH)/ceee/common/common.gyp:ceee_common',
+        '<(DEPTH)/ceee/common/common.gyp:initializing_coclass',
+        '<(DEPTH)/ceee/ie/common/common.gyp:ie_common',
+        '<(DEPTH)/ceee/ie/common/common.gyp:ie_common_settings',
+        '<(DEPTH)/ceee/ie/plugin/toolband/toolband.gyp:toolband_idl',
+        '<(DEPTH)/chrome/chrome.gyp:chrome_version_header',
       ],
       'sources': [
         'base.js',
@@ -38,6 +39,10 @@
         'userscripts_librarian.h',
         'userscripts_docs.h',
       ],
+      'include_dirs': [
+        # For version.h
+        '<(SHARED_INTERMEDIATE_DIR)',
+      ],
     },
     {
       'target_name': 'javascript_bindings',
diff --git a/ceee/ie/plugin/toolband/resource.h b/ceee/ie/plugin/toolband/resource.h
index b3856e5..f9af331 100644
--- a/ceee/ie/plugin/toolband/resource.h
+++ b/ceee/ie/plugin/toolband/resource.h
@@ -21,16 +21,16 @@
 #define IDS_CEEE_NAME                   109
 #define IDS_CEEE_EXECUTOR_CREATOR_NAME  110
 #define IDS_CEEE_EXECUTOR_NAME          111
-
+#define IDS_CEEE_BHO_NAME               112
 
 // Next default values for new objects
 //
 #ifdef APSTUDIO_INVOKED
 #ifndef APSTUDIO_READONLY_SYMBOLS
-#define _APS_NEXT_RESOURCE_VALUE        112
+#define _APS_NEXT_RESOURCE_VALUE        113
 #define _APS_NEXT_COMMAND_VALUE         32768
 #define _APS_NEXT_CONTROL_VALUE         201
-#define _APS_NEXT_SYMED_VALUE           112
+#define _APS_NEXT_SYMED_VALUE           113
 #endif
 #endif
 
diff --git a/ceee/ie/plugin/toolband/toolband.rc b/ceee/ie/plugin/toolband/toolband.rc
index 9a2609a..96a63b0 100644
--- a/ceee/ie/plugin/toolband/toolband.rc
+++ b/ceee/ie/plugin/toolband/toolband.rc
@@ -55,15 +55,14 @@ BEGIN
     BEGIN
         BLOCK "040904e4"
         BEGIN
-            VALUE "CompanyName", "Google Inc."
-            VALUE "FileDescription", "Google Chrome Extensions Execution Environment for IE."
+            VALUE "CompanyName", COMPANY_FULLNAME_STRING
+            VALUE "FileDescription", CEEE_PRODUCT_FULLNAME_STRING
             VALUE "FileVersion", CHROME_VERSION_STRING
             VALUE "LegalCopyright", COPYRIGHT_STRING
             VALUE "InternalName", "ceee_ie.dll"
             VALUE "OriginalFilename", "ceee_ie.dll"
-            VALUE "ProductName", "Google Chrome Extensions Execution Environment"
+            VALUE "ProductName", CEEE_PRODUCT_FULLNAME_STRING
             VALUE "ProductVersion", CHROME_VERSION_STRING
-
         END
     END
     BLOCK "VarFileInfo"
@@ -98,9 +97,10 @@ IDR_GREASEMONKEY_API_JS BINDATA                 "chrome\\renderer\\resources\\gr
 STRINGTABLE
 BEGIN
     IDS_PROJNAME                    "IE"
-    IDS_CEEE_NAME                   "Google Chrome Extensions Execution Environment"
+    IDS_CEEE_NAME                   CEEE_PRODUCT_FULLNAME_STRING
     IDS_CEEE_EXECUTOR_CREATOR_NAME  "Google CEEE Executor Creator"
     IDS_CEEE_EXECUTOR_NAME          "Google CEEE Executor"
+    IDS_CEEE_BHO_NAME               CEEE_PRODUCT_FULLNAME_STRING " Helper"
 END
 
 #endif    // English (U.S.) resources