Do not white list offline archive directory

Do not white list offline archive directory

This is not longer needed after we switch to using offline interceptor

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

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

[jianli, 2016-10-06 00:44:13]

Description was changed from

==========
Do not white list offline archive directory

This is not longer needed after we switch to using offline interceptor

BUG=
==========

to

==========
Do not white list offline archive directory

This is not longer needed after we switch to using offline interceptor

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

[jianli, 2016-10-06 00:44:17]

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

[commit-bot: I haz the power, 2016-10-06 00:45:05]

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

[jianli, 2016-10-06 00:46:00]

jianli@chromium.org changed reviewers:
+ dbeam@chromium.org, mmenke@chromium.org

[jianli, 2016-10-06 00:46:01]

mmenke for chrome_network_delegate.cc
dbeam for offline_internals.js

[Dan Beam, 2016-10-06 00:48:35]

lgtm

[commit-bot: I haz the power, 2016-10-06 02:45:35]

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

[commit-bot: I haz the power, 2016-10-06 02:45:36]

Dry run: This issue passed the CQ dry run.

[mmenke, 2016-10-06 15:28:31]

LGTM, thanks for removing this!

https://codereview.chromium.org/2395763003/diff/1/chrome/browser/net/chrome_n...
File chrome/browser/net/chrome_network_delegate.cc (left):

https://codereview.chromium.org/2395763003/diff/1/chrome/browser/net/chrome_n...
chrome/browser/net/chrome_network_delegate.cc:522:
profile_path_.Append(chrome::kOfflinePageArchivesDirname);
Do we still need the chrome/common/chrome_constants.h include?  If it was added
just for this, I suspect not.

[jianli, 2016-10-06 22:31:33]

https://codereview.chromium.org/2395763003/diff/1/chrome/browser/net/chrome_n...
File chrome/browser/net/chrome_network_delegate.cc (left):

https://codereview.chromium.org/2395763003/diff/1/chrome/browser/net/chrome_n...
chrome/browser/net/chrome_network_delegate.cc:522:
profile_path_.Append(chrome::kOfflinePageArchivesDirname);
On 2016/10/06 15:28:31, mmenke wrote:
> Do we still need the chrome/common/chrome_constants.h include?  If it was
added
> just for this, I suspect not.

Done.

[jianli, 2016-10-06 22:31:43]

The CQ bit was checked by jianli@chromium.org

[jianli, 2016-10-06 22:31:43]

The patchset sent to the CQ was uploaded after l-g-t-m from dbeam@chromium.org,
mmenke@chromium.org
Link to the patchset: https://codereview.chromium.org/2395763003/#ps20001
(title: "Removed unneeded include file")

[commit-bot: I haz the power, 2016-10-06 22:32:47]

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-10-06 23:43:15]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-10-06 23:45:58]

Description was changed from

==========
Do not white list offline archive directory

This is not longer needed after we switch to using offline interceptor

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation
==========

to

==========
Do not white list offline archive directory

This is not longer needed after we switch to using offline interceptor

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:closure_compilation

Committed: https://crrev.com/ca52e6c7bc29818fad9f2422d54530a699a63188
Cr-Commit-Position: refs/heads/master@{#423741}
==========

[commit-bot: I haz the power, 2016-10-06 23:45:59]

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