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

Issue 2148353002: Remove CFX_Edit_Refresh::Analyse (Closed)

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

Description

Remove CFX_Edit_Refresh::Analyse Unused method, removed. Committed: https://pdfium.googlesource.com/pdfium/+/d8f397878b5cd83ba4b75b3e69331986357301b9

Patch Set 1 #

Patch Set 2 : Rebase to master #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -68 lines) Patch
M fpdfsdk/fxedit/fxet_edit.cpp View 1 1 chunk +0 lines, -67 lines 0 comments Download
M fpdfsdk/fxedit/include/fxet_edit.h View 1 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 11 (6 generated)
dsinclair
PTAL.
4 years, 5 months ago (2016-07-14 17:58:32 UTC) #2
Lei Zhang
lgtm
4 years, 5 months ago (2016-07-15 00:03:25 UTC) #3
commit-bot: I haz the power
This CL has an open dependency (Issue 2146993002 Patch 60001). Please resolve the dependency and ...
4 years, 5 months ago (2016-07-15 00:04:28 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2148353002/20001
4 years, 5 months ago (2016-07-18 13:26:21 UTC) #9
commit-bot: I haz the power
4 years, 5 months ago (2016-07-18 13:49:52 UTC) #11
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://pdfium.googlesource.com/pdfium/+/d8f397878b5cd83ba4b75b3e693319863573...

Powered by Google App Engine
This is Rietveld 408576698