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

Side by Side Diff: Source/core/frame/LocalDOMWindow.cpp

Issue 642293004: Use C++11 range-based loop in core/frame (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Use meaningful names Created 6 years, 2 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) 2006, 2007, 2008, 2010 Apple Inc. All rights reserved. 2 * Copyright (C) 2006, 2007, 2008, 2010 Apple Inc. All rights reserved.
3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies) 3 * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
4 * 4 *
5 * Redistribution and use in source and binary forms, with or without 5 * Redistribution and use in source and binary forms, with or without
6 * modification, are permitted provided that the following conditions 6 * modification, are permitted provided that the following conditions
7 * are met: 7 * are met:
8 * 1. Redistributions of source code must retain the above copyright 8 * 1. 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 * 2. Redistributions in binary form must reproduce the above copyright 10 * 2. Redistributions in binary form must reproduce the above copyright
(...skipping 533 matching lines...) Expand 10 before | Expand all | Expand 10 after
544 544
545 void LocalDOMWindow::willDetachFrameHost() 545 void LocalDOMWindow::willDetachFrameHost()
546 { 546 {
547 m_frame->host()->eventHandlerRegistry().didRemoveAllEventHandlers(*this); 547 m_frame->host()->eventHandlerRegistry().didRemoveAllEventHandlers(*this);
548 m_frame->console().messageStorage()->frameWindowDiscarded(this); 548 m_frame->console().messageStorage()->frameWindowDiscarded(this);
549 InspectorInstrumentation::frameWindowDiscarded(m_frame, this); 549 InspectorInstrumentation::frameWindowDiscarded(m_frame, this);
550 } 550 }
551 551
552 void LocalDOMWindow::willDestroyDocumentInFrame() 552 void LocalDOMWindow::willDestroyDocumentInFrame()
553 { 553 {
554 for (WillBeHeapHashSet<RawPtrWillBeWeakMember<DOMWindowProperty> >::const_it erator it = m_properties.begin(); it != m_properties.end(); ++it) 554 for (const auto& domWindowProperty : m_properties)
555 (*it)->willDestroyGlobalObjectInFrame(); 555 domWindowProperty->willDestroyGlobalObjectInFrame();
556 } 556 }
557 557
558 void LocalDOMWindow::willDetachDocumentFromFrame() 558 void LocalDOMWindow::willDetachDocumentFromFrame()
559 { 559 {
560 for (WillBeHeapHashSet<RawPtrWillBeWeakMember<DOMWindowProperty> >::const_it erator it = m_properties.begin(); it != m_properties.end(); ++it) 560 for (const auto& domWindowProperty : m_properties)
561 (*it)->willDetachGlobalObjectFromFrame(); 561 domWindowProperty->willDetachGlobalObjectFromFrame();
562 } 562 }
563 563
564 void LocalDOMWindow::registerProperty(DOMWindowProperty* property) 564 void LocalDOMWindow::registerProperty(DOMWindowProperty* property)
565 { 565 {
566 m_properties.add(property); 566 m_properties.add(property);
567 } 567 }
568 568
569 void LocalDOMWindow::unregisterProperty(DOMWindowProperty* property) 569 void LocalDOMWindow::unregisterProperty(DOMWindowProperty* property)
570 { 570 {
571 m_properties.remove(property); 571 m_properties.remove(property);
(...skipping 1347 matching lines...) Expand 10 before | Expand all | Expand 10 after
1919 FrameDestructionObserver::trace(visitor); 1919 FrameDestructionObserver::trace(visitor);
1920 } 1920 }
1921 1921
1922 v8::Handle<v8::Object> LocalDOMWindow::wrap(v8::Handle<v8::Object> creationConte xt, v8::Isolate* isolate) 1922 v8::Handle<v8::Object> LocalDOMWindow::wrap(v8::Handle<v8::Object> creationConte xt, v8::Isolate* isolate)
1923 { 1923 {
1924 ASSERT_NOT_REACHED(); // LocalDOMWindow has [Custom=ToV8]. 1924 ASSERT_NOT_REACHED(); // LocalDOMWindow has [Custom=ToV8].
1925 return v8::Handle<v8::Object>(); 1925 return v8::Handle<v8::Object>();
1926 } 1926 }
1927 1927
1928 } // namespace blink 1928 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698