New issue
Advanced search Search tips

Issue 811173 link

Starred by 3 users

Issue metadata

Status: Fixed
Owner: ----
Closed: Feb 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

StyleSheet |media|, CSSPageRule |style| IDL attribute should have [PutForwards]

Reported by bhagirat...@samsung.com, Feb 12 2018

Issue description

UserAgent: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:54.0) Gecko/20100101 Firefox/54.0

Steps to reproduce the problem:
1. StyleSheet |media|, CSSPageRule |style| IDL attribute should have [PutForwards]
2. 
3. 

What is the expected behavior?

What went wrong?
StyleSheet |media| should have [PutForwards=mediaText] and CSSPageRule |style| should have [PutForwards=cssText] as per the specification.
https://drafts.csswg.org/cssom/#the-stylesheet-interface
https://drafts.csswg.org/cssom/#the-csspagerule-interface

Did this work before? N/A 

Does this work in other browsers? N/A

Chrome version: <Copy from: 'about:version'>  Channel: n/a
OS Version: 
Flash Version: Shockwave Flash 10.1 r999.<br>Gnash 0.8.11dev, the GNU SWF Player.   Copyright (C) 2006, 2007, 2008, 2009, 2010, 2011   <a href="http://www.fsf.org">Free   Software Foundation</a>, Inc. <br>   Gnash comes with NO WARRANTY, to the extent permitted by law.   You may redistribute copies of Gnash under the terms of the   <a href="http://www.gnu.org/licenses/gpl.html">GNU General Public   License</a>. For more information about Gnash, see <a   href="http://www.gnu.org/software/gnash/">   http://www.gnu.org/software/gnash</a>.   <br>  Compatible Shockwave Flash 10.1 r999.
 
Working on this issue. Thanks !
Project Member

Comment 2 by bugdroid1@chromium.org, Feb 12 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac

commit b7e3a91cb94a9f47a8ebbef7a887d31069c18aac
Author: Bhagirathi Satpathy <bhagirathi.s@samsung.com>
Date: Mon Feb 12 10:50:03 2018

Sync StyleSheet |media|, CSSPageRule |style| IDL attribute as per the spec

Modified StyleSheet |media|and CSSPageRule |style| IDL attribute
to match the specification.

https://drafts.csswg.org/cssom/#the-stylesheet-interface
https://drafts.csswg.org/cssom/#the-csspagerule-interface

Bug:  811173 

Change-Id: Ifa2630e72fb66898fcb82a444162c8f2f41c7ac2
Reviewed-on: https://chromium-review.googlesource.com/903803
Commit-Queue: Bhagirathi Satpathy <bhagirathi.s@samsung.com>
Reviewed-by: Kent Tamura <tkent@chromium.org>
Reviewed-by: Yuki Shiino <yukishiino@chromium.org>
Reviewed-by: Kentaro Hara <haraken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#536051}
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/LayoutTests/external/wpt/css/cssom/cssimportrule.html
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/LayoutTests/external/wpt/css/cssom/interfaces-expected.txt
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/LayoutTests/virtual/stable/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/LayoutTests/webexposed/global-interface-listing-expected.txt
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/Source/core/css/CSSPageRule.idl
[modify] https://crrev.com/b7e3a91cb94a9f47a8ebbef7a887d31069c18aac/third_party/WebKit/Source/core/css/StyleSheet.idl

Comment 3 by e...@chromium.org, Feb 12 2018

Status: Fixed (was: Unconfirmed)

Sign in to add a comment