Bug 79436 - Move HTML-related APIs from DOMWindow.idl to DOMWindowHTML.idl
Summary: Move HTML-related APIs from DOMWindow.idl to DOMWindowHTML.idl
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebCore Misc. (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Kentaro Hara
URL:
Keywords:
Depends on:
Blocks: 79327
  Show dependency tree / graph
 
Reported: 2012-02-23 20:50 PST by Kentaro Hara
Modified: 2012-02-24 05:52 PST (History)
5 users (show)

See Also:

Attachments
Patch (5.47 KB, patch)
2012-02-23 20:52 PST, Kentaro Hara 		no flags Details | Formatted Diff | Diff
patch for landing (5.55 KB, patch)
2012-02-24 01:44 PST, Kentaro Hara 		no flags Details | Formatted Diff | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Kentaro Hara 2012-02-23 20:50:34 PST 
For WebKit modularization, we can move HTML-related APIs from DOMWindow.idl to DOMWindowHTML.idl.
Comment 1 Kentaro Hara 2012-02-23 20:52:19 PST 
Created attachment 128642 [details]
Patch
Comment 2 Adam Barth 2012-02-23 23:30:10 PST 
Comment on attachment 128642 [details]
Patch

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

> Source/WebCore/html/DOMWindowHTML.idl:102
> +#if defined(ENABLE_VIDEO_TRACK) && ENABLE_VIDEO_TRACK

Can we use Conditional?  It's prettier.
Comment 3 Kentaro Hara 2012-02-24 01:44:15 PST 
Created attachment 128686 [details]
patch for landing
Comment 4 Kentaro Hara 2012-02-24 01:44:43 PST 
(In reply to comment #2)
> Can we use Conditional?  It's prettier.

Done.
Comment 5 WebKit Review Bot 2012-02-24 05:50:27 PST 
Comment on attachment 128686 [details]
patch for landing

Clearing flags on attachment: 128686

Committed r108781: <http://trac.webkit.org/changeset/108781>