Revert of DevTools: add the logging aspect into the PerformanceMonitor

Revert of DevTools: add the logging aspect into the PerformanceMonitor (patchset #6 id:100001 of https://codereview.chromium.org/2474073005/ )

Reason for revert:
Speculative revert as sheriff since we see a failure in WorkerDevToolsSanityTest.PauseInSharedWorkerInitialization on Win10.

See link below for details:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win10%20Tests%20x64/builds/5799/steps/browser_tests%20on%20Windows-10-10586/logs/WorkerDevToolsSanityTest.PauseInSharedWorkerInitialization

Original issue's description:
> DevTools: add the logging aspect into the PerformanceMonitor, plumb it over the remote debugging protocol, surface it in DevTools. Also adds proof of concept violation reporting for layout thrashing and long running tasks.
> 
> BUG=662497
> 
> Committed: https://crrev.com/8bf066cc7d7529342fd260e5bf3d98a97c6b133f
> Cr-Commit-Position: refs/heads/master@{#430418}

TBR=panicker@chromium.org,dgozman@chromium.org,caseq@chromium.org,pfeldman@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662497

[henrika (OOO until Aug 14), 2016-11-08 12:07:48]

The CQ bit was checked by henrika@chromium.org

[henrika (OOO until Aug 14), 2016-11-08 12:07:49]

Created Revert of DevTools: add the logging aspect into the PerformanceMonitor

[commit-bot: I haz the power, 2016-11-08 12:08:06]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-11-08 12:08:57]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-08 12:08:58]

Failed to apply patch for
third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp:
While running git apply --index -3 -p1;
  error: patch failed:
third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp:69
  Falling back to three-way merge...
  Applied patch to
'third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp' with
conflicts.
  U third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp

Patch:      
third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
Index: third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
diff --git
a/third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
b/third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
index
ed4e3ee586ad526cab5b7f884d0ea0af6b7f0d5b..1d4dbda772ab9ed214f5c6f7e74b3394b8dd1624
100644
--- a/third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
+++ b/third_party/WebKit/Source/core/inspector/WorkerInspectorController.cpp
@@ -69,8 +69,7 @@
   m_session = new InspectorSession(this, m_instrumentingAgents.get(), 0,
                                    m_debugger->v8Inspector(),
                                    m_debugger->contextGroupId(), nullptr);
-  m_session->append(
-      new InspectorLogAgent(m_thread->consoleMessageStorage(), nullptr));
+  m_session->append(new InspectorLogAgent(m_thread->consoleMessageStorage()));
   m_thread->workerBackingThread().backingThread().addTaskObserver(this);
 }

[dgozman, 2016-11-16 05:51:41]

Description was changed from

==========
Revert of DevTools: add the logging aspect into the PerformanceMonitor (patchset
#6 id:100001 of https://codereview.chromium.org/2474073005/ )

Reason for revert:
Speculative revert as sheriff since we see a failure in
WorkerDevToolsSanityTest.PauseInSharedWorkerInitialization on Win10.

See link below for details:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win10%20Tests%20...

Original issue's description:
> DevTools: add the logging aspect into the PerformanceMonitor, plumb it over
the remote debugging protocol, surface it in DevTools. Also adds proof of
concept violation reporting for layout thrashing and long running tasks.
> 
> BUG=662497
> 
> Committed: https://crrev.com/8bf066cc7d7529342fd260e5bf3d98a97c6b133f
> Cr-Commit-Position: refs/heads/master@{#430418}

TBR=panicker@chromium.org,dgozman@chromium.org,caseq@chromium.org,pfeldman@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662497
==========

to

==========
Revert of DevTools: add the logging aspect into the PerformanceMonitor (patchset
#6 id:100001 of https://codereview.chromium.org/2474073005/ )

Reason for revert:
Speculative revert as sheriff since we see a failure in
WorkerDevToolsSanityTest.PauseInSharedWorkerInitialization on Win10.

See link below for details:

https://uberchromegw.corp.google.com/i/chromium.win/builders/Win10%20Tests%20...

Original issue's description:
> DevTools: add the logging aspect into the PerformanceMonitor, plumb it over
the remote debugging protocol, surface it in DevTools. Also adds proof of
concept violation reporting for layout thrashing and long running tasks.
> 
> BUG=662497
> 
> Committed: https://crrev.com/8bf066cc7d7529342fd260e5bf3d98a97c6b133f
> Cr-Commit-Position: refs/heads/master@{#430418}

TBR=panicker@chromium.org,dgozman@chromium.org,caseq@chromium.org,pfeldman@ch...
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=662497
==========

[dgozman, 2016-11-16 05:51:41]

dgozman@chromium.org changed reviewers:
- dgozman@chromium.org

[caseq, 2016-11-16 19:11:41]

caseq@chromium.org changed reviewers:
- caseq@chromium.org