Make OmniboxEditModel::UpdatePermanentText() fetch the new text itself rather

chrome/browser/ui/omnibox/omnibox_view_browsertest.cc

Index: chrome/browser/ui/omnibox/omnibox_view_browsertest.cc
===================================================================
--- chrome/browser/ui/omnibox/omnibox_view_browsertest.cc	(revision 220446)
+++ chrome/browser/ui/omnibox/omnibox_view_browsertest.cc	(working copy)
@@ -30,6 +30,7 @@
 #include "chrome/browser/ui/omnibox/omnibox_popup_model.h"
 #include "chrome/browser/ui/omnibox/omnibox_view.h"
 #include "chrome/browser/ui/tabs/tab_strip_model.h"
+#include "chrome/browser/ui/toolbar/test_toolbar_model.h"
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/url_constants.h"
 #include "chrome/test/base/in_process_browser_test.h"
@@ -1672,17 +1673,21 @@
 #endif  // defined(TOOLKIT_VIEWS)
 
 IN_PROC_BROWSER_TEST_F(OmniboxViewTest, CopyURLToClipboard) {
+  // Set permanent text thus making sure that omnibox treats 'google.com'
+  // as URL (not as ordinary user input).
+  TestToolbarModel* toolbar_model = new TestToolbarModel;

[msw, 2013/08/30 16:12:16]

Remove |toolbar_model|, rename |toolbar_model_deleter| to |toolbar_model|.

[Peter Kasting, 2013/08/30 18:26:48]

I need a TestToolbarModel type to call set_text(), which doesn't exist on
ToolbarModel.

I could do something like this:

  scoped_ptr<ToolbarModel> toolbar_model(new TestToolbarModel());
  static_cast<TestToolbarModel*>(toolbar_model.get())->set_text(...);

...But I decided that was ugly compared to the current method; casting to
something you _just had_ seems lame.

So I'd rather leave it the current way.

[msw, 2013/08/30 18:32:26]

Ah, I didn't realize it was using test-specific API and I agree. Can you rename
|toolbar_model| to |test_toolbar_model| and then rename |toolbar_model_deleter|
to |toolbar_model|? 'Deleter' seem really odd.

+  scoped_ptr<ToolbarModel> toolbar_model_deleter(toolbar_model);
+  toolbar_model->set_text(ASCIIToUTF16("http://www.google.com/"));
+  browser()->swap_toolbar_models(&toolbar_model_deleter);
+  OmniboxEditModel* edit_model = omnibox_view->model();
+  ASSERT_NE(static_cast<OmniboxEditModel*>(NULL), edit_model);
+  edit_model->UpdatePermanentText();
+
   OmniboxView* omnibox_view = NULL;
   ASSERT_NO_FATAL_FAILURE(GetOmniboxView(&omnibox_view));
   const char* target_url = "http://www.google.com/calendar";
   omnibox_view->SetUserText(ASCIIToUTF16(target_url));
 
-  // Set permanent text thus making sure that omnibox treats 'google.com'
-  // as URL (not as ordinary user input).
-  OmniboxEditModel* edit_model = omnibox_view->model();
-  ASSERT_NE(static_cast<OmniboxEditModel*>(NULL), edit_model);
-  edit_model->UpdatePermanentText(ASCIIToUTF16("http://www.google.com/"));
-
   // Location bar must have focus.
   chrome::FocusLocationBar(browser());
   ASSERT_TRUE(ui_test_utils::IsViewFocused(browser(), VIEW_ID_OMNIBOX));
@@ -1715,17 +1720,21 @@
 }
 
 IN_PROC_BROWSER_TEST_F(OmniboxViewTest, CutURLToClipboard) {
+  // Set permanent text thus making sure that omnibox treats 'google.com'
+  // as URL (not as ordinary user input).
+  TestToolbarModel* toolbar_model = new TestToolbarModel;

[msw, 2013/08/30 16:12:16]

ditto

+  scoped_ptr<ToolbarModel> toolbar_model_deleter(toolbar_model);
+  toolbar_model->set_text(ASCIIToUTF16("http://www.google.com/"));
+  browser()->swap_toolbar_models(&toolbar_model_deleter);
+  OmniboxEditModel* edit_model = omnibox_view->model();
+  ASSERT_NE(static_cast<OmniboxEditModel*>(NULL), edit_model);
+  edit_model->UpdatePermanentText();
+
   OmniboxView* omnibox_view = NULL;
   ASSERT_NO_FATAL_FAILURE(GetOmniboxView(&omnibox_view));
   const char* target_url = "http://www.google.com/calendar";
   omnibox_view->SetUserText(ASCIIToUTF16(target_url));
 
-  // Set permanent text thus making sure that omnibox treats 'google.com'
-  // as URL (not as ordinary user input).
-  OmniboxEditModel* edit_model = omnibox_view->model();
-  ASSERT_NE(static_cast<OmniboxEditModel*>(NULL), edit_model);
-  edit_model->UpdatePermanentText(ASCIIToUTF16("http://www.google.com/"));
-
   // Location bar must have focus.
   chrome::FocusLocationBar(browser());
   ASSERT_TRUE(ui_test_utils::IsViewFocused(browser(), VIEW_ID_OMNIBOX));