Skip to content

Commit

Permalink
Fix lint issues in feature branch (#2970)
Browse files Browse the repository at this point in the history
b/335357085
  • Loading branch information
andrewsavage1 committed Apr 17, 2024
1 parent e889697 commit 4708bf5
Show file tree
Hide file tree
Showing 11 changed files with 16 additions and 37 deletions.
8 changes: 4 additions & 4 deletions cobalt/layout_tests/web_platform_test_parser.cc
Original file line number Diff line number Diff line change
Expand Up @@ -119,12 +119,12 @@ std::vector<WebPlatformTestInfo> EnumerateWebPlatformTests(
&result);

if (!success) {
DLOG(ERROR) << "Failed to evaluate precondition: "
<< "\"" << precondition << "\"";
DLOG(ERROR) << "Failed to evaluate precondition: " << "\"" << precondition
<< "\"";
// Continue to enumerate tests like normal.
} else if (result != "true") {
DLOG(WARNING) << "Skipping Web Platform Tests for "
<< "\"" << top_level << "\"";
DLOG(WARNING) << "Skipping Web Platform Tests for " << "\"" << top_level
<< "\"";
return std::vector<WebPlatformTestInfo>();
}
}
Expand Down
2 changes: 1 addition & 1 deletion cobalt/network/dial/dial_http_server.h
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ class NET_EXPORT DialHttpServer
void Stop();

// HttpServer::Delegate implementation
void OnConnect(int /*conn_id*/) override{};
void OnConnect(int /*conn_id*/) override {};
void OnHttpRequest(int conn_id,
const net::HttpServerRequestInfo& info) override;

Expand Down
23 changes: 0 additions & 23 deletions cobalt/network/disk_cache/impl.cc

This file was deleted.

4 changes: 2 additions & 2 deletions cobalt/renderer/backend/graphics_context.cc
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,8 @@ GraphicsContext::GraphicsContext(GraphicsSystem* system) : system_(system) {
// Verify it's the extension needed.
if (strcmp(graphics_extension_->name, kCobaltExtensionGraphicsName) != 0 ||
graphics_extension_->version < 1) {
LOG(WARNING) << "Not using supplied cobalt graphics extension: "
<< "'" << graphics_extension_->name << "' ("
LOG(WARNING) << "Not using supplied cobalt graphics extension: " << "'"
<< graphics_extension_->name << "' ("
<< graphics_extension_->version << ")";
graphics_extension_ = nullptr;
}
Expand Down
6 changes: 2 additions & 4 deletions cobalt/web/error_event_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -109,11 +109,9 @@ TEST_P(ErrorEventTestWithJavaScript, ConstructorWithEventTypeAndErrorInitDict) {
EXPECT_EQ("rulez", result);

if (!success) {
DLOG(ERROR) << "Failed to evaluate test: "
<< "\"" << result << "\"";
DLOG(ERROR) << "Failed to evaluate test: " << "\"" << result << "\"";
} else {
LOG(INFO) << "Test result : "
<< "\"" << result << "\"";
LOG(INFO) << "Test result : " << "\"" << result << "\"";
}
}

Expand Down
1 change: 1 addition & 0 deletions cobalt/webdriver/dispatcher.h
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@

#include <memory>
#include <string>
#include <unordered_map>
#include <vector>

#include "base/callback.h"
Expand Down
1 change: 1 addition & 0 deletions cobalt/webdriver/protocol/button.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
#include "cobalt/webdriver/protocol/button.h"

#include <memory>
#include <utility>

namespace cobalt {
namespace webdriver {
Expand Down
1 change: 1 addition & 0 deletions cobalt/webdriver/protocol/element_id.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
#include "cobalt/webdriver/protocol/element_id.h"

#include <memory>
#include <utility>

namespace cobalt {
namespace webdriver {
Expand Down
1 change: 1 addition & 0 deletions cobalt/webdriver/protocol/log_entry.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
#include "cobalt/webdriver/protocol/log_entry.h"

#include <memory>
#include <utility>

#include "base/logging.h"

Expand Down
1 change: 1 addition & 0 deletions cobalt/webdriver/protocol/size.cc
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
#include "cobalt/webdriver/protocol/size.h"

#include <memory>
#include <utility>

namespace cobalt {
namespace webdriver {
Expand Down
5 changes: 2 additions & 3 deletions cobalt/websocket/web_socket_impl.cc
Original file line number Diff line number Diff line change
Expand Up @@ -205,9 +205,8 @@ void WebSocketImpl::OnClose(bool was_clean, int error_code,

std::uint16_t close_code = static_cast<std::uint16_t>(error_code);

DLOG(INFO) << "WebSocket is closing."
<< " code[" << close_code << "] reason[" << close_reason << "]"
<< " was_clean: " << was_clean;
DLOG(INFO) << "WebSocket is closing." << " code[" << close_code << "] reason["
<< close_reason << "]" << " was_clean: " << was_clean;

// Queue the deletion of |websocket_channel_|. We would do it here, but this
// function may be called as a callback *by* |websocket_channel_|;
Expand Down

0 comments on commit 4708bf5

Please sign in to comment.