Roll dom_distiller_js

Roll dom_distiller_js

This also changes the way DomDistillerResult is read from base::Values by
updating to the new version of DomDistillerResult::ReadFromValue which
takes in a pointer to DomDistillerResult.

This CL also makes it possible to continue using the
update_domdistiller_js.sh script without modification by moving the
BUILD.gn file for protos to outside of the generated package folder.

Picked up changes:
432b2dc Added support for reading protos into a pointer.
32f096c don't prune a subtree when its only child is a text node.
90dc83c bump up nested-table heuristic
6f59530 Make comment filter a little more restrictive.

BUG=368941, 408292, 409274
Committed: https://crrev.com/276e0ebd79574273c2e32651d70dfa19b9f3ba08
Cr-Commit-Position: refs/heads/master@{#293275}

[nyquist, 2014-08-28 22:10:34]

nyquist@chromium.org changed reviewers:
+ cjhopman@chromium.org

[nyquist, 2014-08-28 22:10:34]

cjhopman: PTAL

Still need to submit the CL in the other repository and roll to that one in this
CL.

[cjhopman, 2014-09-02 20:30:40]

this looks fine, can you add the actual roll to it?

[nyquist, 2014-09-02 20:38:10]

Will submit external repo CL and add the roll as soon as I have submitted the
struct -> proto change. I'll ping you when that happens.

[nyquist, 2014-09-03 18:19:23]

cjhopman: PTAL. Added the roll.

[nyquist, 2014-09-03 18:27:39]

Also, why on earth did the update-script remove the BUILD.gn file? I am guessing
that is unintentional?

[cjhopman, 2014-09-03 19:30:28]

On 2014/09/03 18:27:39, nyquist wrote:
> Also, why on earth did the update-script remove the BUILD.gn file? I am
guessing
> that is unintentional?

I don't see anything in there that would do that. Can you reproduce it? Can you
fix it?

[nyquist, 2014-09-03 20:55:24]

easy to repro. moved BUILD.gn out of generated package folder.

[nyquist, 2014-09-03 20:55:47]

PTAL

[cjhopman, 2014-09-03 21:05:02]

lgtm

[nyquist, 2014-09-03 21:43:07]

The CQ bit was checked by nyquist@chromium.org

[nyquist, 2014-09-03 21:43:16]

The CQ bit was unchecked by nyquist@chromium.org

[nyquist, 2014-09-03 21:44:22]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-09-03 23:23:41]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nyquist@chromium.org/514423002/60002

[commit-bot: I haz the power, 2014-09-04 02:12:26]

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

[commit-bot: I haz the power, 2014-09-04 02:12:27]

Try jobs failed on following builders:
  android_clang_dbg_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_clang_d...)
  android_dbg_tests_recipe on tryserver.chromium.linux
(http://build.chromium.org/p/tryserver.chromium.linux/builders/android_dbg_tes...)

[nyquist, 2014-09-04 06:48:37]

The CQ bit was checked by nyquist@chromium.org

[commit-bot: I haz the power, 2014-09-04 06:49:23]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/nyquist@chromium.org/514423002/60002

[commit-bot: I haz the power, 2014-09-04 06:51:05]

Committed patchset #5 (id:60002) as e02e44e8d35654d43b198ff99859433da437ce93

[commit-bot: I haz the power, 2014-09-10 03:30:28]

Patchset 5 (id:??) landed as
https://crrev.com/276e0ebd79574273c2e32651d70dfa19b9f3ba08
Cr-Commit-Position: refs/heads/master@{#293275}