Views: Draw Textfield selected text in gray when top-level Widget loses focus.

ui/views/controls/textfield/textfield_unittest.cc

 // Copyright 2014 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 "ui/views/controls/textfield/textfield.h"
 
 #include <stddef.h>
 #include <stdint.h>
 
 #include <set>
@@ skipping 312 matching lines @@
     target_->set_controller(this);
   }
 
   views::Textfield* target() { return target_.get(); }
 
   // views::TextfieldController:
   bool HandleKeyEvent(views::Textfield* sender,
                       const ui::KeyEvent& key_event) override {
     if (target_)
       target_->OnBlur();
+    target_->parent()->RemoveChildView(target_.get());

[tapted, 2016/10/27 00:53:42]

This needs a comment (if it's still needed) - the ~View() destructor should be
invoked on the next line, and it does RemoveChildView

[Patti Lor, 2016/10/30 23:26:04]

I added a comment, please let me know if it makes sense to you. This behaviour
seems actually kinda weird to me - maybe it would be better to have a private
View::ViewHierarchyChanged() which calls some sort of virtual
OverriddenViewHierarchyChanged() which subclasses can optionally implement?

     target_.reset();
     return false;
   }
 
  private:
   std::unique_ptr<views::Textfield> target_;
 };
 
 base::string16 GetClipboardText(ui::ClipboardType type) {
   base::string16 text;
@@ skipping 2662 matching lines @@
 
   textfield_->SetTextInputType(ui::TEXT_INPUT_TYPE_PASSWORD);
   ui::AXViewState state_protected;
   textfield_->GetAccessibleState(&state_protected);
   EXPECT_EQ(ui::AX_ROLE_TEXT_FIELD, state_protected.role);
   EXPECT_EQ(ASCIIToUTF16("********"), state_protected.value);
   EXPECT_TRUE(state_protected.HasStateFlag(ui::AX_STATE_PROTECTED));
 }
 
 }  // namespace views