Add Extension Keybinding test for page actions. This is to facilitate an upcoming port of the PageA…

chrome/browser/ui/views/location_bar/location_bar_view.cc

Index: chrome/browser/ui/views/location_bar/location_bar_view.cc
===================================================================
--- chrome/browser/ui/views/location_bar/location_bar_view.cc	(revision 126791)
+++ chrome/browser/ui/views/location_bar/location_bar_view.cc	(working copy)
@@ -1041,7 +1041,7 @@
 
   WebContents* contents = GetWebContentsFromDelegate(delegate_);
   if (!page_action_views_.empty() && contents) {
-    GURL url = GURL(model_->GetText());
+    GURL url = browser()->GetSelectedWebContents()->GetURL();

[Finnur, 2012/03/15 13:49:29]

A note on this:

GetText() from model_ returns the current URL, but does so in a way that strips
http:// from it. This becomes a problem in the tests since the test URLs loaded
are...
GURL("http://127.0.0.1:20000/foo/bar.html") and stripped that becomes...
GURL("127.0.0.1:20000/foo/bar.html")
... which is not a URL that GURL can parse without assistance (thinks everything
in front of the colon is a scheme).

We could fix it with FixupURL but since we just want the actual URL we might as
well just call GetURL(), which returns the real thing.

I'll fix this in GTK when I port keybindings for page actions to GTK and
activate the test for GTK (I already verified it failed the same way there). I
need to see if Mac has the same problem (once keybindings are ported for Mac).

 
     for (PageActionViews::const_iterator i(page_action_views_.begin());
          i != page_action_views_.end(); ++i) {