Prevents windows that cannot snap from snapping using a keyboard accelerator

Prevents windows that cannot snap from snapping using a keyboard accelerator

BUG=386254
TEST=Open a non-resizable app such as a calculator. Alt+] should not snap it.

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=278273

[varkha, 2014-06-18 21:25:55]

oshima@, can you please take a look? This probably was prevented before the
window state events (just a guess). Thanks!

[oshima, 2014-06-18 21:34:11]

lgtm

[varkha, 2014-06-18 21:34:34]

The CQ bit was checked by varkha@chromium.org

[commit-bot: I haz the power, 2014-06-18 21:36:48]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/varkha@chromium.org/342823002/1

[oshima, 2014-06-18 21:36:56]

On 2014/06/18 21:34:34, varkha wrote:
> The CQ bit was checked by mailto:varkha@chromium.org

btw, I think we never have this in window state. We probably should do that, but
I'm ok with this for now because we don't want to record uma when it cannot be
snapped. (or do we?)

[varkha, 2014-06-18 21:42:49]

On 2014/06/18 21:36:56, oshima wrote:
> On 2014/06/18 21:34:34, varkha wrote:
> > The CQ bit was checked by mailto:varkha@chromium.org
> 
> btw, I think we never have this in window state. We probably should do that,
but
> I'm ok with this for now because we don't want to record uma when it cannot be
> snapped. (or do we?)

I think we should not record the unsuccessful attempt to snap in UMA (similar to
how we don't record it when in full screen). At least not record it in the same
way as for successful snapping.

[commit-bot: I haz the power, 2014-06-19 04:34:34]

Change committed as 278273