MD WebUI: fix vulcanize.py for Windows

MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/29c4ba7e0a2d3003db7c7c77c702ed377ab8bceb
Cr-Commit-Position: refs/heads/master@{#439864}

[Dan Beam, 2016-12-20 18:40:40]

Description was changed from

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
==========

to

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-12-20 18:45:08]

Patchset #1 (id:1) has been deleted

[dpapad, 2016-12-20 18:54:08]

lgtm

https://codereview.chromium.org/2594763002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/vulcanize.py (right):

https://codereview.chromium.org/2594763002/diff/20001/chrome/browser/resource...
chrome/browser/resources/vulcanize.py:42: '--redirect',
'"chrome://resources/cr_elements/|%s"' % _CR_ELEMENTS_PATH,
Perhaps add a comment explaining that this is necessary for Win.

[Dan Beam, 2016-12-20 19:03:03]

https://codereview.chromium.org/2594763002/diff/20001/chrome/browser/resource...
File chrome/browser/resources/vulcanize.py (right):

https://codereview.chromium.org/2594763002/diff/20001/chrome/browser/resource...
chrome/browser/resources/vulcanize.py:42: '--redirect',
'"chrome://resources/cr_elements/|%s"' % _CR_ELEMENTS_PATH,
On 2016/12/20 18:54:08, dpapad wrote:
> Perhaps add a comment explaining that this is necessary for Win.

it's necessary for all platforms.  there's a | in the middle.  win just doesn't
get '

[Dan Beam, 2016-12-20 19:03:12]

Description was changed from

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[Dan Beam, 2016-12-20 19:03:14]

The CQ bit was checked by dbeam@chromium.org

[commit-bot: I haz the power, 2016-12-20 19:03:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-20 19:10:26]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482260594697520,
"parent_rev": "81198278e25a679fcb76812ef90fc1db573b7314", "commit_rev":
"53efa584b0317c415a4454707db84e3262640867"}

[commit-bot: I haz the power, 2016-12-20 19:10:53]

Description was changed from

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2594763002
==========

[commit-bot: I haz the power, 2016-12-20 19:10:54]

Committed patchset #1 (id:20001)

[commit-bot: I haz the power, 2016-12-20 19:14:47]

Description was changed from

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Review-Url: https://codereview.chromium.org/2594763002
==========

to

==========
MD WebUI: fix vulcanize.py for Windows

R=dpapad@chromium.org
BUG=673825
NOTRY=true
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/29c4ba7e0a2d3003db7c7c77c702ed377ab8bceb
Cr-Commit-Position: refs/heads/master@{#439864}
==========

[commit-bot: I haz the power, 2016-12-20 19:14:48]

Patchset 1 (id:??) landed as
https://crrev.com/29c4ba7e0a2d3003db7c7c77c702ed377ab8bceb
Cr-Commit-Position: refs/heads/master@{#439864}