Bug 145191

Summary: Update Autofill button icon
Product: WebKit Reporter: Daniel Bates <dbates>
Component: WebCore Misc.Assignee: Daniel Bates <dbates>
Status: RESOLVED FIXED    
Severity: Normal CC: bdakin, buildbot, jberlin, rniwa, sam
Priority: P2 Keywords: InRadar
Version: 528+ (Nightly build)   
Hardware: All   
OS: All   
Attachments:
Description Flags
Patch
none
Archive of layout-test-results from ews102 for mac-mavericks
none
Archive of layout-test-results from ews106 for mac-mavericks-wk2
none
Patch
none
Patch none

Description Daniel Bates 2015-05-19 16:14:43 PDT
Update icon of Autofill button.
Comment 1 Daniel Bates 2015-05-19 16:14:57 PDT
<rdar://problem/20178942>
Comment 2 Daniel Bates 2015-05-19 16:27:19 PDT
Created attachment 253409 [details]
Patch
Comment 3 Build Bot 2015-05-19 17:00:30 PDT
Comment on attachment 253409 [details]
Patch

Attachment 253409 [details] did not pass mac-ews (mac):
Output: http://webkit-queues.appspot.com/results/5391357950558208

New failing tests:
fast/forms/input-auto-fill-button.html
Comment 4 Build Bot 2015-05-19 17:00:35 PDT
Created attachment 253411 [details]
Archive of layout-test-results from ews102 for mac-mavericks

The attached test failures were seen while running run-webkit-tests on the mac-ews.
Bot: ews102  Port: mac-mavericks  Platform: Mac OS X 10.9.5
Comment 5 Build Bot 2015-05-19 17:22:16 PDT
Comment on attachment 253409 [details]
Patch

Attachment 253409 [details] did not pass mac-wk2-ews (mac-wk2):
Output: http://webkit-queues.appspot.com/results/5282933648654336

New failing tests:
fast/forms/input-auto-fill-button.html
Comment 6 Build Bot 2015-05-19 17:22:19 PDT
Created attachment 253412 [details]
Archive of layout-test-results from ews106 for mac-mavericks-wk2

The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews.
Bot: ews106  Port: mac-mavericks-wk2  Platform: Mac OS X 10.9.5
Comment 7 Daniel Bates 2015-05-20 10:39:54 PDT
Created attachment 253446 [details]
Patch

Updated patch that defaults the icon to black and tints it blue on mouse press. Added expected result for OS X Mavericks. Once I land this patch I will fetch the results from the Windows bot and update the expected results for Windows in the respository in a follow up commit.
Comment 8 Daniel Bates 2015-05-20 21:59:37 PDT
Created attachment 253502 [details]
Patch
Comment 9 Daniel Bates 2015-05-21 18:16:02 PDT
Comment on attachment 253502 [details]
Patch

Clearing flags on attachment: 253502

Committed r184745: <http://trac.webkit.org/changeset/184745>
Comment 10 Daniel Bates 2015-05-21 18:16:08 PDT
All reviewed patches have been landed.  Closing bug.
Comment 11 Daniel Bates 2015-05-26 13:41:25 PDT
Committed updated expected results for Windows in <http://trac.webkit.org/changeset/184870>.