Bug 174745 - More NeverDestroyed and related cleanup
Summary: More NeverDestroyed and related cleanup
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit Misc. (show other bugs)
Version: WebKit Nightly Build
Hardware: All All
: P2 Normal
Assignee: Darin Adler
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-21 21:09 PDT by Darin Adler
Modified: 2017-07-24 21:29 PDT (History)
1 user (show)

See Also:


Attachments
Patch (99.24 KB, patch)
2017-07-21 21:09 PDT, Darin Adler
no flags Details | Formatted Diff | Diff
Patch (108.77 KB, patch)
2017-07-22 10:06 PDT, Darin Adler
no flags Details | Formatted Diff | Diff
Patch (108.91 KB, patch)
2017-07-22 10:42 PDT, Darin Adler
no flags Details | Formatted Diff | Diff
Patch (109.21 KB, patch)
2017-07-23 13:20 PDT, Darin Adler
no flags Details | Formatted Diff | Diff
Patch (113.94 KB, patch)
2017-07-23 14:57 PDT, Darin Adler
no flags Details | Formatted Diff | Diff
Patch (114.98 KB, patch)
2017-07-24 09:10 PDT, Darin Adler
achristensen: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Darin Adler 2017-07-21 21:09:13 PDT Comment hidden (obsolete)
Comment 1 Darin Adler 2017-07-21 21:09:39 PDT Comment hidden (obsolete)
Comment 2 Darin Adler 2017-07-22 10:06:47 PDT Comment hidden (obsolete)
Comment 3 Darin Adler 2017-07-22 10:42:37 PDT Comment hidden (obsolete)
Comment 4 Darin Adler 2017-07-23 13:20:51 PDT Comment hidden (obsolete)
Comment 5 Darin Adler 2017-07-23 14:57:32 PDT
Created attachment 316240 [details]
Patch
Comment 6 Darin Adler 2017-07-24 09:10:31 PDT
Created attachment 316293 [details]
Patch
Comment 7 Alex Christensen 2017-07-24 11:43:20 PDT
Comment on attachment 316293 [details]
Patch

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

> Source/WebCore/platform/mac/ThemeMac.mm:736
>          case SquareButtonPart:

This indentation should also be updated.
Comment 8 Darin Adler 2017-07-24 15:49:14 PDT
Comment on attachment 316293 [details]
Patch

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

>> Source/WebCore/platform/mac/ThemeMac.mm:736
>>          case SquareButtonPart:
> 
> This indentation should also be updated.

Thanks for spotting that! I will fix before landing.
Comment 9 Darin Adler 2017-07-24 15:53:27 PDT
Comment on attachment 316293 [details]
Patch

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

> Source/WebCore/platform/mac/ThemeMac.mm:751
> +        // by definition constrained, since we select mini only for small cramped environments.

I should also add a closing parenthesis here.
Comment 10 Darin Adler 2017-07-24 21:29:14 PDT
Committed r219858: <http://trac.webkit.org/changeset/219858>