author | Daniel Holbert <dholbert@cs.stanford.edu> |
Tue, 10 Jan 2012 15:32:58 -0800 | |
changeset 84191 | 5b40b3847195556b7c71c8d2758188c927695cf9 |
parent 84190 | 79eb51bafb9aa229128ab1a4adee845c189c6f1e |
child 84192 | 37ede8ee0d0ec8a7556245d1a76af133757a8ebb |
push id | 21832 |
push user | bmo@edmorley.co.uk |
push date | Wed, 11 Jan 2012 17:04:15 +0000 |
treeherder | mozilla-central@40c9f9ff9fd5 [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | volkmar |
bugs | 717004 |
milestone | 12.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/content/html/content/src/nsGenericHTMLElement.cpp +++ b/content/html/content/src/nsGenericHTMLElement.cpp @@ -1619,26 +1619,26 @@ nsGenericHTMLElement::GetLayoutHistoryAn } bool nsGenericHTMLElement::RestoreFormControlState(nsGenericHTMLElement* aContent, nsIFormControl* aControl) { nsCOMPtr<nsILayoutHistoryState> history; nsCAutoString key; - nsresult rv = GetLayoutHistoryAndKey(aContent, true, - getter_AddRefs(history), key); + GetLayoutHistoryAndKey(aContent, true, + getter_AddRefs(history), key); if (!history) { return false; } nsPresState *state; // Get the pres state for this key - rv = history->GetState(key, &state); - if (state) { + nsresult rv = history->GetState(key, &state); + if (NS_SUCCEEDED(rv) && state) { bool result = aControl->RestoreState(state); history->RemoveState(key); return result; } return false; }