Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(69)

Issue 4822053: Add a test to make sure that clearing all product events clears the values (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
13 years, 11 months ago by Roger Tawa
Modified:
13 years, 11 months ago
Reviewers:
gwilson
CC:
rlz-codereviews_googlegroups.com
Base URL:
http://rlz.googlecode.com/svn/trunk/
Visibility:
Public.

Description

Add a test to make sure that clearing all product events clears the values of stateful events too. This is tested by trying to record a new stateful event after all the events has been cleared; the recording should succeed. Committed: http://code.google.com/p/rlz/source/detail?r=45

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -0 lines) Patch
M win/lib/rlz_lib_test.cc View 1 chunk +10 lines, -0 lines 0 comments Download

Messages

Total messages: 2
Roger Tawa
13 years, 11 months ago (2011-07-29 18:35:53 UTC) #1
gwilson
13 years, 11 months ago (2011-07-29 18:44:39 UTC) #2
LGTM
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b