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

Side by Side Diff: Source/core/html/HTMLMediaElement.cpp

Issue 54273007: Use more references in ContainerNode code (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Fix crashes Created 7 years, 1 month 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
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2008, 2009, 2010, 2011, 2012, 2013 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 2831 matching lines...) Expand 10 before | Expand all | Expand 10 after
2842 KURL mediaURL; 2842 KURL mediaURL;
2843 Node* node; 2843 Node* node;
2844 HTMLSourceElement* source = 0; 2844 HTMLSourceElement* source = 0;
2845 String type; 2845 String type;
2846 String system; 2846 String system;
2847 bool lookingForStartNode = m_nextChildNodeToConsider; 2847 bool lookingForStartNode = m_nextChildNodeToConsider;
2848 bool canUseSourceElement = false; 2848 bool canUseSourceElement = false;
2849 bool okToLoadSourceURL; 2849 bool okToLoadSourceURL;
2850 2850
2851 NodeVector potentialSourceNodes; 2851 NodeVector potentialSourceNodes;
2852 getChildNodes(this, potentialSourceNodes); 2852 getChildNodes(*this, potentialSourceNodes);
2853 2853
2854 for (unsigned i = 0; !canUseSourceElement && i < potentialSourceNodes.size() ; ++i) { 2854 for (unsigned i = 0; !canUseSourceElement && i < potentialSourceNodes.size() ; ++i) {
2855 node = potentialSourceNodes[i].get(); 2855 node = potentialSourceNodes[i].get();
2856 if (lookingForStartNode && m_nextChildNodeToConsider != node) 2856 if (lookingForStartNode && m_nextChildNodeToConsider != node)
2857 continue; 2857 continue;
2858 lookingForStartNode = false; 2858 lookingForStartNode = false;
2859 2859
2860 if (!node->hasTagName(sourceTag)) 2860 if (!node->hasTagName(sourceTag))
2861 continue; 2861 continue;
2862 if (node->parentNode() != this) 2862 if (node->parentNode() != this)
(...skipping 1007 matching lines...) Expand 10 before | Expand all | Expand 10 after
3870 { 3870 {
3871 scheduleLayerUpdate(); 3871 scheduleLayerUpdate();
3872 } 3872 }
3873 3873
3874 bool HTMLMediaElement::isInteractiveContent() const 3874 bool HTMLMediaElement::isInteractiveContent() const
3875 { 3875 {
3876 return fastHasAttribute(controlsAttr); 3876 return fastHasAttribute(controlsAttr);
3877 } 3877 }
3878 3878
3879 } 3879 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698