Issue 905723004: Files.app: Rename a variable in drive_api_util.cc for more sanity.

Issue 905723004: Files.app: Rename a variable in drive_api_util.cc for more sanity. (Closed)

Created:
5 years, 10 months ago by Ben Kwa

Modified:
5 years, 10 months ago

Reviewers:
mtomasz, yoshiki

CC:
chromium-reviews

Base URL:
https://chromium.googlesource.com/chromium/src.git@master

Target Ref:
refs/pending/heads/master

Project:
chromium

Visibility:
Public.

More Reviews

Description

Files.app: Rename a variable in drive_api_util.cc for more sanity. BUG=453880 Committed: https://crrev.com/e9aa71290a8a0d05e385694d533a74b63cce7675 Cr-Commit-Position: refs/heads/master@{#315363}

Patch Set 1 #

Total comments: 1

Created: 5 years, 10 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+4 lines, -4 lines)			Patch
	M	chrome/browser/drive/drive_api_util.cc	View		2 chunks	+4 lines, -4 lines	1 comment	Download

Messages

Total messages: 9 (3 generated)

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/905723004/1

5 years, 10 months ago (2015-02-09 19:24:05 UTC) #5

commit-bot: I haz the power

Patchset 1 (id:??) landed as https://crrev.com/e9aa71290a8a0d05e385694d533a74b63cce7675 Cr-Commit-Position: refs/heads/master@{#315363}

5 years, 10 months ago (2015-02-09 19:28:20 UTC) #7

mtomasz

5 years, 10 months ago (2015-02-25 02:36:30 UTC) #9

Message was sent while issue was closed.

https://codereview.chromium.org/905723004/diff/1/chrome/browser/drive/drive_a...
File chrome/browser/drive/drive_api_util.cc (right):

https://codereview.chromium.org/905723004/diff/1/chrome/browser/drive/drive_a...
chrome/browser/drive/drive_api_util.cc:195: void
FileStreamMd5Digester::OnChunkRead(int bytesRead) {
nit: bytes_read.

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages