[network] Fix response view for wasm requests

The CL https://crrev.com/c/5569109 broke Network > Response for
wasm requests. Since we don't connect NetworkRequest with Script, its
not enough to just disassemble WASM in Script.

This CL re-instates the adhoc disassembling of wasm to the SourceFrame
and adds a better test.

[email protected]

Bug: 40240361
Fixed: 375408034
Change-Id: I41d000ecf9b57f89ac0ea2b2b6b5c0918be01b85
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/5964716
Commit-Queue: Simon Zünd <[email protected]>
Reviewed-by: Philip Pfaffe <[email protected]>

Author: szuend

front_end/core/sdk/Script.ts

@@ -498,7 +498,7 @@ function frameIdForScript(script: Script): Protocol.Page.FrameId|null {
 
 export const sourceURLRegex = /^[\x20\t]*\/\/[@#] sourceURL=\s*(\S*?)\s*$/;
 
-async function disassembleWasm(content: string): Promise<TextUtils.WasmDisassembly.WasmDisassembly> {
+export async function disassembleWasm(content: string): Promise<TextUtils.WasmDisassembly.WasmDisassembly> {
   const worker = Common.Worker.WorkerWrapper.fromURL(
       new URL('../../entrypoints/wasmparser_worker/wasmparser_worker-entrypoint.js', import.meta.url));
   const promise = new Promise<TextUtils.WasmDisassembly.WasmDisassembly>((resolve, reject) => {

front_end/ui/legacy/components/source_frame/SourceFrame.test.ts

@@ -10,6 +10,7 @@ import {
   dispatchPasteEvent,
 } from '../../../../testing/DOMHelpers.js';
 import {describeWithEnvironment} from '../../../../testing/EnvironmentHelpers.js';
+import {expectCall} from '../../../../testing/ExpectStubCall.js';
 import * as Buttons from '../../../components/buttons/buttons.js';
 import * as UI from '../../legacy.js';
 
@@ -125,4 +126,21 @@ describeWithEnvironment('SourceFrame', () => {
     assert.isNull(dialogContainer);
     stub.restore();
   });
+
+  it('disassembles wasm', async () => {
+    const contentData = new TextUtils.ContentData.ContentData(
+        'AGFzbQEAAAABBQFgAAF/AwIBAAcHAQNiYXIAAAoGAQQAQQILACQEbmFtZQAQD3Nob3ctd2FzbS0yLndhdAEGAQADYmFyAgMBAAA=', true,
+        'application/wasm');
+    const sourceFrame = new SourceFrame.SourceFrame.SourceFrameImpl(async () => contentData);
+
+    sourceFrame.markAsRoot();
+    const setContentStub = sinon.stub(sourceFrame, 'setContent');
+    sourceFrame.show(document.body);
+    const content = await expectCall(setContentStub);
+
+    assert.strictEqual(
+        content.toString(), '(module\n  (func $bar (;0;) (export \"bar\") (result i32)\n    i32.const 2\n  )\n)');
+
+    sourceFrame.detach();
+  });
 });

front_end/ui/legacy/components/source_frame/SourceFrame.ts

@@ -33,6 +33,7 @@ import * as Host from '../../../../core/host/host.js';
 import * as i18n from '../../../../core/i18n/i18n.js';
 import * as Platform from '../../../../core/platform/platform.js';
 import * as Root from '../../../../core/root/root.js';
+import * as SDK from '../../../../core/sdk/sdk.js';
 import * as Formatter from '../../../../models/formatter/formatter.js';
 import * as TextUtils from '../../../../models/text_utils/text_utils.js';
 import * as CodeMirror from '../../../../third_party/codemirror.next/codemirror.next.js';
@@ -569,6 +570,11 @@ export class SourceFrameImpl extends Common.ObjectWrapper.eventMixin<EventTypes,
       content = contentData.text;
       isMinified = TextUtils.TextUtils.isMinified(contentData.text);
       this.wasmDisassemblyInternal = null;
+    } else if (contentData.mimeType === 'application/wasm') {
+      // The network panel produces ContentData with raw WASM inside. We have to manually disassemble that
+      // as V8 might not know about it.
+      this.wasmDisassemblyInternal = await SDK.Script.disassembleWasm(contentData.base64);
+      content = CodeMirror.Text.of(this.wasmDisassemblyInternal.lines);
     } else {
       error = i18nString(UIStrings.binaryContentError);
       content = null;