New origin security indicator icons.

components/omnibox/browser/omnibox_view.cc

 // 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.
 
 // This file defines helper functions shared by the various implementations
 // of OmniboxView.
 
 #include "components/omnibox/browser/omnibox_view.h"
 
 #include <utility>
@@ skipping 68 matching lines @@
 }
 
 int OmniboxView::GetIcon() const {
   if (!IsEditingOrEmpty())
     return controller_->GetToolbarModel()->GetIcon();
   int id = AutocompleteMatch::TypeToIcon(model_.get() ?
       model_->CurrentTextType() : AutocompleteMatchType::URL_WHAT_YOU_TYPED);
   return (id == IDR_OMNIBOX_HTTP) ? IDR_LOCATION_BAR_HTTP : id;
 }
 
-gfx::VectorIconId OmniboxView::GetVectorIcon(bool invert) const {
+gfx::VectorIconId OmniboxView::GetVectorIcon() const {
 #if !defined(OS_ANDROID) && !defined(OS_IOS)
   if (!IsEditingOrEmpty()) {
     gfx::VectorIconId id = controller_->GetToolbarModel()->GetVectorIcon();

[Evan Stade, 2016/05/18 21:40:42]

nit: don't need this local var any more

[palmer, 2016/05/19 17:49:23]

Done.

-    if (invert) {
-      if (id == gfx::VectorIconId::LOCATION_BAR_HTTPS_VALID)
-        return gfx::VectorIconId::LOCATION_BAR_HTTPS_VALID_INVERT;
-      if (id == gfx::VectorIconId::LOCATION_BAR_HTTPS_INVALID)
-        return gfx::VectorIconId::LOCATION_BAR_HTTPS_INVALID_INVERT;
-    }
     return id;
   }
   // Reuse the dropdown icons...
   gfx::VectorIconId id = AutocompleteMatch::TypeToVectorIcon(
       model_ ? model_->CurrentTextType()
              : AutocompleteMatchType::URL_WHAT_YOU_TYPED);
-  // but use a tweaked version for the HTTP icon.
+  // but use a different version for the HTTP icon.
   return (id == gfx::VectorIconId::OMNIBOX_HTTP)
              ? gfx::VectorIconId::LOCATION_BAR_HTTP
              : id;
 #else
   NOTIMPLEMENTED();
   return gfx::VectorIconId::VECTOR_ICON_NONE;
 #endif
 }
 
 void OmniboxView::SetUserText(const base::string16& text) {
@@ skipping 66 matching lines @@
   if (client) {
     model_.reset(new OmniboxEditModel(this, controller, std::move(client)));
   }
 }
 
 void OmniboxView::TextChanged() {
   EmphasizeURLComponents();
   if (model_.get())
     model_->OnChanged();
 }