Bug 138265 - Whitespace cleanup in Screen.h
Summary: Whitespace cleanup in Screen.h
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: New Bugs (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Simon Fraser (smfr)
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-10-31 15:38 PDT by Simon Fraser (smfr)
Modified: 2014-10-31 16:21 PDT (History)
2 users (show)

See Also:


Attachments
Patch (2.03 KB, patch)
2014-10-31 15:40 PDT, Simon Fraser (smfr)
cdumez: review+
simon.fraser: commit-queue-
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Simon Fraser (smfr) 2014-10-31 15:38:22 PDT
Whitespace cleanup in Screen.h
Comment 1 Simon Fraser (smfr) 2014-10-31 15:40:08 PDT
Created attachment 240763 [details]
Patch
Comment 2 Chris Dumez 2014-10-31 15:56:38 PDT
Comment on attachment 240763 [details]
Patch

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

> Source/WebCore/page/Screen.h:42
> +class Screen : public ScriptWrappable, public RefCounted<Screen>, public DOMWindowProperty {

nit: we could mark the class as final since we're at it.

> Source/WebCore/page/Screen.h:44
> +    static PassRefPtr<Screen> create(Frame *frame) { return adoptRef(new Screen(frame)); }

nit: Could we fix the star placement as well?
Comment 3 Simon Fraser (smfr) 2014-10-31 16:21:56 PDT
https://trac.webkit.org/r175434