Refactor setIcon to allow for more general use of imageData

extensions/renderer/set_icon_natives.h

Index: extensions/renderer/set_icon_natives.h
diff --git a/extensions/renderer/set_icon_natives.h b/extensions/renderer/set_icon_natives.h
index e5129dd976b3690985a6b2f60f320774ae849540..0def0be51fb886f7be8944b64666d3ec0a4201bc 100644
--- a/extensions/renderer/set_icon_natives.h
+++ b/extensions/renderer/set_icon_natives.h
@@ -24,10 +24,10 @@ class SetIconNatives : public ObjectBackedNativeHandler {
 
  private:
   bool ConvertImageDataToBitmapValue(const v8::Local<v8::Object> image_data,
-                                     base::Value** bitmap_value);
+                                     v8::Local<v8::Value>* image_data_bitmap);
   bool ConvertImageDataSetToBitmapValueSet(
-      const v8::FunctionCallbackInfo<v8::Value>& args,
-      base::DictionaryValue* bitmap_value);
+      v8::Local<v8::Object>& details,
+      v8::Local<v8::Object>* bitmap_set_value);
   void SetIconCommon(const v8::FunctionCallbackInfo<v8::Value>& args);
 
   RequestSender* request_sender_;