<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.webkit.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.4.1"
          urlbase="https://bugs.webkit.org/"
          
          maintainer="admin@webkit.org"
>

    <bug>
          <bug_id>25430</bug_id>
          
          <creation_ts>2009-04-27 14:01:58 -0700</creation_ts>
          <short_desc>Enable loading style sheet for UI controls for media elements in Chromium port</short_desc>
          <delta_ts>2009-04-29 13:14:33 -0700</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>1</classification_id>
          <classification>Unclassified</classification>
          <product>WebKit</product>
          <component>WebKit Misc.</component>
          <version>528+ (Nightly build)</version>
          <rep_platform>PC</rep_platform>
          <op_sys>OS X 10.5</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>P2</priority>
          <bug_severity>Normal</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Hin-Chung Lam">hclam</reporter>
          <assigned_to name="Nobody">webkit-unassigned</assigned_to>
          <cc>dglazkov</cc>
          

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>119020</commentid>
    <comment_count>0</comment_count>
    <who name="Hin-Chung Lam">hclam</who>
    <bug_when>2009-04-27 14:01:58 -0700</bug_when>
    <thetext>Implement a virtual method in RenderTheme to provide style sheet for media controls in Chromium port.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119021</commentid>
    <comment_count>1</comment_count>
      <attachid>29826</attachid>
    <who name="Hin-Chung Lam">hclam</who>
    <bug_when>2009-04-27 14:07:35 -0700</bug_when>
    <thetext>Created attachment 29826
patch + changelog

Just the patch with change log, since &lt;video&gt; and &lt;audio&gt; in chromium is not ready to run through layout tests and the style for default ui is subject to change so no layout tests are provided. This patch is just to enable loading the  style sheet so change the default style sheet would have effects.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119045</commentid>
    <comment_count>2</comment_count>
    <who name="Sam Weinig">sam</who>
    <bug_when>2009-04-27 16:04:48 -0700</bug_when>
    <thetext>In the future, please don&apos;t remove the NOBODY (OOPS!) from the ChangeLog.  We have post commit scripts that check to make sure the lander didn&apos;t forget to update the reviewer by checking for it. </thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>119321</commentid>
    <comment_count>3</comment_count>
    <who name="Eric Seidel (no email)">eric</who>
    <bug_when>2009-04-29 13:14:33 -0700</bug_when>
    <thetext>Committing to http://svn.webkit.org/repository/webkit/trunk ...
	M	WebCore/ChangeLog
	M	WebCore/rendering/RenderThemeChromiumWin.cpp
	M	WebCore/rendering/RenderThemeChromiumWin.h
Committed r43002
</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>29826</attachid>
            <date>2009-04-27 14:07:35 -0700</date>
            <delta_ts>2009-04-27 15:34:15 -0700</delta_ts>
            <desc>patch + changelog</desc>
            <filename>webkit-render-theme.patch</filename>
            <type>text/plain</type>
            <size>1953</size>
            <attacher name="Hin-Chung Lam">hclam</attacher>
            
              <data encoding="base64">SW5kZXg6IFdlYkNvcmUvQ2hhbmdlTG9nCj09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT0KLS0tIFdlYkNvcmUvQ2hhbmdlTG9n
