Hello sgarbiel@,

Re: the fill color, you specify a transparent color so is the color behind it the toolbar color?

Yes

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7a7f2e398b53990858d88bc19115c137630795f3

commit 7a7f2e398b53990858d88bc19115c137630795f3
Author: shrike <shrike@chromium.org>
Date: Wed Jun 01 00:05:21 2016

[Mac][Material Design] Bring Omnibox stroke and MD colors up to spec.

This cl changes the Omnibox's stroke in Incognito mode to match the
recent spec revisions made by sgabriel@.

Changing to the new spec also required switching other MD colors to the
sRGB color space so that everything matched. The plan was to switch to
sRGB (which is correct - the calibrated space is not) eventually, but
to get the Omnibox to look right, now is the time.

R=avi@chromium.org,sky@chromium.org
BUG= 613749 

Review-Url: https://codereview.chromium.org/2013183003
Cr-Commit-Position: refs/heads/master@{#396986}

[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/themes/theme_service_mac.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.h
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/bubble_decoration.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/tabs/tab_view.mm
[modify] https://crrev.com/7a7f2e398b53990858d88bc19115c137630795f3/chrome/browser/ui/cocoa/toolbar/toolbar_button_cocoa.mm

Tested this issue on Mac OS 10.11.5 using chrome latest canary M53-53.0.2754.0. By opening the incognito mode observed the below behavior as seen in the attached screen-shot.

Could anyone please confirm is this is expected behavior of this issue to verify from chrome-TE end.

Thanks!

Deleted: Screen Shot 2016-06-01 at 12.10.25 PM.png 20.2 KB

	Screen Shot 2016-06-01 at 12.10.25 PM.png 20.2 KB View Download

What behavior are you referring to?

We have two feedback complaints that the omnibox fill color is too dark in 53.0.2754.0 canary. I know it's only two complaints so far, but FYI. I'll keep an eye out for more reports.

Your change meets the bar and is auto-approved for M52 (branch: 2743)

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/793018247051e845ad6754e0344f5f892349237f

commit 793018247051e845ad6754e0344f5f892349237f
Author: Jayson Adams <shrike@chromium.org>
Date: Fri Jun 03 18:28:57 2016

[Mac][Material Design] Bring Omnibox stroke and MD colors up to spec.

This cl changes the Omnibox's stroke in Incognito mode to match the
recent spec revisions made by sgabriel@.

Changing to the new spec also required switching other MD colors to the
sRGB color space so that everything matched. The plan was to switch to
sRGB (which is correct - the calibrated space is not) eventually, but
to get the Omnibox to look right, now is the time.

R=avi@chromium.org,sky@chromium.org
BUG= 613749 

Review-Url: https://codereview.chromium.org/2013183003
Cr-Commit-Position: refs/heads/master@{#396986}
(cherry picked from commit 7a7f2e398b53990858d88bc19115c137630795f3)

Review URL: https://codereview.chromium.org/2032143004 .

Cr-Commit-Position: refs/branch-heads/2743@{#207}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/themes/theme_service_mac.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.h
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/bubble_decoration.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/tabs/tab_view.mm
[modify] https://crrev.com/793018247051e845ad6754e0344f5f892349237f/chrome/browser/ui/cocoa/toolbar/toolbar_button_cocoa.mm

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/54bda7172202080f66309be55b4d86a3dcd8b0e7

commit 54bda7172202080f66309be55b4d86a3dcd8b0e7
Author: shrike <shrike@chromium.org>
Date: Fri Jun 03 20:34:23 2016

Revert of [Mac][Material Design] Bring Omnibox stroke and MD colors up to spec. (patchset #1 id:1 of https://codereview.chromium.org/2032143004/ )

Reason for revert:
Broke the continuous build because of a missing symbol.

https://chromegw.corp.google.com/i/official.desktop.continuous/builders/mac%20beta/builds/3236/steps/compile/logs/stdio

FAILED: obj/chrome/browser/ui/cocoa/location_bar/browser_ui.location_bar_view_mac.o
../../third_party/llvm-build/Release+Asserts/bin/clang++ -MMD -MF obj/chrome/browser/ui/cocoa/location_bar/browser_ui.location_bar_view_mac.o.d -DV8_DEPRECATION_WARNINGS -D__ASSERT_MACROS_DEFINE_VERSIONS_WITHOUT_UNDERSCORE=0 -DGOOGLE_CHROME_BUILD -DCR_CLANG_REVISION=269902-1 -DENABLE_RLZ -DUSE_LIBJPEG_TURBO=1 -DENABLE_WEBRTC=1 -DENABLE_MEDIA_ROUTER=1 -DUSE_PROPRIETARY_CODECS -DENABLE_PEPPER_CDMS -DENABLE_NOTIFICATIONS -DENABLE_TOPCHROME_MD=1 -DENABLE_TASK_MANAGER=1 -DENABLE_EXTENSIONS=1 -DENABLE_PDF=1 -DENABLE_PLUGIN_INSTALLATION=1 -DENABLE_PLUGINS=1 -DENABLE_SESSION_SERVICE=1 -DENABLE_THEMES=1 -DENABLE_PROD_WALLET_SERVICE=1 -DENABLE_PRINTING=1 -DENABLE_BASIC_PRINTING=1 -DENABLE_PRINT_PREVIEW=1 -DENABLE_SPELLCHECK=1 -DUSE_BROWSER_SPELLCHECKER=1 -DENABLE_CAPTIVE_PORTAL_DETECTION=1 -DENABLE_SUPERVISED_USERS=1 -DENABLE_SERVICE_DISCOVERY=1 -DENABLE_HANGOUT_SERVICES_EXTENSION=1 -DV8_USE_EXTERNAL_STARTUP_DATA -DFULL_SAFE_BROWSING -DSAFE_BROWSING_CSD -DSAFE_BROWSING_DB_LOCAL -DMOJO_USE_SYSTEM_IMPL -DTOOLKIT_VIEWS=1 -DPROTOBUF_USE_DLLS -DGOOGLE_PROTOBUF_NO_RTTI -DGOOGLE_PROTOBUF_NO_STATIC_INITIALIZER -DSK_SUPPORT_GPU=1 -DSK_IGNORE_LINEONLY_AA_CONVEX_PATH_OPTS -DU_USING_ICU_NAMESPACE=0 -DU_ENABLE_DYLOAD=0 -DU_NOEXCEPT= -DU_STATIC_IMPLEMENTATION -DUSE_CUPS '-DI18N_ADDRESS_VALIDATION_DATA_URL="https://i18napis.appspot.com/ssl-aggregate-address/"' -DI18N_ADDRESSINPUT_USE_BASICTYPES_OVERRIDE=1 -DUSE_LIBPCI=1 -D__STDC_CONSTANT_MACROS -D__STDC_FORMAT_MACROS -DNDEBUG -DOFFICIAL_BUILD -DNVALGRIND -DDYNAMIC_ANNOTATIONS_ENABLED=0 -D_FORTIFY_SOURCE=2 -Igen -I../.. -Iobj/chrome/browser_ui.gen -Iobj/chrome/browser_ui.gen/chrome -I../../skia/config -I../../third_party/khronos -I../../gpu -Igen/angle -I../../third_party/WebKit/Source -Igen/chrome -Igen/protoc_out -I../../third_party/protobuf/src -I../../third_party/dom_distiller_js/dist/proto_gen -Igen/policy -Igen/components -Igen/components/strings -I../../third_party/WebKit -Igen/third_party/WebKit -I../../third_party/skia/include/core -I../../third_party/skia/include/effects -I../../third_party/skia/include/pdf -I../../third_party/skia/include/gpu -I../../third_party/skia/include/lazy -I../../third_party/skia/include/pathops -I../../third_party/skia/include/pipe -I../../third_party/skia/include/ports -I../../third_party/skia/include/utils -I../../third_party/skia/include/utils/mac -I../../skia/ext -I../../third_party/cacheinvalidation/overrides -I../../third_party/cacheinvalidation/src -I../../third_party/cacheinvalidation/google/cacheinvalidation -I../../third_party/icu/source/i18n -I../../third_party/icu/source/common -I../../third_party/zlib -Igen/ui/resources -Igen/content/app/resources/ -I../../third_party/libwebm/source -I../../third_party/opus/src/include -I../../third_party/leveldatabase/src/include -I../../third_party/leveldatabase/src -I../../third_party/leveldatabase -I../../third_party/re2/src -I../../v8/include -I../../third_party/google_toolbox_for_mac -I../../third_party/google_toolbox_for_mac/src -I../../third_party/google_toolbox_for_mac/src/AppKit -I../../third_party/google_toolbox_for_mac/src/DebugUtils -I../../third_party/google_toolbox_for_mac/src/Foundation -Igen/ui/views/resources -I../../third_party/libaddressinput/chromium/override -I../../third_party/libaddressinput/src/cpp/include -Igen/third_party/libaddressinput/ -isysroot /Applications/Xcode5.1.1.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.10.sdk -O2 -gdwarf-2 -fvisibility=hidden -Werror -mmacosx-version-min=10.7 -arch x86_64 -Wall -Wextra -Wno-unused-parameter -Wno-missing-field-initializers -Wno-selector-type-mismatch -Wpartial-availability -Wheader-hygiene -Wno-char-subscripts -Wno-unneeded-internal-declaration -Wno-covered-switch-default -Wstring-conversion -Wno-c++11-narrowing -Wno-deprecated-register -Wno-inconsistent-missing-override -Wno-shift-negative-value -Wno-undefined-var-template -Wno-nonnull -Wexit-time-destructors -std=c++11 -stdlib=libc++ -fno-rtti -fno-exceptions -fvisibility-inlines-hidden -fno-threadsafe-statics -Xclang -load -Xclang /b/build/slave/mac_beta/build/src/third_party/llvm-build/Release+Asserts/lib/libFindBadConstructs.dylib -Xclang -add-plugin -Xclang find-bad-constructs -Xclang -plugin-arg-find-bad-constructs -Xclang check-templates -Xclang -plugin-arg-find-bad-constructs -Xclang follow-macro-expansion -Xclang -plugin-arg-find-bad-constructs -Xclang check-implicit-copy-ctors -fcolor-diagnostics -fno-strict-aliasing -fno-standalone-debug -Wobjc-missing-property-synthesis -fobjc-call-cxx-cdtors  -c ../../chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm -o obj/chrome/browser/ui/cocoa/location_bar/browser_ui.location_bar_view_mac.o
../../chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm:583:29: error: no member named 'GetSecureTextColor' in 'OmniboxViewMac'
            OmniboxViewMac::GetSecureTextColor(security_level, in_dark_mode);
            ~~~~~~~~~~~~~~~~^

Original issue's description:
> [Mac][Material Design] Bring Omnibox stroke and MD colors up to spec.
>
> This cl changes the Omnibox's stroke in Incognito mode to match the
> recent spec revisions made by sgabriel@.
>
> Changing to the new spec also required switching other MD colors to the
> sRGB color space so that everything matched. The plan was to switch to
> sRGB (which is correct - the calibrated space is not) eventually, but
> to get the Omnibox to look right, now is the time.
>
> R=avi@chromium.org,sky@chromium.org
> BUG= 613749 
>
> Review-Url: https://codereview.chromium.org/2013183003
> Cr-Commit-Position: refs/heads/master@{#396986}
> (cherry picked from commit 7a7f2e398b53990858d88bc19115c137630795f3)
>
> Committed: https://chromium.googlesource.com/chromium/src/+/793018247051e845ad6754e0344f5f892349237f

TBR=avi@chromium.org,sky@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG= 613749 

Review-Url: https://codereview.chromium.org/2035153002
Cr-Commit-Position: refs/branch-heads/2743@{#208}
Cr-Branched-From: 2b3ae3b8090361f8af5a611712fc1a5ab2de53cb-refs/heads/master@{#394939}

[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/themes/theme_service_mac.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.h
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/autocomplete_text_field_cell.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/bubble_decoration.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/location_bar_view_mac.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/location_bar/selected_keyword_decoration.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/omnibox/omnibox_popup_cell.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/omnibox/omnibox_view_mac.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/tabs/tab_view.mm
[modify] https://crrev.com/54bda7172202080f66309be55b4d86a3dcd8b0e7/chrome/browser/ui/cocoa/toolbar/toolbar_button_cocoa.mm

Verified the issue on Mac OS 10.11.5 using chrome latest Beta M52-52.0.2743.33 and observed that incognito omnibox is rendering as expected screen-shot provided in the comment #0. Hence adding TE-Verified label.

Thanks!

Deleted: Screen Shot 2016-06-08 at 3.04.00 PM.png 12.4 KB

	Screen Shot 2016-06-08 at 3.04.00 PM.png 12.4 KB View Download