Restyle omnibox popup

chrome/browser/ui/views/omnibox/omnibox_view_delegate.h

Index: chrome/browser/ui/views/omnibox/omnibox_view_delegate.h
diff --git a/chrome/browser/ui/views/omnibox/omnibox_view_delegate.h b/chrome/browser/ui/views/omnibox/omnibox_view_delegate.h
new file mode 100644
index 0000000000000000000000000000000000000000..67660a9b6df217664eba6526f667eb9e1a4fe884
--- /dev/null
+++ b/chrome/browser/ui/views/omnibox/omnibox_view_delegate.h
@@ -0,0 +1,36 @@
+// Copyright (c) 2013 The Chromium Authors. All rights reserved.
+// 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_OMNIBOX_OMNIBOX_VIEW_DELEGATE_H_
+#define CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_VIEW_DELEGATE_H_
+
+#include "ui/gfx/native_widget_types.h"
+
+namespace gfx {
+class Point;
+}
+
+namespace ui {
+class ThemeProvider;
+}
+
+// Delegate interface implemented by the omnibox embedder to provide
+// positioning and rendering information for the popup.
+class OmniboxViewDelegate {

[Peter Kasting, 2013/06/06 18:52:16]

Since the only implementer of this is the LocationBarView, and all the classes
in question are views code that know about LocationBarView (it's in the same
source directory), it seems like we could just add these functions directly to
that class and then have the classes that access them take a LocationBarView*. 
Sure, that makes it less explicit which pieces of the LBV those classes will
need to access, but I think having fewer delegates/interfaces is worth that
tradeoff.

+ public:
+  // Returns the position and width that the popup should be, and also the
+  // left and right margins. The suggestions in the popup will be aligned
+  // inside the margins.
+  virtual void GetPopupPositioningInfo(gfx::Point* top_left_screen_coord,
+                                       int* popup_width,
+                                       int* left_margin,
+                                       int* right_margin) = 0;
+
+  virtual ui::ThemeProvider* GetThemeProviderForPopup() = 0;
+
+  // Returns the native widget to be the parent of the popup.

[Peter Kasting, 2013/06/06 18:52:16]

Nit: to be -> that is?

+  virtual gfx::NativeView GetPopupParent() = 0;
+};
+
+#endif  // CHROME_BROWSER_UI_VIEWS_OMNIBOX_OMNIBOX_VIEW_DELEGATE_H_