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

Issue 1717243002: Fix a crasher in CXFA_LayoutPageMgr::GetAvailHeight() (Closed)

Created:
4 years, 10 months ago by jun_fang
Modified:
4 years, 10 months ago
Reviewers:
Tom Sepez, Lei Zhang
CC:
pdfium-reviews_googlegroups.com, Jim Wang, kai_jing
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 (+12 lines, -12 lines) Patch
M xfa/src/fxfa/src/parser/xfa_layout_pagemgr_new.cpp View 1 chunk +12 lines, -12 lines 0 comments Download

Messages

Total messages: 5 (2 generated)
jun_fang
Hi Tom and Lei, Please help to review this CL.
4 years, 10 months ago (2016-02-22 08:05:52 UTC) #2
Tom Sepez
lgtm
4 years, 10 months ago (2016-02-22 17:18:32 UTC) #3
jun_fang
4 years, 10 months ago (2016-02-23 01:17:50 UTC) #5
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as
6add19c6e183363f543585ce316b54ee9f50592a (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698