New Media Uploader doesn't error when an image is too big

[Ipstenu]

Source me on my iPhone, swearing.

Requirement: PHP file upload size set 'low' (2Megs)

Action: Upload large image (3megs)

Expected Outcome: Upload errors becuase you can't upload a file that big.

Actual Outcome: On existing media uploads, error out with 'Your file is too big, neener.' On new media, it hangs.

I don't think this is a blocker, but given that the iPhone makes images of over 2megs regularly and the default limit for a lot of hosts is 2, this should be in a point release or a hotfix.

[nacin]

Cannot reproduce. Works for me in Mac Chrome 23 (stable), using the Plupload html5, flash, and html4 runtimes.

[nacin]

IRC: ​https://irclogs.wordpress.org/chanlog.php?channel=wordpress-dev&day=2012-12-10&sort=asc#m515178

Ipstenu is able to get a hanging, dead-on-arrival attachment view when uploading multiple images.

[markjaquith]

Uploading three images. One too large. Large one hangs:

[markjaquith]

Uploading single too-large image errors as expected:

[markjaquith]

If all images are larger than the limit, I get an error for each. It's just the "some under the limit, some over the limit" situation that causes the hang.

[koopersmith]

Plupload sends files that have failed to the FilesAdded event, so we need to ignore them.

[markjaquith]

In 23162:

Ignore failed uploads. props koopersmith. fixes #22849

[nacin]

In 23165:

Ignore failed uploads. props koopersmith. fixes #22849. for the 3.5 branch.