diff options
| author | abarth@chromium.org <abarth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-30 05:25:17 +0000 |
|---|---|---|
| committer | abarth@chromium.org <abarth@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-05-30 05:25:17 +0000 |
| commit | f0af6a7ccfde1fb8068f89b190337c1a8ef99890 (patch) | |
| tree | 15ab1d791684e29cf46e9eb8032abb40faf32746 /chrome/renderer/render_view.cc | |
| parent | b664c3c97b36f6cb1a831641ca1b42e90f259b1b (diff) | |
| download | chromium_src-f0af6a7ccfde1fb8068f89b190337c1a8ef99890.zip chromium_src-f0af6a7ccfde1fb8068f89b190337c1a8ef99890.tar.gz chromium_src-f0af6a7ccfde1fb8068f89b190337c1a8ef99890.tar.bz2 | |
Enforce a max length for document titles.
R=deanm
BUG=http://crbug.com/12810
TEST=None
Review URL: http://codereview.chromium.org/113969
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@17282 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/renderer/render_view.cc')
| -rw-r--r-- | chrome/renderer/render_view.cc | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/chrome/renderer/render_view.cc b/chrome/renderer/render_view.cc index 9e669fd..750b9ba 100644 --- a/chrome/renderer/render_view.cc +++ b/chrome/renderer/render_view.cc @@ -23,6 +23,7 @@ #include "build/build_config.h" #include "chrome/common/bindings_policy.h" #include "chrome/common/chrome_switches.h" +#include "chrome/common/chrome_constants.h" #include "chrome/common/jstemplate_builder.h" #include "chrome/common/page_zoom.h" #include "chrome/common/render_messages.h" @@ -1124,8 +1125,13 @@ void RenderView::UpdateURL(WebFrame* frame) { // Tell the embedding application that the title of the active page has changed void RenderView::UpdateTitle(WebFrame* frame, const std::wstring& title) { // Ignore all but top level navigations... - if (webview()->GetMainFrame() == frame) - Send(new ViewHostMsg_UpdateTitle(routing_id_, page_id_, title)); + if (webview()->GetMainFrame() == frame) { + Send(new ViewHostMsg_UpdateTitle( + routing_id_, + page_id_, + title.length() > chrome::kMaxTitleChars ? + title.substr(0, chrome::kMaxTitleChars) : title)); + } } void RenderView::UpdateEncoding(WebFrame* frame, |