[CSSRegions][CSSOM] Make sure all Regions APIs are not visible if CSS_REGIONS is...
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Sep 2012 15:30:43 +0000 (15:30 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 19 Sep 2012 15:30:43 +0000 (15:30 +0000)
https://bugs.webkit.org/show_bug.cgi?id=96300

Patch by Raul Hudea <rhudea@adobe.com> on 2012-09-19
Reviewed by Yury Semikhatsky.

All CSS Regions APIs should be exposed only if CSS_REGIONS is enabled

No new tests because of no behavior changes.

* css/StyleRule.cpp:
(WebCore::StyleRuleBase::reportMemoryUsage): Fix compile without CSS_REGIONS enabled and added an ASSERT_NOT_REACHED as in other functions
* dom/Element.cpp: Compile webkitRegionOverset only if CSS_REGIONS is enabled
(WebCore):
* dom/Element.h:
* dom/Element.idl: Expose webkitRegionOverset only if CSS_REGIONS is enabled

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@129009 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/css/StyleRule.cpp
Source/WebCore/dom/Element.cpp
Source/WebCore/dom/Element.h
Source/WebCore/dom/Element.idl

index ccf5c11..8fbc1cf 100644 (file)
@@ -1,3 +1,21 @@
+2012-09-19  Raul Hudea  <rhudea@adobe.com>
+
+        [CSSRegions][CSSOM] Make sure all Regions APIs are not visible if CSS_REGIONS is not defined
+        https://bugs.webkit.org/show_bug.cgi?id=96300
+
+        Reviewed by Yury Semikhatsky.
+
+        All CSS Regions APIs should be exposed only if CSS_REGIONS is enabled
+
+        No new tests because of no behavior changes.
+
+        * css/StyleRule.cpp:
+        (WebCore::StyleRuleBase::reportMemoryUsage): Fix compile without CSS_REGIONS enabled and added an ASSERT_NOT_REACHED as in other functions
+        * dom/Element.cpp: Compile webkitRegionOverset only if CSS_REGIONS is enabled
+        (WebCore):
+        * dom/Element.h:
+        * dom/Element.idl: Expose webkitRegionOverset only if CSS_REGIONS is enabled
+
 2012-09-19  Patrick Gansterer  <paroga@webkit.org>
 
         Remove all usages of M_PI from WebCore
index 59ed9e5..5ece1b4 100644 (file)
@@ -81,7 +81,10 @@ void StyleRuleBase::reportMemoryUsage(MemoryObjectInfo* memoryObjectInfo) const
     case Unknown:
     case Charset:
     case Keyframe:
-        MemoryClassInfo info(memoryObjectInfo, this, WebCoreMemoryTypes::CSS);
+#if !ENABLE(CSS_REGIONS)
+    case Region:
+#endif
+        ASSERT_NOT_REACHED();
         return;
     }
     ASSERT_NOT_REACHED();
index 0d448e1..3a7c541 100644 (file)
@@ -2033,6 +2033,8 @@ RenderRegion* Element::renderRegion() const
     return 0;
 }
 
+#if ENABLE(CSS_REGIONS)
+
 const AtomicString& Element::webkitRegionOverset() const
 {
     document()->updateLayoutIgnorePendingStylesheets();
@@ -2062,8 +2064,6 @@ const AtomicString& Element::webkitRegionOverset() const
     return undefinedState;
 }
 
-#if ENABLE(CSS_REGIONS)
-
 Vector<RefPtr<Range> > Element::webkitGetRegionFlowRanges() const
 {
     document()->updateLayoutIgnorePendingStylesheets();
index 2f4e925..0cd209d 100644 (file)
@@ -426,8 +426,8 @@ public:
     PassRefPtr<RenderStyle> styleForRenderer();
 
     RenderRegion* renderRegion() const;
-    const AtomicString& webkitRegionOverset() const;
 #if ENABLE(CSS_REGIONS)
+    const AtomicString& webkitRegionOverset() const;
     Vector<RefPtr<Range> > webkitGetRegionFlowRanges() const;
 #endif
 
index 33c11df..4bd64ed 100644 (file)
@@ -143,7 +143,7 @@ module core {
         [Conditional=POINTER_LOCK, V8EnabledAtRuntime=pointerLock] void webkitRequestPointerLock();
 
         // CSS Regions API
-        readonly attribute DOMString webkitRegionOverset;
+        readonly attribute [Conditional=CSS_REGIONS] DOMString webkitRegionOverset;
         [Conditional=CSS_REGIONS] sequence<Range> webkitGetRegionFlowRanges();
 
 #if !defined(LANGUAGE_OBJECTIVE_C) || !LANGUAGE_OBJECTIVE_C