Bug 67687 - REGRESSION(r94593): textfield-overflow-by-value-update.html fails
Summary: REGRESSION(r94593): textfield-overflow-by-value-update.html fails
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Layout and Rendering (show other bugs)
Version: 528+ (Nightly build)
Hardware: Unspecified Unspecified
: P1 Normal
Assignee: Ryosuke Niwa
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-06 17:45 PDT by Ryosuke Niwa
Modified: 2011-09-08 12:02 PDT (History)
4 users (show)

See Also:

Attachments
rebaseline the test (1.35 KB, patch)
2011-09-06 18:51 PDT, Ryosuke Niwa 		no flags Details | Formatted Diff | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Comment 1 Beth Dakin 2011-09-06 17:52:58 PDT 
The test depends on a JS file in the directory, but the contents of the failing test are: 

<!DOCTYPE html>
<head>
<script src="resources/common.js"></script>
<style>
input {
    -webkit-appearance: none;
    font-size: 32px;
    height: 0px;
    width: 100px;
    border-width: 0px;
}
</style>
<body>
<!-- The input value must not be shown outside the input border.
     So, the test result should be just white. -->
<input id="textfield">
<script>
$('textfield').offsetTop;
$('textfield').value = "FAIL";
</script>
</body>
Comment 2 Ryosuke Niwa 2011-09-06 17:59:38 PDT 
Note this test is added by tkent in http://trac.webkit.org/changeset/90379.
Comment 3 Kent Tamura 2011-09-06 18:18:39 PDT 
The text result is not important for this test. If the pixel result is not changed, let's do rebaseline.
Comment 4 Dave Hyatt 2011-09-06 18:20:21 PDT 
This actually doesn't fail for me in ToT Safari.
Comment 5 Ryosuke Niwa 2011-09-06 18:25:29 PDT 
(In reply to comment #3)
> The text result is not important for this test. If the pixel result is not changed, let's do rebaseline.

(In reply to comment #4)
> This actually doesn't fail for me in ToT Safari.

Okay, that sounds great!  I'm going to build ToT and run the pixel test to confirm that there's no regression.
Comment 6 Ryosuke Niwa 2011-09-06 18:49:13 PDT 
I have confirmed that the pixel test still passes.  I'm going to rebaseline it.
Comment 7 Ryosuke Niwa 2011-09-06 18:51:29 PDT 
Created attachment 106532 [details]
rebaseline the test
Comment 8 Kent Tamura 2011-09-06 19:24:00 PDT 
The test has failed on
- "SnowLeopard Intel Debug (Tests)" bot
- "Leopard Intel Debug (Tests)" bot

The test hasn't failed on
- "SnowLeopard Intel Release (tests)" bot
- Chromium ToT bots on chromium.org
- Chromium ToT bots on webkit.org
- my local debug WebKit ToT on Snow Leopard
Comment 9 Ryosuke Niwa 2011-09-06 19:26:51 PDT 
(In reply to comment #8)
> The test has failed on
> - "SnowLeopard Intel Debug (Tests)" bot
> - "Leopard Intel Debug (Tests)" bot
> 
> The test hasn't failed on
> - "SnowLeopard Intel Release (tests)" bot
> - Chromium ToT bots on chromium.org
> - Chromium ToT bots on webkit.org
> - my local debug WebKit ToT on Snow Leopard

Mn... You're right.  That is very weird.
Comment 10 Ryosuke Niwa 2011-09-08 12:02:16 PDT 
It seems like the test is no longer failing.