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

Side by Side Diff: third_party/WebKit/Source/core/html/HTMLMetaElement-in.cpp

Issue 2149893003: Rename Node::inShadowIncludingDocument() to Node::isConnected() (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: addressed Created 4 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
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 Dirk Mueller (mueller@kde.org) 4 * (C) 2001 Dirk Mueller (mueller@kde.org)
5 * Copyright (C) 2003, 2010 Apple Inc. All rights reserved. 5 * Copyright (C) 2003, 2010 Apple Inc. All rights reserved.
6 * 6 *
7 * This library is free software; you can redistribute it and/or 7 * This library is free software; you can redistribute it and/or
8 * modify it under the terms of the GNU Library General Public 8 * modify it under the terms of the GNU Library General Public
9 * License as published by the Free Software Foundation; either 9 * License as published by the Free Software Foundation; either
10 * version 2 of the License, or (at your option) any later version. 10 * version 2 of the License, or (at your option) any later version.
(...skipping 438 matching lines...) Expand 10 before | Expand all | Expand 10 after
449 return InsertionShouldCallDidNotifySubtreeInsertions; 449 return InsertionShouldCallDidNotifySubtreeInsertions;
450 } 450 }
451 451
452 void HTMLMetaElement::didNotifySubtreeInsertionsToDocument() 452 void HTMLMetaElement::didNotifySubtreeInsertionsToDocument()
453 { 453 {
454 process(); 454 process();
455 } 455 }
456 456
457 static bool inDocumentHead(HTMLMetaElement* element) 457 static bool inDocumentHead(HTMLMetaElement* element)
458 { 458 {
459 if (!element->inShadowIncludingDocument()) 459 if (!element->isConnected())
460 return false; 460 return false;
461 461
462 return Traversal<HTMLHeadElement>::firstAncestor(*element); 462 return Traversal<HTMLHeadElement>::firstAncestor(*element);
463 } 463 }
464 464
465 void HTMLMetaElement::process() 465 void HTMLMetaElement::process()
466 { 466 {
467 if (!inShadowIncludingDocument()) 467 if (!isConnected())
468 return; 468 return;
469 469
470 // All below situations require a content attribute (which can be the empty string). 470 // All below situations require a content attribute (which can be the empty string).
471 const AtomicString& contentValue = fastGetAttribute(contentAttr); 471 const AtomicString& contentValue = fastGetAttribute(contentAttr);
472 if (contentValue.isNull()) 472 if (contentValue.isNull())
473 return; 473 return;
474 474
475 const AtomicString& nameValue = fastGetAttribute(nameAttr); 475 const AtomicString& nameValue = fastGetAttribute(nameAttr);
476 if (!nameValue.isEmpty()) { 476 if (!nameValue.isEmpty()) {
477 if (equalIgnoringCase(nameValue, "viewport")) 477 if (equalIgnoringCase(nameValue, "viewport"))
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
514 { 514 {
515 return getAttribute(http_equivAttr); 515 return getAttribute(http_equivAttr);
516 } 516 }
517 517
518 const AtomicString& HTMLMetaElement::name() const 518 const AtomicString& HTMLMetaElement::name() const
519 { 519 {
520 return getNameAttribute(); 520 return getNameAttribute();
521 } 521 }
522 522
523 } 523 }
OLDNEW
« no previous file with comments | « third_party/WebKit/Source/core/html/HTMLMediaElement.cpp ('k') | third_party/WebKit/Source/core/html/HTMLObjectElement.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698