Bug 161891 – Web Inspector: Should be able to pretty print module code (import / export statements)

• Component: Web Inspector
• Status: RESOLVED FIXED

[Joseph Pecoraro]

Summary:
Should be able to pretty print module code (import / export statements).

In preparation for module tags. The pretty printer should be able to handle import / export statements.

Notes
- Esprima has two different parse modes. "script" or "module" based on the "sourceType" option.
- The Formatting tool should be updated to test this.

[Radar WebKit Bug Importer]

<rdar://problem/28272784>

[Joseph Pecoraro]

Created attachment 288666 [details]
[PATCH] Proposed Fix

[Yusuke Suzuki]

Comment on attachment 288666 [details]
[PATCH] Proposed Fix

View in context: https://bugs.webkit.org/attachment.cgi?id=288666&action=review

Nice! r=me.

> LayoutTests/inspector/formatting/resources/javascript-tests/modules-expected.js:19
> +    y = 1;

We can perform destructuring here :)
`export let { x, y, z } = obj`
`export let [x, y, z ] = array`

[Joseph Pecoraro]

Created attachment 288706 [details]
[PATCH] For Landing

[WebKit Commit Bot]

Comment on attachment 288706 [details]
[PATCH] For Landing

Clearing flags on attachment: 288706

Committed r205866: <http://trac.webkit.org/changeset/205866>