Bug 82047

Summary: [CSS Regions] CSS Regions enabled by default on Windows
Product: WebKit Reporter: Ion Rosca <rosca>
Component: CSSAssignee: Ion Rosca <rosca>
Status: RESOLVED DUPLICATE    
Severity: Normal CC: adele, donggwan.kim, hyatt, jberlin, mibalan, rosca, sfalken, shanestephens, WebkitBugTracker
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Windows 7   
Bug Depends on:    
Bug Blocks: 57312    
Attachments:
Description Flags
First patch
buildbot: commit-queue-
win fix eric: review+

Description Ion Rosca 2012-03-23 05:07:11 PDT
CSS Regions 
- cannot be switched on/off through Windows WebKit COM interface.
- cannot be switched on/off using a runtime flag in javascript.
- should be enabled by default

All these problems were solved on mac ( https://bugs.webkit.org/show_bug.cgi?id=78525 )
Comment 1 Ion Rosca 2012-03-23 05:30:42 PDT
Created attachment 133466 [details]
First patch
Comment 2 Build Bot 2012-03-23 06:41:28 PDT
Comment on attachment 133466 [details]
First patch

Attachment 133466 [details] did not pass win-ews (win):
Output: http://queues.webkit.org/results/12120643
Comment 3 Ion Rosca 2012-03-23 08:08:52 PDT
Created attachment 133488 [details]
win fix
Comment 4 Jessie Berlin 2012-03-23 13:07:26 PDT
Comment on attachment 133488 [details]
win fix

This patch does not enable CSS Regions by default. It just adds WK1 API to enable CSS Regions. To enable it by default, you would have to add a line in Source/WebKit/win/WebPreferences.cpp that sets the default to true.

However, I am not sure that it is desirable to turn CSS Regions on by default on Windows.
Comment 5 Ion Rosca 2012-03-23 14:33:41 PDT
(In reply to comment #4)
> (From update of attachment 133488 [details])
> This patch does not enable CSS Regions by default. It just adds WK1 API to enable CSS Regions. To enable it by default, you would have to add a line in Source/WebKit/win/WebPreferences.cpp that sets the default to true.
> 
> However, I am not sure that it is desirable to turn CSS Regions on by default on Windows.

The default true worked for me beeing set in Source\WebKit2\Shared\WebPreferencesStore.h, which is not platform specific.
Mac as well has no specific default value for CSS Regions. 
We could continue without adding default value in WebPreferences.cpp.
Is it ok?
Comment 6 Eric Seidel (no email) 2012-04-19 16:23:54 PDT
Comment on attachment 133488 [details]
win fix

Seems reasonable.  I agree with Jessie, it's not clear to me that CSS regions should be enabled by default on Win.  That's really a decision for Apple.
Comment 7 Mihai Balan 2012-11-22 07:09:11 PST
Actually a duplicate of #89430 that was long gone fixed :)

*** This bug has been marked as a duplicate of bug 89430 ***