Listen OnDisplayMetricsChanged on lock screen

Listen OnDisplayMetricsChanged on lock screen

BUG=453318
TEST=manual

Committed: https://crrev.com/6469449b22b7c5b919ec353ecaa73e0e7766df07
Cr-Commit-Position: refs/heads/master@{#313909}

[Dmitry Polukhin, 2015-01-29 15:15:20]

dpolukhin@chromium.org changed reviewers:
+ oshima@chromium.org

[Dmitry Polukhin, 2015-01-29 15:15:21]

PTAL

[oshima, 2015-01-29 15:29:06]

lgtm with nits

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
File chrome/browser/chromeos/login/lock/webui_screen_locker.h (right):

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
chrome/browser/chromeos/login/lock/webui_screen_locker.h:117: // Overridden from
gfx::DisplayObserver:
nit: can you update these comments to new style?

[Dmitry Polukhin, 2015-01-29 15:39:17]

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
File chrome/browser/chromeos/login/lock/webui_screen_locker.h (right):

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
chrome/browser/chromeos/login/lock/webui_screen_locker.h:117: // Overridden from
gfx::DisplayObserver:
On 2015/01/29 15:29:06, oshima wrote:
> nit: can you update these comments to new style?
> 

Sorry, what do you mean by "new style"? I don't see anything related in style
guide http://dev.chromium.org/developers/coding-style

[oshima, 2015-01-29 16:16:09]

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
File chrome/browser/chromeos/login/lock/webui_screen_locker.h (right):

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
chrome/browser/chromeos/login/lock/webui_screen_locker.h:117: // Overridden from
gfx::DisplayObserver:
On 2015/01/29 15:39:17, Dmitry Polukhin wrote:
> On 2015/01/29 15:29:06, oshima wrote:
> > nit: can you update these comments to new style?
> > 
> 
> Sorry, what do you mean by "new style"? I don't see anything related in style
> guide http://dev.chromium.org/developers/coding-style

// ClassName:

like 

// gfx::DisplayObserver:

no "Overridden from" or "implmenetation."

[Dmitry Polukhin, 2015-01-30 13:08:09]

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
File chrome/browser/chromeos/login/lock/webui_screen_locker.h (right):

https://codereview.chromium.org/882423002/diff/1/chrome/browser/chromeos/logi...
chrome/browser/chromeos/login/lock/webui_screen_locker.h:117: // Overridden from
gfx::DisplayObserver:
On 2015/01/29 16:16:09, oshima wrote:
> On 2015/01/29 15:39:17, Dmitry Polukhin wrote:
> > On 2015/01/29 15:29:06, oshima wrote:
> > > nit: can you update these comments to new style?
> > > 
> > 
> > Sorry, what do you mean by "new style"? I don't see anything related in
style
> > guide http://dev.chromium.org/developers/coding-style
> 
> // ClassName:
> 
> like 
> 
> // gfx::DisplayObserver:
> 
> no "Overridden from" or "implmenetation."

Done.

[Dmitry Polukhin, 2015-01-30 13:08:18]

The CQ bit was checked by dpolukhin@chromium.org

[commit-bot: I haz the power, 2015-01-30 13:08:52]

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

[commit-bot: I haz the power, 2015-01-30 13:59:32]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-01-30 14:02:00]

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