Implement requestIdleCallback API

Source/core/frame/LocalDOMWindow.cpp

 /*
  * Copyright (C) 2006, 2007, 2008, 2010 Apple Inc. All rights reserved.
  * Copyright (C) 2010 Nokia Corporation and/or its subsidiary(-ies)
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 1351 matching lines @@
         return d->requestAnimationFrame(callback);
     return 0;
 }
 
 void LocalDOMWindow::cancelAnimationFrame(int id)
 {
     if (Document* d = document())
         d->cancelAnimationFrame(id);
 }
 
+int LocalDOMWindow::requestIdleFrame(IdleRequestCallback* callback)
+{
+    if (Document* d = document())

[esprehn, 2015/06/10 07:11:42]

document, don't abbreviate.

[rmcilroy, 2015/06/10 18:32:41]

I couldn't call this document because of the function of the same name - instead
I just removed the local variable and used document() in both places (also in
rAF).

+        return d->requestIdleFrame(callback);
+    return 0;
+}
+
+void LocalDOMWindow::cancelIdleFrame(int id)
+{
+    if (Document* d = document())

[esprehn, 2015/06/10 07:11:42]

ditto, also can this really be null? I don't think it can.

[rmcilroy, 2015/06/10 18:32:41]

clearDocument resets m_document. I'm not sure on whether this can have happened
before a requestIdleFrame/cancelIdleFrame though (I was following the same
pattern as rAF above).

+        d->cancelIdleFrame(id);
+}
+
 DOMWindowCSS* LocalDOMWindow::css() const
 {
     if (!m_css)
         m_css = DOMWindowCSS::create();
     return m_css.get();
 }
 
 bool LocalDOMWindow::addEventListener(const AtomicString& eventType, PassRefPtr<EventListener> prpListener, bool useCapture)
 {
     RefPtr<EventListener> listener = prpListener;
@@ skipping 284 matching lines @@
     return m_frameObserver->frame();
 }
 
 v8::Handle<v8::Object> LocalDOMWindow::wrap(v8::Handle<v8::Object> creationContext, v8::Isolate* isolate)
 {
     ASSERT_NOT_REACHED(); // LocalDOMWindow has [Custom=ToV8].
     return v8::Handle<v8::Object>();
 }
 
 } // namespace blink