Move web store data fetching and parsing out of WebstoreStandaloneInstaller.

Move web store data fetching and parsing out of WebstoreStandaloneInstaller.

So that the code could be shared with kiosk apps code.

BUG=173749
TEST=Existing WebstoreStandaloneInstaller test should pass.

R=asargent@chromium.org
TBR=sky@chromium.org for chrome/chrome_browser_extensions.gypi

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=182081

[asargent_no_longer_on_chrome, 2013-02-12 18:40:10]

LGTM

(+cc sergeygs as FYI since he's doing some work in this file currently as well)

https://codereview.chromium.org/12212076/diff/1/chrome/browser/extensions/web...
File chrome/browser/extensions/webstore_data_fetcher.cc (right):

https://codereview.chromium.org/12212076/diff/1/chrome/browser/extensions/web...
chrome/browser/extensions/webstore_data_fetcher.cc:31: //
WebstoreDataFetcher::SafeWebstoreResponseParser
nit: Is this comment needed/helpful? There's only one class in this file. =)

[xiyuan, 2013-02-12 18:49:03]

+sky for TBRing chrome/chrome_browser_extensions.gypi change.

https://codereview.chromium.org/12212076/diff/1/chrome/browser/extensions/web...
File chrome/browser/extensions/webstore_data_fetcher.cc (right):

https://codereview.chromium.org/12212076/diff/1/chrome/browser/extensions/web...
chrome/browser/extensions/webstore_data_fetcher.cc:31: //
WebstoreDataFetcher::SafeWebstoreResponseParser
On 2013/02/12 18:40:10, Antony Sargent wrote:
> nit: Is this comment needed/helpful? There's only one class in this file. =)

Removed.

[commit-bot: I haz the power, 2013-02-12 18:59:05]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xiyuan@chromium.org/12212076/5001

[commit-bot: I haz the power, 2013-02-12 18:59:10]

Failed to apply patch for chrome/chrome_browser_extensions.gypi:
While running patch -p1 --forward --force --no-backup-if-mismatch;
  patching file chrome/chrome_browser_extensions.gypi
  Hunk #1 FAILED at 713.
  1 out of 1 hunk FAILED -- saving rejects to file
chrome/chrome_browser_extensions.gypi.rej

Patch:       chrome/chrome_browser_extensions.gypi
Index: chrome/chrome_browser_extensions.gypi
diff --git a/chrome/chrome_browser_extensions.gypi
b/chrome/chrome_browser_extensions.gypi
index
41f8bd070983a10ad32de7dde4c43cf257787ffe..a831c9440fc5148ae1aa5c40c3d3c783b7cbb4fa
100644
--- a/chrome/chrome_browser_extensions.gypi
+++ b/chrome/chrome_browser_extensions.gypi
@@ -713,6 +713,9 @@
         'browser/extensions/web_intent_callbacks.h',
         'browser/extensions/web_intents_parser.cc',
         'browser/extensions/web_intents_parser.h',
+        'browser/extensions/webstore_data_fetcher.cc',
+        'browser/extensions/webstore_data_fetcher.h',
+        'browser/extensions/webstore_data_fetcher_delegate.h',
         'browser/extensions/webstore_install_helper.cc',
         'browser/extensions/webstore_install_helper.h',
         'browser/extensions/webstore_installer.cc',

[commit-bot: I haz the power, 2013-02-12 21:20:00]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/xiyuan@chromium.org/12212076/11

[commit-bot: I haz the power, 2013-02-13 01:22:01]

Change committed as 182081