CShyZXZpc2lvbiAxNDYwNykKKysrIFdlYkNvcmUvQ2hhbmdlTG9nCSh3b3JraW5nIGNvcHkpCkBA
IC0xLDMgKzEsMTQgQEAKKzIwMDktMDQtMjcgIEFscGhhIExhbSAgPGhjbGFtQGNocm9taXVtLm9y
Zz4KKworICAgICAgICBSZXZpZXdlZCBieSA/LgorCisgICAgICAgIGh0dHBzOi8vYnVncy53ZWJr
aXQub3JnL3Nob3dfYnVnLmNnaT9pZD0yNTQzMAorCisgICAgICAgIFByb3ZpZGUgc3R5bGUgc2hl
ZXQgZm9yIFVJIGNvbnRyb2xzIG9mIG1lZGlhIGVsZW1lbnRzIGZvciBDaHJvbWl1bSBwb3J0Lgor
CisgICAgICAgICogcmVuZGVyaW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uY3BwOgorICAgICAg
ICAoV2ViQ29yZTo6UmVuZGVyVGhlbWVDaHJvbWl1bVdpbjo6ZXh0cmFNZWRpYUNvbnRyb2xzU3R5
bGVTaGVldCk6CisKIDIwMDktMDQtMjMgIEVyaWMgU2VpZGVsICA8ZXJpY0B3ZWJraXQub3JnPgog
CiAgICAgICAgIFJldmlld2VkIGJ5IFNpbW9uIEZyYXNlci4KSW5kZXg6IFdlYkNvcmUvcmVuZGVy
aW5nL1JlbmRlclRoZW1lQ2hyb21pdW1XaW4uaAo9PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09Ci0tLSBXZWJDb3JlL3JlbmRl
cmluZy9SZW5kZXJUaGVtZUNocm9taXVtV2luLmgJKHJldmlzaW9uIDE0NjA3KQorKysgV2ViQ29y
ZS9yZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5oCSh3b3JraW5nIGNvcHkpCkBAIC00
OSw2ICs0OSw5IEBACiAKICAgICAgICAgdmlydHVhbCBTdHJpbmcgZXh0cmFEZWZhdWx0U3R5bGVT
aGVldCgpOwogICAgICAgICB2aXJ0dWFsIFN0cmluZyBleHRyYVF1aXJrc1N0eWxlU2hlZXQoKTsK
KyNpZiBFTkFCTEUoVklERU8pCisgICAgICAgIHZpcnR1YWwgU3RyaW5nIGV4dHJhTWVkaWFDb250
cm9sc1N0eWxlU2hlZXQoKTsKKyNlbmRpZgogCiAgICAgICAgIC8vIEEgbWV0aG9kIGFza2luZyBp
ZiB0aGUgdGhlbWUncyBjb250cm9scyBhY3R1YWxseSBjYXJlIGFib3V0IHJlZHJhd2luZyB3aGVu
IGhvdmVyZWQuCiAgICAgICAgIHZpcnR1YWwgYm9vbCBzdXBwb3J0c0hvdmVyKGNvbnN0IFJlbmRl
clN0eWxlKikgY29uc3QgeyByZXR1cm4gdHJ1ZTsgfQpJbmRleDogV2ViQ29yZS9yZW5kZXJpbmcv
UmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5jcHAKPT09PT09PT09PT09PT09PT09PT09PT09PT09PT09
PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PT09PQotLS0gV2ViQ29yZS9yZW5kZXJp
bmcvUmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5jcHAJKHJldmlzaW9uIDE0NjA3KQorKysgV2ViQ29y
ZS9yZW5kZXJpbmcvUmVuZGVyVGhlbWVDaHJvbWl1bVdpbi5jcHAJKHdvcmtpbmcgY29weSkKQEAg
LTI2MSw2ICsyNjEsMTMgQEAKICAgICByZXR1cm4gU3RyaW5nKHRoZW1lV2luUXVpcmtzVXNlckFn
ZW50U3R5bGVTaGVldCwgc2l6ZW9mKHRoZW1lV2luUXVpcmtzVXNlckFnZW50U3R5bGVTaGVldCkp
OwogfQogCisjaWYgRU5BQkxFKFZJREVPKQorU3RyaW5nIFJlbmRlclRoZW1lQ2hyb21pdW1XaW46
OmV4dHJhTWVkaWFDb250cm9sc1N0eWxlU2hlZXQoKQoreworICAgIHJldHVybiBTdHJpbmcobWVk
aWFDb250cm9sc1VzZXJBZ2VudFN0eWxlU2hlZXQsIHNpemVvZihtZWRpYUNvbnRyb2xzVXNlckFn
ZW50U3R5bGVTaGVldCkpOworfQorI2VuZGlmCisKIGJvb2wgUmVuZGVyVGhlbWVDaHJvbWl1bVdp
bjo6c3VwcG9ydHNGb2N1c1JpbmcoY29uc3QgUmVuZGVyU3R5bGUqIHN0eWxlKSBjb25zdAogewog
ICAgLy8gTGV0IHdlYmtpdCBkcmF3IG9uZSBvZiBpdHMgaGFsbyByaW5ncyBhcm91bmQgYW55IGZv
Y3VzZWQgZWxlbWVudCwK
</data>
<flag name="review"
          id="14934"
          type_id="1"
          status="+"
          setter="dglazkov"
    />
          </attachment>
      

    </bug>

</bugzilla>