[chromedriver] Prevent GetLog from returning more than 100,000 at a time.

chrome/test/chromedriver/logging.h

Index: chrome/test/chromedriver/logging.h
diff --git a/chrome/test/chromedriver/logging.h b/chrome/test/chromedriver/logging.h
index e2797d6cf2c0ee32325c943a18d3d2c7d6ca6a1e..574f77c7c9b387106cd30edfbc44abd4bdc11b9b 100644
--- a/chrome/test/chromedriver/logging.h
+++ b/chrome/test/chromedriver/logging.h
@@ -27,6 +27,7 @@ class WebDriverLog : public Log {
   static const char kBrowserType[];
   static const char kDriverType[];
   static const char kPerformanceType[];
+  static const size_t kMaxReturnedEntries;

[klm, 2017/01/19 20:47:30]

Is this for testing only? I don't believe there is any use for actual callers.
Is there an equivalent of e.g. @VisibleForTesting in Chromium C++, or some
coding pattern for exposing a constant to tests only, without exposing it to
real callers?

[samuong, 2017/01/19 21:15:37]

We could make it private and then friend a test class, wdyt?

[klm, 2017/01/19 21:26:17]

I have no strong opinion, was just pointing out something that I didn't quite
like:) Up to you and Shuotao what to do with it, if anything at all.

 
   // Converts WD wire protocol level name -> Level, false on bad name.
   static bool NameToLevel(const std::string& name, Level* out_level);