Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(10)

Side by Side Diff: Source/core/dom/Element.cpp

Issue 413783002: Use isHTMLPlugInElement to replace checks (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Remove WindowNameCollection change Created 6 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « Source/core/accessibility/AXNodeObject.cpp ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 /* 1 /*
2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org) 2 * Copyright (C) 1999 Lars Knoll (knoll@kde.org)
3 * (C) 1999 Antti Koivisto (koivisto@kde.org) 3 * (C) 1999 Antti Koivisto (koivisto@kde.org)
4 * (C) 2001 Peter Kelly (pmk@post.com) 4 * (C) 2001 Peter Kelly (pmk@post.com)
5 * (C) 2001 Dirk Mueller (mueller@kde.org) 5 * (C) 2001 Dirk Mueller (mueller@kde.org)
6 * (C) 2007 David Smith (catfish.man@gmail.com) 6 * (C) 2007 David Smith (catfish.man@gmail.com)
7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved. 7 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012, 2013 Apple Inc. All rights reserved.
8 * (C) 2007 Eric Seidel (eric@webkit.org) 8 * (C) 2007 Eric Seidel (eric@webkit.org)
9 * 9 *
10 * This library is free software; you can redistribute it and/or 10 * This library is free software; you can redistribute it and/or
(...skipping 72 matching lines...) Expand 10 before | Expand all | Expand 10 after
83 #include "core/frame/csp/ContentSecurityPolicy.h" 83 #include "core/frame/csp/ContentSecurityPolicy.h"
84 #include "core/html/ClassList.h" 84 #include "core/html/ClassList.h"
85 #include "core/html/HTMLCanvasElement.h" 85 #include "core/html/HTMLCanvasElement.h"
86 #include "core/html/HTMLCollection.h" 86 #include "core/html/HTMLCollection.h"
87 #include "core/html/HTMLDocument.h" 87 #include "core/html/HTMLDocument.h"
88 #include "core/html/HTMLElement.h" 88 #include "core/html/HTMLElement.h"
89 #include "core/html/HTMLFormControlsCollection.h" 89 #include "core/html/HTMLFormControlsCollection.h"
90 #include "core/html/HTMLFrameElementBase.h" 90 #include "core/html/HTMLFrameElementBase.h"
91 #include "core/html/HTMLFrameOwnerElement.h" 91 #include "core/html/HTMLFrameOwnerElement.h"
92 #include "core/html/HTMLOptionsCollection.h" 92 #include "core/html/HTMLOptionsCollection.h"
93 #include "core/html/HTMLPlugInElement.h"
93 #include "core/html/HTMLTableRowsCollection.h" 94 #include "core/html/HTMLTableRowsCollection.h"
94 #include "core/html/HTMLTemplateElement.h" 95 #include "core/html/HTMLTemplateElement.h"
95 #include "core/html/parser/HTMLParserIdioms.h" 96 #include "core/html/parser/HTMLParserIdioms.h"
96 #include "core/inspector/InspectorInstrumentation.h" 97 #include "core/inspector/InspectorInstrumentation.h"
97 #include "core/page/Chrome.h" 98 #include "core/page/Chrome.h"
98 #include "core/page/ChromeClient.h" 99 #include "core/page/ChromeClient.h"
99 #include "core/page/FocusController.h" 100 #include "core/page/FocusController.h"
100 #include "core/page/Page.h" 101 #include "core/page/Page.h"
101 #include "core/page/PointerLockController.h" 102 #include "core/page/PointerLockController.h"
102 #include "core/rendering/RenderLayer.h" 103 #include "core/rendering/RenderLayer.h"
(...skipping 3212 matching lines...) Expand 10 before | Expand all | Expand 10 after
3315 return false; 3316 return false;
3316 if (this == document().cssTarget()) 3317 if (this == document().cssTarget())
3317 return false; 3318 return false;
3318 if (isHTMLElement() && toHTMLElement(this)->hasDirectionAuto()) 3319 if (isHTMLElement() && toHTMLElement(this)->hasDirectionAuto())
3319 return false; 3320 return false;
3320 if (hasActiveAnimations()) 3321 if (hasActiveAnimations())
3321 return false; 3322 return false;
3322 // Turn off style sharing for elements that can gain layers for reasons outs ide of the style system. 3323 // Turn off style sharing for elements that can gain layers for reasons outs ide of the style system.
3323 // See comments in RenderObject::setStyle(). 3324 // See comments in RenderObject::setStyle().
3324 // FIXME: Why does gaining a layer from outside the style system require dis abling sharing? 3325 // FIXME: Why does gaining a layer from outside the style system require dis abling sharing?
3325 if (isHTMLFrameElementBase(*this) 3326 if (isHTMLFrameElementBase(*this) || isHTMLPlugInElement(*this) || isHTMLCan vasElement(*this))
3326 || isHTMLEmbedElement(*this)
3327 || isHTMLObjectElement(*this)
3328 || isHTMLAppletElement(*this)
3329 || isHTMLCanvasElement(*this))
3330 return false; 3327 return false;
3331 if (FullscreenElementStack::isActiveFullScreenElement(*this)) 3328 if (FullscreenElementStack::isActiveFullScreenElement(*this))
3332 return false; 3329 return false;
3333 return true; 3330 return true;
3334 } 3331 }
3335 3332
3336 void Element::trace(Visitor* visitor) 3333 void Element::trace(Visitor* visitor)
3337 { 3334 {
3338 #if ENABLE(OILPAN) 3335 #if ENABLE(OILPAN)
3339 if (hasRareData()) 3336 if (hasRareData())
3340 visitor->trace(elementRareData()); 3337 visitor->trace(elementRareData());
3341 visitor->trace(m_elementData); 3338 visitor->trace(m_elementData);
3342 #endif 3339 #endif
3343 ContainerNode::trace(visitor); 3340 ContainerNode::trace(visitor);
3344 } 3341 }
3345 3342
3346 } // namespace blink 3343 } // namespace blink
OLDNEW
« no previous file with comments | « Source/core/accessibility/AXNodeObject.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698