RtcEventLogImpl nits

1. Make |output_period_ms_| optional, so as to clarify where
   it gets assigned a value. (I.e. the value set by the ctor
   is not retained.)
2. Some extra const modifiers.

Bug: webrtc:8111
Change-Id: I9f3ad7ff763cfbc9c9385f7fd4325ba696772765
Reviewed-on: https://webrtc-review.googlesource.com/c/112588
Commit-Queue: Elad Alon <eladalon@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#25877}
diff --git a/logging/BUILD.gn b/logging/BUILD.gn
index 4563306..27be9d5 100644
--- a/logging/BUILD.gn
+++ b/logging/BUILD.gn
@@ -244,6 +244,7 @@
     "../rtc_base:safe_minmax",
     "../rtc_base:sequenced_task_checker",
     "//third_party/abseil-cpp/absl/memory",
+    "//third_party/abseil-cpp/absl/types:optional",
   ]
 
   if (rtc_enable_protobuf) {