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

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

Issue 99233002: Remove TreatNullAs=NullString for HTMLObjectElement (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: rebase Created 7 years 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 | Annotate | Revision Log
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) 2000 Stefan Schimanski (1Stein@gmx.de) 4 * (C) 2000 Stefan Schimanski (1Stein@gmx.de)
5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved. 5 * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2011 Apple Inc. All rights reserved.
6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies) 6 * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
7 * 7 *
8 * This library is free software; you can redistribute it and/or 8 * This library is free software; you can redistribute it and/or
9 * modify it under the terms of the GNU Library General Public 9 * modify it under the terms of the GNU Library General Public
10 * License as published by the Free Software Foundation; either 10 * License as published by the Free Software Foundation; either
(...skipping 326 matching lines...) Expand 10 before | Expand all | Expand 10 after
337 updateDocNamedItem(); 337 updateDocNamedItem();
338 if (inDocument() && !useFallbackContent()) { 338 if (inDocument() && !useFallbackContent()) {
339 setNeedsWidgetUpdate(true); 339 setNeedsWidgetUpdate(true);
340 setNeedsStyleRecalc(); 340 setNeedsStyleRecalc();
341 } 341 }
342 HTMLPlugInElement::childrenChanged(changedByParser, beforeChange, afterChang e, childCountDelta); 342 HTMLPlugInElement::childrenChanged(changedByParser, beforeChange, afterChang e, childCountDelta);
343 } 343 }
344 344
345 bool HTMLObjectElement::isURLAttribute(const Attribute& attribute) const 345 bool HTMLObjectElement::isURLAttribute(const Attribute& attribute) const
346 { 346 {
347 return attribute.name() == dataAttr || (attribute.name() == usemapAttr && at tribute.value().string()[0] != '#') || HTMLPlugInElement::isURLAttribute(attribu te); 347 return attribute.name() == codebaseAttr || attribute.name() == dataAttr
348 || (attribute.name() == usemapAttr && attribute.value().string()[0] != ' #')
349 || HTMLPlugInElement::isURLAttribute(attribute);
348 } 350 }
349 351
350 const AtomicString HTMLObjectElement::imageSourceURL() const 352 const AtomicString HTMLObjectElement::imageSourceURL() const
351 { 353 {
352 return getAttribute(dataAttr); 354 return getAttribute(dataAttr);
353 } 355 }
354 356
355 // FIXME: Remove this hack. 357 // FIXME: Remove this hack.
356 void HTMLObjectElement::reattachFallbackContent() 358 void HTMLObjectElement::reattachFallbackContent()
357 { 359 {
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
505 { 507 {
506 return FormAssociatedElement::form(); 508 return FormAssociatedElement::form();
507 } 509 }
508 510
509 bool HTMLObjectElement::isInteractiveContent() const 511 bool HTMLObjectElement::isInteractiveContent() const
510 { 512 {
511 return fastHasAttribute(usemapAttr); 513 return fastHasAttribute(usemapAttr);
512 } 514 }
513 515
514 } 516 }
OLDNEW
« no previous file with comments | « LayoutTests/fast/dom/element-attribute-js-null-expected.txt ('k') | Source/core/html/HTMLObjectElement.idl » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698