[vr] Clicking on the security icon should prompt the user to bail out of VR

chrome/browser/android/vr_shell/textures/url_bar_texture.h

Index: chrome/browser/android/vr_shell/textures/url_bar_texture.h
diff --git a/chrome/browser/android/vr_shell/textures/url_bar_texture.h b/chrome/browser/android/vr_shell/textures/url_bar_texture.h
index bd646b176c9c698abcb9f79384290bfb275a1087..770acd9a597f1add89303b1a88afe8dde5f72ea3 100644
--- a/chrome/browser/android/vr_shell/textures/url_bar_texture.h
+++ b/chrome/browser/android/vr_shell/textures/url_bar_texture.h
@@ -44,6 +44,7 @@ class UrlBarTexture : public UiTexture {
 
   bool HitsBackButton(const gfx::PointF& position) const;
   bool HitsUrlBar(const gfx::PointF& position) const;
+  bool HitsSecurityIcon(const gfx::PointF& position) const;
 
   void SetHovered(bool hovered);
   void SetPressed(bool pressed);
@@ -62,6 +63,9 @@ class UrlBarTexture : public UiTexture {
   void RenderUrl(const gfx::Size& texture_size, const gfx::Rect& bounds);
   void OnSetMode() override;
   const ColorScheme& color_scheme() const;
+  // Returns the location of the security icon relative to the url bar in

[cjgrant, 2017/06/02 04:16:36]

This comment doesn't add much as it's a private method.  What about just calling
the method security_icon_position_meters()?  That helps at call sites.

[ymalik, 2017/06/02 21:14:31]

Done.

+  // meters.
+  gfx::PointF security_icon_position() const;
 
   gfx::SizeF size_;
   bool hovered_ = false;