Skip to content
This repository was archived by the owner on Sep 2, 2021. It is now read-only.

Commit 224cbec

Browse files
committed
Renamed the variable as per review comments
1 parent 3dc7422 commit 224cbec

2 files changed

Lines changed: 7 additions & 8 deletions

File tree

appshell/appshell_extensions.cpp

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838

3939
extern std::vector<CefString> gDroppedFiles;
4040
extern int g_remote_debugging_port;
41-
extern std::string g_remote_debugging_port_invalid_arg;
41+
extern std::string g_get_remote_debugging_port_error;
4242

4343
namespace appshell_extensions {
4444

@@ -846,12 +846,12 @@ class ProcessMessageDelegate : public ClientHandler::ProcessMessageDelegate {
846846
uberDict->SetList(1, allStats);
847847
responseArgs->SetList(2, uberDict);
848848
} else if (message_name == "GetRemoteDebuggingPort") {
849-
if (g_remote_debugging_port_invalid_arg.empty() && g_remote_debugging_port > 0) {
849+
if (g_get_remote_debugging_port_error.empty() && g_remote_debugging_port > 0) {
850850
responseArgs->SetInt(2, g_remote_debugging_port);
851851
}
852852
else {
853853
responseArgs->SetNull(2);
854-
errInfo = g_remote_debugging_port_invalid_arg;
854+
errInfo = g_get_remote_debugging_port_error;
855855
}
856856
}
857857

@@ -865,7 +865,6 @@ class ProcessMessageDelegate : public ClientHandler::ProcessMessageDelegate {
865865
responseArgs->SetInt(1, error);
866866
}
867867
else {
868-
// Cef is not allowing to set CefDictionaryValue as error-info.
869868
responseArgs->SetString(1, errInfo);
870869
}
871870

appshell/cefclient.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121

2222
CefRefPtr<ClientHandler> g_handler;
2323
int g_remote_debugging_port = 0;
24-
std::string g_remote_debugging_port_invalid_arg;
24+
std::string g_get_remote_debugging_port_error;
2525

2626
#ifdef OS_WIN
2727
bool g_force_enable_acc = false;
@@ -100,8 +100,8 @@ void AppGetSettings(CefSettings& settings, CefRefPtr<CefCommandLine> command_lin
100100
// Enable dev tools
101101
CefString debugger_port = command_line->GetSwitchValue("remote-debugging-port");
102102
if (!debugger_port.empty()) {
103-
g_remote_debugging_port_invalid_arg = debugger_port.ToString();
104-
long port = strtol(g_remote_debugging_port_invalid_arg.c_str(), NULL, 10);
103+
g_get_remote_debugging_port_error = debugger_port.ToString();
104+
long port = strtol(g_get_remote_debugging_port_error.c_str(), NULL, 10);
105105
if (errno == ERANGE) {
106106
errno = port = 0;
107107
}
@@ -110,7 +110,7 @@ void AppGetSettings(CefSettings& settings, CefRefPtr<CefCommandLine> command_lin
110110
if (port > max_reserved_port_num && port < max_port_num) {
111111
g_remote_debugging_port = static_cast<int>(port);
112112
settings.remote_debugging_port = g_remote_debugging_port;
113-
g_remote_debugging_port_invalid_arg.clear();
113+
g_get_remote_debugging_port_error.clear();
114114
}
115115
else {
116116
// Setting debugging port to highest number will disable remote debugging

0 commit comments

Comments
 (0)