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

Unified Diff: content/common/content_client.h

Issue 7922023: Remove webkit_glue::BuildUserAgent(), remove windows spoofing (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix GetUserAgent() to return whether the user agent is overriding, clean up Created 9 years, 3 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: content/common/content_client.h
diff --git a/content/common/content_client.h b/content/common/content_client.h
index 9f5521fee2c54382832cb8e7cd9067f3e89bbc33..fc252384a8f00678b62bc2505116d016655e0c19 100644
--- a/content/common/content_client.h
+++ b/content/common/content_client.h
@@ -76,10 +76,9 @@ class CONTENT_EXPORT ContentClient {
// behalf of a swapped out renderer.
virtual bool CanHandleWhileSwappedOut(const IPC::Message& msg) = 0;
- // Returns the user agent. If mimic_windows is true then the embedder can
- // return a fake Windows user agent. This is a workaround for broken
- // websites.
- virtual std::string GetUserAgent(bool mimic_windows) const = 0;
+ // Returns the user agent and indicates whether this string should
+ // always be used (overriding any hacks to work around browser sniffing).
+ virtual std::string GetUserAgent(bool *overriding) const = 0;
Evan Martin 2011/09/21 20:27:24 star on the left I don't quite get the doc: when
Dirk Pranke 2011/09/21 21:02:35 That is correct. I will attempt to make it clearer
// Returns a string resource given its id.
virtual string16 GetLocalizedString(int message_id) const = 0;

Powered by Google App Engine
This is Rietveld 408576698