Save Image feature us not part of ios web or download component. It is fully implemented in BVC. Marking untriaged as it needs an owner (maybe Kurt who works on context menu for the new architecture).

I couldn't find a better owner for this. Kurt, feel free to reassign if you know a more appropriate owner. 

It looks like Gauthier and Sylvain have worked on this feature in the past, based on git blame for |-saveImageAtURL:referrer:|.  Sending this their way.

+mardini@.
Indeed we do not handle the case where there is no network (or when there are no data). We could display the generic error "Cannot save image" or have something more specific.

+ srahim

We should fix this error message and make it more informative if we know this is because there is no network or no data. 

Shimi: Looking for your input regarding the appropriate message to show in case we cannot save image due to no network or no data. Thanks.

By "no data," do you mean the user's mobile data usage has exceeded their limit? No network and no data feel like very different issues, have you considered a different error message for each error?

In the meantime, here are some rough drafts for iteration:

A) "Can't save image. Either there's no network connection, or you've reached your data limit." 

B) "Can't save image. No network or data is available." 
[I don't think this one is clear enough.]

C) "Can't save image. You're either offline or out of data." 
[This is concise, but the tone may sound a wee bit harsh. But may be best b/c the error dialog will be quite small.]

Thoughts? 

By "no data" I mean, in the code I receive 0 data. I think it would be the same if the user has no network or a bad network. I guess it would be the same if the user reached its data limit or disabled the data on its phone.
I think this alert should display a message explaining more or less the same thing as the message displayed when a page is loaded with no network (dino page). Maybe something like "Can't save image. No internet connection."?

Thanks, Shimi and Gauthier. I'd go for "Can't save image. No internet connection". 

Shimi, if it is ok with you I will implement "Can't save image. No internet connection."

Here is the implementation.

Deleted: Simulator Screen Shot Jun 29, 2017, 1.12.13 PM.png 273 KB

	Simulator Screen Shot Jun 29, 2017, 1.12.13 PM.png 273 KB View Download

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/bbf85c4795b79cbe3285167e3a0d18127ef747a0

commit bbf85c4795b79cbe3285167e3a0d18127ef747a0
Author: gambard <gambard@chromium.org>
Date: Thu Jun 29 15:57:00 2017

No internet connection error when saving image

When the user tries to save an image while having no internet connection
a Cocoa Error.
This CL adds a specific error message to handle this case.

BUG= 730822 

Change-Id: Id44ff57a4401a4d3d378be1d74d20598c3dd846d
Reviewed-on: https://chromium-review.googlesource.com/555170
Reviewed-by: Mark Cogan <marq@chromium.org>
Commit-Queue: Gauthier Ambard <gambard@chromium.org>
Cr-Commit-Position: refs/heads/master@{#483374}
[modify] https://crrev.com/bbf85c4795b79cbe3285167e3a0d18127ef747a0/ios/chrome/app/strings/ios_strings.grd
[modify] https://crrev.com/bbf85c4795b79cbe3285167e3a0d18127ef747a0/ios/chrome/browser/ui/browser_view_controller.mm

verified the issue on the canary build 61.0.3148.0 canary tested in iPad(iOS 10).
Cannot Save Image No internet connection error is displayed on saving an image in no network, works fine.