[OSX, OmniTheatre] URL hiding for Origin Chip.

chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.h"
 
 #include <Carbon/Carbon.h>  // kVK_Return
 
 #include "base/mac/foundation_util.h"
 #include "base/metrics/histogram.h"
 #include "base/strings/string_util.h"
 #include "base/strings/sys_string_conversions.h"
 #include "base/strings/utf_string_conversions.h"
 #include "chrome/browser/autocomplete/autocomplete_input.h"
 #include "chrome/browser/autocomplete/autocomplete_match.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/search/search.h"
 #include "chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.h"
 #import "chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_editor.h"
 #include "chrome/browser/ui/cocoa/omnibox/omnibox_popup_view_mac.h"
 #include "chrome/browser/ui/omnibox/omnibox_edit_controller.h"
 #include "chrome/browser/ui/omnibox/omnibox_popup_model.h"
 #include "chrome/browser/ui/toolbar/toolbar_model.h"
 #include "content/public/browser/web_contents.h"
 #include "extensions/common/constants.h"
 #include "grit/generated_resources.h"
 #include "grit/theme_resources.h"
@@ skipping 170 matching lines @@
     // store existing state before anyone tries to setup the new
     // state.  Anyhow, it would look something like this.
 #if 0
     [[field_ window] makeFirstResponder:field_];
     [[field_ currentEditor] setSelectedRange:state->selection];
 #endif
   }
 }
 
 void OmniboxViewMac::Update() {
+  if (chrome::ShouldDisplayOriginChip() ||

[Justin Donnelly, 2014/02/14 14:57:14]

Any reason not to check just for V2 here like you do below?

[groby-ooo-7-16, 2014/02/15 01:28:14]

That was based on Views, which does the same thing. I'll take it that was an
oversight there, so removed here.

+      chrome::ShouldDisplayOriginChipV2()) {
+    [[field_ cell] setPlaceholderString:
+        base::SysUTF16ToNSString(GetHintText())];
+  }
   if (model()->UpdatePermanentText()) {
     // Something visibly changed.  Re-enable URL replacement.
     controller()->GetToolbarModel()->set_url_replacement_enabled(true);
     model()->UpdatePermanentText();
 
     // Restore everything to the baseline look.
     RevertAll();
 
     // TODO(shess): Figure out how this case is used, to make sure
     // we're getting the selection and popup right.
@@ skipping 545 matching lines @@
       }
     }
   }
 
   return false;
 }
 
 void OmniboxViewMac::OnSetFocus(bool control_down) {
   model()->OnSetFocus(control_down);
   controller()->OnSetFocus();
+
+  // TODO(groby): Not entirely correct, since the chip should only be disabled
+  // after mouseDown: was handled, to allow clicking on the origin chip.
+  if (chrome::ShouldDisplayOriginChipV2()) {
+    controller()->GetToolbarModel()->set_origin_chip_enabled(false);
+    controller()->OnChanged();
+  }
 }
 
 void OmniboxViewMac::OnKillFocus() {
   // Tell the model to reset itself.
   model()->OnWillKillFocus(NULL);
   model()->OnKillFocus();
+
+  // If user input is not in progress, re-enable the origin chip and URL
+  // replacement.  This addresses the case where the URL was shown by a call
+  // to ShowURL().  If the Omnibox acheived focus by other means, the calls to

[Scott Hess - ex-Googler, 2014/02/15 00:55:57]

achieved.

[groby-ooo-7-16, 2014/02/15 01:28:14]

Done.

+  // set_url_replacement_enabled, UpdatePermanentText and RevertAll are not
+  // required (a call to OnChanged would be sufficient) but do no harm.

[Scott Hess - ex-Googler, 2014/02/15 00:55:57]

My general take on things is that calls which scale with user actions (tab,
click, command-L) just have to be fast enough, but have a different
correctness/speed tradeoff.  I think it's fine to over-call here, since the
controller and model don't necessarily bake in cocoa assumptions.

[groby-ooo-7-16, 2014/02/15 01:28:14]

This is taken verbatim from Views, so there's certainly no Cocoaness :)

I'm not sure these are even needed there. ShowURL already does call
UpdatePermanentText, which leaves RevertAll. I'll follow up w/ jdonelly outside
the CL

+  if (chrome::ShouldDisplayOriginChipV2() &&
+      !model()->user_input_in_progress()) {
+    controller()->GetToolbarModel()->set_origin_chip_enabled(true);
+    controller()->GetToolbarModel()->set_url_replacement_enabled(true);
+    model()->UpdatePermanentText();
+    RevertAll();
+  }
 }
 
 void OmniboxViewMac::OnMouseDown(NSInteger button_number) {
   // Restore caret visibility whenever the user clicks in the the omnibox. This
   // is not always covered by OnSetFocus() because when clicking while the
   // omnibox has invisible focus does not trigger a new OnSetFocus() call.
   if (button_number == 0 || button_number == 1)
     model()->SetCaretVisibility(true);
 }
 
@@ skipping 166 matching lines @@
 
 NSUInteger OmniboxViewMac::GetTextLength() const {
   return [field_ currentEditor] ?  [[[field_ currentEditor] string] length] :
                                    [[field_ stringValue] length];
 }
 
 bool OmniboxViewMac::IsCaretAtEnd() const {
   const NSRange selection = GetSelectedRange();
   return NSMaxRange(selection) == GetTextLength();
 }