Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(380)

Unified Diff: chrome/browser/android/vr_shell/textures/url_bar_texture.h

Issue 2902783002: VR: Add URL bar security-related coloring and emphasis. (Closed)
Patch Set: Tweak the back button icon size to match UX mocks. Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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 c6ebf975f196f75fa63d6693d17f01f197c58518..308c855d1b07bf2e476b84761e525d82d299b728 100644
--- a/chrome/browser/android/vr_shell/textures/url_bar_texture.h
+++ b/chrome/browser/android/vr_shell/textures/url_bar_texture.h
@@ -10,6 +10,7 @@
#include "base/macros.h"
#include "chrome/browser/android/vr_shell/textures/ui_texture.h"
+#include "components/security_state/core/security_state.h"
#include "url/gurl.h"
namespace gfx {
@@ -32,7 +33,7 @@ class UrlBarTexture : public UiTexture {
gfx::SizeF GetDrawnSize() const override;
void SetURL(const GURL& gurl);
- void SetSecurityLevel(int level);
+ void SetSecurityLevel(security_state::SecurityLevel level);
bool HitsBackButton(const gfx::PointF& position) const;
bool HitsUrlBar(const gfx::PointF& position) const;
@@ -44,14 +45,21 @@ class UrlBarTexture : public UiTexture {
void Draw(SkCanvas* canvas, const gfx::Size& texture_size) override;
float ToPixels(float meters) const;
bool HitsTransparentRegion(const gfx::PointF& meters, bool left) const;
+ void RenderUrl(const gfx::Size& texture_size, const gfx::Rect& bounds);
+ void ApplyUrlStyling(const base::string16& formatted_url,
+ const url::Parsed& parsed,
+ gfx::RenderText* render_text);
gfx::SizeF size_;
- int security_level_;
bool hovered_ = false;
bool pressed_ = false;
+
GURL gurl_;
+ security_state::SecurityLevel security_level_;
+
+ std::unique_ptr<gfx::RenderText> url_render_text_;
GURL last_drawn_gurl_;
- std::vector<std::unique_ptr<gfx::RenderText>> gurl_render_texts_;
+ security_state::SecurityLevel last_drawn_security_level_;
DISALLOW_COPY_AND_ASSIGN(UrlBarTexture);
};

Powered by Google App Engine
This is Rietveld 408576698