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

Issue 1897973004: Rename CodexTest.cpp to CodecTest.cpp (Closed)

Created:
4 years, 8 months ago by scroggo
Modified:
4 years, 8 months ago
Reviewers:
msarett
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Rename CodexTest.cpp to CodecTest.cpp CodexTest confuses me almost every time I try to open it. Renaming it to what my brain thinks it should be called. GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1897973004 Committed: https://skia.googlesource.com/skia/+/2477594a3953d93c7fe22cfd03289abfba5a2d82

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+-1 lines, -973 lines) Patch
A + tests/CodecTest.cpp View 0 chunks +-1 lines, --1 lines 0 comments Download
D tests/CodexTest.cpp View 1 chunk +0 lines, -974 lines 0 comments Download

Messages

Total messages: 8 (4 generated)
scroggo
CLs on this test have died down, so I assume there won't be any conflicts ...
4 years, 8 months ago (2016-04-19 14:46:56 UTC) #3
msarett
LGTM
4 years, 8 months ago (2016-04-19 14:47:26 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1897973004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1897973004/1
4 years, 8 months ago (2016-04-19 15:11:17 UTC) #6
commit-bot: I haz the power
4 years, 8 months ago (2016-04-19 15:27:43 UTC) #8
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/2477594a3953d93c7fe22cfd03289abfba5a2d82

Powered by Google App Engine
This is Rietveld 408576698