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

Issue 301033003: Removed unnecessary map. (Closed)

Created:
6 years, 6 months ago by Vitaly Buka (NO REVIEWS)
Modified:
6 years, 6 months ago
Reviewers:
gene
CC:
chromium-reviews
Visibility:
Public.

Description

Patch Set 1 #

Patch Set 2 : Wed 05/28/2014 22:06:08.94 #

Patch Set 3 : Wed 05/28/2014 22:09:12.72 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+6 lines, -18 lines) Patch
M pdf/document_loader.h View 2 chunks +0 lines, -2 lines 0 comments Download
M pdf/document_loader.cc View 1 2 2 chunks +6 lines, -16 lines 0 comments Download

Messages

Total messages: 8 (0 generated)
Vitaly Buka (NO REVIEWS)
6 years, 6 months ago (2014-05-29 00:43:07 UTC) #1
gene
lgtm Could you please add comment explaining strategy? For example // For first 10 requests, ...
6 years, 6 months ago (2014-05-29 01:57:24 UTC) #2
Vitaly Buka (NO REVIEWS)
The CQ bit was checked by vitalybuka@chromium.org
6 years, 6 months ago (2014-05-29 05:10:00 UTC) #3
Vitaly Buka (NO REVIEWS)
The CQ bit was unchecked by vitalybuka@chromium.org
6 years, 6 months ago (2014-05-29 05:10:10 UTC) #4
Vitaly Buka (NO REVIEWS)
On 2014/05/29 01:57:24, gene wrote: > lgtm > > Could you please add comment explaining ...
6 years, 6 months ago (2014-05-29 05:10:12 UTC) #5
Vitaly Buka (NO REVIEWS)
The CQ bit was checked by vitalybuka@chromium.org
6 years, 6 months ago (2014-05-29 05:10:18 UTC) #6
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-status.appspot.com/cq/vitalybuka@chromium.org/301033003/40001
6 years, 6 months ago (2014-05-29 05:10:49 UTC) #7
commit-bot: I haz the power
6 years, 6 months ago (2014-05-29 20:41:33 UTC) #8
Message was sent while issue was closed.
Change committed as 273580

Powered by Google App Engine
This is Rietveld 408576698