OLD | NEW |
---|---|
1 /* | 1 /* |
2 * Copyright (C) 2010 Google Inc. All rights reserved. | 2 * Copyright (C) 2010 Google Inc. All rights reserved. |
3 * Copyright (C) 2011 Apple Inc. All rights reserved. | 3 * Copyright (C) 2011 Apple Inc. All rights reserved. |
4 * | 4 * |
5 * Redistribution and use in source and binary forms, with or without | 5 * Redistribution and use in source and binary forms, with or without |
6 * modification, are permitted provided that the following conditions are | 6 * modification, are permitted provided that the following conditions are |
7 * met: | 7 * met: |
8 * | 8 * |
9 * * Redistributions of source code must retain the above copyright | 9 * * Redistributions of source code must retain the above copyright |
10 * notice, this list of conditions and the following disclaimer. | 10 * notice, this list of conditions and the following disclaimer. |
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
182 input->dispatchFormControlInputEvent(); | 182 input->dispatchFormControlInputEvent(); |
183 input->dispatchFormControlChangeEvent(); | 183 input->dispatchFormControlChangeEvent(); |
184 break; | 184 break; |
185 } | 185 } |
186 | 186 |
187 case DispatchNoEvent: | 187 case DispatchNoEvent: |
188 break; | 188 break; |
189 } | 189 } |
190 | 190 |
191 if (!input->focused()) | 191 if (!input->focused()) |
192 input->setTextAsOfLastFormControlChangeEvent(sanitizedValue); | 192 input->setTextAsOfLastFormControlChangeEvent(sanitizedValue.isNull() ? v isibleValue() : sanitizedValue); |
tkent
2014/11/10 01:54:39
Why is it visibleValue(), instead of defaultValue(
Habib Virji
2014/11/13 15:00:56
As defaultValue() and sanitizedValue both hold nul
tkent
2014/11/14 00:23:56
Using visibleValue() is semantically incorrect. I
Habib Virji
2014/11/24 11:05:32
Done.
| |
193 } | 193 } |
194 | 194 |
195 void TextFieldInputType::handleKeydownEvent(KeyboardEvent* event) | 195 void TextFieldInputType::handleKeydownEvent(KeyboardEvent* event) |
196 { | 196 { |
197 if (!element().focused()) | 197 if (!element().focused()) |
198 return; | 198 return; |
199 if (Chrome* chrome = this->chrome()) { | 199 if (Chrome* chrome = this->chrome()) { |
200 chrome->client().handleKeyboardEventOnTextField(element(), *event); | 200 chrome->client().handleKeyboardEventOnTextField(element(), *event); |
201 return; | 201 return; |
202 } | 202 } |
(...skipping 346 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
549 return shouldSpinButtonRespondToMouseEvents() && element().focused(); | 549 return shouldSpinButtonRespondToMouseEvents() && element().focused(); |
550 } | 550 } |
551 | 551 |
552 void TextFieldInputType::spinButtonDidReleaseMouseCapture(SpinButtonElement::Eve ntDispatch eventDispatch) | 552 void TextFieldInputType::spinButtonDidReleaseMouseCapture(SpinButtonElement::Eve ntDispatch eventDispatch) |
553 { | 553 { |
554 if (eventDispatch == SpinButtonElement::EventDispatchAllowed) | 554 if (eventDispatch == SpinButtonElement::EventDispatchAllowed) |
555 element().dispatchFormControlChangeEvent(); | 555 element().dispatchFormControlChangeEvent(); |
556 } | 556 } |
557 | 557 |
558 } // namespace blink | 558 } // namespace blink |
OLD | NEW |