Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(270)

Side by Side Diff: Source/core/html/shadow/DateTimeEditElement.cpp

Issue 152623002: Make StyleChange parameter explicit in setNeedsStyleRecalc. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2012 Google Inc. All rights reserved. 2 * Copyright (C) 2012 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 667 matching lines...) Expand 10 before | Expand all | Expand 10 after
678 if (DateTimeFieldElement* field = fieldAt(std::min(focusedFieldIndex, m_ fields.size() - 1))) 678 if (DateTimeFieldElement* field = fieldAt(std::min(focusedFieldIndex, m_ fields.size() - 1)))
679 field->focus(); 679 field->focus();
680 } 680 }
681 681
682 if (lastChildToBeRemoved) { 682 if (lastChildToBeRemoved) {
683 for (Node* childNode = fieldsWrapper->firstChild(); childNode; childNode = fieldsWrapper->firstChild()) { 683 for (Node* childNode = fieldsWrapper->firstChild(); childNode; childNode = fieldsWrapper->firstChild()) {
684 fieldsWrapper->removeChild(childNode); 684 fieldsWrapper->removeChild(childNode);
685 if (childNode == lastChildToBeRemoved) 685 if (childNode == lastChildToBeRemoved)
686 break; 686 break;
687 } 687 }
688 setNeedsStyleRecalc(); 688 setNeedsStyleRecalc(SubtreeStyleChange);
689 } 689 }
690 } 690 }
691 691
692 AtomicString DateTimeEditElement::localeIdentifier() const 692 AtomicString DateTimeEditElement::localeIdentifier() const
693 { 693 {
694 return m_editControlOwner ? m_editControlOwner->localeIdentifier() : nullAto m; 694 return m_editControlOwner ? m_editControlOwner->localeIdentifier() : nullAto m;
695 } 695 }
696 696
697 void DateTimeEditElement::fieldDidChangeValueByKeyboard() 697 void DateTimeEditElement::fieldDidChangeValueByKeyboard()
698 { 698 {
(...skipping 97 matching lines...) Expand 10 before | Expand all | Expand 10 after
796 { 796 {
797 DateTimeFieldsState dateTimeFieldsState; 797 DateTimeFieldsState dateTimeFieldsState;
798 for (size_t fieldIndex = 0; fieldIndex < m_fields.size(); ++fieldIndex) 798 for (size_t fieldIndex = 0; fieldIndex < m_fields.size(); ++fieldIndex)
799 m_fields[fieldIndex]->populateDateTimeFieldsState(dateTimeFieldsState); 799 m_fields[fieldIndex]->populateDateTimeFieldsState(dateTimeFieldsState);
800 return dateTimeFieldsState; 800 return dateTimeFieldsState;
801 } 801 }
802 802
803 } // namespace WebCore 803 } // namespace WebCore
804 804
805 #endif 805 #endif
OLDNEW
« no previous file with comments | « Source/core/html/forms/TextFieldInputType.cpp ('k') | Source/core/html/shadow/DateTimeFieldElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698