Alternate NTP: Add search token to omnibox

chrome/browser/ui/cocoa/location_bar/separator_decoration.mm

Index: chrome/browser/ui/cocoa/location_bar/separator_decoration.mm
diff --git a/chrome/browser/ui/cocoa/location_bar/separator_decoration.mm b/chrome/browser/ui/cocoa/location_bar/separator_decoration.mm
new file mode 100644
index 0000000000000000000000000000000000000000..31111c6e8a797c2181d96fba4428cec7fa84a9fe
--- /dev/null
+++ b/chrome/browser/ui/cocoa/location_bar/separator_decoration.mm
@@ -0,0 +1,37 @@
+// Copyright (c) 2013 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.
+
+#import "chrome/browser/ui/cocoa/location_bar/separator_decoration.h"
+
+#import "chrome/browser/ui/cocoa/nsview_additions.h"
+
+SeparatorDecoration::SeparatorDecoration() {
+}
+
+SeparatorDecoration::~SeparatorDecoration() {
+}
+
+void SeparatorDecoration::DrawInFrame(NSRect frame, NSView* control_view) {
+  // Inset by 1 from top and bottom to avoid drawing on top of the the omnibox
+  // bezel.
+  NSRect rect = NSInsetRect(frame, 0, 1);
+  rect.size.width = [control_view cr_lineWidth];
+  rect.origin.x = NSMaxX(frame) - NSWidth(rect);
+  [SeparatorColor(control_view) set];
+  NSRectFillUsingOperation(rect, NSCompositeSourceOver);
+}
+
+CGFloat SeparatorDecoration::GetWidthForSpace(CGFloat width,
+                                              CGFloat text_width) {
+  return 2;
+}
+
+SeparatorDecoration* SeparatorDecoration::AsSeparatorDecoration() {
+  return this;
+}
+
+NSColor* SeparatorDecoration::SeparatorColor(NSView* view) const {
+  const CGFloat lineWidth = [view cr_lineWidth];
+  return [[NSColor lightGrayColor] colorWithAlphaComponent:0.25 / lineWidth];

[Scott Hess - ex-Googler, 2013/01/29 23:28:36]

This seems like an odd and dangerous formulation to me.  I think it would be
clearer to have two alpha values, one for the special case and one for all
others, with a ternary operator or whatever works best.

[sail, 2013/01/30 01:40:55]

I agree that this is weird but this is how we do it through out the code base.
Here are a few examples:
https://code.google.com/searchframe#OAMlx_jo-ck/src/chrome/browser/ui/cocoa/s...
https://code.google.com/searchframe#OAMlx_jo-ck/src/chrome/browser/ui/cocoa/n...

[Scott Hess - ex-Googler, 2013/01/30 23:48:42]

Not so great if |view| ends up nil in any of those places, though...

[sail, 2013/01/31 03:08:00]

Done.
Used an if statement instead.

+}