Add a feature observer for RenderDeprecatedFlexibleBox
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2013 19:19:53 +0000 (19:19 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 20 Mar 2013 19:19:53 +0000 (19:19 +0000)
https://bugs.webkit.org/show_bug.cgi?id=112740

Patch by Christian Biesinger <cbiesinger@chromium.org> on 2013-03-20
Reviewed by Tony Chang.

No new tests - only adding measurements.

* page/FeatureObserver.h:
Add constants for RenderDeprecatedFlexibleBox, and add separate
constants for its use in the UI and in extensions.
* rendering/RenderDeprecatedFlexibleBox.cpp:
(WebCore::RenderDeprecatedFlexibleBox::RenderDeprecatedFlexibleBox):
Call FeatureObserver. Check the document()->url() to see which
constant to use.

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

Source/WebCore/ChangeLog
Source/WebCore/page/FeatureObserver.h
Source/WebCore/rendering/RenderDeprecatedFlexibleBox.cpp

index eec1252..338fd8c 100644 (file)
@@ -1,3 +1,20 @@
+2013-03-20  Christian Biesinger  <cbiesinger@chromium.org>
+
+        Add a feature observer for RenderDeprecatedFlexibleBox
+        https://bugs.webkit.org/show_bug.cgi?id=112740
+
+        Reviewed by Tony Chang.
+
+        No new tests - only adding measurements.
+
+        * page/FeatureObserver.h:
+        Add constants for RenderDeprecatedFlexibleBox, and add separate
+        constants for its use in the UI and in extensions.
+        * rendering/RenderDeprecatedFlexibleBox.cpp:
+        (WebCore::RenderDeprecatedFlexibleBox::RenderDeprecatedFlexibleBox):
+        Call FeatureObserver. Check the document()->url() to see which
+        constant to use.
+
 2013-03-20  Tim Horton  <timothy_horton@apple.com>
 
         Autosize should use documentRect height instead of scrollHeight
index 033f964..cd36837 100644 (file)
@@ -104,6 +104,9 @@ public:
         XFrameOptions,
         XFrameOptionsSameOrigin,
         XFrameOptionsSameOriginWithBadAncestorChain,
+        DeprecatedFlexboxWebContent,
+        DeprecatedFlexboxChrome,
+        DeprecatedFlexboxChromeExtension,
         // Add new features above this line. Don't change assigned numbers of each items.
         NumberOfFeatures, // This enum value must be last.
     };
index 845d831..7df3c70 100644 (file)
@@ -25,6 +25,7 @@
 #include "config.h"
 #include "RenderDeprecatedFlexibleBox.h"
 
+#include "FeatureObserver.h"
 #include "Font.h"
 #include "LayoutRepainter.h"
 #include "RenderLayer.h"
@@ -124,6 +125,13 @@ RenderDeprecatedFlexibleBox::RenderDeprecatedFlexibleBox(Element* element)
 {
     setChildrenInline(false); // All of our children must be block-level
     m_stretchingChildren = false;
+    const KURL& url = document()->url();
+    if (url.protocolIs("chrome"))
+        FeatureObserver::observe(document(), FeatureObserver::DeprecatedFlexboxChrome);
+    else if (url.protocolIs("chrome-extension"))
+        FeatureObserver::observe(document(), FeatureObserver::DeprecatedFlexboxChromeExtension);
+    else
+        FeatureObserver::observe(document(), FeatureObserver::DeprecatedFlexboxWebContent);
 }
 
 RenderDeprecatedFlexibleBox::~RenderDeprecatedFlexibleBox()