From 5b47dbfe85e8db4122089afa6d00264f1e973af2 Mon Sep 17 00:00:00 2001 From: Tom Sepez Date: Fri, 27 Feb 2015 13:48:38 -0800 Subject: Remove unused IDS_ constants from pdfium. Along the way, I noticed that one of these is used in a header file that isn't included anywhere and describes no actual code (JS_Console.h). Also add missing header guards, and IWYU to resources.h Also move a static function from a header to the new resources.cpp file. Also fix the grammar on some of the error messages. Most of these appear to be JS error messages destined for a JS error handler or (someday) a console (which can remain in english), and oddly, some appear to be about failed form validation, which smells user-facing. Also check that the message makes sense for the point the C++ code wants to use it, and reword accordingly. As it turns out, these currently don't make it back to JS due to the unimplemented JS_Error() function; this is an enabling step towards implementing it. R=thestig@chromium.org Review URL: https://codereview.chromium.org/955273003 --- BUILD.gn | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'BUILD.gn') diff --git a/BUILD.gn b/BUILD.gn index 3c6c74ab11..d51c563982 100644 --- a/BUILD.gn +++ b/BUILD.gn @@ -705,7 +705,6 @@ static_library("javascript") { "fpdfsdk/include/javascript/Icon.h", "fpdfsdk/include/javascript/IJavaScript.h", "fpdfsdk/include/javascript/JavaScript.h", - "fpdfsdk/include/javascript/JS_Console.h", "fpdfsdk/include/javascript/JS_Context.h", "fpdfsdk/include/javascript/JS_Define.h", "fpdfsdk/include/javascript/JS_EventHandler.h", @@ -735,6 +734,7 @@ static_library("javascript") { "fpdfsdk/src/javascript/JS_Value.cpp", "fpdfsdk/src/javascript/PublicMethods.cpp", "fpdfsdk/src/javascript/report.cpp", + "fpdfsdk/src/javascript/resource.cpp", "fpdfsdk/src/javascript/util.cpp", ] -- cgit v1.2.3