wine/gecko
0
0
Fork 0
mirror of https://gitlab.winehq.org/wine/wine-gecko.git synced 2024-09-13 09:24:08 -07:00
Code Issues Packages Projects Releases Wiki Activity

Bug 1088158 - Update the visibility state of placeholders for textareas correctly when they get their anonymous content; r=roc

Browse Source
This commit is contained in:
Ehsan Akhgari 2014-10-28 09:44:26 -04:00
parent 86b9567292
commit a7f04dfe6e
4 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
editor/reftests/1088158-ref.html Normal file
View File

@ -0,0 +1,2 @@
<!DOCTYPE html>
<textarea placeholder="placeholder"></textarea>

8
editor/reftests/1088158.html Normal file
View File

@ -0,0 +1,8 @@
<!DOCTYPE html>
<script>
onload = function() {
var t = document.createElement('textarea');
t.placeholder = "placeholder";
document.body.appendChild(t.cloneNode(true));
}
</script>

1
editor/reftests/reftest.list
View File

@ -127,3 +127,4 @@ needs-focus == spellcheck-contenteditable-focused-reframe.html spellcheck-conten
== spellcheck-contenteditable-property-dynamic-override-inherit.html spellcheck-contenteditable-disabled-ref.html
needs-focus == 969773.html 969773-ref.html
== 997805.html 997805-ref.html
== 1088158.html 1088158-ref.html

7
layout/forms/nsTextControlFrame.cpp
View File

@ -351,6 +351,13 @@ nsTextControlFrame::CreateAnonymousContent(nsTArray<ContentInfo>& aElements)
placeholderStyleContext))) {
return NS_ERROR_OUT_OF_MEMORY;
}
if (!IsSingleLineTextControl()) {
// For textareas, UpdateValueDisplay doesn't initialize the visibility
// status of the placeholder because it returns early, so we have to
// do that manually here.
txtCtrl->UpdatePlaceholderVisibility(true);
}
}
rv = UpdateValueDisplay(false);