Web Inspector: console.assert messages shouldn't end in an exclamation point

[Matt Baker]

No need to be so dramatic.

[Matt Baker]

Created attachment 321046 [details]
Patch

[Joseph Pecoraro]

Comment on attachment 321046 [details]
Patch

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

r=me

> Source/WebInspectorUI/UserInterface/Views/Resizer.js:76
> +        console.assert(false, `Unexpected Resizer orientation: ${this._orientation}.`);

I'd prefer including the orientation as its own parameter instead of an inline string.

    console.assert(false, "Unexpected Resizer orientation", this._orientation);

[Matt Baker]

Created attachment 321120 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 321120 [details]
Patch for landing

Clearing flags on attachment: 321120

Committed r222174: <http://trac.webkit.org/changeset/222174>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Radar WebKit Bug Importer]

<rdar://problem/34693261>