Make calls to AtomicString(const String&) explicit in html/

Source/core/html/forms/FormController.cpp

Index: Source/core/html/forms/FormController.cpp
diff --git a/Source/core/html/forms/FormController.cpp b/Source/core/html/forms/FormController.cpp
index a3cd9b02100adaf59888746bc9f6ae9155ff087d..17a299086f6eec07e9cca61593e26d81c86277e9 100644
--- a/Source/core/html/forms/FormController.cpp
+++ b/Source/core/html/forms/FormController.cpp
@@ -221,7 +221,7 @@ PassOwnPtr<SavedFormState> SavedFormState::deserialize(const Vector<String>& sta
         FormControlState state = FormControlState::deserialize(stateVector, index);
         if (type.isEmpty() || type.find(isNotFormControlTypeCharacter) != kNotFound || state.isFailure())
             return nullptr;
-        savedFormState->appendControlState(name, type, state);
+        savedFormState->appendControlState(AtomicString(name), AtomicString(type), state);
     }
     return savedFormState.release();
 }
@@ -453,7 +453,7 @@ void FormController::formStatesFromStateVector(const Vector<String>& stateVector
         return;
 
     while (i + 1 < stateVector.size()) {
-        AtomicString formKey = stateVector[i++];
+        AtomicString formKey = AtomicString(stateVector[i++]);

[eseidel, 2013/12/31 23:22:35]

I guess the stateVector should eventually be Vector<AtomicSTring> but I can also
understand the desire to kepe this change small.

[Inactive, 2014/01/01 19:26:29]

I took a look at it and it looks like this change may be big. I'd rather leave
it out of this CL and address it separately.

         OwnPtr<SavedFormState> state = SavedFormState::deserialize(stateVector, i);
         if (!state) {
             i = 0;