Bug 60564

Summary: [Chromium] Add rebaselines for video-zoom-controls.
Product: WebKit Reporter: imasaki
Component: New BugsAssignee: imasaki
Status: RESOLVED INVALID    
Severity: Normal CC: abarth, aroben, commit-queue, dbates, eric, jianli, levin, ojan, vrk
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch
none
Patch
none
Patch
none
Patch none

imasaki
Reported 2011-05-10 11:20:40 PDT
[Chromium] Add rebaselines for video-zoom-controls.
Attachments
Patch (423.79 KB, patch)
2011-05-10 11:23 PDT, imasaki 		no flags
DetailsFormatted DiffDiff
Patch (423.79 KB, patch)
2011-05-10 11:54 PDT, imasaki 		no flags
DetailsFormatted DiffDiff
Patch (425.13 KB, patch)
2011-05-10 14:04 PDT, imasaki 		no flags
DetailsFormatted DiffDiff
Patch (424.53 KB, patch)
2011-05-11 13:54 PDT, imasaki 		no flags
DetailsFormatted DiffDiff
Show Obsolete (3) View All Add attachment proposed patch, testcase, etc.
imasaki
Comment 1 2011-05-10 11:23:47 PDT
Created attachment 92988 [details] Patch
imasaki
Comment 2 2011-05-10 11:54:05 PDT
Created attachment 92990 [details] Patch
imasaki
Comment 3 2011-05-10 11:55:39 PDT
Comment on attachment 92990 [details] Patch set auto-props
David Levin
Comment 4 2011-05-10 11:57:23 PDT
(In reply to comment #3) > (From update of attachment 92990 [details]) > set auto-props I don't think that worked here. When I click "review patch" the image doesn't show up as an image and when I look at details, the props aren't there. I think you need to add the file *after* setting up the auto-props. Did you do that?
imasaki
Comment 5 2011-05-10 14:04:39 PDT
Created attachment 93004 [details] Patch
Eric Seidel (no email)
Comment 6 2011-05-11 10:33:27 PDT
Dan Bates knows more about the auto-props stuff than I. Ojan, Aroben, and Adam know more about the review tool these days than I.
imasaki
Comment 7 2011-05-11 13:54:51 PDT
Created attachment 93177 [details] Patch
Note You need to log in before you can comment on or make changes to this bug.