Remove wstring from TableModel....

chrome/browser/notifications/notification_exceptions_table_model.h

Index: chrome/browser/notifications/notification_exceptions_table_model.h
===================================================================
--- chrome/browser/notifications/notification_exceptions_table_model.h	(revision 70272)
+++ chrome/browser/notifications/notification_exceptions_table_model.h	(working copy)
@@ -9,6 +9,7 @@
 #include <string>

[viettrungluu, 2010/12/29 23:41:17]

I don't think it's needed.

(I don't think you need to include string16.h either. In case you are (or anyone
else is) wondering, my rule of thumb is to not include string16.h (for example)
if string16 is only used in a way which implies that string16.h must already be
included by one of our includes. In this case, we have only an override of a
method returning string16, so the superclass's header must have included
string16.h.

[Avi (use Gerrit), 2010/12/30 00:04:15]

Done.

 #include <vector>
 
+#include "base/compiler_specific.h"
 #include "chrome/browser/notifications/desktop_notification_service.h"
 #include "chrome/browser/remove_rows_table_model.h"
 #include "chrome/common/notification_observer.h"
@@ -26,9 +27,9 @@
   virtual void RemoveAll();
 
   // Overridden from TableModel:
-  virtual int RowCount();
-  virtual std::wstring GetText(int row, int column_id);
-  virtual void SetObserver(TableModelObserver* observer);
+  virtual int RowCount() OVERRIDE;
+  virtual string16 GetText(int row, int column_id) OVERRIDE;
+  virtual void SetObserver(TableModelObserver* observer) OVERRIDE;
 
   // Overridden from NotificationObserver:
   virtual void Observe(NotificationType type,