Index: content/renderer/render_view_impl.cc |
diff --git a/content/renderer/render_view_impl.cc b/content/renderer/render_view_impl.cc |
index e66153301e131007be323524a5fb471c5d6fb51e..97f2d8fe5f061f4ec60b64cf2d896a5f5ddd0b47 100644 |
--- a/content/renderer/render_view_impl.cc |
+++ b/content/renderer/render_view_impl.cc |
@@ -1690,9 +1690,11 @@ void RenderViewImpl::printPage(WebLocalFrame* frame) { |
void RenderViewImpl::saveImageFromDataURL(const blink::WebString& data_url) { |
// Note: We should basically send GURL but we use size-limited string instead |
// in order to send a larger data url to save a image for <canvas> or <img>. |
- if (data_url.length() < kMaxLengthOfDataURLString) |
- Send(new ViewHostMsg_SaveImageFromDataURL( |
- GetRoutingID(), GetMainRenderFrame()->GetRoutingID(), data_url.utf8())); |
+ if (data_url.length() < kMaxLengthOfDataURLString) { |
+ Send(new FrameHostMsg_SaveImageFromDataURL( |
+ GetRoutingID(), GetMainRenderFrame()->GetRoutingID(), |
Charlie Reis
2016/03/29 04:41:49
This looks problematic-- we probably shouldn't be
brettw
2016/03/29 05:44:39
It seems to fix the repro steps. I looked through
Charlie Reis
2016/03/29 18:17:13
Ok. CC'ing rdsmith (who added the RFH ID, I think
asanka
2016/03/29 20:56:05
AFAICT the reason why it's not crashing is:
* Non
|
+ data_url.utf8())); |
+ } |
} |
bool RenderViewImpl::enumerateChosenDirectory( |