I forget cpplint every time
authorPaul Betts <paul@paulbetts.org>
Thu, 25 Jun 2015 21:09:25 +0000 (15:09 -0600)
committerPaul Betts <paul@paulbetts.org>
Thu, 25 Jun 2015 21:09:25 +0000 (15:09 -0600)
atom/browser/api/atom_api_window.cc
atom/browser/api/atom_api_window.h
atom/browser/native_window.cc
atom/browser/native_window_observer.h

index be615c9..9e8076f 100644 (file)
@@ -162,7 +162,7 @@ void Window::OnDevToolsClosed() {
   devtools_web_contents_.Reset();
 }
 
-void Window::OnExecuteWindowsCommand(std::string& command_name) {
+void Window::OnExecuteWindowsCommand(const std::string& command_name) {
   Emit("app-command", command_name);
 }
 
index 248abf6..5867b1e 100644 (file)
@@ -71,7 +71,7 @@ class Window : public mate::TrackableObject<Window>,
   void OnDevToolsFocus() override;
   void OnDevToolsOpened() override;
   void OnDevToolsClosed() override;
-  void OnExecuteWindowsCommand(std::string& command_name) override;
+  void OnExecuteWindowsCommand(const std::string& command_name) override;
 
  private:
   // APIs for NativeWindow.
index 308a484..38bbedf 100644 (file)
@@ -596,7 +596,7 @@ void NativeWindow::OnCapturePageDone(const CapturePageCallback& callback,
 }
 
 const char* NativeWindow::AppCommandToString(int command_id) {
-  switch(command_id) {
+  switch (command_id) {
   case APPCOMMAND_BROWSER_BACKWARD:
     return "browser-backward";
   case APPCOMMAND_BROWSER_FORWARD:
index c8a2c5e..5b0a0c5 100644 (file)
@@ -67,7 +67,7 @@ class NativeWindowObserver {
   virtual void OnRendererResponsive() {}
 
   // Called on Windows when App Commands arrive (WM_APPCOMMAND)
-  virtual void OnExecuteWindowsCommand(std::string& command_name) {}
+  virtual void OnExecuteWindowsCommand(const std::string& command_name) {}
 };
 
 }  // namespace atom