FBTF: Move code from headers into cc files.

webkit/glue/form_field.h

Index: webkit/glue/form_field.h
diff --git a/webkit/glue/form_field.h b/webkit/glue/form_field.h
index 2a1ffcfeaf9536360950cbcc632e95e227cf29bc..56d89697a5293d556858fddd68f3d13a10d00985 100644
--- a/webkit/glue/form_field.h
+++ b/webkit/glue/form_field.h
@@ -22,6 +22,7 @@ class FormField {
             const string16& value,
             const string16& form_control_type,
             int size);
+  ~FormField();

[tony, 2010/08/27 20:24:48]

Nit: In the case of a non-virtual empty destructor, is it just for completeness?
 I imagine it would be faster to compile/link to not explicitly declare
(although marginally so).

[jamesr, 2010/08/27 20:28:44]

If it's not explicitly declared then you get the compiler's default version
(with definition).

[Elliot Glaysher, 2010/08/27 21:02:45]

This is not an empty destructor. The destructor, even though declared as {},
will make five function calls minimum.

[darin (slow to review), 2010/08/27 21:28:35]

Agreed.  This is a good case to move into the .cc file.  Just look at the member
vars!

 
   const string16& label() const { return label_; }
   const string16& name() const { return name_; }