Fix missing mandoline deps.

mandoline/app/BUILD.gn

 # Copyright 2015 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import("//third_party/mojo/src/mojo/public/mojo.gni")
 
 group("app") {
   deps = [
     "//mandoline/ui",
   ]
 
   if (is_android) {
     deps += [ "//mandoline/app/android" ]
   } else {
     deps += [ "//mandoline/app/desktop" ]
   }
 
   data_deps = [
-    "//components/html_viewer",

[msw, 2015/05/15 16:53:10]

Why does moving this to mandoline/app/desktop help?
If we need html_viewer for android and desktop, isn't this right?

[jam, 2015/05/18 14:21:51]

the issue is that one when builds "mandoline", that builds the
mandoline/mandoline_apk targets in mandoline/app/{android,desktop}. the android
apk depended on html_viewer already. the desktop executable target didn't. they
don't depend on mandoline/app group.

[msw, 2015/05/18 18:04:42]

Ah ok, then shouldn't we move these other data deps to the mandoline targets as
well? Building "mandoline" from a clean output dir should suffice. I think it'd
be nice if this "app" target itself were renamed to be a cross-platform
"mandoline" target, listing the shared desktop/android deps and the
platform-specific executable target.

[jam, 2015/05/20 15:45:28]

I don't know why exactly this app group was added. we should figure that out
asynchronously, but it seems we should land this now because we need building
"mandoline" to work correctly.

i can verify that building mandoline alone with this change fixes things.

     "//mandoline/ui/browser",
     "//mandoline/services/core_services",
     "//mojo/runner:mojo_runner",
   ]
 }