Bug 79603

Summary: [EFL] Support mutation observers
Product: WebKit Reporter: ChangSeok Oh <kevin.cs.oh>
Component: WebKit EFLAssignee: Nobody <webkit-unassigned>
Status: RESOLVED FIXED    
Severity: Normal CC: gyuyoung.kim, lucas.de.marchi, rakuco, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch none

ChangSeok Oh
Reported 2012-02-26 02:10:30 PST
This bug is for supporting DOM mutation observers. If you need more information about mutation observers, please see http://www.w3.org/TR/dom/#mutation-observers.
Attachments
Patch (4.44 KB, patch)
2012-02-26 07:39 PST, ChangSeok Oh
no flags
Patch (4.38 KB, patch)
2012-02-27 07:14 PST, ChangSeok Oh
no flags
ChangSeok Oh
Comment 1 2012-02-26 07:39:20 PST
Raphael Kubo da Costa (:rakuco)
Comment 2 2012-02-26 13:27:25 PST
Comment on attachment 128914 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=128914&action=review > Source/cmake/OptionsEfl.cmake:85 > +WEBKIT_FEATURE(ENABLE_MUTATION_OBSERVERS "Enable HTML5 mutation observers support" DEFAULT OFF) Any reason for the default not being ON here (it is enabled by default by build-webkit)?
ChangSeok Oh
Comment 3 2012-02-26 18:07:58 PST
(In reply to comment #2) > (From update of attachment 128914 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=128914&action=review > > > Source/cmake/OptionsEfl.cmake:85 > > +WEBKIT_FEATURE(ENABLE_MUTATION_OBSERVERS "Enable HTML5 mutation observers support" DEFAULT OFF) > > Any reason for the default not being ON here (it is enabled by default by build-webkit)? Nope. Let me change it. :)
Gyuyoung Kim
Comment 4 2012-02-26 18:49:22 PST
Comment on attachment 128914 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=128914&action=review >>> Source/cmake/OptionsEfl.cmake:85 >>> +WEBKIT_FEATURE(ENABLE_MUTATION_OBSERVERS "Enable HTML5 mutation observers support" DEFAULT OFF) >> >> Any reason for the default not being ON here (it is enabled by default by build-webkit)? > > Nope. Let me change it. :) To run layout test, you should enable this option in build-webkit.
ChangSeok Oh
Comment 5 2012-02-27 07:14:56 PST
Eric Seidel (no email)
Comment 6 2012-02-27 15:11:45 PST
Comment on attachment 129033 [details] Patch rs=me.
WebKit Review Bot
Comment 7 2012-02-27 16:36:30 PST
Comment on attachment 129033 [details] Patch Clearing flags on attachment: 129033 Committed r109039: <http://trac.webkit.org/changeset/109039>
WebKit Review Bot
Comment 8 2012-02-27 16:36:35 PST
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.