Bug 35129 - [Chromium] autofill should not be shown for read only text fields
Summary: [Chromium] autofill should not be shown for read only text fields
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKit API (show other bugs)
Version: 528+ (Nightly build)
Hardware: PC OS X 10.5
: P2 Normal
Assignee: Jay Campan
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-02-18 13:31 PST by Jay Campan
Modified: 2010-02-24 21:24 PST (History)
3 users (show)

See Also:


Attachments
Bug fix patch (1.40 KB, patch)
2010-02-18 13:39 PST, Jay Campan
eric: review-
Details | Formatted Diff | Diff
Adding a manual test. (3.74 KB, patch)
2010-02-24 09:57 PST, Jay Campan
levin: review-
levin: commit-queue-
Details | Formatted Diff | Diff
Fixed bad line endings (3.71 KB, patch)
2010-02-24 10:47 PST, Jay Campan
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jay Campan 2010-02-18 13:31:56 PST
When a read only input text field is clicked twice, the autofill popup is shown.
It should not.
Comment 1 Jay Campan 2010-02-18 13:39:14 PST
Created attachment 49039 [details]
Bug fix patch
Comment 2 Eric Seidel (no email) 2010-02-22 14:09:07 PST
Comment on attachment 49039 [details]
Bug fix patch

We can write a manual test for this, no?  Should go in WebCore/manual-tests/
Comment 3 Jay Campan 2010-02-24 09:57:10 PST
Created attachment 49403 [details]
Adding a manual test.

Added a manual test in WebCore/manual-tests/chromium.
Comment 4 WebKit Review Bot 2010-02-24 10:18:06 PST
Attachment 49403 [details] did not pass style-queue:

Failed to run "WebKitTools/Scripts/check-webkit-style" exit_code: 1
WebCore/manual-tests/chromium/no-autofill-on-readonly.html:1:  One or more unexpected \r (^M) found; better to use only a \n  [whitespace/carriage_return] [1]
Suppressing further [whitespace/carriage_return] reports for this file.
Total errors found: 32 in 4 files


If any of these errors are false positives, please file a bug against check-webkit-style.
Comment 5 David Levin 2010-02-24 10:38:47 PST
Comment on attachment 49403 [details]
Adding a manual test.

r=me except for the style issue and since you aren't a committer r- to fix that up.
Comment 6 Jay Campan 2010-02-24 10:47:56 PST
Created attachment 49411 [details]
Fixed bad line endings
Comment 7 WebKit Commit Bot 2010-02-24 21:24:15 PST
Comment on attachment 49411 [details]
Fixed bad line endings

Clearing flags on attachment: 49411

Committed r55219: <http://trac.webkit.org/changeset/55219>
Comment 8 WebKit Commit Bot 2010-02-24 21:24:19 PST
All reviewed patches have been landed.  Closing bug.