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

Issue 837533003: Cleanup: Get rid of CRLF line endings. (Closed)

Created:
5 years, 11 months ago by Lei Zhang
Modified:
5 years, 11 months ago
Reviewers:
brucedawson
CC:
pdfium-reviews_googlegroups.com
Base URL:
https://pdfium.googlesource.com/pdfium.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+393 lines, -393 lines) Patch
M AUTHORS View 1 chunk +33 lines, -33 lines 0 comments Download
M core/src/fpdfapi/fpdf_page/fpdf_page_colors.cpp View 2 chunks +9 lines, -9 lines 0 comments Download
M core/src/fpdfapi/fpdf_page/fpdf_page_doc.cpp View 2 chunks +47 lines, -47 lines 0 comments Download
M core/src/fpdfapi/fpdf_page/fpdf_page_pattern.cpp View 1 chunk +1 line, -1 line 0 comments Download
M core/src/fxcodec/codec/fx_codec_jpx_opj.cpp View 2 chunks +51 lines, -51 lines 0 comments Download
M core/src/fxge/freetype/fx_freetype.c View 1 chunk +169 lines, -169 lines 0 comments Download
M fpdfsdk/src/javascript/global.cpp View 1 chunk +18 lines, -18 lines 0 comments Download
M third_party/freetype/src/psaux/psaux.c View 1 chunk +34 lines, -34 lines 0 comments Download
M third_party/macros.h View 1 chunk +31 lines, -31 lines 0 comments Download

Messages

Total messages: 6 (2 generated)
Lei Zhang
Noticed https://codereview.chromium.org/792043005 added some. Might as well remove them all. On http://crbug.com/420796, someone is working ...
5 years, 11 months ago (2015-01-06 00:35:14 UTC) #3
brucedawson
lgtm But clearly I need to figure out how to get VC++ to stop adding ...
5 years, 11 months ago (2015-01-06 00:39:12 UTC) #4
Lei Zhang
Committed patchset #1 (id:20001) manually as 256ef88a26cff56fc7c23119d2d9e1b41468bd1a (presubmit successful).
5 years, 11 months ago (2015-01-06 00:54:08 UTC) #5
brucedawson
5 years, 11 months ago (2015-01-06 21:12:56 UTC) #6
Message was sent while issue was closed.
Merge to xfa branch?

Powered by Google App Engine
This is Rietveld 408576698