Have InputType factories take an HTMLInputElement reference in parameter

Source/core/html/forms/CheckboxInputType.cpp

 /*
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
@@ skipping 22 matching lines @@
 #include "core/html/forms/CheckboxInputType.h"
 
 #include "core/events/KeyboardEvent.h"
 #include "core/html/HTMLInputElement.h"
 #include "core/html/forms/InputTypeNames.h"
 #include "platform/text/PlatformLocale.h"
 #include "wtf/PassOwnPtr.h"
 
 namespace WebCore {
 
-PassRefPtr<InputType> CheckboxInputType::create(HTMLInputElement* element)
+PassRefPtr<InputType> CheckboxInputType::create(HTMLInputElement& element)
 {
     return adoptRef(new CheckboxInputType(element));
 }
 
 const AtomicString& CheckboxInputType::formControlType() const
 {
     return InputTypeNames::checkbox();
 }
 
 bool CheckboxInputType::valueMissing(const String&) const
 {
-    return element()->isRequired() && !element()->checked();
+    return element().isRequired() && !element().checked();
 }
 
 String CheckboxInputType::valueMissingText() const
 {
     return locale().queryString(WebKit::WebLocalizedString::ValidationValueMissingForCheckbox);
 }
 
 void CheckboxInputType::handleKeyupEvent(KeyboardEvent* event)
 {
     const String& key = event->keyIdentifier();
     if (key != "U+0020")
         return;
     dispatchSimulatedClickIfActive(event);
 }
 
 PassOwnPtr<ClickHandlingState> CheckboxInputType::willDispatchClick()
 {
     // An event handler can use preventDefault or "return false" to reverse the checking we do here.
     // The ClickHandlingState object contains what we need to undo what we did here in didDispatchClick.
 
     OwnPtr<ClickHandlingState> state = adoptPtr(new ClickHandlingState);
 
-    state->checked = element()->checked();
-    state->indeterminate = element()->indeterminate();
+    state->checked = element().checked();
+    state->indeterminate = element().indeterminate();
 
     if (state->indeterminate)
-        element()->setIndeterminate(false);
+        element().setIndeterminate(false);
 
-    element()->setChecked(!state->checked, DispatchChangeEvent);
+    element().setChecked(!state->checked, DispatchChangeEvent);
 
     return state.release();
 }
 
 void CheckboxInputType::didDispatchClick(Event* event, const ClickHandlingState& state)
 {
     if (event->defaultPrevented() || event->defaultHandled()) {
-        element()->setIndeterminate(state.indeterminate);
-        element()->setChecked(state.checked);
+        element().setIndeterminate(state.indeterminate);
+        element().setChecked(state.checked);
     }
 
     // The work we did in willDispatchClick was default handling.
     event->setDefaultHandled();
 }
 
 bool CheckboxInputType::isCheckbox() const
 {
     return true;
 }
 
 bool CheckboxInputType::supportsIndeterminateAppearance() const
 {
     return true;
 }
 
 } // namespace WebCore