Implement the Instant extended API startMargin, endMargin, and rtl properties and the onmarginchang…

chrome/browser/ui/omnibox/omnibox_view.h

Index: chrome/browser/ui/omnibox/omnibox_view.h
diff --git a/chrome/browser/ui/omnibox/omnibox_view.h b/chrome/browser/ui/omnibox/omnibox_view.h
index 692144f48fc5aeb8585e4c18ccab27b1dd8c29a8..cf9e9a71c4fe4f1d02cb5a5a9a339eaaaaea8e91 100644
--- a/chrome/browser/ui/omnibox/omnibox_view.h
+++ b/chrome/browser/ui/omnibox/omnibox_view.h
@@ -200,6 +200,9 @@ class OmniboxView {
   // Returns true if the user is composing something in an IME.
   virtual bool IsImeComposing() const = 0;
 
+  // Get the bounds of the widget containing the omnibox.
+  virtual gfx::Rect GetContainerBounds() const = 0;

[sky, 2012/12/04 23:06:36]

It's not clear what you mean by 'container' here, and your implementation isn't
really doing what the description says.

It seems you only use this in OmniboxEditModel::OnOmniboxBoundsChanged. Instead
of adding a new method make OnOmniboxBoundsChanged take the region you need
(this echoes my earlier comment).

[melevin, 2012/12/06 23:13:00]

I've removed both functions. The LocationBarView will now compute and pass the
margins directly to Instant.

+
 #if defined(TOOLKIT_VIEWS)
   virtual int GetMaxEditWidth(int entry_width) const = 0;