Select when disabled should not allow value change and popup is disabled

Source/core/html/HTMLOptionElement.cpp

 /*
  * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
  *           (C) 1999 Antti Koivisto (koivisto@kde.org)
  *           (C) 2001 Dirk Mueller (mueller@kde.org)
  *           (C) 2006 Alexey Proskuryakov (ap@nypop.com)
  * Copyright (C) 2004, 2005, 2006, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Google Inc. All rights reserved.
  * Copyright (C) 2011 Motorola Mobility, Inc.  All rights reserved.
  *
  * This library is free software; you can redistribute it and/or
@@ skipping 309 matching lines @@
     ContainerNode* parent = parentNode();
     if (parent && parent->hasTagName(optgroupTag))
         return "    " + text();
     return text();
 }
 
 bool HTMLOptionElement::isDisabledFormControl() const
 {
     if (ownElementDisabled())
         return true;
+    bool status = false;
+    ContainerNode* select = parentNode();
     if (Element* parent = parentElement())
-        return parent->hasTagName(optgroupTag) && parent->isDisabledFormControl();
-    return false;
+        status = parent->hasTagName(optgroupTag) && parent->isDisabledFormControl();
+    if (!status && select && select->hasTagName(selectTag))
+        status = ownerSelectElement()->isDisabledOrReadOnly();

[tkent, 2014/02/12 08:15:43]

We can't do this change.  This function should match to the HTML specification.

http://www.whatwg.org/specs/web-apps/current-work/multipage/common-idioms.htm...
http://www.whatwg.org/specs/web-apps/current-work/multipage/the-button-elemen...
An option in <select disabled><option>...</select> is NOT disabled according to
the specification.

+    return status;
 }
 
 Node::InsertionNotificationRequest HTMLOptionElement::insertedInto(ContainerNode* insertionPoint)
 {
     if (HTMLSelectElement* select = ownerSelectElement()) {
         select->setRecalcListItems();
         // Do not call selected() since calling updateListItemSelectedStates()
         // at this time won't do the right thing. (Why, exactly?)
         // FIXME: Might be better to call this unconditionally, always passing m_isSelected,
         // rather than only calling it if we are selected.
@@ skipping 22 matching lines @@
 
 HTMLFormElement* HTMLOptionElement::form() const
 {
     if (HTMLSelectElement* selectElement = ownerSelectElement())
         return selectElement->formOwner();
 
     return 0;
 }
 
 } // namespace WebCore