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

Side by Side Diff: Source/bindings/v8/V8WindowShell.cpp

Issue 280123002: Oilpan: move LiveNodeList collections to the heap. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Have NodeRareData clear out NodeListsNodeData instead. Created 6 years, 7 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) 2008, 2009, 2011 Google Inc. All rights reserved. 2 * Copyright (C) 2008, 2009, 2011 Google 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 are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 400 matching lines...) Expand 10 before | Expand all | Expand 10 after
411 return; 411 return;
412 updateDocumentProperty(); 412 updateDocumentProperty();
413 updateSecurityOrigin(m_frame->document()->securityOrigin()); 413 updateSecurityOrigin(m_frame->document()->securityOrigin());
414 } 414 }
415 415
416 static v8::Handle<v8::Value> getNamedProperty(HTMLDocument* htmlDocument, const AtomicString& key, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate) 416 static v8::Handle<v8::Value> getNamedProperty(HTMLDocument* htmlDocument, const AtomicString& key, v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
417 { 417 {
418 if (!htmlDocument->hasNamedItem(key) && !htmlDocument->hasExtraNamedItem(key )) 418 if (!htmlDocument->hasNamedItem(key) && !htmlDocument->hasExtraNamedItem(key ))
419 return v8Undefined(); 419 return v8Undefined();
420 420
421 RefPtr<HTMLCollection> items = htmlDocument->documentNamedItems(key); 421 RefPtrWillBeRawPtr<HTMLCollection> items = htmlDocument->documentNamedItems( key);
422 if (items->isEmpty()) 422 if (items->isEmpty())
423 return v8Undefined(); 423 return v8Undefined();
424 424
425 if (items->hasExactlyOneItem()) { 425 if (items->hasExactlyOneItem()) {
426 Element* element = items->item(0); 426 Element* element = items->item(0);
427 ASSERT(element); 427 ASSERT(element);
428 Frame* frame = 0; 428 Frame* frame = 0;
429 if (isHTMLIFrameElement(*element) && (frame = toHTMLIFrameElement(*eleme nt).contentFrame())) 429 if (isHTMLIFrameElement(*element) && (frame = toHTMLIFrameElement(*eleme nt).contentFrame()))
430 return toV8(frame->domWindow(), creationContext, isolate); 430 return toV8(frame->domWindow(), creationContext, isolate);
431 return toV8(element, creationContext, isolate); 431 return toV8(element, creationContext, isolate);
(...skipping 52 matching lines...) Expand 10 before | Expand all | Expand 10 after
484 484
485 void V8WindowShell::updateSecurityOrigin(SecurityOrigin* origin) 485 void V8WindowShell::updateSecurityOrigin(SecurityOrigin* origin)
486 { 486 {
487 ASSERT(m_world->isMainWorld()); 487 ASSERT(m_world->isMainWorld());
488 if (!isContextInitialized()) 488 if (!isContextInitialized())
489 return; 489 return;
490 setSecurityToken(origin); 490 setSecurityToken(origin);
491 } 491 }
492 492
493 } // WebCore 493 } // WebCore
OLDNEW
« no previous file with comments | « Source/bindings/tests/results/V8TestSpecialOperations.cpp ('k') | Source/bindings/v8/custom/V8HTMLAllCollectionCustom.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698