commit 996c93068bfc8b443c77b735bc6400285bc8a407
author Lei Zhang <thestig@chromium.org> Fri Apr 13 15:44:36 2018 +0000
committer Chromium commit bot <commit-bot@chromium.org> Fri Apr 13 15:44:36 2018 +0000
tree 13b6d18f62f15c4ba04edec3dd2349f7ae3c1d54
parent 7afdad5ab7a1bd54ddf6f2a823be30d4b8e39567

Rename the other CPDF_Color::SetValue() variant.

Rename it to SetValueForNonePattern() and combine the components
parameters into a std::vector. Fix the callers to use std::vector as
well.

Change-Id: I0edd7e7876e47b56821e63cc5073fc21fd4098ee
Reviewed-on: https://pdfium-review.googlesource.com/30470
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>

fpdfsdk/fpdf_editpage.cpp

diff --git a/fpdfsdk/fpdf_editpage.cpp b/fpdfsdk/fpdf_editpage.cpp
index bc494f8..20d8894 100644
--- a/fpdfsdk/fpdf_editpage.cpp
+++ b/fpdfsdk/fpdf_editpage.cpp
@@ -9,6 +9,7 @@
 #include <algorithm>
 #include <memory>
 #include <utility>
+#include <vector>
 
 #include "core/fpdfapi/edit/cpdf_pagecontentgenerator.h"
 #include "core/fpdfapi/page/cpdf_form.h"
@@ -399,11 +400,11 @@
   if (!page_object || R > 255 || G > 255 || B > 255 || A > 255)
     return false;
 
-  float rgb[3] = {R / 255.f, G / 255.f, B / 255.f};
+  std::vector<float> rgb = {R / 255.f, G / 255.f, B / 255.f};
   auto* pPageObj = CPDFPageObjectFromFPDFPageObject(page_object);
   pPageObj->m_GeneralState.SetFillAlpha(A / 255.f);
   pPageObj->m_ColorState.SetFillColor(
-      CPDF_ColorSpace::GetStockCS(PDFCS_DEVICERGB), rgb, 3);
+      CPDF_ColorSpace::GetStockCS(PDFCS_DEVICERGB), rgb);
   pPageObj->SetDirty(true);
   return true;
 }