Always initialize WindowFeatures.{x|y|width|height}

Source/core/page/WindowFeatures.cpp

 /*
  *  Copyright (C) 2000 Harri Porten (porten@kde.org)
  *  Copyright (C) 2006 Jon Shier (jshier@iastate.edu)
  *  Copyright (C) 2003, 2004, 2005, 2006, 2007, 2010 Apple Inc. All rights reseved.
  *  Copyright (C) 2006 Alexey Proskuryakov (ap@webkit.org)
  *
  *  This library is free software; you can redistribute it and/or
  *  modify it under the terms of the GNU Lesser General Public
  *  License as published by the Free Software Foundation; either
  *  version 2 of the License, or (at your option) any later version.
@@ skipping 19 matching lines @@
 
 namespace WebCore {
 
 // Though isspace() considers \t and \v to be whitespace, Win IE doesn't when parsing window features.
 static bool isWindowFeaturesSeparator(UChar c)
 {
     return c == ' ' || c == '\t' || c == '\n' || c == '\r' || c == '=' || c == ',' || c == '\0';
 }
 
 WindowFeatures::WindowFeatures(const String& features)
-    : xSet(false)
+    : x(0)
+    , xSet(false)
+    , y(0)
     , ySet(false)
+    , width(0)
     , widthSet(false)
+    , height(0)
     , heightSet(false)
+    , resizable(true)
     , fullscreen(false)
     , dialog(false)
 {
     /*
      The IE rule is: all features except for channelmode and fullscreen default to YES, but
      if the user specifies a feature string, all features default to NO. (There is no public
      standard that applies to this method.)
 
      <http://msdn.microsoft.com/workshop/author/dhtml/reference/methods/open_0.asp>
      We always allow a window to be resized, which is consistent with Firefox.
      */
 
-    if (features.length() == 0) {
+    if (features.isEmpty()) {
         menuBarVisible = true;
         statusBarVisible = true;
         toolBarVisible = true;
         locationBarVisible = true;
         scrollbarsVisible = true;
-        resizable = true;
         return;
     }
 
     menuBarVisible = false;
     statusBarVisible = false;
     toolBarVisible = false;
     locationBarVisible = false;
     scrollbarsVisible = false;
-    resizable = true;
 
     // Tread lightly in this code -- it was specifically designed to mimic Win IE's parsing behavior.
     int keyBegin, keyEnd;
     int valueBegin, valueEnd;
 
     int i = 0;
     int length = features.length();
     String buffer = features.lower();
     while (i < length) {
         // skip to first non-separator, but don't skip past the end of the string
@@ skipping 174 matching lines @@
         if (separatorPosition != kNotFound) {
             value = featureString.substring(separatorPosition + 1).stripWhiteSpace().lower();
             value = value.left(value.find(' '));
         }
 
         map.set(key, value);
     }
 }
 
 } // namespace WebCore