CSSGradientValue::getStopColors(): unnecessary use of HeapVector<>.

third_party/WebKit/Source/core/inspector/InspectorCSSAgent.cpp

 /*
  * Copyright (C) 2010, Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1.  Redistributions of source code must retain the above copyright
  *     notice, this list of conditions and the following disclaimer.
  * 2.  Redistributions in binary form must reproduce the above copyright
  *     notice, this list of conditions and the following disclaimer in the
@@ skipping 126 matching lines @@
     LayoutPoint center = rect.center();
     unsigned leftPadding, rightPadding, topPadding, bottomPadding;
     leftPadding = rightPadding = rect.width() / 2;
     topPadding = bottomPadding = rect.height() / 2;
     HitTestResult result(request, center, topPadding, rightPadding, bottomPadding, leftPadding);
     document.frame()->contentLayoutItem().hitTest(result);
     return document.elementsFromHitTestResult(result);
 }
 
 // Blends the colors from the given gradient with the existing colors.
-void blendWithColorsFromGradient(CSSGradientValue* gradient, HeapVector<Color>& colors, bool& foundNonTransparentColor, bool& foundOpaqueColor, const LayoutObject& layoutObject)
+void blendWithColorsFromGradient(CSSGradientValue* gradient, Vector<Color>& colors, bool& foundNonTransparentColor, bool& foundOpaqueColor, const LayoutObject& layoutObject)
 {
-    HeapVector<Color> stopColors;
+    Vector<Color> stopColors;
     gradient->getStopColors(stopColors, layoutObject);
 
     if (colors.isEmpty()) {
         colors.appendRange(stopColors.begin(), stopColors.end());
     } else {
         if (colors.size() > 1) {
             // Gradient on gradient is too complicated, bail out
             colors.clear();
             return;
         }
 
         Color existingColor = colors.first();
         colors.clear();
         for (auto stopColor : stopColors) {
             foundNonTransparentColor = foundNonTransparentColor || (stopColor.alpha() != 0);
             colors.append(existingColor.blend(stopColor));
         }
     }
     foundOpaqueColor = foundOpaqueColor || gradient->knownToBeOpaque(layoutObject);
 }
 
 // Gets the colors from an image style, if one exists and it is a gradient.
-void addColorsFromImageStyle(const ComputedStyle& style, HeapVector<Color>& colors, bool& foundOpaqueColor, bool& foundNonTransparentColor, const LayoutObject& layoutObject)
+void addColorsFromImageStyle(const ComputedStyle& style, Vector<Color>& colors, bool& foundOpaqueColor, bool& foundNonTransparentColor, const LayoutObject& layoutObject)
 {
     const FillLayer& backgroundLayers = style.backgroundLayers();
     if (!backgroundLayers.hasImage())
         return;
 
     StyleImage* styleImage = backgroundLayers.image();
     // hasImage() does not always indicate that this is non-null
     if (!styleImage)
         return;
 
@@ skipping 10 matching lines @@
         CSSGradientValue* gradient = toCSSGradientValue(imageCSS);
         blendWithColorsFromGradient(gradient, colors, foundNonTransparentColor, foundOpaqueColor, layoutObject);
     }
     return;
 }
 
 // Get the background colors behind the given rect in the given document, by
 // walking up all the elements returned by a hit test (but not going beyond
 // |topElement|) covering the area of the rect, and blending their background
 // colors.
-bool getColorsFromRect(LayoutRect rect, Document& document, Element* topElement, HeapVector<Color>& colors)
+bool getColorsFromRect(LayoutRect rect, Document& document, Element* topElement, Vector<Color>& colors)
 {
     HeapVector<Member<Element>> elementsUnderRect = elementsFromRect(rect, document);
 
     bool foundOpaqueColor = false;
     bool foundTopElement = false;
 
     for (auto e = elementsUnderRect.rbegin(); !foundTopElement && e != elementsUnderRect.rend(); ++e) {
         const Element* element = *e;
         if (element == topElement)
             foundTopElement = true;
@@ skipping 1809 matching lines @@
         return;
 
     for (const LayoutObject* child = elementLayout->slowFirstChild(); child; child = child->nextSibling()) {
         if (!child->isText())
             continue;
         textBounds.unite(LayoutRect(child->absoluteBoundingBoxRect()));
     }
     if (textBounds.size().isEmpty())
         return;
 
-    HeapVector<Color> colors;
+    Vector<Color> colors;
     FrameView* view = element->document().view();
     if (!view) {
         *errorString = "No view.";
         return;
     }
     Document& document = element->document();
     bool isMainFrame = !document.ownerElement();
     bool foundOpaqueColor = false;
     if (isMainFrame && !view->isTransparent()) {
         // Start with the "default" page color (typically white).
@@ skipping 29 matching lines @@
     visitor->trace(m_cssStyleSheetToInspectorStyleSheet);
     visitor->trace(m_documentToCSSStyleSheets);
     visitor->trace(m_invalidatedDocuments);
     visitor->trace(m_nodeToInspectorStyleSheet);
     visitor->trace(m_documentToViaInspectorStyleSheet);
     visitor->trace(m_inspectorUserAgentStyleSheet);
     InspectorBaseAgent::trace(visitor);
 }
 
 } // namespace blink