----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6608/ -----------------------------------------------------------
(Updated Aug. 27, 2012, 11:52 a.m.) Review request for cloudstack and Kelven Yang. Changes ------- Modified consoleproxyajaximagehandler to send the thumbnail to fit in CP ui Description ------- When a viewer is launched, the images are getting stored in AjaxImageFifoCache class. When request for thumbnail, we need to pass "key" param to retreive the image and its not getting passed so thumbnail is requested its failing with NPE. Issue is fixed by passing key param with default value 0 and Image is retrieved with the current key value. This addresses bug CS-15778. Diffs (updated) ----- console-proxy/src/com/cloud/consoleproxy/AjaxFIFOImageCache.java c9b5b60 console-proxy/src/com/cloud/consoleproxy/ConsoleProxyAjaxImageHandler.java 1fdda2e server/src/com/cloud/servlet/ConsoleProxyServlet.java a8519fe Diff: https://reviews.apache.org/r/6608/diff/ Testing ------- Thanks, Rajesh Battala