Cleanup everything but net/ for our first clang plugins.

webkit/glue/webmenuitem.h

Index: webkit/glue/webmenuitem.h
diff --git a/webkit/glue/webmenuitem.h b/webkit/glue/webmenuitem.h
index c767bea7f7a7cfebd81db9d436457c3365f21456..c2e5f1d4eb654b83556be4edd933d880b3f2293d 100644
--- a/webkit/glue/webmenuitem.h
+++ b/webkit/glue/webmenuitem.h
@@ -21,23 +21,17 @@ struct WebMenuItem {
     SUBMENU  // This is currently only used by Pepper, not by WebKit.
   };
 
+  WebMenuItem();
+  WebMenuItem(const WebKit::WebMenuItemInfo& item);
+  WebMenuItem(const WebMenuItem& item);

[Nico, 2011/02/02 01:00:23]

This one is new?

[Elliot Glaysher, 2011/02/02 01:09:26]

Correct. std::vector<> calls the copy constructor and if we're publicly exposing
a std::vector<WebMenuItem>, we'll probably want to use this instead of the
compiler's autogenerated one.

+  ~WebMenuItem();
+
   string16 label;
   Type type;
   unsigned action;
   bool enabled;
   bool checked;
   std::vector<WebMenuItem> submenu;
-
-  WebMenuItem() : type(OPTION), action(0), enabled(false), checked(false) {
-  }
-
-  WebMenuItem(const WebKit::WebMenuItemInfo& item)
-      : label(item.label),
-        type(static_cast<Type>(item.type)),
-        action(item.action),
-        enabled(item.enabled),
-        checked(item.checked) {
-  }
 };
 
 #endif  // WEBMENUITEM_H_