Bug 164706 – Remove many includes of ExceptionCode.h

• Component: Bindings
• Status: RESOLVED FIXED

[Darin Adler]

Remove many includes of ExceptionCode.h

[Darin Adler]

Created attachment 294689 [details]
Patch

[Darin Adler]

Created attachment 294690 [details]
Patch

[Darin Adler]

Created attachment 294693 [details]
Patch

[Alex Christensen]

Comment on attachment 294693 [details]
Patch

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

> Source/WebCore/Modules/webdatabase/Database.cpp:181
> +using GuidDatabaseMap = HashMap<DatabaseGUID, std::unique_ptr<HashSet<Database*>>>;

Could we get rid of the unique_ptr here?

[WebKit Commit Bot]

Comment on attachment 294693 [details]
Patch

Rejecting attachment 294693 [details] from commit-queue.

Failed to run "['/Volumes/Data/EWS/WebKit/Tools/Scripts/webkit-patch', '--status-host=webkit-queues.webkit.org', '--bot-id=webkit-cq-03', 'land-attachment', '--force-clean', '--non-interactive', '--parent-command=commit-queue', 294693, '--port=mac']" exit_code: 2 cwd: /Volumes/Data/EWS/WebKit

Last 500 characters of output:
0-ab3c-d52691b4dbfc ...
Currently at 208680 = 60ab07e7402f70b0feef683c6d95233ae67b963a
r208681 = 20904491d96b304c2b42cfd0c9dac23089ca51a4
r208682 = f0fcd5a65b62bb57ab12a7c5e39d9d1dc2828bf4
r208683 = da335709346b23bbb626212f35cccde87784b565
r208684 = 9bcba2339b23e84b6fa2bb2907e906485062586d
Done rebuilding .git/svn/refs/remotes/origin/master/.rev_map.268f45cc-cd09-0410-ab3c-d52691b4dbfc
First, rewinding head to replay your work on top of it...
Fast-forwarded master to refs/remotes/origin/master.

Full output: http://webkit-queues.webkit.org/results/2514166

[Darin Adler]

Comment on attachment 294693 [details]
Patch

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

>> Source/WebCore/Modules/webdatabase/Database.cpp:181
>> +using GuidDatabaseMap = HashMap<DatabaseGUID, std::unique_ptr<HashSet<Database*>>>;
> 
> Could we get rid of the unique_ptr here?

Sure. The hard thing for me when doing something like this is deciding where to stop. I often see things like unnecessary heap allocations or even algorithms that iterate an entire collection rather than keeping something that’s indexed, and it’s hard for me to know which to change in a given patch. I suppose the downside is that the map might be larger because empty cells would be considerably larger.

[Darin Adler]

Committed r208688: <http://trac.webkit.org/changeset/208688>