Move execute_code_function to extensions.

extensions/common/constants.cc

Index: extensions/common/constants.cc
diff --git a/extensions/common/constants.cc b/extensions/common/constants.cc
index c9fe31c3a45fb29772c09eb99c23b5a0247e8ff1..d2f0e312ac74e107770bbead923a98b10dc515d5 100644
--- a/extensions/common/constants.cc
+++ b/extensions/common/constants.cc
@@ -57,6 +57,11 @@ const char kWebStoreAppId[] = "ahfgeienlihckogmohjhadlkjgocpleb";
 const char kMimeTypeJpeg[] = "image/jpeg";
 const char kMimeTypePng[] = "image/png";
 
+const char kNoCodeOrFileToExecuteError[] = "No source code or file specified.";
+const char kMoreThanOneValuesError[] = "Code and file should not be specified "

[Ken Rockot(use gerrit already), 2014/09/04 22:54:47]

Could you fix this constant name now that it's moving to general extensions
constants? How about kCodeAndFileNotAllowedError or something. And maybe the
text could be clarified "... should not both be specified."


Actually, on second look, are any of these constants referenced outside of
execute_code_function.cc? If not, please just move them into that file in an
anonymous namespace.

[Fady Samuel, 2014/09/04 22:59:59]

I don't want to block this CL but should these strings be internationalized?

+    "at the same time in the second argument.";
+const char kLoadFileError[] = "Failed to load file: \"*\". ";
+
 }  // namespace extensions
 
 namespace extension_misc {