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

Unified Diff: content/renderer/v8_value_converter_impl.cc

Issue 14081010: Cleanup: Remove unnecessary ".get()" from scoped_ptrs<>. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix some gtk issues Created 7 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 side-by-side diff with in-line comments
Download patch
Index: content/renderer/v8_value_converter_impl.cc
diff --git a/content/renderer/v8_value_converter_impl.cc b/content/renderer/v8_value_converter_impl.cc
index b1a063c5a59d4f9b233b0da4050978b05672b71d..e8b256303df7b0fa3e9f9e127928e28d6ba89b8e 100644
--- a/content/renderer/v8_value_converter_impl.cc
+++ b/content/renderer/v8_value_converter_impl.cc
@@ -263,12 +263,12 @@ base::BinaryValue* V8ValueConverterImpl::FromV8Buffer(
scoped_ptr<WebKit::WebArrayBuffer> array_buffer(
WebKit::WebArrayBuffer::createFromV8Value(val));
scoped_ptr<WebKit::WebArrayBufferView> view;
- if (array_buffer.get()) {
+ if (array_buffer) {
data = reinterpret_cast<char*>(array_buffer->data());
length = array_buffer->byteLength();
} else {
view.reset(WebKit::WebArrayBufferView::createFromV8Value(val));
- if (view.get()) {
+ if (view) {
data = reinterpret_cast<char*>(view->baseAddress()) + view->byteOffset();
length = view->byteLength();
}
@@ -322,7 +322,7 @@ Value* V8ValueConverterImpl::FromV8Object(
}
scoped_ptr<base::Value> child(FromV8ValueImpl(child_v8, unique_map));
- if (!child.get())
+ if (!child)
// JSON.stringify skips properties whose values don't serialize, for
// example undefined and functions. Emulate that behavior.
continue;
« no previous file with comments | « content/renderer/renderer_webkitplatformsupport_impl.cc ('k') | content/renderer/v8_value_converter_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698