fix(ext/webstorage): make getOwnPropertyDescriptor with symbol return undefined

[crowlKats]

Closes #13347

[crowlKats]

probably should add a wpt for this

[kitsonk]

LGTM

Yeah, I would raise an issue on the WPT tests for it.

[ah-yu]

deno/ext/webstorage/01_webstorage.js

Lines 133 to 136 in 79b698f

	has(target, p) {
	return p === SymbolFor("Deno.customInspect") ||
	(typeof target.getItem(p)) === "string";
	},

It seems Symbol('foo') in localStorage would throw same error

[bartlomieju]

@crowlKats can this be landed for 1.19?

[crowlKats]

@bartlomieju I need to look into this again; functionality wise its correct, but no tests currently so i'd like to hold off for a later release

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

[lucacasonato]

@crowlKats can you add a test?