View Background Image Feature

BUG=746
TEST=none

Initial proof of concept for the right-click context menu "View Background Image" feature.  This patch
is intended to confirm if I'm on the right track and get feedback on things I could focus on.  

Also, I only focused on Windows for this particular implementation. I don't have a MAC and I haven't
yet fired up any Linux distro, so I would probably need help there.

[Peter Kasting, 2009-08-11 17:35:25]

Drive-by: In the future please put BUG=, TEST= and similar at the bottom of your
message rather than the top, so that the summary line in the email that gets
sent out is more informative.

[steveisok, 2009-08-11 17:37:29]

Sorry!  I noticed that after the fact. Will change in the future.

On Tue, Aug 11, 2009 at 1:35 PM, <pkasting@chromium.org> wrote:

> Drive-by: In the future please put BUG=, TEST= and similar at the bottom
> of your message rather than the top, so that the summary line in the
> email that gets sent out is more informative.
>
> http://codereview.chromium.org/164279
>

[Peter Kasting, 2009-08-11 17:37:38]

Also: I question whether we actually want this.  CCed Ben on the bug in hopes
he'll make an up-or-down call.

[brettw, 2009-08-11 17:47:33]

In my opinion, we don't want this feature.

If we do, I don't think we should have the duplicate context menu function in
the webview delegate. The additional parameter should be on the existing
function.

[steveisok, 2009-08-11 18:05:34]

Agreed on the duplicate context menu function... I was only trying to get a
proof of concept spun up quickly. Didn't want to deal with the fallout of
breaking the contract.
As for the feature itself, it's my opinion that little features like these
make a difference.  As an average user if I want to grab a nifty background
image through chrome, then I have to go through a rather convoluted process
in order to get it. Chances are I'm firing up IE or Firefox.  On some level
Chrome loses credibility by not providing convenience to the user.

Now, how many people would actually use this? I would guess a decent
percentage.  Enough to not ignore the utility aspect of the feature.


On Tue, Aug 11, 2009 at 1:47 PM, <brettw@chromium.org> wrote:

> In my opinion, we don't want this feature.
>
> If we do, I don't think we should have the duplicate context menu
> function in the webview delegate. The additional parameter should be on
> the existing function.
>
>
> http://codereview.chromium.org/164279
>

[Peter Kasting, 2009-08-21 01:18:22]

Closed because bug is WontFix.

[heavymark, 2009-11-04 16:36:03]

For the rest of the web developer world who does need this feature like all
Firefox users, can we use these files to install it on our own? Or turn it into
a plugin for web developers to install?