Update prepare-ChangeLog to work with ES6 Class syntax

• Component: Tools / Tests
• Status: RESOLVED FIXED

[Joseph Pecoraro]

* SUMMARY
Update prepare-ChangeLog to work with ES6 Class syntax. Currently it doesn't understand method syntax, so any modifications to Web Inspector files with class syntax produces nothing.

[Joseph Pecoraro]

Created attachment 249465 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

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

I just realized there are tests. Let me add some quick tests.

[Joseph Pecoraro]

Created attachment 249468 [details]
[PATCH] Proposed Fix

[Joseph Pecoraro]

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

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

> Tools/Scripts/prepare-ChangeLog:1393
> +                    $currentMethod =~ s/\.{2,}/\./g; # Removes consecutive periods.

I have a feeling this line is not needed, insect later when this is pop'd it'll do this. So cq- and I can fix locally if that is the case.

[Joseph Pecoraro]

Created attachment 249469 [details]
[PATCH] Proposed Fix

Nope, that caused issue. But I still tweaked things just a bit to remove duplicated work.

[WebKit Commit Bot]

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

Clearing flags on attachment: 249469

Committed r181996: <http://trac.webkit.org/changeset/181996>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.