Enable compiler -Werror and -Wall on our webkit code for mac and linux.

This fixes some initialization order, removes unused varibles,
and fixes a couple signed/unsigned comparison errors.  Please
check the signed/unsigned cases.

We can't enable it for windows yet because MSVC complains
about headers pulled in from WebCore.  We'll have to wait
until the API is upstreamed.

BUG=21140

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


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

1 files changed, 2 insertions, 2 deletions

diff --git a/webkit/glue/glue_serialize.cc b/webkit/glue/glue_serialize.cc
index 7ec7f81..e0f662da 100644
--- a/webkit/glue/glue_serialize.cc
+++ b/webkit/glue/glue_serialize.cc
@@ -104,7 +104,7 @@ inline double ReadReal(const SerializeObject* obj) {
   const void* tmp;
   int length = 0;
   ReadData(obj, &tmp, &length);
-  if (length > 0 && length >= sizeof(0.0))
+  if (length > 0 && length >= static_cast<int>(sizeof(0.0)))
     return *static_cast<const double*>(tmp);
   else
     return 0.0;
@@ -319,7 +319,7 @@ static WebHistoryItem ReadHistoryItem(
   // The extra referrer string is read for backwards compat.
   const WebHTTPBody& http_body = ReadFormData(obj);
   const WebString& http_content_type = ReadString(obj);
-  const WebString& unused_referrer = ReadString(obj);
+  ALLOW_UNUSED const WebString& unused_referrer = ReadString(obj);
   if (include_form_data) {
     item.setHTTPBody(http_body);
     item.setHTTPContentType(http_content_type);