Chromium Code Reviews
Help | Chromium Project | Gerrit Changes | Sign in
(4)

Issue 3012263002: Move log_count_ out of RtcEventLogImpl (Closed)

Created:
2 months, 1 week ago by eladalon
Modified:
2 months, 1 week ago
Reviewers:
terelius
CC:
webrtc-reviews_webrtc.org, tterriberry_mozilla.com
Target Ref:
refs/heads/master
Project:
webrtc
Visibility:
Public.

Description

Move log_count_ out of RtcEventLogImpl The limit on logs is not specific to the implementation of the logs, but is rather shared between all possible logs. Also, by making it local to the .cc, not a member, we reduce the necessity of making RtcEventLog::Create a friend of the implementation. This necessity is removed completely by a following CL. BUG=webrtc:8111

Patch Set 1 #

Messages

Total messages: 3 (1 generated)
eladalon
PTAL
2 months, 1 week ago (2017-09-13 14:37:50 UTC) #2
eladalon
2 months, 1 week ago (2017-09-14 14:54:24 UTC) #3

Powered by Google App Engine
This is Rietveld efc10ee0f