Remove ShowURL/HideURL from OmniboxView and LocationBarView

components/omnibox/browser/omnibox_view.cc

Index: components/omnibox/browser/omnibox_view.cc
diff --git a/components/omnibox/browser/omnibox_view.cc b/components/omnibox/browser/omnibox_view.cc
index c92730f03337167fa96512a737b685be95034197..ab062677182c790717b90860959396593e13833f 100644
--- a/components/omnibox/browser/omnibox_view.cc
+++ b/components/omnibox/browser/omnibox_view.cc
@@ -112,20 +112,6 @@ void OmniboxView::SetUserText(const base::string16& text,
   SetWindowTextAndCaretPos(text, text.length(), update_popup, true);
 }
 
-void OmniboxView::ShowURL() {
-  SetFocus();
-  controller_->GetToolbarModel()->set_url_replacement_enabled(false);

[Peter Kasting, 2016/08/18 08:32:19]

This was the only place outside tests that set this to false, so you should be
good to rip out this setter and remove the corresponding tests now.

That will also mean that RevertAll() will simply become a call to
RvertWithoutResettingSearchTermReplacement(), so the two should just be
collapsed into one RevertAll() method at the same time.

[Marc Treib, 2016/08/18 09:10:38]

Ah, I hadn't noticed that yet, thanks! Next CL incoming soon :)
Acknowledged.

-  model_->UpdatePermanentText();
-  RevertWithoutResettingSearchTermReplacement();
-  SelectAll(true);
-}
-
-void OmniboxView::HideURL() {
-  controller_->GetToolbarModel()->set_url_replacement_enabled(true);
-  model_->UpdatePermanentText();
-  RevertWithoutResettingSearchTermReplacement();
-}
-
 void OmniboxView::RevertAll() {
   controller_->GetToolbarModel()->set_url_replacement_enabled(true);
   RevertWithoutResettingSearchTermReplacement();