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

Unified Diff: printing/printing_utils.cc

Issue 1212883003: Include user email into document title before spooling (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Mon Jul 6 10:13:42 PDT 2015 Created 5 years, 5 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: printing/printing_utils.cc
diff --git a/printing/printing_utils.cc b/printing/printing_utils.cc
index 3d91de1bf0bfb0cf6cfbeb65e546b4605c0b4ce6..02520382be39c0faf4f7b3fa849742eca05970bf 100644
--- a/printing/printing_utils.cc
+++ b/printing/printing_utils.cc
@@ -9,20 +9,21 @@
#include "third_party/icu/source/common/unicode/uchar.h"
#include "ui/gfx/text_elider.h"
-namespace {
-const int kMaxDocumentTitleLength = 50;
-}
-
namespace printing {
-base::string16 SimplifyDocumentTitle(const base::string16& title) {
+base::string16 SimplifyDocumentTitleWithLength(const base::string16& title,
+ size_t length) {
base::string16 no_controls(title);
no_controls.erase(
std::remove_if(no_controls.begin(), no_controls.end(), &u_iscntrl),
no_controls.end());
base::string16 result;
- gfx::ElideString(no_controls, kMaxDocumentTitleLength, &result);
+ gfx::ElideString(no_controls, static_cast<int>(length), &result);
return result;
}
+base::string16 SimplifyDocumentTitle(const base::string16& title) {
+ return SimplifyDocumentTitleWithLength(title, kMaxDocumentTitleLength);
+}
+
} // namespace printing

Powered by Google App Engine
This is Rietveld 408576698