[Toolbar Views] Move toolbar files to a new subdirectory.

chrome/browser/ui/views/toolbar/wrench_menu_observer.h

Index: chrome/browser/ui/views/toolbar/wrench_menu_observer.h
diff --git a/chrome/browser/ui/views/wrench_menu_observer.h b/chrome/browser/ui/views/toolbar/wrench_menu_observer.h
similarity index 58%
rename from chrome/browser/ui/views/wrench_menu_observer.h
rename to chrome/browser/ui/views/toolbar/wrench_menu_observer.h
index eccee388e8830339ba7b5e6c727d38b53b1f8d7b..d59ee8fd4cb1d3d6efa2eb65d4a73eb7a6a92136 100644
--- a/chrome/browser/ui/views/wrench_menu_observer.h
+++ b/chrome/browser/ui/views/toolbar/wrench_menu_observer.h
@@ -2,9 +2,10 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#ifndef CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_OBSERVER_H_
-#define CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_OBSERVER_H_
+#ifndef CHROME_BROWSER_UI_VIEWS_TOOLBAR_WRENCH_MENU_OBSERVER_H_
+#define CHROME_BROWSER_UI_VIEWS_TOOLBAR_WRENCH_MENU_OBSERVER_H_
 
+// TODO(gbillock): Make this an inner class of WrenchMenu. (even needed?)

[tfarina, 2013/11/22 22:21:30]

why did you add this TODO?

Quoting from chromium coding style:

"Prefer putting delegate classes in their own header files. Implementors of the
delegate interface will often be included elsewhere, which will often cause more
coupling with the header of the main class."

http://www.chromium.org/developers/coding-style - Code formatting section

We also discussed about this in https://codereview.chromium.org/79773005/

 class WrenchMenuObserver {
  public:
   // Invoked when the WrenchMenu is about to be destroyed (from its destructor).
@@ -14,4 +15,4 @@ class WrenchMenuObserver {
   virtual ~WrenchMenuObserver() {}
 };
 
-#endif  // CHROME_BROWSER_UI_VIEWS_WRENCH_MENU_OBSERVER_H_
+#endif  // CHROME_BROWSER_UI_VIEWS_TOOLBAR_WRENCH_MENU_OBSERVER_H_