Enabling a disabled input[type=file] in a disabled fieldset and then enabling the disabled fieldset result in the input[type=file] keeps disabled.
Reported by
x.gow...@gmail.com,
Jun 21 2016
|
|||
Issue description
Chrome Version : 51.0.2704.103 (Official Build) (64-bit)
URLs (if applicable) : <see the demo file attched>
Other browsers tested:
Add OK or FAIL, along with the version, after other browsers where you
have tested this issue:
Firefox:OK
IE:OK
What steps will reproduce the problem?
(1)Enable a disabled input[type=file] in a disabled fieldset.
(2)Enable the disabled fieldset.
What is the expected result?
The input[type=file] becomes enabled.
What happens instead?
The input[type=file] keeps disabled, although its "disabled" attribute is removed normally.
Additional information:
Please refer to the file attached.
,
Jun 23 2016
Only [Choose File] button keeps disabled. Clicking on "No file chosen" text or typing Enter key triggers a file chooser. Probably we need to call layoutObject()->updateFromElement() in FileInputType::disabledAttributeChanged() in order to update 'active' state.
,
Jun 23 2016
i'm interesting this issue. i would like to fix it. thank you.
,
Jun 26 2016
could you review this commit? https://codereview.chromium.org/2093383002/.
,
Jun 28 2016
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/3849599529aec6e8f8048086ddb6fd9ef8a9e554 commit 3849599529aec6e8f8048086ddb6fd9ef8a9e554 Author: hs1217.lee <hs1217.lee@gmail.com> Date: Tue Jun 28 22:08:49 2016 Implement disabledAttributeChanged() in HTMLInputElement. when called disabledAttributeChanged() at of HTMLFieldSetElement, call the m_inputTypeView->disabledAttributeChanged() for update at HTMLInputElement. BUG= 621870 Review-Url: https://codereview.chromium.org/2093383002 Cr-Commit-Position: refs/heads/master@{#402562} [add] https://crrev.com/3849599529aec6e8f8048086ddb6fd9ef8a9e554/third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-disable-expected.html [add] https://crrev.com/3849599529aec6e8f8048086ddb6fd9ef8a9e554/third_party/WebKit/LayoutTests/fast/forms/fieldset/fieldset-disable.html [modify] https://crrev.com/3849599529aec6e8f8048086ddb6fd9ef8a9e554/third_party/WebKit/Source/core/html/HTMLInputElement.cpp [modify] https://crrev.com/3849599529aec6e8f8048086ddb6fd9ef8a9e554/third_party/WebKit/Source/core/html/HTMLInputElement.h
,
Jun 28 2016
|
|||
►
Sign in to add a comment |
|||
Comment 1 by rnimmagadda@chromium.org
, Jun 23 2016Components: Blink>Forms>Fieldset
Labels: -Pri-3 M-52 OS-Linux OS-Mac OS-Windows Pri-2
Status: Untriaged (was: Unconfirmed)