Commit e4ac69da authored by philn@webkit.org's avatar philn@webkit.org
Browse files

2011-01-18 Philippe Normand <pnormand@igalia.com>

        Unreviewed, rolling out r76002.
        http://trac.webkit.org/changeset/76002
        https://bugs.webkit.org/show_bug.cgi?id=49790

        breaks a bunch of fast/forms tests on mac and GTK

        * fast/forms/select-option-background-color-change.html: Removed.
        * fast/forms/select-option-color-change.html: Removed.
        * fast/forms/select-option-visibility-change.html: Removed.
        * platform/mac/fast/forms/select-option-background-color-change-expected.checksum: Removed.
        * platform/mac/fast/forms/select-option-background-color-change-expected.png: Removed.
        * platform/mac/fast/forms/select-option-background-color-change-expected.txt: Removed.
        * platform/mac/fast/forms/select-option-color-change-expected.checksum: Removed.
        * platform/mac/fast/forms/select-option-color-change-expected.png: Removed.
        * platform/mac/fast/forms/select-option-color-change-expected.txt: Removed.
        * platform/mac/fast/forms/select-option-visibility-change-expected.checksum: Removed.
        * platform/mac/fast/forms/select-option-visibility-change-expected.png: Removed.
        * platform/mac/fast/forms/select-option-visibility-change-expected.txt: Removed.
2011-01-18  Philippe Normand  <pnormand@igalia.com>

        Unreviewed, rolling out r76002.
        http://trac.webkit.org/changeset/76002
        https://bugs.webkit.org/show_bug.cgi?id=49790

        breaks a bunch of fast/forms tests on mac and GTK

        * html/HTMLOptionElement.cpp:
        (WebCore::HTMLOptionElement::setRenderStyle):

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@76006 268f45cc-cd09-0410-ab3c-d52691b4dbfc
parent 938aa80d
2011-01-18 Philippe Normand <pnormand@igalia.com>
Unreviewed, rolling out r76002.
http://trac.webkit.org/changeset/76002
https://bugs.webkit.org/show_bug.cgi?id=49790
breaks a bunch of fast/forms tests on mac and GTK
* fast/forms/select-option-background-color-change.html: Removed.
* fast/forms/select-option-color-change.html: Removed.
* fast/forms/select-option-visibility-change.html: Removed.
* platform/mac/fast/forms/select-option-background-color-change-expected.checksum: Removed.
* platform/mac/fast/forms/select-option-background-color-change-expected.png: Removed.
* platform/mac/fast/forms/select-option-background-color-change-expected.txt: Removed.
* platform/mac/fast/forms/select-option-color-change-expected.checksum: Removed.
* platform/mac/fast/forms/select-option-color-change-expected.png: Removed.
* platform/mac/fast/forms/select-option-color-change-expected.txt: Removed.
* platform/mac/fast/forms/select-option-visibility-change-expected.checksum: Removed.
* platform/mac/fast/forms/select-option-visibility-change-expected.png: Removed.
* platform/mac/fast/forms/select-option-visibility-change-expected.txt: Removed.
2011-01-18 Csaba Osztrogonác <ossy@webkit.org>
 
Unreviewed.
......
<html>
<head>
<script>
function test()
{
if (window.layoutTestController)
layoutTestController.waitUntilDone();
document.getElementById('option').style.backgroundColor = 'green';
if (window.layoutTestController)
layoutTestController.notifyDone();
}
</script>
</head>
</html>
<body onload="setTimeout(test, 0)">
<p>This tests changing the background color of an option element updates the listbox</p>
<select multiple="multiple">
<option id="option" value="A">A</option>
</select>
</body>
</html>
<html>
<head>
<script>
function test()
{
if (window.layoutTestController)
layoutTestController.waitUntilDone();
document.getElementById('option').style.color = 'green';
if (window.layoutTestController)
layoutTestController.notifyDone();
}
</script>
</head>
</html>
<body onload="setTimeout(test, 0)">
<p>This tests changing the color of an option element updates the listbox</p>
<select multiple="multiple">
<option id="option" value="A">A</option>
</select>
</body>
</html>
<html>
<head>
<script>
function test()
{
if (window.layoutTestController)
layoutTestController.waitUntilDone();
document.getElementById('option').style.visibility = 'hidden';
if (window.layoutTestController)
layoutTestController.notifyDone();
}
</script>
</head>
</html>
<body onload="setTimeout(test, 0)">
<p>This tests changing the visibility of an option element updates the listbox</p>
<select multiple="multiple">
<option id="option" value="A">A</option>
</select>
</body>
</html>
layer at (0,0) size 800x600
RenderView at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
RenderBlock {P} at (0,0) size 784x18
RenderText {#text} at (0,0) size 513x18
text run at (0,0) width 513: "This tests changing the background color of an option element updates the listbox"
RenderBlock (anonymous) at (0,34) size 784x61
RenderListBox {SELECT} at (2,2) size 25x57 [bgcolor=#FFFFFF] [border: (1px inset #808080)]
RenderText {#text} at (0,0) size 0x0
b49a6a3ceaaa2b191e5ae3ddaad26d45
\ No newline at end of file
layer at (0,0) size 800x600
RenderView at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
RenderBlock {P} at (0,0) size 784x18
RenderText {#text} at (0,0) size 434x18
text run at (0,0) width 434: "This tests changing the color of an option element updates the listbox"
RenderBlock (anonymous) at (0,34) size 784x61
RenderListBox {SELECT} at (2,2) size 25x57 [bgcolor=#FFFFFF] [border: (1px inset #808080)]
RenderText {#text} at (0,0) size 0x0
layer at (0,0) size 800x600
RenderView at (0,0) size 800x600
layer at (0,0) size 800x600
RenderBlock {HTML} at (0,0) size 800x600
RenderBody {BODY} at (8,8) size 784x584
RenderBlock {P} at (0,0) size 784x18
RenderText {#text} at (0,0) size 456x18
text run at (0,0) width 456: "This tests changing the visibility of an option element updates the listbox"
RenderBlock (anonymous) at (0,34) size 784x61
RenderListBox {SELECT} at (2,2) size 25x57 [bgcolor=#FFFFFF] [border: (1px inset #808080)]
RenderText {#text} at (0,0) size 0x0
2011-01-18 Philippe Normand <pnormand@igalia.com>
Unreviewed, rolling out r76002.
http://trac.webkit.org/changeset/76002
https://bugs.webkit.org/show_bug.cgi?id=49790
breaks a bunch of fast/forms tests on mac and GTK
* html/HTMLOptionElement.cpp:
(WebCore::HTMLOptionElement::setRenderStyle):
2011-01-18 MORITA Hajime <morrita@google.com>
 
Unreviewed windows build fix.
......
......@@ -221,9 +221,6 @@ String HTMLOptionElement::label() const
void HTMLOptionElement::setRenderStyle(PassRefPtr<RenderStyle> newStyle)
{
m_style = newStyle;
if (HTMLSelectElement* select = ownerSelectElement())
if (RenderObject* renderer = select->renderer())
renderer->repaint();
}
RenderStyle* HTMLOptionElement::nonRendererRenderStyle() const
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment