Chromium Code Reviews| Index: chrome/browser/net/net_log_logger.h |
| =================================================================== |
| --- chrome/browser/net/net_log_logger.h (revision 171385) |
| +++ chrome/browser/net/net_log_logger.h (working copy) |
| @@ -5,7 +5,12 @@ |
| #ifndef CHROME_BROWSER_NET_NET_LOG_LOGGER_H_ |
| #define CHROME_BROWSER_NET_NET_LOG_LOGGER_H_ |
| +#include <string> |
| +#include <vector> |
| + |
| +#include "base/file_path.h" |
| #include "base/memory/scoped_handle.h" |
| +#include "base/memory/scoped_ptr.h" |
| #include "net/base/net_log.h" |
| class FilePath; |
| @@ -35,8 +40,25 @@ |
| // net::NetLog::ThreadSafeObserver implementation: |
| virtual void OnAddEntry(const net::NetLog::Entry& entry) OVERRIDE; |
| + // Start collecting NetLog data into a temporary file. |
| + static void StartNetLog(); |
|
qsr
2012/12/10 10:51:07
Did you see with any of the Owners if that is fine
|
| + |
| + // Stop collecting NetLog data into a temporary file. |
| + static void StopNetLog(); |
| + |
| + // Send NetLog data that is in the temporary file. |
| + static void SendNetLog(); |
| + |
| private: |
| + // Open the file specified by |log_path_|. |
| + void OpenFile(); |
| + |
| + // Send NetLog via email. |
| + static void SendEmail(const FilePath::StringType& file_to_attch); |
|
qsr
2012/12/10 10:51:07
1. attach
2. Why isn't this a const FilePath& ?
|
| + |
| ScopedStdioHandle file_; |
| + FilePath log_path_; |
| + std::vector<FilePath::StringType> files_to_delete_; |
|
qsr
2012/12/10 10:51:07
Same thing here, why isn't those FilePath?
|
| DISALLOW_COPY_AND_ASSIGN(NetLogLogger); |
| }; |