Option Groups with display: none should not show the children option elements

public/platform/WebVector.h

Index: public/platform/WebVector.h
diff --git a/public/platform/WebVector.h b/public/platform/WebVector.h
index e83a2d0d2cd56d627ae2b4d50fed0c0e3fcfa591..b22327bba676842cee2f6a7b08f71356936463d4 100644
--- a/public/platform/WebVector.h
+++ b/public/platform/WebVector.h
@@ -152,6 +152,14 @@ public:
         std::swap(m_size, other.m_size);
     }
 
+    void shrink(size_t size)

[tkent, 2014/08/06 12:46:07]

Why is this necessary?
WebVector has no rich operations intentionally. We should not add this.

[spartha, 2014/08/06 13:31:09]

The function pre-allocates the vector for efficiency. This worked fine earlier
as the size was fixed. Now with this patch, the final size can change and we
have to resize the vector for the patch to work. I wanted to avoid looping
through the list again to make a copy, hence added the method. If the goal is to
keep WebVector simple, then I can check for alternative approach. Please let me
know your opinion.

+    {
+        BLINK_ASSERT(size <= m_size);
+        for (size_t index = m_size - 1; index >= size; --index)
+            m_ptr[index].~T();
+        m_size = size;
+    }
+
 private:
     void initialize(size_t size)
     {