Allow embedded_test_server to serve from multiple directories

Allow embedded_test_server to serve from multiple directories

Add a new method to EmbeddedTestServer:
ServeFilesFromDirectories(const std::vector<base::FilePath>& dirs)

According to the order in dirs, the first matched file would be served.

TEST=net_unittests --gtest_filter='EmbeddedTestServerTest.*'

[wychen, 2015-04-22 04:22:29]

wychen@chromium.org changed reviewers:
+ phajdan.jr@chromium.org, satorux@chromium.org

[wychen, 2015-04-22 04:22:29]

PTAL

[satorux1, 2015-04-22 04:26:25]

satorux@chromium.org changed reviewers:
+ mtomasz@chromium.org
- satorux@chromium.org

[satorux1, 2015-04-22 04:26:25]

defer to mtomasz@

[mtomasz, 2015-04-22 11:03:16]

lgtm

[wychen, 2015-04-22 16:55:46]

The CQ bit was checked by wychen@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2015-04-22 16:58:06]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1093993003/1

[commit-bot: I haz the power, 2015-04-22 19:06:40]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-22 19:06:42]

Dry run: Try jobs failed on following builders:
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[wychen, 2015-04-22 22:43:16]

The CQ bit was checked by wychen@chromium.org

[wychen, 2015-04-22 22:43:16]

The patchset sent to the CQ was uploaded after l-g-t-m from mtomasz@chromium.org
Link to the patchset: https://codereview.chromium.org/1093993003/#ps40001
(title: "fix iOS")

[commit-bot: I haz the power, 2015-04-22 22:45:04]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1093993003/40001

[wychen, 2015-04-23 00:45:41]

wychen@chromium.org changed reviewers:
+ asanka@chromium.org

[wychen, 2015-04-23 00:45:41]

Asanka, can you take a look as an owner? Thanks!

[commit-bot: I haz the power, 2015-04-23 01:56:03]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-04-23 01:56:05]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[Paweł Hajdan Jr., 2015-04-23 23:23:25]

Why is this needed?

[wychen, 2015-04-23 23:27:16]

On 2015/04/23 23:23:25, Paweł Hajdan Jr. wrote:
> Why is this needed?

Sorry for missing the context. This CL below relies on this change.
https://codereview.chromium.org/1009703002/

In that test, files from multiple directories are needed in the embedded test
server, but unfortunately we couldn't use symlink, so serving from multiple
directories is the alternative way to do it.

[asanka, 2015-04-23 23:53:25]

On 2015/04/23 at 23:27:16, wychen wrote:
> On 2015/04/23 23:23:25, Paweł Hajdan Jr. wrote:
> > Why is this needed?
> 
> Sorry for missing the context. This CL below relies on this change.
> https://codereview.chromium.org/1009703002/
> 
> In that test, files from multiple directories are needed in the embedded test
server, but unfortunately we couldn't use symlink, so serving from multiple
directories is the alternative way to do it.

You should be able to call ServeFilesFromDirectory() multiple times to add
directories.

[wychen, 2015-04-24 00:08:32]

On 2015/04/23 23:53:25, asanka wrote:
> You should be able to call ServeFilesFromDirectory() multiple times to add
> directories.

You are right. I'll just close this CL. Thanks for pointing this out.