Files.app: Add private API to check if PiexLoader is enabled or not.

Files.app: Add private API to check if PiexLoader is enabled or not.

PiexLoader is enabled only for official build.

BUG=396702
TEST=None
R=isherman@chromium.org

Committed: https://crrev.com/a54475bee8ff34ec3ada07413938fd973950e906
Cr-Commit-Position: refs/heads/master@{#321933}

[hirono, 2015-03-23 09:32:03]

hirono@chromium.org changed reviewers:
+ mtomasz@chromium.org

[hirono, 2015-03-23 09:32:03]

PTAL, thanks!

[mtomasz, 2015-03-23 09:44:39]

https://codereview.chromium.org/1026733003/diff/1/chrome/browser/chromeos/ext...
File chrome/browser/chromeos/extensions/file_manager/private_api_misc.h (right):

https://codereview.chromium.org/1026733003/diff/1/chrome/browser/chromeos/ext...
chrome/browser/chromeos/extensions/file_manager/private_api_misc.h:195: bool
RunSync() override;
RunSync is deprecated, lets use Run(). See:
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser...

[hirono, 2015-03-24 01:25:38]

https://codereview.chromium.org/1026733003/diff/1/chrome/browser/chromeos/ext...
File chrome/browser/chromeos/extensions/file_manager/private_api_misc.h (right):

https://codereview.chromium.org/1026733003/diff/1/chrome/browser/chromeos/ext...
chrome/browser/chromeos/extensions/file_manager/private_api_misc.h:195: bool
RunSync() override;
On 2015/03/23 09:44:39, mtomasz wrote:
> RunSync is deprecated, lets use Run(). See:
>
https://code.google.com/p/chromium/codesearch#chromium/src/extensions/browser...

Done.

[hirono, 2015-03-24 01:30:47]

hirono@chromium.org changed reviewers:
+ fukino@chromium.org, isherman@chromium.org

[hirono, 2015-03-24 01:30:48]

@fukino - PTAL extern/file_manager_private.js
@isherman - PTAL extensions/browser/extension_function_histogram_value.h and
tools/metrics/histograms/histograms.xml

Thanks!

[mtomasz, 2015-03-24 01:39:23]

lgtm

[fukino, 2015-03-24 01:49:59]

lgtm

[hirono, 2015-03-24 02:27:13]

On 2015/03/24 01:49:59, fukino wrote:
> lgtm

@isherman - Let me do TBR for adding API entry to the histogram definition.

[hirono, 2015-03-24 02:27:16]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-03-24 02:27:49]

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

[commit-bot: I haz the power, 2015-03-24 02:35:12]

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

[commit-bot: I haz the power, 2015-03-24 02:35:14]

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...)

[Ilya Sherman, 2015-03-24 03:51:28]

histograms.xml lgtm

[hirono, 2015-03-24 03:53:27]

The CQ bit was checked by hirono@chromium.org

[commit-bot: I haz the power, 2015-03-24 03:53:33]

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

[commit-bot: I haz the power, 2015-03-24 04:38:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-03-24 04:39:07]

Patchset 3 (id:??) landed as
https://crrev.com/a54475bee8ff34ec3ada07413938fd973950e906
Cr-Commit-Position: refs/heads/master@{#321933}