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

Side by Side Diff: chrome/renderer/extensions/chrome_v8_extension_handler.h

Issue 1053663003: Cleanup: Remove unused variables in chrome/ found by Scythe. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_ 5 #ifndef CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_
6 #define CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_ 6 #define CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/basictypes.h" 10 #include "base/basictypes.h"
11 #include "ipc/ipc_listener.h" 11 #include "ipc/ipc_listener.h"
12 #include "v8/include/v8.h" 12 #include "v8/include/v8.h"
13 13
14 namespace extensions { 14 namespace extensions {
15 class ScriptContext;
16 15
17 // Base class for context-scoped handlers used with ChromeV8Extension. 16 // Base class for context-scoped handlers used with ChromeV8Extension.
18 // TODO(koz): Rename/refactor this somehow. Maybe just pull it into 17 // TODO(koz): Rename/refactor this somehow. Maybe just pull it into
19 // ChromeV8Extension. 18 // ChromeV8Extension.
20 class ChromeV8ExtensionHandler : public IPC::Listener { 19 class ChromeV8ExtensionHandler : public IPC::Listener {
21 public: 20 public:
22 ~ChromeV8ExtensionHandler() override; 21 ~ChromeV8ExtensionHandler() override;
23 22
24 // IPC::Listener 23 // IPC::Listener
25 bool OnMessageReceived(const IPC::Message& message) override = 0; 24 bool OnMessageReceived(const IPC::Message& message) override = 0;
26 25
27 protected: 26 protected:
28 explicit ChromeV8ExtensionHandler(ScriptContext* context); 27 ChromeV8ExtensionHandler();
29 int GetRoutingID(); 28 int GetRoutingID();
30 void Send(IPC::Message* message); 29 void Send(IPC::Message* message);
31 ScriptContext* context_;
32 30
33 private: 31 private:
34 int routing_id_; 32 int routing_id_;
35 DISALLOW_COPY_AND_ASSIGN(ChromeV8ExtensionHandler); 33 DISALLOW_COPY_AND_ASSIGN(ChromeV8ExtensionHandler);
36 }; 34 };
37 35
38 } // namespace extensions 36 } // namespace extensions
39 37
40 #endif // CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_ 38 #endif // CHROME_RENDERER_EXTENSIONS_CHROME_V8_EXTENSION_HANDLER_H_
OLDNEW
« no previous file with comments | « chrome/renderer/extensions/app_bindings.cc ('k') | chrome/renderer/extensions/chrome_v8_extension_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698