Project: chromium Issues People Development process History Sign in
New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.
Issue 229421 Disable 'iframe@seamless' attribute in M27.
Starred by 7 users Project Member Reported by mkwst@chromium.org, Apr 9 2013 Back to list
Status: Archived
Owner:
Closed: Aug 2015
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: All
Pri: 2
Type: Bug



Sign in to add a comment
In https://src.chromium.org/viewvc/blink?view=rev&revision=147704, we hid the 'seamless' attribute of 'iframe' elements behind the seamless iframes runtime flag. I'd like to merge the single line of that change (https://src.chromium.org/viewvc/blink/trunk/Source/WebCore/html/HTMLIFrameElement.idl?r1=147704&r2=147703&pathrev=147704) with that effect back to M27 to ensure that we don't accidentally expose unwanted attributes to the web.

 
Comment 1 by kareng@google.com, Apr 9 2013
Labels: -Merge-Requested Merge-Approved
Comment 2 by mkwst@chromium.org, Apr 16 2013
Labels: -Merge-Approved Merge-Merged
Merged in http://src.chromium.org/viewvc/blink?view=revision&revision=147958.
Project Member Comment 3 by bugdroid1@chromium.org, Apr 18 2013
Labels: merge-merged-chromium
------------------------------------------------------------------------
r147958 | mkwst@chromium.org | 2013-04-09T19:40:19.037871Z

Changed paths:
   M http://src.chromium.org/viewvc/blink/branches/chromium/1453/Source/WebCore/html/HTMLIFrameElement.idl?r1=147958&r2=147957&pathrev=147958

Disable 'iframe@seamless' for M27.

Seamless IFrames are hidden behind a runtime flag in M27, but the 'seamless'
attribute on the 'iframe' element isn't propetly guarded. This patch merges
the one-line change to HTMLIFrameElement.idl from r147704 to add the V8
annotation for that attribute.

BUG= 229421 

------------------------------------------------------------------------
Status: Archived
Slated to an old branch and no activity in the bug in over a year suggest that this can be archived.
Sign in to add a comment