New issue
Advanced search Search tips

Issue 913231 link

Starred by 2 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 2
Type: Bug



Sign in to add a comment

Merge [Allow the caller of startEventLogging to specify the output period] into M72

Project Member Reported by eladalon@chromium.org, Dec 9

Issue description

Project Member

Comment 1 by sheriffbot@chromium.org, Dec 10

Labels: -Merge-Request-72 Hotlist-Merge-Approved Merge-Approved-72
Your change meets the bar and is auto-approved for M72. Please go ahead and merge the CL to branch 3626 manually. Please contact milestone owner if you have questions.
Owners: govind@(Android), kariahda@(iOS), djmm@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 2 by bugdroid1@chromium.org, Dec 10

Labels: -merge-approved-72 merge-merged-3626
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6

commit 1178b7e6cd0e4afd83264c26ea64d4ead93d88a6
Author: Elad Alon <eladalon@chromium.org>
Date: Mon Dec 10 20:47:55 2018

Allow the caller of startEventLogging to specify the output period

(cherry picked from commit a085b1d39d62fc2246f771132604b5ae096cfc17)

Bug: 775415, 913231
Change-Id: I4828416070d2fcd5955939edd319cd468d7c77c3
Reviewed-on: https://chromium-review.googlesource.com/c/1358438
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Henrik Grunell <grunell@chromium.org>
Commit-Queue: Elad Alon <eladalon@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#614985}
Reviewed-on: https://chromium-review.googlesource.com/c/1369943
Reviewed-by: Elad Alon <eladalon@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#233}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_api.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/extensions/api/webrtc_logging_private/webrtc_logging_private_apitest.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager_common.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager_common.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager_remote.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager_remote.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_event_log_manager_unittest.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_logging_handler_host.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/media/webrtc/webrtc_logging_handler_host.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/policy/policy_browsertest.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/resources/hangout_services/manifest.json
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/browser/resources/hangout_services/thunk.js
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/chrome/common/extensions/api/webrtc_logging_private.idl
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/browser/renderer_host/render_process_host_impl.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/browser/renderer_host/render_process_host_impl.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/common/media/peer_connection_tracker_messages.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/public/browser/render_process_host.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/public/test/mock_render_process_host.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/public/test/mock_render_process_host.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/renderer/media/webrtc/peer_connection_tracker.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/renderer/media/webrtc/peer_connection_tracker.h
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/renderer/media/webrtc/rtc_peer_connection_handler.cc
[modify] https://crrev.com/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6/content/renderer/media/webrtc/rtc_peer_connection_handler.h

Labels: Merge-Merged-72-3626
The following revision refers to this bug: 
https://chromium.googlesource.com/chromium/src.git/+/1178b7e6cd0e4afd83264c26ea64d4ead93d88a6

Commit: 1178b7e6cd0e4afd83264c26ea64d4ead93d88a6
Author: eladalon@chromium.org
Commiter: eladalon@chromium.org
Date: 2018-12-10 20:47:55 +0000 UTC

Allow the caller of startEventLogging to specify the output period

(cherry picked from commit a085b1d39d62fc2246f771132604b5ae096cfc17)

Bug: 775415, 913231
Change-Id: I4828416070d2fcd5955939edd319cd468d7c77c3
Reviewed-on: https://chromium-review.googlesource.com/c/1358438
Reviewed-by: Guido Urdaneta <guidou@chromium.org>
Reviewed-by: Ken Buchanan <kenrb@chromium.org>
Reviewed-by: Avi Drissman <avi@chromium.org>
Reviewed-by: Devlin <rdevlin.cronin@chromium.org>
Reviewed-by: Henrik Grunell <grunell@chromium.org>
Commit-Queue: Elad Alon <eladalon@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#614985}
Reviewed-on: https://chromium-review.googlesource.com/c/1369943
Reviewed-by: Elad Alon <eladalon@chromium.org>
Cr-Commit-Position: refs/branch-heads/3626@{#233}
Cr-Branched-From: d897fb137fbaaa9355c0c93124cc048824eb1e65-refs/heads/master@{#612437}

Sign in to add a comment