Summary: | [EFL] Remove WebCore::currentTime() | ||||||||
---|---|---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Patrick R. Gansterer <paroga> | ||||||
Component: | Platform | Assignee: | Patrick R. Gansterer <paroga> | ||||||
Status: | RESOLVED FIXED | ||||||||
Severity: | Normal | CC: | antognolli+webkit, gyuyoung.kim, kenneth, leandro, lucas.de.marchi, ryuan.choi, tonikitoo | ||||||
Priority: | P2 | ||||||||
Version: | 528+ (Nightly build) | ||||||||
Hardware: | Other | ||||||||
OS: | Other | ||||||||
Attachments: |
|
Description
Patrick R. Gansterer
2011-02-06 11:53:44 PST
Created attachment 81423 [details]
Patch
add WebKit EFL reviewers/committers to CC. It looks OK to me. LGTM. Comment on attachment 81423 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=81423&action=review > Source/WebCore/ChangeLog:8 > + There is no declaration for it and we use WTF::currentTime() everywherey. typo: everywhereY. > Source/WebCore/platform/efl/SystemTimeEfl.cpp:2 > - * Copyright (C) 2008 INdT. All rights reserved. > - * Copyright (C) 2009-2010 ProFUSION embedded systems > - * Copyright (C) 2009-2010 Samsung Electronics > + * Copyright (C) 2009 Apple Inc. All Rights Reserved. Why this? Created attachment 83069 [details] Patch (In reply to comment #4) > > Source/WebCore/ChangeLog:8 > > + There is no declaration for it and we use WTF::currentTime() everywherey. > > typo: everywhereY. Fixed. > > Source/WebCore/platform/efl/SystemTimeEfl.cpp:2 > > - * Copyright (C) 2008 INdT. All rights reserved. > > - * Copyright (C) 2009-2010 ProFUSION embedded systems > > - * Copyright (C) 2009-2010 Samsung Electronics > > + * Copyright (C) 2009 Apple Inc. All Rights Reserved. > > Why this? Now it's the same file as the other SystemTime*.cpp files of the other ports. Comment on attachment 83069 [details] Patch Clearing flags on attachment: 83069 Committed r79122: <http://trac.webkit.org/changeset/79122> All reviewed patches have been landed. Closing bug. |