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

Issue 1565803003: Return const-ref object in a few CPWL_Wnd getters. (Closed)

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

Description

Return const-ref object in a few CPWL_Wnd getters. Also remove a bunch of dead code. R=tsepez@chromium.org Committed: https://pdfium.googlesource.com/pdfium/+/cbed99098ad4cdc3f9f260cc1e8521f67c12d35d

Patch Set 1 #

Patch Set 2 : rebase #

Patch Set 3 : remove dead code #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -56 lines) Patch
M fpdfsdk/include/pdfwindow/PWL_Utils.h View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M fpdfsdk/include/pdfwindow/PWL_Wnd.h View 1 2 3 chunks +3 lines, -10 lines 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Utils.cpp View 1 2 1 chunk +0 lines, -1 line 0 comments Download
M fpdfsdk/src/pdfwindow/PWL_Wnd.cpp View 1 2 9 chunks +8 lines, -44 lines 0 comments Download

Messages

Total messages: 8 (3 generated)
Lei Zhang
4 years, 11 months ago (2016-01-06 22:08:58 UTC) #2
Tom Sepez
We're sure about object lifetimes?
4 years, 11 months ago (2016-01-06 22:14:18 UTC) #3
Lei Zhang
On 2016/01/06 22:14:18, Tom Sepez wrote: > We're sure about object lifetimes? Yes. And trying ...
4 years, 11 months ago (2016-01-06 22:55:22 UTC) #5
Tom Sepez
lgtm
4 years, 11 months ago (2016-01-06 22:56:45 UTC) #6
Lei Zhang
4 years, 11 months ago (2016-01-06 22:58:54 UTC) #8
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
cbed99098ad4cdc3f9f260cc1e8521f67c12d35d (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698