Bug 67270

Summary: Rename closeColorChooser to cleanupColorChooser
Product: WebKit Reporter: Keishi Hattori <keishi>
Component: FormsAssignee: Keishi Hattori <keishi>
Severity: Normal CC: tkent, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Description Flags
fixed changelog none

Description Keishi Hattori 2011-08-31 01:18:25 PDT
Since closeColorChooser won't be closing the color chooser on Mac OS X, I am renaming it to cleanupColorChooser.
This will cleanup any color chooser listeners and on other platforms other than Macs the color chooser will close.
Comment 1 Keishi Hattori 2011-08-31 01:22:03 PDT
Will be uploading patch when Bug 67267 lands
Comment 2 Keishi Hattori 2011-08-31 19:19:13 PDT
Created attachment 105884 [details]
Comment 3 Kent Tamura 2011-08-31 19:30:20 PDT
Comment on attachment 105884 [details]

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

> Source/WebCore/ChangeLog:12
> +        I need to take care of any color chooser listeners, I am renaming this to cleanup. This will close the color

Using the word 'I' in ChangeLog looks strange.

> Source/WebCore/ChangeLog:25
> +        * page/ChromeClient.h:
> +
> +2011-08-31  Keishi Hattori  <keishi@webkit.org>
> +
> +        Remove closeColorChooser call from FrameLoader::transitionToCommitted
> +        https://bugs.webkit.org/show_bug.cgi?id=67267

There are two ChangeLog entries.
Comment 4 Keishi Hattori 2011-08-31 20:20:49 PDT
Created attachment 105889 [details]
fixed changelog
Comment 5 Kent Tamura 2011-08-31 20:25:38 PDT
Comment on attachment 105889 [details]
fixed changelog

Comment 6 WebKit Review Bot 2011-08-31 22:32:41 PDT
Comment on attachment 105889 [details]
fixed changelog

Clearing flags on attachment: 105889

Committed r94270: <http://trac.webkit.org/changeset/94270>
Comment 7 WebKit Review Bot 2011-08-31 22:32:45 PDT
All reviewed patches have been landed.  Closing bug.