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

Side by Side Diff: Source/core/workers/WorkerGlobalScope.cpp

Issue 216523002: Oilpan: Replace most of RefPtrs for Event objects with oilpan's transition types (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 8 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 | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2008 Apple Inc. All Rights Reserved. 2 * Copyright (C) 2008 Apple Inc. All Rights Reserved.
3 * Copyright (C) 2009, 2011 Google Inc. All Rights Reserved. 3 * Copyright (C) 2009, 2011 Google Inc. All Rights Reserved.
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 218 matching lines...) Expand 10 before | Expand all | Expand 10 after
229 scriptLoader->loadSynchronously(executionContext(), *it, AllowCrossOrigi nRequests); 229 scriptLoader->loadSynchronously(executionContext(), *it, AllowCrossOrigi nRequests);
230 230
231 // If the fetching attempt failed, throw a NetworkError exception and ab ort all these steps. 231 // If the fetching attempt failed, throw a NetworkError exception and ab ort all these steps.
232 if (scriptLoader->failed()) { 232 if (scriptLoader->failed()) {
233 exceptionState.throwDOMException(NetworkError, "The script at '" + i t->elidedString() + "' failed to load."); 233 exceptionState.throwDOMException(NetworkError, "The script at '" + i t->elidedString() + "' failed to load.");
234 return; 234 return;
235 } 235 }
236 236
237 InspectorInstrumentation::scriptImported(executionContext(), scriptLoade r->identifier(), scriptLoader->script()); 237 InspectorInstrumentation::scriptImported(executionContext(), scriptLoade r->identifier(), scriptLoader->script());
238 238
239 RefPtr<ErrorEvent> errorEvent; 239 RefPtrWillBeRawPtr<ErrorEvent> errorEvent = nullptr;
haraken 2014/04/03 10:39:31 This '= nullptr' was necessary.
240 m_script->evaluate(ScriptSourceCode(scriptLoader->script(), scriptLoader ->responseURL()), &errorEvent); 240 m_script->evaluate(ScriptSourceCode(scriptLoader->script(), scriptLoader ->responseURL()), &errorEvent);
241 if (errorEvent) { 241 if (errorEvent) {
242 m_script->rethrowExceptionFromImportedScript(errorEvent.release()); 242 m_script->rethrowExceptionFromImportedScript(errorEvent.release());
243 return; 243 return;
244 } 244 }
245 } 245 }
246 } 246 }
247 247
248 EventTarget* WorkerGlobalScope::errorEventTarget() 248 EventTarget* WorkerGlobalScope::errorEventTarget()
249 { 249 {
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
303 { 303 {
304 visitor->trace(m_console); 304 visitor->trace(m_console);
305 visitor->trace(m_location); 305 visitor->trace(m_location);
306 visitor->trace(m_navigator); 306 visitor->trace(m_navigator);
307 #if ENABLE(OILPAN) 307 #if ENABLE(OILPAN)
308 HeapSupplementable<WorkerGlobalScope>::trace(visitor); 308 HeapSupplementable<WorkerGlobalScope>::trace(visitor);
309 #endif 309 #endif
310 } 310 }
311 311
312 } // namespace WebCore 312 } // namespace WebCore
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698