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

Issue 284203012: Bring in pdfium code into third_party/pdfium. (Closed)

Created:
6 years, 7 months ago by jam
Modified:
6 years, 7 months ago
Reviewers:
brettw, Nico
CC:
chromium-reviews
Visibility:
Public.

Description

Bring in pdfium code into third_party/pdfium. R=brettw@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271449

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -0 lines) Patch
M .gitignore View 1 chunk +1 line, -0 lines 0 comments Download
M DEPS View 3 chunks +8 lines, -0 lines 0 comments Download
M tools/licenses.py View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
jam
6 years, 7 months ago (2014-05-19 18:03:39 UTC) #1
brettw
lgtm
6 years, 7 months ago (2014-05-19 18:05:09 UTC) #2
Nico
Is it possible to pull in the src to third_party/pdfium/src and check in the gyp ...
6 years, 7 months ago (2014-05-19 20:52:03 UTC) #3
Nico
On Mon, May 19, 2014 at 1:52 PM, <thakis@chromium.org> wrote: > Is it possible to ...
6 years, 7 months ago (2014-05-19 20:54:23 UTC) #4
jam
6 years, 7 months ago (2014-05-19 21:18:22 UTC) #5
Message was sent while issue was closed.
On 2014/05/19 20:54:23, Nico wrote:
> On Mon, May 19, 2014 at 1:52 PM, <mailto:thakis@chromium.org> wrote:
> 
> > Is it possible to pull in the src to third_party/pdfium/src and check in
> > the gyp
> > file directly into chromium's repo? The few repos that have their gyp
> > files not
> > in chromium's repo always cause me grief when doing compiler changes.
> 
> 
> (Since this repo builds without -Werror, it'll probably be less of an issue
> here, but if there's no good reason to keep the gyp file upstream having it
> in chromium would still make things a bit simpler.)

yeah we'd like the gyp to be in that project; as it'll be used by more than
chromium. other projects will generate their own build files from gyp.

also, we are aiming for a separate buildbot for pdfium that runs self-contained
tests, so we'll need the build files there.

Powered by Google App Engine
This is Rietveld 408576698