diff options
| author | deepak.m1 <deepak.m1@samsung.com> | 2015-01-08 21:16:03 -0800 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2015-01-09 05:16:45 +0000 |
| commit | d209afa5561213383fa0bc18423a28c684dbe142 (patch) | |
| tree | 6e5d06067c4ce88aaacd564c9fad61ee96f48b90 /pdf | |
| parent | 8b8e1fac50acf7f2245b96b1f30353fb41ac8de8 (diff) | |
| download | chromium_src-d209afa5561213383fa0bc18423a28c684dbe142.zip chromium_src-d209afa5561213383fa0bc18423a28c684dbe142.tar.gz chromium_src-d209afa5561213383fa0bc18423a28c684dbe142.tar.bz2 | |
copying to clipboard should add a newline before the content of the next page.
When we selected some text that spreads in multiple pages then content of
pages should be separated by new line as happening in the Adobe Reader.
code added for checking selection in newpage, then add a newline char.
BUG=84387
Review URL: https://codereview.chromium.org/824363002
Cr-Commit-Position: refs/heads/master@{#310714}
Diffstat (limited to 'pdf')
| -rw-r--r-- | pdf/pdfium/pdfium_engine.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/pdf/pdfium/pdfium_engine.cc b/pdf/pdfium/pdfium_engine.cc index 2a3ec26..de50c79 100644 --- a/pdf/pdfium/pdfium_engine.cc +++ b/pdf/pdfium/pdfium_engine.cc @@ -2251,11 +2251,14 @@ void PDFiumEngine::InvalidateAllPages() { std::string PDFiumEngine::GetSelectedText() { base::string16 result; + base::string16 new_line_char = base::UTF8ToUTF16("\n"); for (size_t i = 0; i < selection_.size(); ++i) { if (i > 0 && selection_[i - 1].page_index() > selection_[i].page_index()) { - result = selection_[i].GetText() + result; + result = selection_[i].GetText() + new_line_char + result; } else { + if (i > 0) + result.append(new_line_char); result.append(selection_[i].GetText()); } } |