[system health] Add desktop browsing story for facebook and change url of twitter story.

[system health] Add desktop browsing story for facebook and change url of twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035
Committed: https://crrev.com/0a7a65b967c020c7380cba3488b8316b56708ed7
Cr-Commit-Position: refs/heads/master@{#406267}

[ulan, 2016-07-18 11:07:26]

ulan@chromium.org changed reviewers:
+ petrcermak@chromium.org

[ulan, 2016-07-18 11:07:26]

ptal

https://codereview.chromium.org/2162473002/diff/1/tools/perf/page_sets/system...
File tools/perf/page_sets/system_health/browsing_stories.py (left):

https://codereview.chromium.org/2162473002/diff/1/tools/perf/page_sets/system...
tools/perf/page_sets/system_health/browsing_stories.py:98: # directly in the
main timeline.
The comment was not entirely correct: photo posts are opened in pop-up window.

https://codereview.chromium.org/2162473002/diff/1/tools/perf/page_sets/system...
File tools/perf/page_sets/system_health/browsing_stories.py (right):

https://codereview.chromium.org/2162473002/diff/1/tools/perf/page_sets/system...
tools/perf/page_sets/system_health/browsing_stories.py:161: URL =
'https://www.twitter.com/nasa'
It is difficult to get work appropriate twitter comment contents with the old
url :)

[petrcermak, 2016-07-18 11:24:35]

LGTM. Please add a bug ID, or remove the "BUG=" line.

Thanks,
Petr

[ulan, 2016-07-18 11:30:03]

Description was changed from

==========
Add desktop browsing story for facebook and change url of twitter story.

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

to

==========
Add desktop browsing story for facebook and change url of twitter story.
==========

[ulan, 2016-07-19 10:42:53]

Description was changed from

==========
Add desktop browsing story for facebook and change url of twitter story.
==========

to

==========
Add desktop browsing story for facebook and change url of twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter
==========

[ulan, 2016-07-19 10:43:45]

Description was changed from

==========
Add desktop browsing story for facebook and change url of twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter
==========

to

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter
==========

[ulan, 2016-07-19 10:44:15]

Description was changed from

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter
==========

to

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035
==========

[ulan, 2016-07-19 10:47:36]

The CQ bit was checked by ulan@chromium.org

[ulan, 2016-07-19 10:47:36]

The patchset sent to the CQ was uploaded after l-g-t-m from
petrcermak@chromium.org
Link to the patchset: https://codereview.chromium.org/2162473002/#ps20001
(title: "record wpr, disable facebook")

[commit-bot: I haz the power, 2016-07-19 10:47:49]

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-07-19 10:55:22]

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

[commit-bot: I haz the power, 2016-07-19 10:55:23]

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

[ulan, 2016-07-19 13:30:46]

ulan@chromium.org changed reviewers:
+ nednguyen@google.com

[ulan, 2016-07-19 13:30:47]

Ned, could you please rubber-stamp as an owner?

[nednguyen, 2016-07-19 13:46:25]

On 2016/07/19 13:30:47, ulan wrote:
> Ned, could you please rubber-stamp as an owner?

lgtm sorry

[ulan, 2016-07-19 13:54:33]

The CQ bit was checked by ulan@chromium.org

[commit-bot: I haz the power, 2016-07-19 13:54:54]

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-07-19 13:59:42]

Description was changed from

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035
==========

to

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035
==========

[commit-bot: I haz the power, 2016-07-19 13:59:43]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-07-19 13:59:47]

CQ bit was unchecked.

[commit-bot: I haz the power, 2016-07-19 14:01:03]

Description was changed from

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035
==========

to

==========
[system health] Add desktop browsing story for facebook and change url of
twitter story.

*** NOTE TO SHERRIF ***
Regressions are expected in
- system_health.memory_mobile
- system_health.memory_desktop
- v8.browsing_mobile
- v8.browsing_desktop
for the following stories:
- load:social:twitter
- browse:social:twitter

BUG=chromium:621035

Committed: https://crrev.com/0a7a65b967c020c7380cba3488b8316b56708ed7
Cr-Commit-Position: refs/heads/master@{#406267}
==========

[commit-bot: I haz the power, 2016-07-19 14:01:04]

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