Migrates battery_status from content/renderer/ to WebKit/platform/

third_party/WebKit/Source/platform/base/Bind.h

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-#include "platform/graphics/paint/DisplayItemList.h"
+#ifndef Bind_h
+#define Bind_h
 
-#include "platform/graphics/paint/PaintChunk.h"
+#include "base/bind.h"
+#include "mojo/public/cpp/bindings/callback.h"
+
+// This file provides limited access to base::Bind() function. base::Bind()
+// could be dangerous if it's used across threads, so we don't want to allow
+// general use of base::Bind().
+// See also "platform/ThreadSafeFunctional.h".

[haraken, 2016/02/09 12:01:21]

I'd move the contents of this file to ThreadSafeFunctional.h.

[Yuki, 2016/02/12 12:36:14]

We'd like to limit the access to base/bind.h.  Note that ThreadSafeFunctional.h
is placed at platform/, and we don't want to allow all the code in platform/ to
use base/bind.h.  So I dug this subdirectory and placed a new DEPS.

 
 namespace blink {
 
-DisplayItemList::Range<DisplayItemList::iterator>
-DisplayItemList::itemsInPaintChunk(const PaintChunk& paintChunk)
+// Binds an instance of a class to its member function. Does not bind anything
+// else.
+template<typename Class, typename ReturnType, typename... Args>
+mojo::Callback<ReturnType(Args...)>
+bindInstanceToMethodForMojo(ReturnType (Class::*method)(Args...), Class* instance)

[haraken, 2016/02/09 12:01:21]

tzik@: In your refactoring plan, this method is going to be WTF::sameThreadBind,
right? If so, we can rename this to WTF::sameThreadBind.

Also it would be great if we could insert an assert to check that the bind
function is not used across threads. You don't need to address it in this CL.

[Yuki, 2016/02/12 12:36:14]

tzik@, is there an easy way to check whether we're running in the same thread or
not?

 {
-    return Range<iterator>(begin() + paintChunk.beginIndex, begin() + paintChunk.endIndex);
-}
-
-DisplayItemList::Range<DisplayItemList::const_iterator>
-DisplayItemList::itemsInPaintChunk(const PaintChunk& paintChunk) const
-{
-    return Range<const_iterator>(begin() + paintChunk.beginIndex, begin() + paintChunk.endIndex);
+    return base::Bind(method, base::Unretained(instance));
 }
 
 } // namespace blink
+
+#endif // Bind_h