Revert "Reland "DevTools: Support CPU profiles in OOPIFs""

This reverts commit 250e7ef1.

Reason for revert: oopif-performance-cpu-profiles.js failing
consistently in:
https://ci.chromium.org/buildbot/chromium.webkit/WebKit%20Linux%20Trusty%20MSAN/7812

Original change's description:
> Reland "DevTools: Support CPU profiles in OOPIFs"
> 
> This is a reland of 91d5ec53
> 
> Original change's description:
> > DevTools: Support CPU profiles in OOPIFs
> >
> > BUG=832313,820250
> >
> > Change-Id: I53d730d4ff7e747eb0ccfe9a055738c1fe9858cf
> > Reviewed-on: https://chromium-review.googlesource.com/1011350
> > Commit-Queue: Alexei Filippov <alph@chromium.org>
> > Reviewed-by: Dmitry Gozman <dgozman@chromium.org>
> > Cr-Commit-Position: refs/heads/master@{#553688}
> 
> TBR=dgozman
> 
> Bug: 832313, 820250
> Change-Id: I177e0d404b8efe595ddc680368b21c32160cd5de
> Reviewed-on: https://chromium-review.googlesource.com/1065230
> Commit-Queue: Alexei Filippov <alph@chromium.org>
> Reviewed-by: Alexei Filippov <alph@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#560049}

TBR=dgozman@chromium.org,alph@chromium.org

# Not skipping CQ checks because original CL landed > 1 day ago.

Bug: 832313, 820250
Change-Id: Idf9a6925438105817159658fd8fa19ba00d1b530
Reviewed-on: https://chromium-review.googlesource.com/1065539Reviewed-by: calamity <calamity@chromium.org>
Commit-Queue: calamity <calamity@chromium.org>
Cr-Commit-Position: refs/heads/master@{#560229}

third_party/WebKit/LayoutTests/http/tests/devtools/oopif/oopif-performance-cpu-profiles-expected.txt

-Test CPU profiles are recorded for OOPIFs.
-
-name: CrRendererMain
-url: http://127.0.0.1:8000/devtools/oopif/resources/page.html
-has CpuProfile: true
-
-name: CrRendererMain
-url: http://devtools.oopif.test:8000/devtools/oopif/resources/inner-iframe.html?second
-has CpuProfile: true
-
-

third_party/WebKit/LayoutTests/http/tests/devtools/oopif/oopif-performance-cpu-profiles.js

-// Copyright 2018 The Chromium Authors. All rights reserved.
-// Use of this source code is governed by a BSD-style license that can be
-// found in the LICENSE file.
-
-(async function() {
-  TestRunner.addResult(`Test CPU profiles are recorded for OOPIFs.\n`);
-
-  await TestRunner.loadModule('performance_test_runner');
-  await TestRunner.showPanel('timeline');
-
-  await PerformanceTestRunner.startTimeline();
-  await TestRunner.navigatePromise('resources/page.html');
-  await PerformanceTestRunner.stopTimeline();
-
-  for (const track of PerformanceTestRunner.timelineModel().tracks().sort((a, b) => a.url > b.url)) {
-    if (track.type !== TimelineModel.TimelineModel.TrackType.MainThread)
-      continue;
-    TestRunner.addResult(`name: ${track.name}`);
-    TestRunner.addResult(`url: ${track.url}`);
-    TestRunner.addResult(`has CpuProfile: ${track.thread.events().some(e => e.name === 'CpuProfile')}\n`);
-  }
-
-  TestRunner.completeTest();
-})();

third_party/blink/renderer/devtools/front_end/platform/utilities.js

@@ -1204,10 +1204,13 @@ Multimap.prototype = {
 
   /**
    * @param {K} key
-   * @return {!Set<!V>}
+   * @return {!Set.<!V>}
    */
   get: function(key) {
-    return this._map.get(key) || new Set();
+    let result = this._map.get(key);
+    if (!result)
+      result = new Set();
+    return result;
   },
 
   /**

third_party/blink/renderer/devtools/front_end/sdk/TracingModel.js

@@ -292,14 +292,6 @@ SDK.TracingModel = class {
     return this._processByName.get(name);
   }
 
-  /**
-   * @param {number} pid
-   * @return {?SDK.TracingModel.Process}
-   */
-  processById(pid) {
-    return this._processById.get(pid) || null;
-  }
-
   /**
    * @param {string} processName
    * @param {string} threadName

third_party/blink/renderer/devtools/front_end/timeline/TimelineController.js

@@ -26,10 +26,6 @@ Timeline.TimelineController = class {
     SDK.targetManager.observeModels(SDK.CPUProfilerModel, this);
   }
 
-  dispose() {
-    SDK.targetManager.unobserveModels(SDK.CPUProfilerModel, this);
-  }
-
   /**
    * @return {!SDK.Target}
    */
@@ -228,74 +224,20 @@ Timeline.TimelineController = class {
     this._tracingModel.addEvents([cpuProfileEvent]);
   }
 
-  /**
-   * @return {?Map<string, number>}
-   */
-  _buildTargetToProcessIdMap() {
-    const metadataEventTypes = TimelineModel.TimelineModel.DevToolsMetadataEvent;
-    const metadataEvents = this._tracingModel.devToolsMetadataEvents();
-    const browserMetaEvent = metadataEvents.find(e => e.name === metadataEventTypes.TracingStartedInBrowser);
-    if (!browserMetaEvent)
-      return null;
-
-    /** @type {!Multimap<string, string>} */
-    const pseudoPidToFrames = new Multimap();
-    /** @type {!Map<string, number>} */
-    const targetIdToPid = new Map();
-    const frames = browserMetaEvent.args.data['frames'];
-    for (const frameInfo of frames)
-      targetIdToPid.set(frameInfo.frame, frameInfo.processId);
-    for (const event of metadataEvents) {
-      const data = event.args.data;
-      switch (event.name) {
-        case metadataEventTypes.FrameCommittedInBrowser:
-          if (data.processId)
-            targetIdToPid.set(data.frame, data.processId);
-          else
-            pseudoPidToFrames.set(data.processPseudoId, data.frame);
-          break;
-        case metadataEventTypes.ProcessReadyInBrowser:
-          for (const frame of pseudoPidToFrames.get(data.processPseudoId) || [])
-            targetIdToPid.set(frame, data.processId);
-          break;
-      }
-    }
-    const mainFrame = frames.find(frame => !frame.parent);
-    const mainRendererProcessId = mainFrame.processId;
-    const mainProcess = this._tracingModel.processById(mainRendererProcessId);
-    if (mainProcess)
-      targetIdToPid.set(SDK.targetManager.mainTarget().id(), mainProcess.id());
-    return targetIdToPid;
-  }
-
   _injectCpuProfileEvents() {
     if (!this._cpuProfiles)
       return;
 
     const metadataEventTypes = TimelineModel.TimelineModel.DevToolsMetadataEvent;
     const metadataEvents = this._tracingModel.devToolsMetadataEvents();
+    const mainMetaEvent =
+        metadataEvents.filter(event => event.name === metadataEventTypes.TracingStartedInPage).peekLast();
+    if (!mainMetaEvent)
+      return;
 
-    const targetIdToPid = this._buildTargetToProcessIdMap();
-    if (targetIdToPid) {
-      for (const [id, profile] of this._cpuProfiles) {
-        const pid = targetIdToPid.get(id);
-        if (!pid)
-          continue;
-        const process = this._tracingModel.processById(pid);
-        const thread = process && process.threadByName(TimelineModel.TimelineModel.RendererMainThreadName);
-        if (thread)
-          this._injectCpuProfileEvent(pid, thread.id(), profile);
-      }
-    } else {
-      // Legacy backends support.
-      const mainMetaEvent =
-          metadataEvents.filter(event => event.name === metadataEventTypes.TracingStartedInPage).peekLast();
-      if (mainMetaEvent) {
-        const pid = mainMetaEvent.thread.process().id();
-        const mainCpuProfile = this._cpuProfiles.get(this._tracingManager.target().id());
-        this._injectCpuProfileEvent(pid, mainMetaEvent.thread.id(), mainCpuProfile);
-      }
-    }
+    const pid = mainMetaEvent.thread.process().id();
+    const mainCpuProfile = this._cpuProfiles.get(this._tracingManager.target().id());
+    this._injectCpuProfileEvent(pid, mainMetaEvent.thread.id(), mainCpuProfile);
 
     const workerMetaEvents =
         metadataEvents.filter(event => event.name === metadataEventTypes.TracingSessionIdForWorker);

third_party/blink/renderer/devtools/front_end/timeline/TimelinePanel.js

@@ -498,7 +498,6 @@ Timeline.TimelinePanel = class extends UI.Panel {
     this._setState(Timeline.TimelinePanel.State.StopPending);
     this._performanceModel = await this._controller.stopRecording();
     this._setUIControlsEnabled(true);
-    this._controller.dispose();
     this._controller = null;
   }