[cleanup, battery] Don't include unneeded headers.

third_party/WebKit/Source/modules/battery/NavigatorBattery.h

Index: third_party/WebKit/Source/modules/battery/NavigatorBattery.h
diff --git a/third_party/WebKit/Source/modules/battery/NavigatorBattery.h b/third_party/WebKit/Source/modules/battery/NavigatorBattery.h
index 0f9630d8f210dd2d4a663cc0cca90a448933b40d..7826e49a4cb2733ea58439e94c31aa484554ef03 100644
--- a/third_party/WebKit/Source/modules/battery/NavigatorBattery.h
+++ b/third_party/WebKit/Source/modules/battery/NavigatorBattery.h
@@ -5,8 +5,6 @@
 #ifndef NavigatorBattery_h
 #define NavigatorBattery_h
 
-#include "bindings/core/v8/ScriptPromise.h"
-#include "core/frame/Navigator.h"
 #include "platform/Supplementable.h"
 #include "platform/heap/Handle.h"
 
@@ -14,6 +12,8 @@ namespace blink {
 
 class BatteryManager;
 class Navigator;
+class ScriptPromise;
+class ScriptState;

[timvolodine, 2017/06/09 12:06:19]

Probably need to include ScriptPromise and ScriptState in NavigatorBattery.cc?

[riju_, 2017/06/09 13:36:11]

NavigatorBattery.cpp is including BatteryManager.h, which includes
ScriptPromiseProperty, which includes ScriptPromise.
ScriptPromise includes ScriptValue.h, which includes ScriptState.h

[timvolodine, 2017/06/09 14:00:14]

I think we generally avoid relying on transitive includes (because
implementations can change which will cause potential breakage in files that
depend on their includes).

https://google.github.io/styleguide/cppguide.html#Names_and_Order_of_Includes

 
 class NavigatorBattery final : public GarbageCollected<NavigatorBattery>,
                                public Supplement<Navigator> {