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

Issue 3012263002: Move log_count_ out of RtcEventLogImpl (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 days, 20 hours ago by eladalon
Modified:
5 days, 20 hours 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
6 days, 20 hours ago (2017-09-13 14:37:50 UTC) #2
eladalon
5 days, 20 hours ago (2017-09-14 14:54:24 UTC) #3
Sign in to reply to this message.

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