Strip leading "javascript:" schemas from text pasted or dropped into the omnibox.

chrome/browser/ui/omnibox/omnibox_view.h

Index: chrome/browser/ui/omnibox/omnibox_view.h
===================================================================
--- chrome/browser/ui/omnibox/omnibox_view.h	(revision 85569)
+++ chrome/browser/ui/omnibox/omnibox_view.h	(working copy)
@@ -15,8 +15,11 @@
 #include <string>
 
 #include "base/string16.h"
+#include "base/string_util.h"
+#include "base/utf_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "content/common/page_transition_types.h"
+#include "content/common/url_constants.h"
 #include "ui/gfx/native_widget_types.h"
 #include "webkit/glue/window_open_disposition.h"
 
@@ -25,13 +28,15 @@
 class GURL;
 class TabContents;
 
+namespace views {
 #if defined(TOOLKIT_VIEWS)
-namespace views {
 class DropTargetEvent;
 class View;
-}  // namespace views
 #endif
 
+string16 StripJavascriptSchemas(const string16&);

[Peter Kasting, 2011/05/24 00:15:18]

?? Why did you move this here?  It belonged in OmniboxView where you had it
before.

[Cris Neckar, 2011/05/24 05:23:33]

?? It was never in OmniboxView. It can't be as I need to call it from outside of
that context (specifically toolbar_controller.mm). Check the previous patches.

[Peter Kasting, 2011/05/24 17:40:47]

I don't understand why that matters.  I think this should be a public static
function in OmniboxView.  That's callable everywhere.

It definitely shouldn't be in the views namespace, as that's not used on Mac.

[Cris Neckar, 2011/05/24 18:48:11]

Done.

+}  // namespace views
+
 class OmniboxView {
  public:
   // Used by the automation system for getting at the model from the view.