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

Side by Side Diff: Source/bindings/core/v8/custom/V8BlobCustomHelpers.cpp

Issue 1011093002: [bindings] Pass NormalConversion to to[U]Int{8/16/32/64}(...) in v8_types.py and reduce conversion … (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Review comments Created 5 years, 9 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 /* 1 /*
2 * Copyright (C) 2013 Google Inc. All rights reserved. 2 * Copyright (C) 2013 Google Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions are 5 * modification, are permitted provided that the following conditions are
6 * met: 6 * met:
7 * 7 *
8 * * Redistributions of source code must retain the above copyright 8 * * Redistributions of source code must retain the above copyright
9 * notice, this list of conditions and the following disclaimer. 9 * notice, this list of conditions and the following disclaimer.
10 * * Redistributions in binary form must reproduce the above 10 * * Redistributions in binary form must reproduce the above
(...skipping 80 matching lines...) Expand 10 before | Expand all | Expand 10 after
91 } 91 }
92 m_contentType = m_contentType.lower(); 92 m_contentType = m_contentType.lower();
93 } 93 }
94 94
95 if (!m_hasFileProperties) 95 if (!m_hasFileProperties)
96 return true; 96 return true;
97 97
98 v8::Local<v8::Value> lastModified; 98 v8::Local<v8::Value> lastModified;
99 TONATIVE_DEFAULT(bool, containsLastModified, DictionaryHelper::get(dictionar y, "lastModified", lastModified), false); 99 TONATIVE_DEFAULT(bool, containsLastModified, DictionaryHelper::get(dictionar y, "lastModified", lastModified), false);
100 if (containsLastModified) { 100 if (containsLastModified) {
101 TONATIVE_DEFAULT(long long, lastModifiedInt, toInt64(isolate, lastModifi ed), false); 101 NonThrowableExceptionState exceptionState;
haraken 2015/03/17 15:08:12 Ditto. Can we use the existing |exceptionState| ?
102 TONATIVE_DEFAULT(long long, lastModifiedInt, toInt64(isolate, lastModifi ed, NormalConversion, exceptionState), false);
102 setLastModified(static_cast<double>(lastModifiedInt)); 103 setLastModified(static_cast<double>(lastModifiedInt));
103 } else { 104 } else {
104 setDefaultLastModified(); 105 setDefaultLastModified();
105 } 106 }
106 107
107 return true; 108 return true;
108 } 109 }
109 110
110 bool processBlobParts(v8::Isolate* isolate, v8::Local<v8::Object> blobParts, boo l normalizeLineEndingsToNative, BlobData& blobData) 111 bool processBlobParts(v8::Isolate* isolate, v8::Local<v8::Object> blobParts, boo l normalizeLineEndingsToNative, BlobData& blobData)
111 { 112 {
(...skipping 21 matching lines...) Expand all
133 TOSTRING_DEFAULT(V8StringResource<>, stringValue, item, false); 134 TOSTRING_DEFAULT(V8StringResource<>, stringValue, item, false);
134 blobData.appendText(stringValue, normalizeLineEndingsToNative); 135 blobData.appendText(stringValue, normalizeLineEndingsToNative);
135 } 136 }
136 } 137 }
137 return true; 138 return true;
138 } 139 }
139 140
140 } // namespace V8BlobCustomHelpers 141 } // namespace V8BlobCustomHelpers
141 142
142 } // namespace blink 143 } // namespace blink
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698