| OLD | NEW |
| 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. | 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 | 4 |
| 5 #include "chrome/browser/extensions/webstore_install_helper.h" | 5 #include "chrome/browser/extensions/webstore_install_helper.h" |
| 6 | 6 |
| 7 #include "base/bind.h" | 7 #include "base/bind.h" |
| 8 #include "base/values.h" | 8 #include "base/values.h" |
| 9 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h" | 9 #include "chrome/browser/bitmap_fetcher/bitmap_fetcher.h" |
| 10 #include "components/safe_json_parser/safe_json_parser.h" | 10 #include "components/safe_json/safe_json_parser.h" |
| 11 #include "content/public/browser/browser_thread.h" | 11 #include "content/public/browser/browser_thread.h" |
| 12 #include "net/base/load_flags.h" | 12 #include "net/base/load_flags.h" |
| 13 #include "net/url_request/url_request.h" | 13 #include "net/url_request/url_request.h" |
| 14 | 14 |
| 15 using content::BrowserThread; | 15 using content::BrowserThread; |
| 16 | 16 |
| 17 namespace { | 17 namespace { |
| 18 | 18 |
| 19 const char kImageDecodeError[] = "Image decode failed"; | 19 const char kImageDecodeError[] = "Image decode failed"; |
| 20 | 20 |
| (...skipping 21 matching lines...) Expand all Loading... |
| 42 | 42 |
| 43 void WebstoreInstallHelper::Start() { | 43 void WebstoreInstallHelper::Start() { |
| 44 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); | 44 CHECK(BrowserThread::CurrentlyOn(BrowserThread::UI)); |
| 45 | 45 |
| 46 // No existing |json_parser_| to avoid unbalanced AddRef(). | 46 // No existing |json_parser_| to avoid unbalanced AddRef(). |
| 47 CHECK(!json_parser_.get()); | 47 CHECK(!json_parser_.get()); |
| 48 AddRef(); // Balanced in OnJSONParseSucceeded()/OnJSONParseFailed(). | 48 AddRef(); // Balanced in OnJSONParseSucceeded()/OnJSONParseFailed(). |
| 49 // Use base::Unretained so that base::Bind won't AddRef() on us. Otherwise, | 49 // Use base::Unretained so that base::Bind won't AddRef() on us. Otherwise, |
| 50 // we'll have two callbacks holding references to us, only one of which will | 50 // we'll have two callbacks holding references to us, only one of which will |
| 51 // ever be called. | 51 // ever be called. |
| 52 json_parser_ = new safe_json_parser::SafeJsonParser( | 52 json_parser_ = new safe_json::SafeJsonParser( |
| 53 manifest_, | 53 manifest_, |
| 54 base::Bind(&WebstoreInstallHelper::OnJSONParseSucceeded, | 54 base::Bind(&WebstoreInstallHelper::OnJSONParseSucceeded, |
| 55 base::Unretained(this)), | 55 base::Unretained(this)), |
| 56 base::Bind(&WebstoreInstallHelper::OnJSONParseFailed, | 56 base::Bind(&WebstoreInstallHelper::OnJSONParseFailed, |
| 57 base::Unretained(this))); | 57 base::Unretained(this))); |
| 58 json_parser_->Start(); | 58 json_parser_->Start(); |
| 59 | 59 |
| 60 if (icon_url_.is_empty()) { | 60 if (icon_url_.is_empty()) { |
| 61 icon_decode_complete_ = true; | 61 icon_decode_complete_ = true; |
| 62 } else { | 62 } else { |
| (...skipping 59 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 122 if (!icon_decode_complete_ || !manifest_parse_complete_) | 122 if (!icon_decode_complete_ || !manifest_parse_complete_) |
| 123 return; | 123 return; |
| 124 | 124 |
| 125 if (error_.empty() && parsed_manifest_) | 125 if (error_.empty() && parsed_manifest_) |
| 126 delegate_->OnWebstoreParseSuccess(id_, icon_, parsed_manifest_.release()); | 126 delegate_->OnWebstoreParseSuccess(id_, icon_, parsed_manifest_.release()); |
| 127 else | 127 else |
| 128 delegate_->OnWebstoreParseFailure(id_, parse_error_, error_); | 128 delegate_->OnWebstoreParseFailure(id_, parse_error_, error_); |
| 129 } | 129 } |
| 130 | 130 |
| 131 } // namespace extensions | 131 } // namespace extensions |
| OLD | NEW |